Attiny13A, programmieren von verschiedenen abläufen

k, x, i sind nicht gerade aussagekräftige Dateinamen. Nutze den Platz 
für längere Namen.

Auch die Port-Zugriffe kann man mit Hilfe von Definitionen und 
Funktionen lesbarer machen. Beispiele:

1

#define BEREITSCHAFTS_LED PB3

2

PORTB |= (1<<BEREITSCHAFTS_LED);   // ein schalten

3

PORTB &= ~ (1<<BEREITSCHAFTS_LED); // aus schalten

4

5

#define Bereitschafts_LED_Ein PORTB |= (1<<PB3);

6

#define Bereitschafts_LED_Aus PORTB &= ~(1<<PB3);

7

Bereitschafts_LED_Ein;

8

Bereitschafts_LED_Aus;

9

10

static inline void Bereitschafts_LED_Ein()

11

{

12

    PORTB |= (1<<PB3);

13

}

14

15

static inline void Bereitschafts_LED_Aus()

16

{

17

    PORTB |= &= ~(1<<PB3);

18

}

19

20

Bereitschafts_LED_Ein();

21

Bereitschafts_LED_Aus();

Anstelle dieser langen "if"-Ketten halte ich "switch" für eleganter.

Bist du sicher dass die Stromversorgung Ok ist? Hast du 
Abblock-Kondensatoren? Zeige mal den Schaltplan und Fotos.

Wozu dient die Variable i?

Ich sehe da ein Problem:

Wenn k zum Beispiel gerade 14 ist und due dann von Programm X=1 nach x=2 
umschaltest, wird es ziemlich lange dauern, bis der Zähler k überläuft 
und bei 0 beginnt. Ich würde entweder bei jedem Umschalten des Programms 
k auf 0 setzen oder den letzten if Ausruck ändern.

anstatt: if (k==51) {k = 0;}
besser: if (k>=51) {k = 0;}

oder noch besser mit switch:

1

switch(k)

2

{

3

    case 1:  PORTB |=  (1 << PB0); break;

4

    case 6:  PORTB &= ~(1 << PB0); break;

5

    case 11: PORTB |=  (1 << PB0); break;

6

    case 16: PORTB &= ~(1 << PB0); break;     

7

    case 26: PORTB |=  (1 << PB0); break;

8

    case 31: PORTB &= ~(1 << PB0); break;

9

    case 36: PORTB |=  (1 << PB0); break;

10

    case 41: PORTB &= ~(1 << PB0); break;

11

    default: k = 0;

12

}

Tobias B. schrieb:
> Da mir der letzte Spannungsteiler hops gegangen ist

Stromversorgung über einen Spannungsteiler ist schon mal ein riesen 
Fehler. Verwende entweder einen aktiven Spannungsregler (wie den LM7805) 
oder schließe den Mikrocontroller direkt ans Netzteil/Batterie an, falls 
die Spannung passt.

Und nie vergessen: 100nF direkt an jedem IC von VCC nach GND.

Zeige mal den Schaltplan und Fotos.

Tobias B. schrieb:
> Was meint ihr? Ist der 100µF übertrieben? Der ist als SMD Bauteil doch
> recht groß.

Schau ins Datenblatt des Spannungsregler. Da wird immer ein konkreter 
Wert empfohlen und manchmal ein maximale erlaubter Wert angegeben.

Ich glaube, dass Schlüsselwort EEMEM soll vor dem Namen der Variable 
stehen.

Warum benutzt du für x eine 16 Bit (int) Variable, kopierst sie aber zum 
Speichern von und zu 8 Bit um? Vermutlich ist das jetzt nicht die 
Problemursache, aber es ist zumindest merkwürdig.

Wenn du diese EEprom Zelle aufgrund vorheriger falscher Versuche schon 
tausende male beschrieben hast, ist sie vielleicht einfach schon 
verschlissen. Probiere mal einen anderen Speicherplatz. Anstelle der 
EEMEM Variable kannst du bei eeprom_write_byte() und eeprom_read_byte() 
einfach die Nummer (=Adresse) der Speicherzelle angeben.

Zeige den Quelltext mit switch/case.
Benutze längere Variablen Namen.
Benutze #define und/oder Funktionen um den Code besser lesbar zu machen.

Ich habe keine Lust, diese Wirrwarr noch öfter analysieren zu müssen.

> Da immer das if (x < 1) {x = 1;} greift.

Woher weißt du, dass genau diese Zeile greift? Hast du einen Debugger 
verwendet, oder wie analysierst du das? Ich würde das gerne wissen, weil 
man die weitere Vorgehensweise darauf aufsetzen kann.

Noch ein möglicher Fehler:

Beim Programmstart aktivierst du den internen Pull-Up Widerstand vom 
Taster und fragst in ohne nennenswerte Verzögerung danach ab.

Je nachdem, wie hoch die parasitäre Kapazität vom Steckbrett, Leitungen 
und Taster sind, ist der pin dann noch dabei, von Low nach High hoch zu 
kommen.

Benutze einen externen Pull-Up Widerstand mit 1 bis 4,7k Ohm und baue 
eine kleine Warteschleife ein, bevor du den Taster zum ersten mal 
abfragst.

> default: k = 0;

Das war falsch. Das greift ja schon bei all den Zwischenständen von k, 
wo NICHTS passieren soll. So müsste es gehen:

1

switch(k)

2

{

3

    case 1:  PORTB |=  (1 << PB0); break;

4

    case 6:  PORTB &= ~(1 << PB0); break;

5

    case 11: PORTB |=  (1 << PB0); break;

6

    case 16: PORTB &= ~(1 << PB0); break;     

7

    case 26: PORTB |=  (1 << PB0); break;

8

    case 31: PORTB &= ~(1 << PB0); break;

9

    case 36: PORTB |=  (1 << PB0); break;

10

    case 41: PORTB &= ~(1 << PB0); break;

11

    case 51: k = 0;

12

}

Dann aber nicht vergessen, k zurück auf 0 zu setzen, wenn der Taster zum 
Programmwechsel gedrückt wird.

Tobias B. schrieb:
> nein ich habe kein Debugger, live zu sehen was der Attiny macht wäre
> schon schön...

Also hast du das nur vermutet. Das solltest du auch deutlich 
hinschreiben, sonst führt deine (Fehl-)Diagnose die Helfer in die Irre.

Es ist immer Hilfreich, solche Vermutungen durch zusätzliche LED 
Anzeigen oder serielle Ausgaben zu verifizieren. Dafür brauchst du nur 
einen freien Pin. Siehe http://stefanfrings.de/avr_hello_world/

> Wie meint ihr das, den Code lesbarer zu machen? Sorry für diese Frage..

Habe ich doch geschrieben. Versuche es mal so (ungetesteter Code):

1

#define F_CPU 4800000UL //4,8MHz

2

#include <avr/io.h>

3

#include <util/delay.h>

4

#include <avr/eeprom.h>

5

6

// Die LED ist an PB0 und GND angeschlossen

7

#define LED_EIN { PORTB |= (1 << PB0); }

8

#define LED_AUS { PORTB &= ~(1 << PB0); }

9

10

// Der Taster ist an PB1 und GND angeschlossen (erfordert Pull-Up)

11

#define TASTER_GEDRUECKT (!(PINB & (1<<PB1)))

12

13

uint8_t eeBlinkMusterNr EEMEM;  // Um das Blinkmuster im EEprom zu speichern

14

uint8_t blinkMusterNr;          // Aktuell laufendes Blinkmuster (1 oder 2)

15

uint8_t step=0;                 // Schrittzähler für die Blinkmuster

16

17

18

void setup()

19

{

20

    // I/O Pins initialisieren

21

    DDRB  = 0b00000001; // PB0=Ausgang für die LED

22

    PORTB = 0b00000010; // PB1=Eingang mit Pull-Up für den Taster

23

24

    // Konfiguration aus EEprom lesen

25

    blinkMusterNr = eeprom_read_byte(&eeBlinkMusterNr);

26

}

27

28

29

void blinkmuster_1()

30

{

31

    switch (step)

32

    {

33

        case 10:  LED_EIN; break;

34

        case 20:  LED_AUS; break;

35

        case 30:  LED_EIN; break;

36

        case 40:  LED_AUS; break;

37

        case 80:  LED_EIN; break;

38

        case 90:  LED_AUS; break;

39

        case 100: LED_EIN; break;

40

        case 110: LED_AUS; break;

41

        case 150: step=0;

42

    }

43

}

44

45

46

void blinkmuster_2()

47

{

48

    switch (step)

49

    {

50

        case 1:   LED_EIN; break;

51

        case 6:   LED_AUS; break;

52

        case 11:  LED_EIN; break;

53

        case 16:  LED_AUS; break;

54

        case 26:  LED_EIN; break;

55

        case 31:  LED_AUS; break;

56

        case 36:  LED_EIN; break;

57

        case 41:  LED_AUS; break;

58

        case 51: step=0;

59

    }

60

}

61

62

63

int main(void)

64

{

65

    setup();

66

67

    // Main loop, repeats every 25ms

68

    while(1)

69

    {

70

        _delay_ms(25);

71

72

        // Wenn Taste gedrückt...

73

        if (TASTER_GEDRUECKT)

74

        {

75

            // dann auf das nächste Blinkmuster umschalten

76

            blinkMusterNr++;  

77

            step=0;          

78

79

            // Neue Blinkmuster Nr im EEprom speichern

80

            eeprom_write_byte(&eeBlinkMusterNr, blinkMusterNr);

81

82

            // Ein bisschen warten 

83

            // Anm. stefanus: das würde ich etwas "intelligenter" machen

84

            _delay_ms(500);

85

        }

86

87

        // Blinkmuster anzeigen    

88

        switch (blinkMusterNr)

89

        {

90

            case 1: blinkmuster_1(); break;

91

            case 2: blinkmuster_2(); break;

92

            default: blinkMusterNr=1;

93

        }

94

        step++;

95

    }

96

}

Du solltest deiner LED dringend einen Vorwiderstand spendieren, wenn du 
sie nicht killen willst. Wenn du das mit vielen LEDs so (ohne 
Widerstand) machst, brennt dir sogar der Mikrocontroller durch.

Stefanus, vielen Dank.
Vorwiderstand hab ich dran, 220Ohm.

Hmm woran liegt das mit PB5?

Deine code ist sehr übersichtlich, ich werde mir das wohl auch so 
angewöhnen, danke :-)

Tobias schrieb:
> Hmm woran liegt das mit PB5?

PB5 ist der Reset Eingang.

Versuche nicht, die entsprechende Fuse zu ändern, denn ohne Reset 
funktioniert die ganze ISP Schnittstelle danach nicht mehr!

Das Programm hat eine unschöne Stelle:

Wenn blinkMusterNr=2 aktiv ist und du dann die Taste drückst, wird die 
Variable auf 3 erhöht und das dann auch so im EEprom gespeichert.

3 ist ungültig, deswegen wird das direkt danach von den default case auf 
1 geändert.

Letztendlich wird im EEprom immer 2 oder 3 stehen, obwohl in 
Wirklichkeit die Blinkmuster 2 oder 1 aktiv sind. Ist egal, oder?