AVR Project – Relay Timer with ATmega8 AVR MCU

[Sebastian M. (basti87)]

Guten Morgen liebe Forum Gemeinde,

ich habe mich an den vergangenen Tagen mal mit einem Projekt, welches 
ich im Internet gefunden habe auseinander gesetzt.

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

Dazu habe ich alle nötigen Daten heruntergeladen und im AVR Studio 
eingefügt, sowie die Schaltung auf meinen Steckbrettern 
zusammengesteckt. Siehe Foto.

Leider funktioniert es aber nicht. Ich habe aber darauf geachtet, die 
Fuses umzustellen.

Anschließend habe ich es versucht mit AVR Simulator IDE zu simulieren, 
auch ohne Erfolg. (Aber ohne die Fuses).

Kann mir jemand Tipps geben wie ich es zum laufen bekomme?

Hier der Code aus LCD_RELAY_TIMER.c:

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

A Simple Device Timer project designed using ATmega8
AVR MVU. The Timer is usefull for keeping a device
"ON" for a specific period of time. After the set time
elapse the timer automatically turns the load off.

The Timer uses a standart 16x2 lcd module for user interface
UI. User can set the time using a 3 button keypad.

After that Timer is started. While count down is in 
progress, the time left is displayed on screen.

The program use our LCD driver library more details
of which can be found in Website.

Use avr-gcc + AVR Studio to compile.

Author: Avinash Gupta
E:Mail: [e-mail]
Web: www.eXtremeElectronics.co.in

*** THIS PROJECT IS PROVIDED FOR EDUCATION/HOBBY USE ONLY  ***

*** NO PROTION OF THIS WORK CAN BE USED IN COMMERIAL       ***
*** APPLICATION WITHOUT WRITTEN PERMISSION FROM THE AUTHOR ***

EVERYONE IS FREE TO POST/PUBLISH THIS ARTICLE IN
PRINTED OR ELECTRONIC FORM IN FREE/PAID WEBSITES/MAGAZINES/BOOKS
IF PROPER CREDIT TO ORIGINAL AUTHOR IS MENTIONED WITH LINKS TO
ORIGINAL ARTICLE 




Copyright (C) 2008-2009 eXtreme Electronics, India.

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

#include <avr/io.h>
#include <avr/interrupt.h>

#include "lcd.h"

//Connection of Load
#define LOAD_DDR DDRC
#define LOAD_PORT PORTC
#define LOAD_POS PC0

//Global variable for the clock system
volatile unsigned int   clock_millisecond=0;
volatile char       clock_second=0;
volatile char       clock_minute=0;
volatile char       clock_hour=0;


void Wait(uint8_t n)
{

  uint8_t i,temp;
  temp=n*28;

  for(i=0;i<temp;i++)
    _delay_loop_2(0);
}

void LoadOn()
{
  LOAD_PORT|=(1<<LOAD_POS);
}

void LoadOff()
{
  LOAD_PORT&=(~(1<<LOAD_POS));
}
main()
{

  while(1)
  {
    LOAD_DDR|=(1<<LOAD_POS);

    LoadOff();

    //Enable Pullups on Keypad
    PORTB|=((1<<PB2)|(1<<PB1)|(1<<PB0));

    int8_t hr,min;  //Target Time
    hr=min=0;

    //Initialize the LCD Subsystem
    InitLCD(0);
    //Clear the display
    LCDClear();

    //Set up the timer1 as described in the
    //tutorial
    TCCR1B=(1<<WGM12)|(1<<CS11)|(1<<CS10);
    OCR1A=250;

    //Enable the Output Compare A interrupt
    TIMSK|=(1<<OCIE1A);

    //Enable interrupts globally
    sei();

    LCDClear();
    LCDWriteString("    Welcome     ");
    LCDWriteStringXY(0,1,"   Relay Timer  ");

    Wait(4);

    LCDClear();
    LCDWriteString("Set Time - 00:00");
    LCDWriteStringXY(0,1," Start     ^");

    uint8_t selection=1;
    uint8_t old_pinb=PINB;

    while(1)
    {
      while((PINB & 0b00000111) == (old_pinb & 0b00000111));
    
      //Input received


      if(!(PINB & (1<<PINB2)) && (old_pinb & (1<<PB2)))
      {
        //Selection key Pressed
        selection++;
        if(selection==3)
          selection =0;
      }

      if(!(PINB & (1<<PINB1)) && (old_pinb & (1<<PB1)))
      {
        //Up Key Pressed
        if(selection == 1)
        {
        
          //Hour is selected so increment it
          hr++;

          if(hr == 100)
            hr =0;
        }

        if(selection == 2)
        {

          //Min is selected so increment it
          min++;

          if(min == 60)
            min =0;
        }

        if(selection == 0)
        {
          //Start Selected
          break;
        }


      }

      if(!(PINB & (1<<PINB0)) && (old_pinb & (1<<PB0)))
      {
      //Down Key Pressed
        if(selection == 1)
        {

          //Hour is selected so decrement it
          hr--;

          if(hr == -1)
            hr =99;
        }

        if(selection == 2)
        {

          //Min is selected so decrement it
          min--;

          if(min == -1)
            min =59;
        }

        if(selection == 0)
        {
          //Start Selected
          break;
        }

      }

    
      old_pinb=PINB;

    

      //Update Display
      LCDClear();
      LCDWriteString("Set Time - 00:00");
      LCDWriteStringXY(0,1," Start    ");

      //Hour
      LCDWriteIntXY(11,0,hr,2);

      //Minute
      LCDWriteIntXY(14,0,min,2);

      if(selection == 0)
        LCDWriteStringXY(0,1,">");
    
      if(selection == 1)
        LCDWriteStringXY(11,1,"^");

      if(selection == 2)
        LCDWriteStringXY(14,1,"^");

      _delay_loop_2(0);
      _delay_loop_2(0);
      _delay_loop_2(0);
      _delay_loop_2(0);

      _delay_loop_2(0);
      _delay_loop_2(0);
      _delay_loop_2(0);
      _delay_loop_2(0);
    }

    //Start the Load
    LoadOn();

    //Now start the timer
    clock_hour = hr;
    clock_minute = min;
    clock_second =0;

    LCDClear();
    LCDWriteString("  Power Off In ");

    while(1)
    {
      LCDWriteIntXY(4,1,clock_hour,2);
      LCDWriteString(":");
      LCDWriteIntXY(7,1,clock_minute,2);
      LCDWriteString(":");
      LCDWriteIntXY(10,1,clock_second,2);

      if((clock_hour == 0) && (clock_minute == 0) && (clock_second == 0))
      {
        //Time Out
        LoadOff();

        LCDClear();
        LCDWriteString("Load Turned Off");
      
        while(1)
        {
          LCDWriteStringXY(0,1,"*Press Any Key*");

          Wait(1);

          LCDWriteStringXY(0,1,"                ");

          Wait(1);

          if((~PINB) & 0b00000111)
            break;

        }

        break;

      

      }

      _delay_loop_2(0);
      _delay_loop_2(0);
      _delay_loop_2(0);
      _delay_loop_2(0);
    }
  //Continue again
  }


}

//The output compate interrupt handler
//We set up the timer in such a way that
//this ISR is called exactly at 1ms interval
ISR(TIMER1_COMPA_vect)
{
  clock_millisecond++;
  if(clock_millisecond==1000)
  {
    clock_second--;
    clock_millisecond=0;
    if(clock_second==-1)
    {
      clock_minute--;
      clock_second=59;

      if(clock_minute==-1)
      {
        clock_hour--;
        clock_minute=59;
      }
    }
  }
}

[Karl H. (kbuchegg)]

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.

[Sebastian M. (basti87)]

Hallo,

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

[Karl H. (kbuchegg)]

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

[Sebastian M. (basti87)]

Ja die Ports und Pins stimmen soweit, nur bei

#define LS_BLINK 0B00000001
#define LS_ULINE 0B00000010

bin ich mir unsicher.

[Karl H. (kbuchegg)]

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


sagst du.
Photo?


>

> #define LS_BLINK 0B00000001
> #define LS_ULINE 0B00000010
>

>
> bin ich mir unsicher.

Braucht dich erst mal nicht zu interessieren.

[Karl H. (kbuchegg)]

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.

[Sebastian M. (basti87)]

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

/************************************************
  LCD CONNECTIONS
*************************************************/

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

#define LCD_E D //Enable OR strobe signal
#define LCD_E_POS PD4  //Position of enable in above port

#define LCD_RS D  
#define LCD_RS_POS PD6

#define LCD_RW D
#define LCD_RW_POS PD5


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

#define LS_BLINK 0B00000001
#define LS_ULINE 0B00000010

[Karl H. (kbuchegg)]

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?

[Karl H. (kbuchegg)]

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

[Sebastian M. (basti87)]

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.

[Karl H. (kbuchegg)]

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.

[Sebastian M. (basti87)]

Hier die Bilder...

[Sebastian M. (basti87)]

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. (basti87)]

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...

[Karl H. (kbuchegg)]

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.

[Karl H. (kbuchegg)]

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.

[Sebastian M. (basti87)]

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?

[Sebastian M. (basti87)]

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

[Sebastian M. (basti87)]

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

Warning  1  
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.:

...
void LoadOff()
{
  LOAD_PORT&=(~(1<<LOAD_POS));
}
main()
{

  while(1)
  {
    LOAD_DDR|=(1<<LOAD_POS);

    LoadOff();

    //Enable Pullups on Keypad
    PORTB|=((1<<PB2)|(1<<PB1)|(1<<PB0));

    int8_t hr,min;  //Target Time
    hr=min=0;
...

[Sebastian M. (basti87)]

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".

[Cyblord -. (cyblord)]

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

Ist auch kein Fehler, sondern eine Warnung.

Sollte man troztdem beheben.

[Sebastian M. (basti87)]

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?

[Cyblord -. (cyblord)]

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?

[Sebastian M. (basti87)]

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/

[Sebastian M. (basti87)]

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?

[Cyblord -. (cyblord)]

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.

[Sebastian M. (basti87)]

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.

[Karl H. (kbuchegg)]

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

int main()
{
  InitLCD(0);
  LCDWriteString("    Welcome     ");

  while( 1 )
    ;
}

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.

[Sebastian M. (basti87)]

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.

[Karl H. (kbuchegg)]

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.

[Sebastian M. (basti87)]

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

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

[Karl H. (kbuchegg)]

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

int main()
{
  lcd_init( 0 );

  PORTB |= ( 1 << PB0 );

  while( 1 )
  {
    if( PINB & ( 1 << PB0 ) )
    {
      LCDGotoXY( 0, 0 );
      LCDWriteString("Oben ");
    }
    else {
      LCDGotoXY( 0, 0 );
      LCDWriteString("Unten");
    }
  }
}

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

int main()
{
  uint8_t oldPin;
  uint8_t newPin;

  uitn8_t cnt = 0;

  lcd_init( 0 );

  PORTB |= ( 1 << PB0 );

  oldPin = PINB & ( 1 << PB0 );

  LCDWriteIntXY( 4, 1, cnt, 2 );

  while( 1 )
  {
    newPin = PINB & ( 1 << PB0 );

    if( newPin != oldPin )   // gibts einen Unterschied zu 'vorher'
                             // wenn ja, dann ist die Taste entweder
    {                        // gedrückt oder losgelassen worden
      cnt++;
      LCDWriteIntXY( 4, 1, cnt, 2 );
    }
    oldPin = newPin;

    _delay_ms( 10 );
  }
}

(@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.

[Sebastian M. (basti87)]

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

[Karl H. (kbuchegg)]

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 :-)

[Sebastian M. (basti87)]

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?

[Karl H. (kbuchegg)]

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.

[Karl H. (kbuchegg)]

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.

[Sebastian M. (basti87)]

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

PORTB |= ( 1 << PB0 );

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

[Karl H. (kbuchegg)]

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
>

> PORTB |= ( 1 << PB0 );
>

>
> 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.

[Sebastian M. (basti87)]

int main()
{
  uint8_t oldPin; // Variable deklarieren
  uint8_t newPin;// Variable deklarieren

  uitn8_t cnt = 0; //setzte cnt=00

  lcd_init( 0 ); // LCD initialisieren

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

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

  LCDWriteIntXY( 4, 1, cnt, 2 ); // schreibe an LCD zeile 1 spalte 4 
                                  //dem wert von cnt, zweistellig

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

    if( newPin != oldPin )   // gibts einen Unterschied zu 'vorher'
                             // wenn ja, dann ist die Taste entweder
    {                        // gedrückt oder losgelassen worden
      cnt++;                 //erhöhe cnt um 1
      LCDWriteIntXY( 4, 1, cnt, 2 ); // schreibe an LCD zeile 1 spalte 4 
                                      //dem wert von cnt, zweistellig

    }
    oldPin = newPin;   // gleichsetzen
 
    _delay_ms( 10 );   // 10ms warten
  }
}

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?

[Karl H. (kbuchegg)]

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)

[Sebastian M. (basti87)]

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.

[Sebastian M. (basti87)]

Guten Morgen,

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

while(1)
    {
      // ### Zeit einstellen ###  

      
      
      
      // Select Key
      newPin = PINB & ( 1 << PB2 );    
      
      if (newPin != oldPin)
      {
        selection ++;
        
        if (selection ==3)
        {
          selection=0;
        }
          
      }

      // --- Up Key ---
      newPin = PINB & ( 1 << PB1 );
      
      if (newPin != oldPin)
      {
        
        if (selection ==1)      // Auswahl ist auf Stunden
        {
          hr++;
          if (hr==100)      // Wenn Stunde =100 setze wieder 0
          {
            hr=0;
          }
        }
        
        if (selection ==2)      // Auswahl ist auf Minuten
        {
          min++;
          if (min==60)      // Wenn Minute =60 setze wieder 0
          {
            min=0;
          }
        }
      
        
        if (selection ==0)
        {
          break;
        }
      }
        // --- --- ---
        

      // --- Down Key ---
      newPin = PINB & ( 1 << PB0 );
        
      if (newPin != oldPin)
      {
          
        if (selection ==1)      // Auswahl ist auf Stunden
        {
          hr--;
          if (hr==-1)      // Wenn Stunde =100 setze wieder 0
          {
              hr=99;
          }
        }
          
        if (selection ==2)      // Auswahl ist auf Minuten
        {
          min++;
          if (min==-1)      // Wenn Minute =60 setze wieder 0
          {
            min=59;
          }
        }
          
        if (selection ==0)
        {
          break;
        }
        // --- --- ---            
      }
        
        
    }

[Sebastian M. (basti87)]

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

// --- Down Key ---
      newPin = PINB & ( 1 << PB0 );
        
      if (newPin != oldPin)
      {
          
        if (selection ==1)      // Auswahl ist auf Stunden
        {
          hr--;
          if (hr==-1)      // Wenn Stunde =100 setze wieder 0
          {
            hr=99;
          }
        LCDWriteIntXY(11,0,hr,2);
        }
          
        if (selection ==2)      // Auswahl ist auf Minuten
        {
          min++;
          if (min==-1)      // Wenn Minute =60 setze wieder 0
          {
            min=59;
          }
        LCDWriteIntXY(11,0,min,2);
        }
          
        if (selection ==0)
        {
          break;
        }
        // --- --- ---

Jetzt fehlen noch die Curserpositionen und die eigentliche 
Timerfunktion.

[Sebastian M. (basti87)]

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:

Anzeige:
00:02:55
...
00:02:00
00:02:99
00:02:98
00:02:97
...
00:02:00
00:02:99
00:02:98

etc...

liegt das vielleicht an dem Timer dings?

ISR(TIMER1_COMPA_vect)
{
  clock_millisecond++;
  if(clock_millisecond==1000)
  {
    clock_second--;
    clock_millisecond=0;
    if(clock_second==-1) //-1
    {
      clock_minute--;
      clock_second=59;

      if(clock_minute==-1) //-1
      {
        clock_hour--;
        clock_minute=59;
      }
    }
  }
}

[Karl H. (kbuchegg)]

Hier sitzt der Übeltäter

volatile char       clock_second=0;
volatile char       clock_minute=0;
volatile char       clock_hour=0;

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

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

müssen das signed Werte sein.

Also

volatile signed char       clock_second=0;
volatile signed char       clock_minute=0;
volatile signed char       clock_hour=0;

[Ich (Gast)]

Wahrscheinlich es liegt an die Timer Dinge, es ist die Timer
Funktionalität die Probleme gibt.

Es scheint dass das Problem ist das deine Sekunden Counter in eine
Variable ohne Vorzeichen gespeichert ist.

Also, im General jede Variable ist nur ein Sammlung von Bits die
irgendwo in den Arbeitsspeicher des Mikrocontroller liegen. Wie die Bits
interpretiert werden stellt man ein durch den Typ Deklaration.


Zum Beispiel:

char a = 0;  // 8 bit variable (mit oder ohne Vorzeichen abhängig von Kompilator/Plattform) und Anfangswert = 0 
unsigned char b = 0; // 8 bit variable ohne Vorzeichen und Anfangswert = 0 
signed char c = 0; // 8 bit variable mit Vorzeichen und Anfangswert = 0

was passiert wenn man eins subtrahiert:

a--;  // a hat jetzt ein Wert von -1 oder 255 (hängt von ob der Kompilator char als ein Typ mit oder ohne Vorzeichen nimnt  )  
b--; // b hat jetzt ein Wert von 255 ( Typ ohne Vorzeichen, 0 ist die Minimalwert)  
c--; // a hat jetzt ein Wert von -1 ( Typ mit Vorzeichen, -127 ist die Minimalvert. Analogisch zum der Fall mit Variable b, wenn b ein wert von -127 hätte und eins davon subtrahiert wird, dann würde c ein Wert von +127

Wieso sieht man 00:02:99 statt 00:02:255? Ich nehme an, dass wenn deine
Anzeige Funktion ein Wert mit Mehr Stellen als erwarten kriegt dann
Zeigt die die Maximale Wert die man mit die gewünschte Stellenmenge
anzeigen kann. Ich bin aber nicht sicher, ich hab fast kein Ehrfahrung
mit avr gcc und überall kein mit diese LCD Code.

Wenn du Englisch verstehen kannst (es ist sowieso extrem schwer
Programmieren zu Lernen ohne Englisch zu verstehen zu können) lies mal
den Artikel hier: https://en.wikipedia.org/wiki/C_data_types

Bisschen offtopic:

Ich weiss dass das ganze Ding an Anfang sehr verwirrend aussehen kann.
Man muss gleichzeitig auf Software und Hardware aufpassen: Dinge wie:
versorgt den Spannungsquelle eine gleiche Spannung? Auch under Last?
Sind die Kondensatoren für den Quarz die Richtige wert? Hab ich die
richtige Fuses am Kontroller programiert? Benutzt den Projektcode
irgendwelche nicht Standarte Erweiterungen und werden die von den
Kompilator die ich benutze Unterstützt? und viele viele mehr ähnliche
Fragen...

Wenn aber in die Ende man sein Projekt zu eine korrekt funktionierende
Zustand bringt alle diese Schwierigkeiten blenden in Vergangenheit aus.
Mindestens bis man genug zum Spielen mit seine neue Gadget hatte und ein
neues und mehr kompliziertes Projekt annimmt.

[Sebastian M. (basti87)]

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

...
#define LS_BLINK 0B00000001 //geändert auf 0B00000000
#define LS_ULINE 0B00000010 //geändert auf 0B00000000
...
#define LCDClear() LCDCmd(0b00000001)    //geändert auf 0b00000000
#define LCDHome() LCDCmd(0b00000010);    //geändert aud 0b00000000

ohne Erfolg.

LCD.c

...
//Now the LCD is in 4-bit mode

  LCDCmd(0b00001100|style);    //geändert auf ohne |style
  LCDCmd(0b00101000);      //function set 4-bit,2 line 5x7 dot format
...

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. (basti87)]

Sebastian M. schrieb:
> wunder bar

=wunderbar