Keine Reaktion auf Eingabe

[Michael W. (michael_w71)]

Servus,

hatte vor ein kleines Projekt zum Lernen zu bauen. Das zieht sich 
inzwischen schon seit ein paar Wochen, dem ein oder anderen dürfte die 
Schaltung bekannt vorkommen.

Da die Schaltung mit dieser Community entwickelt wurde, nehme ich 
einfach an, dass sie soweit stimmen müsste (was nicht im Bild ist, ist 
auch nicht angeschlossen). Es muss also an dem Programm liegen; wenn ich 
die Batterie anschließe läuft die Uhrzeit normal ab und die Namen werden 
richtig angezeigt. Wenn ich einen der Schalter drücke, passiert aber 
nichts (die Uhr läuft aber weiter), es wird nicht einmal das '+' 
angezeigt, was ich ich zum Test (Zeile 141) reinschreibe, wenn irgendein 
Input existiert.

LCD wird angesteuert mit der Fleury-Lib (funktioniert auch, sonst wären 
die Ausgaben ja nicht da)

Vielen Dank im Voraus für Hilfe.

Grüße
Michael

[Karl H. (kbuchegg)]

Michael W. schrieb:
> Servus,
>
> hatte vor ein kleines Projekt zum Lernen zu bauen. Das zieht sich
> inzwischen schon seit ein paar Wochen, dem ein oder anderen dürfte die
> Schaltung bekannt vorkommen.
>
> Da die Schaltung mit dieser Community entwickelt wurde, nehme ich
> einfach an, dass sie soweit stimmen müsste (was nicht im Bild ist, ist
> auch nicht angeschlossen).

Bitte zeig ALLES.

Da du den ADC Port benutzt: Hast du an AVcc die Spannungsversorgung 
angeschlossen?

[Karl H. (kbuchegg)]

ZU deinem Programm.

Gerade WEIL du Anfänger bist: schreibe dein Programm so einfach wie 
möglich!

Dein Code ist einfach nur grauenhaft zu lesen.

[Karl H. (kbuchegg)]

  while(1){
    // Knopfbehandlung
    if(pre_input != PORTD){

Hä?
Wieso PORTD?
Wie und wo und vor allen Dingen warum kommt da jetzt der PORTD ins 
Spiel?

[Karl H. (kbuchegg)]

Die Uhrenweiterschaltung sollte als ganzes in die ISR

Dafür solltest du dir als Ausgleich mal ein paar LCD Ausgabefunktionen 
machen, damit du die Ausgaben in der Hauptschleife vernünftig erledigen 
kannst, ohne gleich mal eine halbe Bildschirmseite Code in der 
Hauptschleife dafür zu brauchen.

[Karl H. (kbuchegg)]

> #define BUTTON_PORT (&PORTC)
> #define BUTTON_DDR (&DDRC)

Ist es an irgendeiner STelle wichtig, dass du die Adresse benutzen 
kannst? Wenn nein: dann lass es. Das verschafft dir nur zusätzlichen 
Aufwand bei der Benutzung von BUTTON_PORT. Aufwand für nichts.

[Karl H. (kbuchegg)]

Da du sowieso einen Timer im Bereich 1 Millisekunde am laufen hast, 
würde es sich anbieten da die PeDa Tasten-Entprellung und Auswertung mit 
einzubauen.

[Karl H. (kbuchegg)]

>       if(((*BUTTON_PORT)&BUTTON_MASK_ADJUST)


Eingaben von aussen, zb Tasten, werden abgefragt, indem man das PIN (!) 
Register ausliest und nicht das PORT Register!


An dieser Stelle hör ich auf. das hat keinen Sinn.
Lies das Tutorial und mach ein paar Vorübungen.
Den ganzen Code müsste man nehmen, wegschmeissen und nochmal von vorne 
anfangen. Da ist kaum Struktur drinnen, der ganze Code ist verworren und 
unübersichtlich.

[Karl H. (kbuchegg)]

>     // U-Umlaut senden
>     lcd_command(0x48); // CG-RAM 1-0

Kann denn dein LCD keine Umlaute anzeigen?
Mag ich kaum glauben.
Gut im regulären ASCII Datensatz sind sie nicht drinnen, aber in der 
Erweiterung der Zeichen mit gesetztem Bit 7 sind neben einem ganzen 
Haufen anderer Zeichen sicher auch Umlaute mit drinn, die vom LCD 
angezeigt werden ohne dass man dafür die benutzerdefinierten Zeichen 
angreifen muss.

[Karl H. (kbuchegg)]

Probier mal dieses Programm

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

#define KEY_DDR   DDRC
#define KEY_PORT  PORTC
#define KEY_PIN   PINC

#define KEY_ADJUST 1
#define KEY_DISP   0

int main()
{
  uint8_t keysPrev;
  uint8_t keysNow;

  lcd_init(LCD_DISP_ON_CURSOR);

  KEY_DDR &= ~(( 1 << KEY_ADJUST ) | ( 1 << KEY_DISP ));
  KEY_PORT |= ( 1 << KEY_ADJUST ) | ( 1 << KEY_DISP );

  keysPrev = ~KEY_PIN;   // das bewirkt, dass beim ersten Vergleich
                         // mit der aktuellen Tastenstellung auf jeden
                         // Fall ein Unterschied festgestellt wird und damit
                         // beim Programmstart der erste Update des LCD
                         // erzwungen wird.

  while(1) {

    keysNow = KEY_PIN;

    if( keysNow != keysPrev ) {
      keysNow = keysPrev;

      lcd_gotoxy( 0, 0 );

      if( keysNow & ( 1 << KEY_ADJUST ) )
        lcd_puts( "Adjust gedrueckt" );

      else if( keysNow & ( 1 << KEY_DISP ) )
        lcd_puts( "Disp   gedrueckt" );

      else
        lcd_puts( "nichts gedrueckt" );
    }
  }
}

Bei Tastendruck muss sich was auf deinem LCD tun.

[Karl H. (kbuchegg)]

Edit: Hab noch ein paar Änderungen am Programm gemacht. Eine davon ist 
eine wichtige. Also wenn du den Code testest, dann übernimm ihn jetzt 
nochmal.

[Michael W. (michael_w71)]

OK, ich schreib mal um, ihr habt Recht: der Code ist echt 
unübersichtlich. Wenn ich fertig bin, probiere ich und schreib dann 
nochmal.

[Karl H. (kbuchegg)]

Michael W. schrieb:
> OK, ich schreib mal um, ihr habt Recht: der Code ist echt
> unübersichtlich. Wenn ich fertig bin, probiere ich und schreib dann
> nochmal.

Ne. Du sollst zuerst den Code probieren, damit man hier mal weiß ob 
deine Verschaltung grundsätzlich funktioniert.
Denn ich bau die nächsten Beispiele nicht auf ungetester Hardware auf. 
Erst mal will ich wissen, ob das da funktioniert, ehe ich dir Programme 
für einen µC gebe, den ich nicht vor mir liegen habe und den ich blind 
programmieren muss ohne eine Chance zu haben, die Programme zu testen, 
ehe du sie zu Gesicht bekommst.

[Michael W. (michael_w71)]

Habe mal deinen Code geflasht: es scheint doch an der Schaltung zu 
liegen. Mir ist aufgefallen, dass etwa 1,8V permanent am Anschluss 
anliegen, kann das an dem Poti in Kombination mit dem/den 
22K-Widerständen liegen?

(Ausgabe ist übrigens "Adjust gedrueckt")
Habe mal dein Programm modifiziert und einen Fall eingeführt, wenn beide 
Taster gedrückt sind: Beide sind 1.

Was tun?

[Karl H. (kbuchegg)]

Oops.
Reiss die Schalter runter. Die sind schlecht verdrahtet.
Du hängst sie so an


    µC-Pin  -------------+
                         |
                       \
                        \
                         |
                         |
                    -----+---- GND

Also einfach nur den Schalter dazu benutzen, den µC Pin bei Tastendruck 
mit GND zu verbinden. Nicht mehr. Den Rest macht die Software.

Und dann änderst du noch

 .....

     if( !( keysNow & ( 1 << KEY_ADJUST ) ) )
        lcd_puts( "Adjust gedrueckt" );

     else if( !( keysNow & ( 1 << KEY_DISP ) ) )
....

[Michael W. (michael_w71)]

So,
habe es gerade umgelötet: Dein Programm funkioniert fast, es zeigt das 
richtige an, wenn ich strom einschalte. Danach tut sich nichts mehr, 
wenn ich einen oder Beide Knöpfe drücke oder loslasse.

[Karl H. (kbuchegg)]

Arrgh.
Mein Fehler

Das ist falsch

    if( keysNow != keysPrev ) {
      keysNow = keysPrev;

so ist es richtig

    if( keysNow != keysPrev ) {
      keysPrev = keysNow;

Im ganzen Satz :-)

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

#define KEY_DDR   DDRC
#define KEY_PORT  PORTC
#define KEY_PIN   PINC

#define KEY_ADJUST 1
#define KEY_DISP   0

int main()
{
  uint8_t keysPrev;
  uint8_t keysNow;

  lcd_init(LCD_DISP_ON_CURSOR);

  KEY_DDR &= ~(( 1 << KEY_ADJUST ) | ( 1 << KEY_DISP ));
  KEY_PORT |= ( 1 << KEY_ADJUST ) | ( 1 << KEY_DISP );

  keysPrev = ~KEY_PIN;   // das bewirkt, dass beim ersten Vergleich
                         // mit der aktuellen Tastenstellung auf jeden
                         // Fall ein Unterschied festgestellt wird und damit
                         // beim Programmstart der erste Update des LCD
                         // erzwungen wird.

  while(1) {

    keysNow = KEY_PIN;

    if( keysNow != keysPrev ) {
      keysPrev = keysNow;

      lcd_gotoxy( 0, 0 );

     if( !( keysNow & ( 1 << KEY_ADJUST ) ) )
        lcd_puts( "Adjust gedrueckt" );

     else if( !( keysNow & ( 1 << KEY_DISP ) ) )
        lcd_puts( "Disp   gedrueckt" );

      else
        lcd_puts( "nichts gedrueckt" );
    }
  }
}

(Es gibt eben nichts was so einfach wäre, dass man es doch verbocken 
kann)

[Michael W. (michael_w71)]

das ärgert mich auch, hätte ich eigentlich auch sehen müssen. Vielen 
Dank trotzdem.

Frage noch: Warum funktioniert die Schaltung auf dem Pollin-Board; bzw. 
Warum kann ich dort über PORTD (angeschlossen an den Schaltern auf dem 
Pollin-Board) nicht-invertiert lesen?

Grüße
Michael

[Karl H. (kbuchegg)]

Michael W. schrieb:

> Frage noch: Warum funktioniert die Schaltung auf dem Pollin-Board; bzw.
> Warum kann ich dort über PORTD (angeschlossen an den Schaltern auf dem
> Pollin-Board) nicht-invertiert lesen?

Weil auf dem Pollin-Board die Schalter anders rum angeschlossen sind.
Das hat aber Nachteile. Unter anderem den, dass du nicht die eingebauten 
Pullup-Widerstände des µC benutzen kannst.

Wozu externe Pull-Down Widerstände verbauen, wenn der µC Pullup mithat?

(Zudem hat das Pollin Board noch ein anderes Problem, was letzten Endes 
dazu führt, dass man die Taster sowieso auf die hier benutzte 
Standard-Konfiguration umbaut)

[Karl H. (kbuchegg)]

OK. Jetzt funktionieren erst mal die Taster.

Nächster Punkt. Die Uhr einbauen.

[Karl H. (kbuchegg)]

Probier mal.
Bei Druck auf den DISP Taster müsste die Anzeige von Uhrzeit auf 
Wochentag/Woche umwechseln

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

#ifndef TRUE
#define TRUE 1
#define FALSE 0
#endif


#define KEY_DDR   DDRC
#define KEY_PORT  PORTC
#define KEY_PIN   PINC

#define KEY_ADJUST 1
#define KEY_DISP   0

struct time
{
  uint8_t week_;
  uint8_t day_;
  uint8_t hour_;
  uint8_t min_;
  uint8_t sec_;
};

volatile uint8_t timeTick;      // wird nach jeder Sekunde TRUE
volatile struct time actTime;   // die aktuelle Zeit


char* dayString[] = { "Mo", "Di", "Mi", "Do", "Fr", "Sa", "So" };

int main()
{
  uint8_t keysPrev;
  uint8_t keysNow;
  uint8_t dispMode = 0;

  lcd_init(LCD_DISP_ON_CURSOR);

  //
  // Timer 1
  //   CTC Modus
  //   Prescaler + OCR1A   -> 1 ms
  OCR1A = (1000*(CPU_FREQ/1000000))-1; // Timer löst alle 1000 Counts aus
  TCCR1A = 0x00; // CTC-Modus
  TCCR1B = (1<<WGM12)|(1<<CS10); // Prescaler 1 -> (1.000.000/1)/1000 = 1000 mal pro Sekunde
  TIMSK = (1<<OCIE1A);

  // Tasten 
  KEY_DDR &= ~(( 1 << KEY_ADJUST ) | ( 1 << KEY_DISP ));
  KEY_PORT |= ( 1 << KEY_ADJUST ) | ( 1 << KEY_DISP );

  keysPrev = ~KEY_PIN;   // das bewirkt, dass beim ersten Vergleich
                         // mit der aktuellen Tastenstellung auf jeden
                         // Fall ein Unterschied festgestellt wird und damit
                         // beim Programmstart der erste Update des LCD
                         // erzwungen wird.

  while(1) {

    keysNow = KEY_PIN;

    if( keysNow != keysPrev ) {
      keysPrev = keysNow;

      if( !( keysNow & ( 1 << KEY_DISP ) ) )
        mode = 1;
      else
        mode = 0;
    }
    
    if( mode == 0 )
      lcd_time( 0, 0, &actTime );
    else
      lcd_date( 0, 0, &actTime );
  }
}

void lcd_time( uint8_t x, uint8_t y, struct time* tim )
{
  char buf[20];
  
  sprintf( buf, "%2d:%02d:%02d", (int)tim->hour_, (int)tim->min_, (int)tim->sec_ );
  lcd_gotoxy( x, y );
  lcd_puts( buf );
}

void lcd_date( uint8_t x, uint8_t y, struct time* tim )
{
  char buf[20];
  
  sprintf( buf, "%02d - %s  ", (int)tim->week_, dayString[tim->day_] );
  lcd_gotoxy( x, y );
  lcd_puts( buf );
}

ISR(TIMER1_COMPA_vect)
{
  static uint16_t milliSec;
  
  milliSec++;
  if( milliSec == 1000 ) {  
    milliSec = 0;
    timeTick = TRUE;

    actTime.sec_++;
    if( actTime.sec_ == 60 ) {
      actTime.sec_ = 0;
      
      actTime.min_++;
      if( actTime.min_ == 60 ) {
        actTime.min_ = 0;
        
        actTime.hour_++;
        if( actTime.hour_ == 24 ) {
          actTime.hour_ = 0;
          
          actTime.day_++;
          if( actTime.day_ == 7 ) {
            actTime.day_ = 0;
            
            actTime.week_++;
          }
        }
      }
    }
  }
}

[Michael W. (michael_w71)]

Danke, aber kurz mal bitte Stopp. Ich möchte versuchen das noch einmal 
selbst zu programmieren. Ich werde aber bestimmt was aus deinem Code 
übernehmen.

Warum legst du für die einzelnen Elemente der Zeit extra eine Struktur 
anlegen, wenn das ganze Projekt im Prinzip eine erweiterte Uhr ist?

[Michael W. (michael_w71)]

So, hab ich jetzt noch einmal Komplett neu geschrieben. Habe übrigens im 
Datenblatt des Displays nachgesehen: es gibt kein "ü" in der 
Code-Tabelle. Es ist angeblich aber trotzdem ein HD47780.

Mal der jetzige Code im Anhang.

XTAL ist in der Fleury-Lib definiert und das gleiche wie CPU_FREQ.

Habe noch folgende Beobachtung in der letzten Stunde gemacht: die Uhr 
läuft zu langsam. Ich werd mich mal dran machen und einen 
Korrekturfaktor einbauen (also OCR1A veränderbar machen).

Vielleicht wird der dann im EEPROM gespeichert, würde zumindest nicht 
schaden. (Der Wert wird im Idealfall ja nur ein paar mal geschrieben)

Grüße
Michael

[Karl H. (kbuchegg)]

Michael W. schrieb:

> Warum legst du für die einzelnen Elemente der Zeit extra eine Struktur
> anlegen, wenn das ganze Projekt im Prinzip eine erweiterte Uhr ist?

Weil es erstens nichts kostet
ich zweitens dann die Uhrzeit wie im Beispiel in einfacher Art und Weise 
an Funktionen übergeben kann
drittens du selber sagst, dass es eine erweiterte Uhr ist, da also noch 
Dinge dazu kommen
von denen ich viertens denke, dass da irgendwann mal zusätzliche 
einstellbare Zeiten dazu kommen werden

Aber der Hauptpunkt ist: weil es mich nichts kostet und ich so die Dinge 
beisammen halte, die zusammen gehören. Eine Uhrzeit besteht nun mal aus 
diesen Dingen.

[Karl H. (kbuchegg)]

Michael W. schrieb:
> So, hab ich jetzt noch einmal Komplett neu geschrieben. Habe übrigens im
> Datenblatt des Displays nachgesehen: es gibt kein "ü" in der
> Code-Tabelle. Es ist angeblich aber trotzdem ein HD47780.

Interessant.
Es gibt da verschiedene Versionen.
Normalerweise hat man UMlaute. Allerdings nur in Form von 
Kleinbuchstaben. Für Grossbuchstaben mit Umlaut reicht der Platz nicht.

http://de.wikipedia.org/w/index.php?title=Datei:Charset.gif&filetimestamp=20090712201008


> Habe noch folgende Beobachtung in der letzten Stunde gemacht: die Uhr
> läuft zu langsam.

:-)
Die 1Mhz sind bei dir vom internen RC-Oszillator?
Dann allerdings darfst du dich nicht wundern. Der ist so genau wie der 
Wasserstand der Donau.
FÜr eine Uhr brauchst du einen Quarz. Ohne wird das nichts.
Und diesen Quarz musst du auch ausmessen. Denn selbst wenn am Quarz 4MHz 
druafsteht, bedeutet das nicht, das er exakt 4000000 Schwingungen in der 
Sekunde macht.

AVR - Die genaue Sekunde / RTC

[Georg G. (df2au)]

Karl Heinz Buchegger schrieb:
>> Datenblatt des Displays nachgesehen: es gibt kein "ü" in der
>> Code-Tabelle. Es ist angeblich aber trotzdem ein HD47780.
>
> Interessant.
> Es gibt da verschiedene Versionen.
> Normalerweise hat man UMlaute. Allerdings nur in Form von
> Kleinbuchstaben.

Probier mal das Zeichen 0xf5. Das ist in allen mir vorliegenden 
Datenblättern das kleine "ü".

[Michael W. (michael_w71)]

Georg G. schrieb:
> Karl Heinz Buchegger schrieb:
>>> Datenblatt des Displays nachgesehen: es gibt kein "ü" in der
>>> Code-Tabelle. Es ist angeblich aber trotzdem ein HD47780.
>>
>> Interessant.
>> Es gibt da verschiedene Versionen.
>> Normalerweise hat man UMlaute. Allerdings nur in Form von
>> Kleinbuchstaben.
>
> Probier mal das Zeichen 0xf5. Das ist in allen mir vorliegenden
> Datenblättern das kleine "ü".

Hab irgendwann mal probiert (nicht jetzt), da kommt irgenwas anderes 
raus, ich weiß jetzt aber nicht mehr was. Habe auch nach anderen 
HD47780-Datenblättern geschaut, hat aber alles nichts geracht. Naja, 
dann hab ich halt das "ü" selbst rein geschrieben.

Ja, ist mir klar, dass das auf gar keinen Fall annähernd genau ist. Da 
die Schaltung auf der Platine aber schon mehr oder weniger den gesamten 
Platz einnimmt und ich dann isolierte Drahtbrücken bräuchte (wäre dann 
3. Ebene), möchte ich lieber nur den Korrekturfaktor verwenden. Aber 
fürs nächste Projekt (was auch immer das sein wird) weiß ich ja jetzt 
Bescheid.

Das mit dem Korrektur-Faktor dürfte in diesem speziellen Fall auch 
funktionieren, da Quarze ja bekanntlich von der Temperatur abhängig sind 
- und die ist im Einsatzraum ziemlich konstant 21°C. Und wenn die Uhr 
mal ein Paar Minuten falsch geht, ist das kein Problem, ich kann sie ja 
nachstellen.

Beim nächsten Projekt werde ich auch einen Programmier-Anschluss (ISP) 
anlöten; Erstens muss man beim raus-hebeln sehr vorsichtig sein und 
zweitens geht es deutlich schneller (mit ISP-Anschluss), wenn man 
einfach was probieren möchte. Kann ich den direkt an die Ports hängen 
(siehe Pollin-Board) oder muss ich etwas zwischendran hängen (weil das 
Pollin-Board in dem Punkt ebenfalls nicht ganz stimmt)?

Grüße
Michael

[Karl H. (kbuchegg)]

Michael W. schrieb:

> Hab irgendwann mal probiert (nicht jetzt), da kommt irgenwas anderes
> raus, ich weiß jetzt aber nicht mehr was. Habe auch nach anderen
> HD47780-Datenblättern geschaut, hat aber alles nichts geracht. Naja,
> dann hab ich halt das "ü" selbst rein geschrieben.

Im Zweifelsfall hilft immer das Experiment weiter.
Es ist relativ trivial, sich ein Programm zu schreiben, welches 
nacheinander zb jeweils 16 Zeichen mit aufsteigenden Codes auf dem LCD 
ausgibt.
Im AVR-Tutorial ist zb ein derartiges Programm in einer etwas anderen 
Form, aber mit der gleichen Funktionalität in Assembler geschrieben 
(allerdings im Zusammenhang mit der Erklärung, wie man fachgerecht 
Tastendrücke erkennt).

> Ja, ist mir klar, dass das auf gar keinen Fall annähernd genau ist. Da
> die Schaltung auf der Platine aber schon mehr oder weniger den gesamten
> Platz einnimmt und ich dann isolierte Drahtbrücken bräuchte

Welche Drahtbrücken.
Du scheinst da etwas misszuverstehen: Der Quarz muss sowieso dicht an 
den µC herangesetzt werden. Und das du diesen Platz nicht mehr hast, 
kann ich ehrlich gesagt kaum glauben.

> 3. Ebene), möchte ich lieber nur den Korrekturfaktor verwenden.

Tja.
Dann wird deine Uhr immer nur eine nette (und durchaus lehrreiche) 
Spielerei bleiben, die für praktische Zwecke nicht zu gebrauchen ist.
Solange du damit leben kannst, ist das ok.


> Das mit dem Korrektur-Faktor dürfte in diesem speziellen Fall auch
> funktionieren, da Quarze ja bekanntlich von der Temperatur abhängig sind

Langsam. Quarze sind temperaturabhängig. Ja.
Aber die Frage ist: um welche Größenordnungen geht es!

Dein RC-Schwingkreis hat Abweichungen, die sind um einige Größenordnung 
größer als das, was dir selbst der grindigste Quarz auf Anhieb bietet.

[Michael W. (michael_w71)]

OK. Die Tastenentprellung von Herrn Dannegger ist relativ interressant 
gelöst.

> Tja.
> Dann wird deine Uhr immer nur eine nette (und durchaus lehrreiche)
> Spielerei bleiben, die für praktische Zwecke nicht zu gebrauchen ist.
> Solange du damit leben kannst, ist das ok.

Mehr sollte es auch nicht sein (der Putzplan könnte für uns drei hier 
auch locker auf ner A4-Seite stehen, was er noch tut).
Leider habe ich auf der Lochrasterplatine zwischen Controller (Auf der 
Seite wo XTAL1 und 2 liegen) und Ende der Platine 6 Löcher im 3-Punkt. 
Davon sind aber zwei nebeneinander nicht benutzbar (liegen schon 
Brücken) und Masse ist relativ weit entfernt und nicht eingeplant war. 
Das hängt auch damit zusammen, dass ich die Platine bestimmt fünf mal 
umgelötet habe.
Wie gesagt: fürs nächste mal weiß ich Bescheid. Hätte aber auch nicht 
gedacht, dass der interne Oszillator/Taktgeber so schwankt.

Vielen Dank für deine Hilfe!

Grüße
Michael