AVR Project – Relay Timer with ATmega8 AVR MCU

Sebastian M. schrieb:

> Leider funktioniert es aber nicht.

Was heißt funktioniert nicht?

Fang mit was einfacherem an.
Steck mal eine LED an einen Port Pin und sieh nach, ob der µC überhaupt 
läuft, indem du die LED blinken lässt.
Im Moment sind da zu viele ungetestete Systeme im Spiel, so dass man 
nicht wirklich eine Aussage darüber treffen kann, was das Problem ist. 
Also muss man abspecken und mit einem möglichst einfachen Testprogramm 
System für System, Komponente für Komponente in Betrieb nehmen.

Hallo,

ja die LED blinkt. Mit funktioniert nicht meine ich, dass ich keine 
anzeige am LCD Display bekomme.

Dann überprüf mal, wie du das LCD angeshclossen hast, und ob das mit der 
bei dir eingetragenen Konfiguration in lcd.h übereinstimmt.

Ja die Ports und Pins stimmen soweit, nur bei

1

#define LS_BLINK 0B00000001

2

#define LS_ULINE 0B00000010

bin ich mir unsicher.

Sebastian M. schrieb:
> Ja die Ports und Pins stimmen soweit


sagst du.
Photo?


>

1

> #define LS_BLINK 0B00000001

2

> #define LS_ULINE 0B00000010

3

>

>
> bin ich mir unsicher.

Braucht dich erst mal nicht zu interessieren.

Autsch.
Der LCD Code ist furchtbar. So wie der geschrieben ist, trau ich dem 
nicht über den Weg.

Hol dir vom P.Fleury die LCD Library, konfigurier sie und mach damit mal 
ein Testprogramm.

PS: Auch der restliche Code ist furchtbar. Aber das ist ein anderes 
Thema.

wovon möchtest du ein Foto haben, von der Hardware?

1

/************************************************

2

  LCD CONNECTIONS

3

*************************************************/

4

5

#define LCD_DATA D  //Port PD0-PD3 are connected to D4-D7

6

7

#define LCD_E D //Enable OR strobe signal

8

#define LCD_E_POS PD4  //Position of enable in above port

9

10

#define LCD_RS D  

11

#define LCD_RS_POS PD6

12

13

#define LCD_RW D

14

#define LCD_RW_POS PD5

15

16

17

//************************************************

18

19

#define LS_BLINK 0B00000001

20

#define LS_ULINE 0B00000010

Sebastian M. schrieb:
> wovon möchtest du ein Foto haben, von der Hardware?
Nein, von deiner Schwester.

Natürlich von deiner Hardware!
Was denkst du wieviele hier schon nach Hilfe gefragt haben und Bein und 
Stein geschworen haben, ihre Anschlüsse wären korrekt und dann stellte 
sich nach Stunden heraus, dass sie sich zb bei den Pinnummern verzählt 
haben?

Sebastian M. schrieb:

> #define LCD_RS D
> #define LCD_RS_POS PD6
>
> #define LCD_RW D
> #define LCD_RW_POS PD5

Das kann nicht stimmen.

Wenn am Port D die Datenbits auf den Pins PD4 bis PD7 liegen, dann kann 
die RS Leitung nicht auf PD6 liegen

Karl Heinz Buchegger schrieb:
> Das kann nicht stimmen.
>
> Wenn am Port D die Datenbits auf den Pins PD4 bis PD7 liegen, dann kann
> die RS Leitung nicht auf PD6 liegen

Die Ports am Controller sind PD0 - PD3 das sind die Anschlüsse am
LCD D4 - D7.

Sebastian M. schrieb:
> Karl Heinz Buchegger schrieb:
>> Das kann nicht stimmen.
>>
>> Wenn am Port D die Datenbits auf den Pins PD4 bis PD7 liegen, dann kann
>> die RS Leitung nicht auf PD6 liegen
>
> Die Ports am Controller sind PD0 - PD3 das sind die Anschlüsse am
> LCD D4 - D7.

Ja, habs schon gemerkt.
In deinem Projekt ist eine etwas andere Version der LCD Routinen als 
die, die ich mir von der angegebenen Web-Site gezogen habe.

Hier die Bilder...

Karl Heinz Buchegger schrieb:
> In deinem Projekt ist eine etwas andere Version der LCD Routinen als
> die, die ich mir von der angegebenen Web-Site gezogen habe.

Wie meinst du das? Habe diese runtergeladen und eingebunden und 
eigentlich nichts verändert.

Sebastian M. schrieb:
> Hier die Bilder...

Ach ja, der Aufbau ist nicht mehr komplett, hatte bereits angefangen mit 
dem Rückbau. LCD ist aber noch dran gewesen...

Sebastian M. schrieb:
> Karl Heinz Buchegger schrieb:
>> In deinem Projekt ist eine etwas andere Version der LCD Routinen als
>> die, die ich mir von der angegebenen Web-Site gezogen habe.
>
> Wie meinst du das? Habe diese runtergeladen und eingebunden und
> eigentlich nichts verändert.

Die hier
http://extremeelectronics.co.in/avr-tutorials/using-lcd-module-with-avrs/

sind ein wenig anders gemacht.

Sebastian M. schrieb:

> Ach ja, der Aufbau ist nicht mehr komplett, hatte bereits angefangen mit
> dem Rückbau. LCD ist aber noch dran gewesen...

Da ist aber nicht wirklich irgendwas nachvollziehbar zu erkennen.

Karl Heinz Buchegger schrieb:
> Da ist aber nicht wirklich irgendwas nachvollziehbar zu erkennen.

Das habe ich mir schon fast gedacht. Ich hatte es bereits mehrmals 
kontrolliert und immer wieder neu zusammengesteckt.

Welche unterschiede hast du denn in den Routinen gefunden?

Soeben ist meine Bestellung gekommen. Ich werde mal ein anderes Quarz 
sowie andere Kondensatoren versuchen. Melde mich dann zurück.

Ich bekomme jetzt im AVR Studio folgende Warnung nach dem start von 
Build Solution.

1

Warning  1  

2

return type defaults to 'int' [enabled by default]  ...RELAY_TIMER_DOWNLOAD\LCD_RELAY_TIMER.c  76  1  RELAY_TIMER_DOWNLOAD

Das ist dieser Codeabschnitt in der LCD_RELAY_TIMER.c der Fehler soll in 
der Zeile main() sein.:

1

...

2

void LoadOff()

3

{

4

  LOAD_PORT&=(~(1<<LOAD_POS));

5

}

6

main()

7

{

8

9

  while(1)

10

  {

11

    LOAD_DDR|=(1<<LOAD_POS);

12

13

    LoadOff();

14

15

    //Enable Pullups on Keypad

16

    PORTB|=((1<<PB2)|(1<<PB1)|(1<<PB0));

17

18

    int8_t hr,min;  //Target Time

19

    hr=min=0;

20

...

Habe jetzt auch Unterschiede festgestellt zwischen dem Quellcode der auf 
der Homepage abgedruckt ist und der Datei die gedownloaded wird. Dabei 
ist meiner Meinung nach die Abgedruckte version die "fehlerhafte".

main() hat keinen Rückgabetyp und deshalb wird der automatisch zu int.

Ist auch kein Fehler, sondern eine Warnung.

Sollte man troztdem beheben.

cyblord ---- schrieb:
> Ist auch kein Fehler, sondern eine Warnung.

das klingt so einfach. Spielt es keine rolle, wie ich main() deklariere? 
Was schlägst du vor?

Sebastian M. schrieb:
> cyblord ---- schrieb:
>> Ist auch kein Fehler, sondern eine Warnung.
>
> das klingt so einfach. Spielt es keine rolle, wie ich main() deklariere?
> Was schlägst du vor?

int

Deshalb sollte auch ans Ende der main() ein return 0.
So macht man das sauber. Auch wenns im Controller keine Rolle spielt.


Und Rückgabetyp weglassen ist nie gut. Wieso fehlt der überhaupt?

cyblord ---- schrieb:
> Und Rückgabetyp weglassen ist nie gut. Wieso fehlt der überhaupt?

kann ich dir auch nicht sagen, dieses Projekt stammt von einer Homepage, 
ich versuche nur zur Übung das zum laufen zu bekommen.

http://extremeelectronics.co.in/avr-projects/avr-project-relay-timer-with-atmega8-avr-mcu/

cyblord ---- schrieb:
> nt
>
> Deshalb sollte auch ans Ende der main() ein return 0.

nur zum Verständnis, da ich den rückgabetyp nicht weiterverarbeite, ist 
es egal wie ich diesen Deklariere?

Sebastian M. schrieb:
> cyblord ---- schrieb:
>> nt
>>
>> Deshalb sollte auch ans Ende der main() ein return 0.
>
> nur zum Verständnis, da ich den rückgabetyp nicht weiterverarbeite, ist
> es egal wie ich diesen Deklariere?

Nach irgendwelchen C Standards (die ich grade nicht genau benennen kann) 
sollte main() einen INT zurückgeben. Ich würde dabei bleiben. Auf einem 
Laufzeitsystem, ist das wichtig, damit ein programm einen errocode 
rückgeben kann. Darum auch return 0, da 0=kein fehler, <>0 = Fehler.
Du hast kein Laufzeitystem auf dem Controller, darum egal. Trotzdem 
würde ich mich bei C daran halten.

cyblord ---- schrieb:
> Nach irgendwelchen C Standards (die ich grade nicht genau benennen kann)
> sollte main() einen INT zurückgeben. Ich würde dabei bleiben. Auf einem
> Laufzeitsystem, ist das wichtig, damit ein programm einen errocode
> rückgeben kann. Darum auch return 0, da 0=kein fehler, <>0 = Fehler.
> Du hast kein Laufzeitystem auf dem Controller, darum egal. Trotzdem
> würde ich mich bei C daran halten.

Ok, danke für den Tipp.

Ich werde jetzt die Schaltung im Keller erneut aufbauen. Mal schauen ob 
es mit den neuen Komponenten klappt.

Sebastian M. schrieb:

> kann ich dir auch nicht sagen, dieses Projekt stammt von einer Homepage,
> ich versuche nur zur Übung das zum laufen zu bekommen.

Konzentrier dich erst mal nur auf das LCD und lass den Timer-Code weg.

Ein einfaches

1

int main()

2

{

3

  InitLCD(0);

4

  LCDWriteString("    Welcome     ");

5

6

  while( 1 )

7

    ;

8

}

reicht, um das LCD in Betrieb zu nehmen und den Funktionsnachweis zu 
erbringen.
Alles andere sind erst mal nur zusätzliche Möglichkeiten, Fehler zu 
machen.
Und so prickelnd gut ist der Code von diesem Timer dann auch wieder 
nicht geschrieben.

So habe die Schaltung erneut zusammengesteckt, leider ohne Funktion. Als 
ich dann das Quarz gegen ein frisch geliefertes ausgetauscht habe, habe 
ich endlich eine Anzeige bekommen.

=> Quarz defekt <=

Nun gibt es aber ein neues Problem, welches dieses mal nicht an der 
Hardware liegen kann.

Nach dem Einschalten ist folgendes zu sehen:
("||||||||||||||||") = schwarze Rechtecke
("                ")

Nach dem Drücken von "Select" erscheint:
("    Welcome     ")
("   Relay Timer  ")

dann passiert nach 4 sek. nichts, erst durch drücken von
"select" oder "up" oder "down" für eine dauer von 3 sek. danach ist 
folgendes Bild:
("Set Time - 00:00")
(" Start     ^    ");

Jedes drücken einer Taste führt wieder zum Bild:
("    Welcome     ")
("   Relay Timer  ")



Karl Heinz Buchegger schrieb:
> Und so prickelnd gut ist der Code von diesem Timer dann auch wieder
> nicht geschrieben.

Leider bin ich aber nicht in der Lage, selbst einen Code zu schreiben. 
Somit bleibt mir nur noch die Möglichkeit, einen zu suchen, und diesen 
dann nach meinen Wünschen anzupassen.

Sebastian M. schrieb:

> Leider bin ich aber nicht in der Lage, selbst einen Code zu schreiben.
> Somit bleibt mir nur noch die Möglichkeit, einen zu suchen, und diesen
> dann nach meinen Wünschen anzupassen.

Ich denke du willst üben?

Dann analysier den Anfang des Codes, sieh nach was passiert (die ganze 
Timer Sache lässt du erst mal weg), und analysiere wie die Sache mit den 
Tasten funktioniert.

Und dann schreibst du den ganzen Teil, der sich um LCD, Tasten und 
erhöhen/erniedrigen der einzelnen Zahlenwerte dreht neu und nach deinen 
Vorstellungen.

Eine bessere Übung gibt es nicht.

Wenn der Teil dann erst mal zufriedenstellend funktioniert, dann können 
wir uns über den Timer unterhalten, bzw. wie man dann die eingestellte 
Zeit tatsächlich ablaufen lässt. Aber erst mal hast du mit LCD, Tasten 
und deren Auswertung genug zu tun.

Sorry. Aber wer etwas selber bauen will, der muss auch selber ran. Das 
war schon immer die Maxime hier im Forum.

Karl Heinz Buchegger schrieb:
> Eine bessere Übung gibt es nicht.

Hast du auch einen Vorschlag wie man dort am besten ran geht?

Sebastian M. schrieb:
> Karl Heinz Buchegger schrieb:
>> Eine bessere Übung gibt es nicht.
>
> Hast du auch einen Vorschlag wie man dort am besten ran geht?

Sicher.
Ignorier erst mal die komplette Timer Sache.

Du hast 2 Variablen, die Stunden und die Minuten.
Und du hast 3 Tasten.
Mit einer Taste wählst du aus, welchen Zahlenwert (Stunden oder Minuten) 
du verändern willst und mit den anderen beiden erhöhst bzw. erniedrigst 
du den Wert. Und dann werden die neuen Werte wieder am LCD angezeigt.

Analysier den Code. Genau diese Funktionalität ist da im Grunde schon 
programmiert, wenn auch etwas ... seltsam und langatmig. D.h. du hast 
schon Code, an dem du dir ansehen kannst, wie man zb den Wert einer 
Variablen ausgibt. Du hast schon Code der, ähm, Tastenabfragen 
realisiert. Nur bitte nimm nicht diese besch.... delay Funktionen des 
Autors, sondern wenigstens die im AVR-GCC bereits vorhandenen.

Fürs erste reicht es ja auch, wenn du nach dem Drücken einer Taste erst 
mal 2 unterschiedliche Texte anzeigst

1

int main()

2

{

3

  lcd_init( 0 );

4

5

  PORTB |= ( 1 << PB0 );

6

7

  while( 1 )

8

  {

9

    if( PINB & ( 1 << PB0 ) )

10

    {

11

      LCDGotoXY( 0, 0 );

12

      LCDWriteString("Oben ");

13

    }

14

    else {

15

      LCDGotoXY( 0, 0 );

16

      LCDWriteString("Unten");

17

    }

18

  }

19

}

Dann gehts eben weiter. Nicht etwas tun solange eine Taste gedrückt ist, 
sondern eine gedrückte Taste nur einmal auswerten. Um beim Original zu 
beliben, zb so

1

int main()

2

{

3

  uint8_t oldPin;

4

  uint8_t newPin;

5

6

  uitn8_t cnt = 0;

7

8

  lcd_init( 0 );

9

10

  PORTB |= ( 1 << PB0 );

11

12

  oldPin = PINB & ( 1 << PB0 );

13

14

  LCDWriteIntXY( 4, 1, cnt, 2 );

15

16

  while( 1 )

17

  {

18

    newPin = PINB & ( 1 << PB0 );

19

20

    if( newPin != oldPin )   // gibts einen Unterschied zu 'vorher'

21

                             // wenn ja, dann ist die Taste entweder

22

    {                        // gedrückt oder losgelassen worden

23

      cnt++;

24

      LCDWriteIntXY( 4, 1, cnt, 2 );

25

    }

26

    oldPin = newPin;

27

28

    _delay_ms( 10 );

29

  }

30

}

(@Alle: Ja ich weiß, die Entprellung ist nicht gut gelöst. Aber bei den 
Voraussetzungen ist alles andere noch zu schwer für ihn.)


Und so geht das eben weiter. Eines führt zum anderen.
Als nächstes brauchst du dann eine zweite Taste, so dass du den Wert 
nicht nur erhöhen, sondern auch verringern kannst.
Klappt das, dann kommt eine 2-te Variable ins Spiel, die verändert wird.
Und du musst irgendwie zwischen den beiden umschalten, so dass du beide 
voneinander unabhängig erhöhen/erniedrigen kannst.
Dazu brauchst du eine weitere (3-te) Taste, die die Umschaltung macht. 
Und du musst deinem Benutzer irgendwie anzeigen, welche der beiden 
Zahlen er gerade verändert.

Aber Grundvoraussetzung ist es nun mal, dass du Tasten auswerten und 
Aktionen daran knüpfen kannst. Perfekt um das zu üben.

Puh, ein ganz schönes Brett. Ich mach mich gleich dran...

Sebastian M. schrieb:
> Puh, ein ganz schönes Brett. Ich mach mich gleich dran...

Tja. So ist das nun mal, wenn man etwas selber bauen will.
Man muss es lernen.
Aber dafür kriegt man dann auch exakt das, was man sich vorstellt :-)

Karl Heinz Buchegger schrieb:
> lcd_init( 0 );

das ist doch aber nicht der Aufruf der funktion in meinem Projekt oder? 
Hier lautet deiser Befehl InitLCD(0).

Wollen wir von vorne rein gleich die bessere LCD Routine nehmen?

Sebastian M. schrieb:
> Karl Heinz Buchegger schrieb:
>> lcd_init( 0 );
>
> das ist doch aber nicht der Aufruf der funktion in meinem Projekt oder?
> Hier lautet deiser Befehl InitLCD(0).

OK. Dann eben so.
Ich hab nicht bei jedem Aufruf der vorgefertigten Funktionen nachgesehen 
(*), ob sich die Funktion wirklich so schreibt. Dazu hast du ja die 
'Vorlage' mit dem Originalcode, in dem man schnell und einfach nachsehen 
kann.

(*) das ist von hier aus nämlich recht mühsam zu bewerkstelligen. Aber 
du hast ja den Code vor dir - sieh nach, wie die Funktion wirklich 
heißt. Aus dem Zusammenhang sollte klar sein, welche Funktion gemeint 
ist.

Sebastian M. schrieb:

> Wollen wir von vorne rein gleich die bessere LCD Routine nehmen?

Wenn die hier jetzt funktionieren, dann nimm sie auch. Die sind momentan 
das kleinste Problem, so wie ich das sehe.

Ehrlich gesagt, verstehe ich jetzt nicht was dass oben für 
Codeausschnitte sind? Der erste ist ja nur ein Test, knopf drücken = 
Anzeige.

Aber sagt

1

PORTB |= ( 1 << PB0 );

nicht aus , dass der Ausgang auf logisch 1 gesetzt werden soll?

Sebastian M. schrieb:
> Ehrlich gesagt, verstehe ich jetzt nicht was dass oben für
> Codeausschnitte sind? Der erste ist ja nur ein Test, knopf drücken =
> Anzeige.
>
> Aber sagt
>

1

> PORTB |= ( 1 << PB0 );

2

>

>
> nicht aus , dass der Ausgang auf logisch 1 gesetzt werden soll?


Nein, nicht nicht in diesem Fall.
Denn der Pin PB0 am Port B wurde ja nicht mittels DDRB auf Ausgang 
programmiert. Daher ist er nach wie vor ein Eingang und daher schaltet 
diese Anweisung den Pullup Widerstand für diesen Pin ein.

AVR-GCC-Tutorial

> Der erste ist ja nur ein Test
Aber offenbar einer, den du in seiner Einfachheit nicht komplett 
verstehst.
Üben!
Solche Dinge, wie die Zusammenhänge der DDRx, PORTx und PINx Register 
dürfen kein Problem sein. Sowas musst du einwandfrei erkennen können, 
wenn du es siehst.
Denn sonst unterhalten wir uns über eine einfache Blinddarmoperation, 
dessen ausführender Arzt Schwierigkeiten beim Aufkleben eines Pflasters 
hat. Keine guten Voraussetzungen.

1

int main()

2

{

3

  uint8_t oldPin; // Variable deklarieren

4

  uint8_t newPin;// Variable deklarieren

5

6

  uitn8_t cnt = 0; //setzte cnt=00

7

8

  lcd_init( 0 ); // LCD initialisieren

9

10

  PORTB |= ( 1 << PB0 ); // Pin0 pull up ein

11

12

  oldPin = PINB & ( 1 << PB0 ); // oldPin ist =1 wenn pb0=1

13

14

  LCDWriteIntXY( 4, 1, cnt, 2 ); // schreibe an LCD zeile 1 spalte 4 

15

                                  //dem wert von cnt, zweistellig

16

17

  while( 1 )

18

  {

19

    newPin = PINB & ( 1 << PB0 );// newPin ist =1 wenn pb0=1

20

21

    if( newPin != oldPin )   // gibts einen Unterschied zu 'vorher'

22

                             // wenn ja, dann ist die Taste entweder

23

    {                        // gedrückt oder losgelassen worden

24

      cnt++;                 //erhöhe cnt um 1

25

      LCDWriteIntXY( 4, 1, cnt, 2 ); // schreibe an LCD zeile 1 spalte 4 

26

                                      //dem wert von cnt, zweistellig

27

28

    }

29

    oldPin = newPin;   // gleichsetzen

30

31

    _delay_ms( 10 );   // 10ms warten

32

  }

33

}

das ist ja alles schön und gut, aber weiterbringen tut mich das doch 
auch nicht. was bringt das? an welcher stelle ist das sinnvoll?

Sagtest du nicht, du willst üben?

Dann übe! Und fang endlich an, nicht nur "malen nach Zahlen" zu 
betreiben, sondern deine Programme selbst zu entwickeln!
Den Grundstein hab ich dir gelegt um daraus das Programm zu entwickeln. 
Die weitere Entwicklung wirst du aber selber machen müssen. Denn letzten 
Endes willst DU ja dieses Projekt stemmen, nicht ich. Ich helfe dir 
nur, wenn du nicht mehr weiterkommst (und glaub mir, es wäre für mich 
wesentlich einfacher das Programm einfach zu schreiben als dich da durch 
die Entwicklung zu begleiten. Die 30 Minuten, die ich für das 
vollständige Programm brauchen würde, sind schon lange um)

Karl Heinz Buchegger schrieb:
> Die 30 Minuten, die ich für das
> vollständige Programm brauchen würde, sind schon lange um

Respekt... ich werde wahrscheinlich 3000 minuten brauchen... :-/, da ich 
das mit den schaltern nicht schnalle.

Guten Morgen,

ich habe mich mal um die taster gekümmert, bitte mal durchschauen.
Habs nochmal verbessert.

1

while(1)

2

    {

3

      // ### Zeit einstellen ###  

4

5

6

7

8

      // Select Key

9

      newPin = PINB & ( 1 << PB2 );    

10

11

      if (newPin != oldPin)

12

      {

13

        selection ++;

14

15

        if (selection ==3)

16

        {

17

          selection=0;

18

        }

19

20

      }

21

22

      // --- Up Key ---

23

      newPin = PINB & ( 1 << PB1 );

24

25

      if (newPin != oldPin)

26

      {

27

28

        if (selection ==1)      // Auswahl ist auf Stunden

29

        {

30

          hr++;

31

          if (hr==100)      // Wenn Stunde =100 setze wieder 0

32

          {

33

            hr=0;

34

          }

35

        }

36

37

        if (selection ==2)      // Auswahl ist auf Minuten

38

        {

39

          min++;

40

          if (min==60)      // Wenn Minute =60 setze wieder 0

41

          {

42

            min=0;

43

          }

44

        }

45

46

47

        if (selection ==0)

48

        {

49

          break;

50

        }

51

      }

52

        // --- --- ---

53

54

55

      // --- Down Key ---

56

      newPin = PINB & ( 1 << PB0 );

57

58

      if (newPin != oldPin)

59

      {

60

61

        if (selection ==1)      // Auswahl ist auf Stunden

62

        {

63

          hr--;

64

          if (hr==-1)      // Wenn Stunde =100 setze wieder 0

65

          {

66

              hr=99;

67

          }

68

        }

69

70

        if (selection ==2)      // Auswahl ist auf Minuten

71

        {

72

          min++;

73

          if (min==-1)      // Wenn Minute =60 setze wieder 0

74

          {

75

            min=59;

76

          }

77

        }

78

79

        if (selection ==0)

80

        {

81

          break;

82

        }

83

        // --- --- ---            

84

      }

85

86

87

    }

So, habe jetzt auch noch, wie ich glaube,die LCD Aktualisierung ergänzt. 
Das siht dann für dem DOWN KEY exemplarisch so aus:

1

// --- Down Key ---

2

      newPin = PINB & ( 1 << PB0 );

3

4

      if (newPin != oldPin)

5

      {

6

7

        if (selection ==1)      // Auswahl ist auf Stunden

8

        {

9

          hr--;

10

          if (hr==-1)      // Wenn Stunde =100 setze wieder 0

11

          {

12

            hr=99;

13

          }

14

        LCDWriteIntXY(11,0,hr,2);

15

        }

16

17

        if (selection ==2)      // Auswahl ist auf Minuten

18

        {

19

          min++;

20

          if (min==-1)      // Wenn Minute =60 setze wieder 0

21

          {

22

            min=59;

23

          }

24

        LCDWriteIntXY(11,0,min,2);

25

        }

26

27

        if (selection ==0)

28

        {

29

          break;

30

        }

31

        // --- --- ---

Jetzt fehlen noch die Curserpositionen und die eigentliche 
Timerfunktion.

Hey,

zur Info, ich habe jetzt zumindest den Ursprünglichen Code zum laufen 
bekommen. In der Schaltung von der Homepage fehlte die Verbindung von 
Pin 22 (AGND) zu GND und 21(AREF) zu +5V.

Wenn ich jetzt eine Zeit einstelle und starte:

2min eingestellt und gestartet:

1

Anzeige:

2

00:02:55

3

...

4

00:02:00

5

00:02:99

6

00:02:98

7

00:02:97

8

...

9

00:02:00

10

00:02:99

11

00:02:98

12

13

etc...

liegt das vielleicht an dem Timer dings?

1

ISR(TIMER1_COMPA_vect)

2

{

3

  clock_millisecond++;

4

  if(clock_millisecond==1000)

5

  {

6

    clock_second--;

7

    clock_millisecond=0;

8

    if(clock_second==-1) //-1

9

    {

10

      clock_minute--;

11

      clock_second=59;

12

13

      if(clock_minute==-1) //-1

14

      {

15

        clock_hour--;

16

        clock_minute=59;

17

      }

18

    }

19

  }

20

}

Hier sitzt der Übeltäter

1

volatile char       clock_second=0;

2

volatile char       clock_minute=0;

3

volatile char       clock_hour=0;

hier ist nicht definiert ob ein char signed oder unsigned sein soll.
So wie die Vergleiche hier sind

1

...

2

    if(clock_second==-1) //-1

3

...

müssen das signed Werte sein.

Also

1

volatile signed char       clock_second=0;

2

volatile signed char       clock_minute=0;

3

volatile signed char       clock_hour=0;

wunder bar, danke euch beiden.

Bin gerade am rumspielen, klappt auch ganz gut.

aber ein problem bekomm ich nicht weg, und zwar ist das der blinkende 
Cursor.

Ich habe folgende Änderungen versucht:

LCD.h

1

...

2

#define LS_BLINK 0B00000001 //geändert auf 0B00000000

3

#define LS_ULINE 0B00000010 //geändert auf 0B00000000

4

...

5

#define LCDClear() LCDCmd(0b00000001)    //geändert auf 0b00000000

6

#define LCDHome() LCDCmd(0b00000010);    //geändert aud 0b00000000

ohne Erfolg.

LCD.c

1

...

2

//Now the LCD is in 4-bit mode

3

4

  LCDCmd(0b00001100|style);    //geändert auf ohne |style

5

  LCDCmd(0b00101000);      //function set 4-bit,2 line 5x7 dot format

6

...

ohne erfolg.

Ich habe sowohl alles einzeln, als auch in Kombination probiert.
Habt ihr noch eine Idee?

PS: Die Anweisungen habe ich von hier
http://www.mikrocontroller.net/articles/AVR-Tutorial:_LCD

Sebastian M. schrieb:
> wunder bar

=wunderbar