Programm bleibt stehen

[Pierre G. (bqube)]

Guten Abend liebe Gemeinde,

ich habe ein kleines Uhren Programm geschrieben was immer stehen bleibt 
sobald auf der Stunden und Minuten Portseite also PORTD und PORTB 
jeweils 4 PORTS gleichzeitig Low sind.

Es ist auch egal welche PORTS es sind.

Stunde       Minute

E  Z         E  Z

0  0         0  0
0  x         0  x
0  x         0  x
x  x         x  x


0  0         0  0
0  x         x  0
0  x         x  0
x  x         x  x


0  0         0  0
0  x         x  0
0  x         x  x
x  x         x  0

Könnte einer von euch bitte mal einen Blick auf mein Programm werfen 
danke schonmal.

/*
 * Wecker_01.c
 *
 * Created: 15.06.2014 15:37:42
 *  Author: Werkstadt
 */ 

#include <stdio.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#ifndef F_CPU
#warning "F_CPU war noch nicht definiert, wird nun mit 1000000 definiert"
#define F_CPU 1000000UL
#endif
#include <util/delay.h>

#define STR 4 // Schalter RUN
#define STP 5 // Schalter PROG
#define STA 00 // Schalter ALARM
#define TTS 00 // Taster SNOOZE
#define TTMIN 0 // Taster +MIN
#define TTST 1 // Taster +ST

int    sek=0;
int    min=0;
int    st=0;
int    min_s=0;
int    st_s=0;
int    p1=0;
int    p2=0;
int    p3=0;
int    t01=0;
int    RUN=0;


int tast=0;
int min_8=0;
int st_8=0;
int foo=0;
int foo2=0;




void show (void)
{
  min_s=min;
  while(foo==0)
  {
    if(min_s > 9)
    {
      min_8=(min_8+0x10);
      min_s=(min_s-10);
    }
    else
    {
      min_8=(min_8+min_s);
      ++foo;
    }
  }
  foo=0;
  min_s=0;
  
  st_s=st;
  while(foo==0)
  {
    if(st_s > 9)
    {
      st_8=(st_8+0x10);
      st_s=(st_s-10);
    }
    else
    {
      st_8=(st_8+st_s);
      ++foo;
    }
  }
  foo=0;
  st_s=0;
  
  p1=min_8;
  min_8=~p1; // Wert Minuten invertieren
  p2=st_8; 
  st_8=~p2; // Wert Stunden invertieren
  
  PORTD=min_8;
  PORTB=st_8;
  
  min_8=0;
  st_8=0;
}

void prog (void)
{
  while(foo==0)
  {
  if(!(PINC & (1<<0))) // Abfrage Taster +Minute
  {
    ++min;
    if(min>59)
    {
      min=0;
    }
    _delay_ms(2);
    while(foo2==0)
    {
      if(PINC & (1<<0)) // Taster entprellen
      {
        ++foo2;
        _delay_ms(2);
      }
    }
    foo2=0;
  }
  
  if(!(PINC & (1<<1))) // Abfrage Taster +Stunde
  {
    ++st;
    if(st>=24)
    {
      st=0;
    }
    _delay_ms(2);
    while(foo2==0)
    {
      if(PINC & (1<<1)) // Taster entprellen
      {
        ++foo2;
        _delay_ms(2);
      }
    }
    foo2=0;
  }  
  show();
  if(PINC & (1<<5))
  {
    ++foo;
  }
  }
  foo=0;
}

void run (void) // Ablauf Programm
{
  
  RUN=1;
  while(foo==0)
  {
    
    show();
    if(PINC & (1<<4))
    {
      ++foo;
      RUN=0;
    }
  }
  
  foo=0;
}


int main(void)
{
  DDRB = 0b00111111; // Ausgang LED's Stunden
  DDRD = 0xFF; // Ausgang LED's Minuten
  DDRC = 0x00; // Eingang Taster / Schalter
  PORTB = 0xFF; // LED's Stunden Aus
  PORTD = 0xFF; // LED's Minuten Aus
  PORTC = 0xFF; // Eingang Pullup Aktive
  
ASSR |= (1<<AS2);      //async operation
TCCR2 |= (1<< CS22) | (1<< CS20);  //Prescaler auf 128
TCNT2 = 0x00;        //Init counter
TIMSK |= (1<<TOIE2);      //Overflow Interrupt Enable
sei();
  
    while(1)
    {
  
  if(!(PINC & (1<<5))) // Wenn Schalter Programmierung Gesetz ist
  {
    prog();
  }
  
  if(!(PINC & (1<<4))) // Wenn Schalter Programm Laufen Gesetz ist
  {
    run();
  }

  
  
  }
  
  return 0;
}
  
  ISR (TIMER2_OVF_vect)
  {
    if(RUN>=1)
    {
      sek++;
    }
    
      if(sek>=60)
      {
        sek=0;
        min++;
      }
      
        if(min>=60)
        {
          min=0;
          st++;
        }
        
          if(st>=24)
          {
            st=0;
          }
          while(ASSR & (1<<TCN2UB | 1<<OCR2UB | 1<<TCR2UB));
    
  }

[Karl H. (kbuchegg)]

Ich hab zwar die Hälfte deiner Programmlogik nicht auf Anhieb 
verstanden, aber ich denke mal, du hast dich mit deinen globalen 
Variablen selbst aufs Glatteis geführt.
Insbesondere foo verwendest du nach Lust und Laune und wie es dir 
beliebt.

zb benutzt du hier

void run (void) // Ablauf Programm
{
  
  RUN=1;
  while(foo==0)
  {
    
    show();

ein foo, (ohne es mit einem vernünftigen Wert zu versehen), dasselbe foo 
wird allerdings auch in show() benutzt.

Das ist immer ein und dieselbe Variable!

Damit einem genau so was nicht passiert, gibt es funktionslokale 
Variablen

void foo()
{
  int i;

  for( i = 0; i < 5; i++ )
    ... mach was
}

void bar()
{
  int i;

  for( i = 0; i < 8; i++ )
    foo();
}

das i in foo() und das i in bar() sind 2 verschiedene Variablen und 
kommen sich nicht in die Quere. Sie sind voneinander vollkommen 
unabhängig. Eine Veränderung von i in foo() verändert nicht die Logik in 
bar().

So, und jetzt muss ich mal ergründen, was du eigentlich in show() 
beabsichtigst. Das hier

  while(foo==0)
  {
    if(min_s > 9)
    {
      min_8=(min_8+0x10);
      min_s=(min_s-10);
    }
    else
    {
      min_8=(min_8+min_s);
      ++foo;
    }
  }

sieht alles mehr als merkwürdig aus.

[Marvin (Gast)]

Hi,

Kannte erst mal alle Vars aus der ISR volatile deklarieren...


Gruß

[Karl H. (kbuchegg)]

Ach. Das soll eine BCD Zerlegung sein.

Das kannst du viel einfacher schreiben.

//
// Aufteilen der Zahl in Zehner und Einer
// Die Zehner werden in die oberen 4 Bits des Ergebnisses geschoben
//
uint8_t toBCD( uint8_t dezimalWert )
{
  uint8_t Ergebnis;

  Ergebnis = ( dezimalWert / 10 ) << 4;
  Ergebnis |= ( dezimalWert % 10 );

  return Ergebnis;
}

void show()
{
  PORTD = toBCD( min );
  PORTB = toBCD( st );
}

[Pierre G. (bqube)]

Also der teil hier

min_s=min;
  while(foo==0)
  {
    if(min_s > 9)
    {
      min_8=(min_8+0x10);
      min_s=(min_s-10);
    }
    else
    {
      min_8=(min_8+min_s);
      ++foo;
    }
  }
  foo=0;
  min_s=0;

Sorgt dafür das ich aus z.b DEZ 46(Minuten) die in der Variable "min"
steckt den BCD Code erzeuge so das ich auf einem PORT im
oberen Nibble Zehner und im unteren die Einer stehen hab.

Zum schluss geb ich das nur auf den jeweiliegen Port aus und hab meine 
Zeit.

[Karl H. (kbuchegg)]

Marvin hat ja volatile schon angemahnt.

Wenn du den Code dann vereinfachst, dann fallen auch die meisten anderen 
Variablen auch alle weg.

Aber auf eines möchte ich noch eingehen.
Ich hab keine Ahnung, wo du die Sache mit dem foo her hast. Aber du 
schiesst dir damit ins Knie.

So etwas

void run (void) // Ablauf Programm
{
  
  RUN=1;
  while(foo==0)
  {
    
    show();
    if(PINC & (1<<4))
    {
      ++foo;
      RUN=0;
    }
  }
  
  foo=0;
}

ist ausser unübersichtlich nur noch unübersichtlich. Es mag Fälle geben, 
in denen man eine Schleife so abbricht und das die einfachste Lösugn 
darstellt. Aber in den meisten Fällen solltest du um solche Dinge einen 
Bogen machen.
Wann soll denn diese Schleife abbrechen? Na, wenn am PINC  das Bit 4 auf 
1 geht. D.h. die Schleife soll laufen, solange das Bit 4 am PINC auf 0 
ist. Und genau so programmierst du das auch

void run (void) // Ablauf Programm
{ 
  RUN = 1;

  while( !(PINC & (1<<4) )
  {
    show();
  }

  RUN = 0;
}

Viel simpler. Viel einfacher zu durchschauen. Wesentlich weniger 
Potential, sich selbst ins Knie zu schiessen. Wenn du show mindestens 1 
mal aufgerufen haben willst, dann eben so

void run (void) // Ablauf Programm
{ 
  RUN = 1;

  do
  {
    show();
  } while( !(PINC & (1<<4) );

  RUN = 0;
}

aber nicht mit 200 Flag-Variablen, die dann auch noch global sind. Da 
ist ein Schuss ins Knie quasi vorprogrammiert.

[Pierre G. (bqube)]

Karl Heinz schrieb:
> Ach. Das soll eine BCD Zerlegung sein.
>
> Das kannst du viel einfacher schreiben.
>
>

> //
> // Aufteilen der Zahl in Zehner und Einer
> // Die Zehner werden in die oberen 4 Bits des Ergebnisses geschoben
> //
> uint8_t toBCD( uint8_t dezimalWert )
> {
>   uint8_t Ergebnis;
> 
>   Ergebnis = ( dezimalWert / 10 ) << 4;
>   Ergebnis |= ( dezimalWert % 10 );
> 
>   return Ergebnis;
> }
> 
> void show()
> {
>   PORTD = toBCD( min );
>   PORTB = toBCD( st );
> }
>

Danke schön ;) wenn ich da meinen weg vergleiche ist meiner viel zu 
umständlich.

Also kann es durch meine Globalen Variablen dazu kommen das sie sich 
überschneiden und dadurch der Programmablauf durcheinander kommt ?

[Karl H. (kbuchegg)]

Pierre Gnauck schrieb:

> Also kann es durch meine Globalen Variablen dazu kommen das sie sich
> überschneiden und dadurch der Programmablauf durcheinander kommt ?

Ich denke mal: ja.
Auslöser dürfte sein, dass du in show das foo erhöhst, und damit dann 
aus der Schleife in run rausfällst.

Aber ich hab das nicht weiter gross analysiert. Erspars mir bitte. Du 
hast da vieles in deinem Programm wesentlich zu umständlich und 
langatmig gelöst, so dass es schwer ist, da noch den Überblick zu 
behalten.

[Karl H. (kbuchegg)]

Karl Heinz schrieb:
> Pierre Gnauck schrieb:
>
>> Also kann es durch meine Globalen Variablen dazu kommen das sie sich
>> überschneiden und dadurch der Programmablauf durcheinander kommt ?
>
> Ich denke mal: ja.
> Auslöser dürfte sein, dass du in show das foo erhöhst, und damit dann
> aus der Schleife in run rausfällst.

Nein, ist auch nicht richtig.
Du setzt foo am Ende von show wieder auf 0.

> Erspars mir bitte.
(Gemeint ist natürlich die Analyse, was da passiert)

[Pierre G. (bqube)]

Gut dann werde ich mal ein wenig aufräumen, ja das mit den umständlichen 
kommt denk ich mal aus der asm Welt von der ich gerade in C einsteige.

Mfg Bqube

[Karl H. (kbuchegg)]

In prog dasselbe wie in run()

Mach das nicht! Die ganzen Schleifen lassen sich alle viel einfacher 
formulieren, wenn du die Abbruchbedingung direkt ins while 
reinschreibst. Gleichzeitig bist du dann auch alle Kopfstände und 
Kopfzerbrechen los, wer wann welche globale Variable foo oder foo2 oder 
... verändert.

Aus dem hier

  if(!(PINC & (1<<1))) // Abfrage Taster +Stunde
  {
    ++st;
    if(st>=24)
    {
      st=0;
    }
    _delay_ms(2);
    while(foo2==0)
    {
      if(PINC & (1<<1)) // Taster entprellen
      {
        ++foo2;
        _delay_ms(2);
      }
    }
    foo2=0;
  }

wird dann das hier

  if(!(PINC & (1<<1))) // Abfrage Taster +Stunde
  {
    ++st;
    if(st>=24)
    {
      st=0;
    }

    _delay_ms(2);

    while( PINC & ( 1 << 1 ) )
      _delay_ms(2);
  }

einfacher, kürzer, leichter zu überschauen, keine globalen Variablen 
mehr (ausser st)

[Karl H. (kbuchegg)]

Wenn du dir dann noch ein #define machst

#define STUNDEN_TASTER    1

dann verändert sich der Code weiter zu

  if(!(PINC & (1 << STUNDEN_TASTER)) )
  {
    ++st;
    if(st>=24)
    {
      st=0;
    }

    _delay_ms(2);

    while( PINC & ( 1 << STUNDEN_TASTER ) )
      _delay_ms(2);
  }

dann brauchst du erstens den Kommentar nicht mehr und es ist zweitens 
viel leichter zu kontrollieren, dass du beim if und beim while jeweils 
denselben Taster benutzt hast. Ausserdem steckt jetzt im Code ein 
direkter Zusammenhang drinnen, dass du in beiden Fällen wirklich mit 
voller Absicht denselben Taster gemeint hast. Diese Absicht ist auch 
dann zu sehen, wenn man nicht analysiert, was das Programm hier 
eigentlich macht.

Keine derartigen 'magischen Konstanten'! Dafür gibt es #define, damit 
man derartigen Zahlenwerten einen 'Namen' geben kann.
Das hat dann auch den netten Nebeneffekt, dass man bei einer Änderung 
des Zahlenwertes, dieses nur an einer einzigen Stelle machen muss. 
Nämlich beim #define. Der Compiler passt dann alle anderen Stellen 
anhand dieses #define an, in dem er den dort verwendeten Namen durch 
genau diese Zahl ersetzt. Im Endeffekt wird also genau der gleiche Code 
wie deiner übersetzt. Nur ist er für mich als Programmierer viel 
leichter zu lesen, zu verstehen und zu ändern.

[Karl H. (kbuchegg)]

Pierre Gnauck schrieb:
> Gut dann werde ich mal ein wenig aufräumen, ja das mit den umständlichen
> kommt denk ich mal aus der asm Welt von der ich gerade in C einsteige.

gut.
Dann fang gleich mal damit an, zu akzeptieren, dass der Datentyp der 
Wahl für ein Byte ein uint8_t ist und nicht einfach alles als int 
gemacht wird.

generell alles als int zu nehmen ist auf Desktopsystemen in Ordnung.
Da du aus der Assemblerwelt kommst, ist dir geläufig, dass du deinem AVR 
mit 16 Bit Arithmetik nicht unwesentlich mehr Arbeit aufhalst. Wenn also 
ein Byte völlig ausreicht (weil Stunden oder Minuten nun mal locker in 1 
Byte passen), dann verpasst du deinem AVR eine Fleissaufgabe, indem du 
alles als int definierst, wo es ein uint8_t auch getan hätte.

[Pierre G. (bqube)]

So ich hab das ganze ein wenig aufgeräumt nur die DEZ zu BCD geschichte 
haut mit der Funktion von dir nicht hin ?

Wenn ich es mit ihr versuche sind alle LED's an...

Hier der veränderte Teil.

/*
 * Wecker_01.c
 *
 * Created: 15.06.2014 15:37:42
 *  Author: Werkstadt
 */ 

#include <stdio.h>
#include <math.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#ifndef F_CPU
#warning "F_CPU war noch nicht definiert, wird nun mit 1000000 definiert"
#define F_CPU 1000000UL
#endif
#include <util/delay.h>

#define Schalter_Run 4
#define Schalter_Prog 5
// #define Schalter_Alarm 00
// #define Taster_Snooze 00
#define Taster_Minute 0
#define Taster_Stunde 1

uint8_t sek = 0;
uint8_t min = 0;
uint8_t st  = 0;





void show ()
{
  uint8_t min_h = 0;
  uint8_t st_h  = 0;
  
  min_h  = ( min / 10 ) << 4;
  min_h |= ( min % 10 );
  
  st_h   = ( st / 10 ) << 4;
  st_h  |= ( st % 10 );
  
  PORTD = ~ min_h;
  PORTB = ~ st_h;
  
  
  
}

void prog (void)
{
  if(!(PINC & (1<<Taster_Minute)))
  {
    min++;
    
    if(min>59)
    {
      min=0;
    }
    
    _delay_ms(20);
    
    while(!(PINC & (1<<Taster_Minute)))
    {
        _delay_ms(20);
    }
  }
  
  if(!(PINC & (1<<Taster_Stunde)))
  {
    st++;
    if(st>=24)
    {
      st=0;
    }
    _delay_ms(20);
    while(!(PINC & (1<<Taster_Stunde)))
    {
        _delay_ms(20);
    }
  }
}




int main(void)
{
  DDRB = 0xFF;
  DDRD = 0xFF;
  DDRC = 0x00;
  PORTB = 0xFF;
  PORTD = 0xFF;
  PORTC = 0xFF;
  

  TIMSK &=~ ((1<<OCIE2)|(1<<TOIE2));
  ASSR  = (1<<AS2);
  TCNT2=0;
  OCR2=0;
  TCCR2 |= ((1<<CS22)|(1<<CS20));
  TCCR2 &=~ (1<<CS21);
  while(ASSR & (1<<TCN2UB | 1<<OCR2UB | 1<<TCR2UB));
  TIFR  = 0;
  TIMSK |= (1<<TOIE2);
  sei();

  
  
    while(1)
    {
  
  if(!(PINC & (1<<Schalter_Prog)))
  {
    prog();
    show();
  }
  
  if(!(PINC & (1<<Schalter_Run)))
  {
    if(sek>59)
    {
      sek=0;
      min++;
    }
    
    if(min>59)
    {
      min=0;
      st++;
    }
    
    if(st>=24)
    {
      st=0;
      sek=0;
      min=0;
    }
    show();
  }
  }
  return 0;
}
  
  ISR (TIMER2_OVF_vect)
  {
    
      sek++;
    
    
      
  while(ASSR & (1<<TCN2UB | 1<<OCR2UB | 1<<TCR2UB));
    
  }

[Karl H. (kbuchegg)]

volatile uint8_t sek = 0;
volatile uint8_t min = 0;
volatile uint8_t st  = 0;

ok, sek müsste nicht volatile sein. Aber min und st.


FAQ: Was hat es mit volatile auf sich

[Karl H. (kbuchegg)]

Wieso hast du denn hier

  ISR (TIMER2_OVF_vect)
  {
    
      sek++;
    
    
      
  while(ASSR & (1<<TCN2UB | 1<<OCR2UB | 1<<TCR2UB));
    
  }

das weiterzählen von min und st rausgenommen?

Das war keine gute Idee. Die würde ich da drinn lassen. Denn min und st 
müssen auch dann weiterhin hochgezählt werden, wenn dein Programm mit 
irgendwas ganz anderem stundenlang beschäftigt ist. Das Hochzählen einer 
Uhr ist als 'unteilbare EInheit' anzusehen.

[Karl H. (kbuchegg)]

Karl Heinz schrieb:

> das weiterzählen von min und st rausgenommen?


In diesem Sinne:
vergiss mein vorheriges Posting.
Jetzt muss sek volatile sein.


Die Variable, auf die du von der ISR und von irgendwo anders aus 
zugreifst.

[Pierre G. (bqube)]

Jetzt hast du mich ein wenig verwirrt :)

[Pierre G. (bqube)]

Aber ich glaube so langsam das ist ein Hardware Problem besitzte denn 
der effekt ist gleich geblieben.

[Karl H. (kbuchegg)]

Abgesehen vom volatile kann ich allerdings nichts mehr entdecken.
Müsste eigentlich laufen.

[Karl H. (kbuchegg)]

Pierre Gnauck schrieb:
> Aber ich glaube so langsam das ist ein Hardware Problem besitzte denn
> der effekt ist gleich geblieben.

NIcht so schnell.

Speck mal ordentlich ab.

....

int main()
{

  .... die ganzen Initialisierungen
  .... aber noch kein sei(), also keine Interrupts

  while( 1 ) {

    min++;
    if( min == 60 ) {
      st++;
      min = 0;
      if( st == 24 ) {
        st = 0;
      }
    }

    show();
    _delay_ms( 1000 );
  }
}

jetzt wollen wir mal sehen, ob deine LED funktionieren.

(Nicht alles auf einmal schreiben!)

[Pierre G. (bqube)]

Sobald ich versuche beim Zeiteinstellen auf beiden Seiten Stunden / 
Minuten 4 ausgänge gleichzeitig Low setzte steht alles.

[Karl H. (kbuchegg)]

Pierre Gnauck schrieb:
> Sobald ich versuche beim Zeiteinstellen auf beiden Seiten Stunden /
> Minuten 4 ausgänge gleichzeitig Low setzte steht alles.

Lass uns systematisch vorgehen. OK?

[Pierre G. (bqube)]

Ich hatte schneller geschrieben wie  gelesen

[Pierre G. (bqube)]

Wie meinst du das mit deiner Änderung sei(); ist doch bei mir vorhanden 
und zwar hier

TIMSK &=~ ((1<<OCIE2)|(1<<TOIE2));
  ASSR  = (1<<AS2);
  TCNT2=0;
  OCR2=0;
  TCCR2 |= ((1<<CS22)|(1<<CS20));
  TCCR2 &=~ (1<<CS21);
  while(ASSR & (1<<TCN2UB | 1<<OCR2UB | 1<<TCR2UB));
  TIFR  = 0;
  TIMSK |= (1<<TOIE2);
  sei();

[Karl H. (kbuchegg)]

Pierre Gnauck schrieb:
> Wie meinst du das mit deiner Änderung sei(); ist doch bei mir vorhanden
> und zwar hier

Ja.
Aber ich möchte für den ersten Test die Interrupts mal aus dem Spiel 
nehmen.

Im Grunde braucht das Tetsprogramm nur die Ports und sonst nix. Ich will 
dir aber nicht zumuten, dass du den Timercode da rausnimmst. Ist ja doch 
etwas umfangreicher.
Daher: von mir aus soll der Timer im Hintergrund laufen. Aber er soll 
mir beim Rest nicht in die QUere kommen. Beim ersten Test brauchen wir 
ihn nicht. Jede Sekunde (realisiert durch _delay_ms) wird min um 1 
erhöht und show aufgerufen. D.h. deine LED müssen im Sekundentakt die 
Zeit hochzählen.

(1000 ist vielleicht etwas lang, 500 werdens auch tun. Wir wollen ja nur 
wissen, ob irgendetwas ungewöhnliches passiert, wenn die LEDs ihre 
Muster durchgehen. Zb Spannungseinbruch am Netzteil/Batterie)


Und damit sich der Timer aus dem Rest raushält, wird der sei() einfach 
auskommentiert.

[Dieter F. (Gast)]

Betreibst Du die LED über Vorwiderstände - wenn ja, welche Werte?

[Pierre G. (bqube)]

Es sind Extra Helle LED's 20mA aber da mir das doch ein wenig zu hell 
war laufen sie pauschal über 1kohm.

[Karl H. (kbuchegg)]

Ich überleg schon die ganze Zeit, aber ich werd und werd aus dem hier

Stunde       Minute

E  Z         E  Z

0  0         0  0
0  x         0  x
0  x         0  x
x  x         x  x


0  0         0  0
0  x         x  0
0  x         x  0
x  x         x  x

nicht schlau :-)

[Pierre G. (bqube)]

So nun sind die Minuten an dem besagten Punkt vorbei und laufen fröhlich 
vor sich hin.

[Karl H. (kbuchegg)]

Pierre Gnauck schrieb:
> So nun sind die Minuten an dem besagten Punkt vorbei und laufen fröhlich
> vor sich hin.

gut.

Hast du Sekundentakt oder Halbsekunden Takt?

[Pierre G. (bqube)]

> Stunde       Minute
>
> E  Z         E  Z
>
> 0  0         0  0
> 0  x         0  x
> 0  x         0  x
> x  x         x  x
>
>
> 0  0         0  0
> 0  x         x  0
> 0  x         x  0
> x  x         x  x


Ganz einfach die Nullen sind die LED's es sind immer 4 damit die Optik 
Symetrisch ist ^^ und die Kreuze Stellen die situation nach (LED AN)

[Karl H. (kbuchegg)]

Vergiss die Frage, das erleben wir heut sowieso nicht mehr, wenn die 
Stunden soweit sind.

Änderung:

int main()
{

.... wie jetzt auch

  while( 1 ) {

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

    st = min % 24;

    _delay_ms( 500 );
  }
}

Das geht ein wenig schneller, so dass wir dann alle LED durch haben :-)

[Pierre G. (bqube)]

Interesannt jetzt Passiert nix mehr

[Karl H. (kbuchegg)]

Pierre Gnauck schrieb:
> Interesannt jetzt Passiert nix mehr

:-)

mein Fehler.
show() Aufruf vergessen.

[Pierre G. (bqube)]

Läuft glücklich vor sich hin

[Karl H. (kbuchegg)]

Ich geh mal davon aus, dass auch das nichts ungewöhnliches mehr an den 
Tag bringen wird.

D.h. ich würde dann gerne mal die Stelltasten mit dazu nehmen. Interrupt 
bleibt nach wie vor abgeschaltet

.

int main()
{
  ...

  while( 1 ) {
    prog();
    show();
  }
}

Die prog Funktion hab ich durchgesehen. Aufgefallen ist mir da jetzt 
nicht wirklich was schlimmes.

[Pierre G. (bqube)]

Aber es steht :)

[Karl H. (kbuchegg)]

Wie sind deine Tasten angeschlossen.

Laut Programm: einfache Taster, die nach GND schalten. Nichts 
ausergewöhnliches und auch nichts zusätzliches. Nur die Taster.

[Pierre G. (bqube)]

Genau Masse geht dann geschaltet auf den jeweiliegen Eingang vom PORTC

[Karl H. (kbuchegg)]

Kopfkratz

Diese Funktion

void prog (void)
{
  if(!(PINC & (1<<Taster_Minute)))
  {
    min++;
    
    if(min>59)
    {
      min=0;
    }
    
    _delay_ms(20);
    
    while(!(PINC & (1<<Taster_Minute)))
    {
        _delay_ms(20);
    }
  }
  
  if(!(PINC & (1<<Taster_Stunde)))
  {
    st++;
    if(st>=24)
    {
      st=0;
    }
    _delay_ms(20);
    while(!(PINC & (1<<Taster_Stunde)))
    {
        _delay_ms(20);
    }
  }
}

und du drückst auf den Taster am Portpin PORTC / 0



(Verezeih wenn ich alles 3 mal nachfrage. Aber ich muss mir hier ein 
Bild von der Situation bei dir machen. Und bei Fehlersuche stelle ich 
grundsätzlich alles in Frage)

[Karl H. (kbuchegg)]

Wie oft kannst du drücken, ehe alles steht?
Oder steht alles sofort?

[Pierre G. (bqube)]

Ich drücke jetzt PINC.1 für Stunden so lange (Beispiel) das ich DEZ 
17:00 eingestellt hab

 Stunde       Minute
 E  Z         E  Z

 0  0         0  0
 0  x         0  0
 0  x         0  0
 x  x         0  0

Nun drücke ich auf PINC.0 für Minuten so lange das ich bis zur Zeit 
17:17 komme.

 Stunde       Minute
>
> E  Z         E  Z
>
> 0  0         0  0
> 0  x         0  x
> 0  x         0  x
> x  x         x  x

Und Jetzt passiert nix mehr.

[Karl H. (kbuchegg)]

Das gibts doch gar nicht. Hmmmmmmm

Vielleicht doch was mit der Stomversorgung.

Bei den LED Spielchen. Hattest du da den Fall, dass wirklich alle LED 
geleuchet haben?

Wenn nicht, sollten wir das unbedingt ausprobieren und gezielt 
herbeiführen.

[Karl H. (kbuchegg)]

So zum Beispiel
(auf die Idee hätt ich auch schon vorher kommen können. Silly me)

...

int main()
{

.... wie jetzt auch

  st = 17;

  while( 1 ) {

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

    show();

    _delay_ms( 500 );
  }
}

kommt er da über die Hürde drüber?

[Pierre G. (bqube)]

Wo wir hoch gezählt hatten sind wir ja bei dieser Mischung 17:17 
angekommen und weiter gekommen er bleibt echt nur beim Analogen drücken 
da stehen.

Netzteil 8V 2A --> Lm317 5V --> Schaltung mit Osci überwacht... aber nix 
zu erkennen.

[Pierre G. (bqube)]

Ist doch zum Mäuse melken ja er Zählt drüber hinaus ???????

[Karl H. (kbuchegg)]

Pierre Gnauck schrieb:
> Wo wir hoch gezählt hatten sind wir ja bei dieser Mischung 17:17
> angekommen und weiter gekommen er bleibt echt nur beim Analogen drücken
> da stehen.

Hmm. Im Moment bin ich ratlos.
Im Code seh ich nichts, was das erklären könnte. Es ist auch die Zahl 17 
nicht irgendwie jetzt aussergewöhnlich oder würde irgendwo vorkommen 
oder hätte jetzt sonst irgendeinen Bezug.

[Pierre G. (bqube)]

Gut mein Hund schaut mich sowieso schon an als ob ich einen an der 
Waffel hätte, ich würde sagen wir machen da morgen weiter.

Ich schau mir nochmal ganz genau die Schaltung an überprüfe alle und 
melde mich dann gegen abend wieder.

Ich danke dir aber schonmal für deine Mühne :)

[Karl H. (kbuchegg)]

Irgendwelche vertauschte LED oder Pins?

Obwohl: das wär dir beim Zählen aufgefallen

[Karl H. (kbuchegg)]

Pierre Gnauck schrieb:

> Ich schau mir nochmal ganz genau die Schaltung an überprüfe alle und
> melde mich dann gegen abend wieder.

Freitag abend ist bei mir eher schlecht. Werd zusehen, dass ich online 
gehen kann.

> Ich danke dir aber schonmal für deine Mühne :)

Keine Ursache. Das interessiert mich jetzt selber was da los ist.
Sowas hab ich auch noch nie gesehen.

[Mark B. (markbrandis)]

Pierre Gnauck schrieb:
> Gut mein Hund schaut mich sowieso schon an als ob ich einen an der
> Waffel hätte

Pictures please :)

[Ausgeloggt (Gast)]

Probier mal so

//add all #includes here...

#define IS_PRESSED(key)  !(PINC & (1 << key))
#define KEY_HOURS   1
#define KEY_MINUTES 2

#define WAIT_DEBOUNCE 20               //in ms

volatile uint8_t hours = 0;
volatile uint8_t minutes = 0;
volatile uint8_t seconds = 0;

static void init(void)
{
   DDRB  = 0b00111111;    // Ausgang LED's Stunden
   DDRD  = 0xFF;          // Ausgang LED's Minuten
   DDRC  = 0x00;          // Eingang Taster / Schalter
   PORTB = 0xFF;          // LED's Stunden Aus
   PORTD = 0xFF;          // LED's Minuten Aus
   PORTC = 0xFF;          // Eingang Pullup Aktive
   
   TIMSK &=~ ((1<<OCIE2)|(1<<TOIE2));
   ASSR  = (1<<AS2);
   TCNT2=0;
   OCR2=0;
   TCCR2 |= ((1<<CS22)|(1<<CS20));
   TCCR2 &=~ (1<<CS21);
   while(ASSR & (1<<TCN2UB | 1<<OCR2UB | 1<<TCR2UB));
   TIFR  = 0;
   TIMSK |= (1<<TOIE2);
   sei();
}

static uint8_t toBCD(uint8_t value)
{
   return ((value / 10) << 4) | (value % 10);
}

static void nextHour(void)
{
   hours ++;
   if(hours >= 24)
   {
      hours = 0;
   }
}

static void nextMinute(void)
{
   minutes ++;
   if(minutes >= 60)
   {
      minutes = 0;
   }
}

int main(void)
{
   init();
   
   do
   {
      // Show time
      PORTB = toBCD(hours);
      PORTD = toBCD(minutes);

      // Check key presses
      if(IS_PRESSED(KEY_HOURS))
      {
         nextHour();
         
         do
         {
            _delay_ms(WAIT_DEBOUNCE);
         } while(IS_PRESSED(KEY_HOURS));
      }
      
      if(IS_PRESSED(KEY_MINUTES))
      {
         nextMinute();
         
         do
         {
            _delay_ms(WAIT_DEBOUNCE);
         } while(IS_PRESSED(KEY_MINUTES));
      }
   } while(1);
   
   return 0;
}

ISR(TIMER2_OVF_vect)
{
   seconds ++;
   if(seconds >= 60)
   {
      seconds = 0;
      nextMinute();
      if(minutes == 0)
      {
         nextHour();
      }
   }
   while(ASSR & (1<<TCN2UB | 1<<OCR2UB | 1<<TCR2UB))
      ;
}

[Pierre G. (bqube)]

Nabend Karl Heinz,

ich habe nun endlich mal Zeit für meinen Wecker / Uhr gefunden und siehe 
da es war kein Software sondern ein Hardware fehler, den (Schande über 
mein Haupt) ich erst nach 2 Stunden gefunden hatte.

Ich hatte alle Ausgänge und Eingänge über 1K Widerstände miteinander 
verbunden und diese sollten dann gemeinsam auf VCC gelegt werden, nur 
diese Verbindung hatte ich vergessen :-).

Mfg Bqube

[Pierre G. (bqube)]

Mark Brandis schrieb:
> Pierre Gnauck schrieb:
>> Gut mein Hund schaut mich sowieso schon an als ob ich einen an der
>> Waffel hätte
>
> Pictures please :)

Hier :-)

[Mark B. (markbrandis)]

Pierre Gnauck schrieb:
>> Pictures please :)
>
> Hier :-)

Danke :-)

Oder eher so: :-----)

Bei der langen Nase die er hat ;-)