If Problem Sekundenänderung

[Matthias T. (matthias_199)]

Servus,

Ich baue grad an meiner Uhr und habe ein sehr stranges Problem.

Ich bekomme den AVR nicht überredet das er wenn sich die sekunde ändert 
er doch bitte meinen counter erhöht und alles in meinen Latch schiebt.

Das Finde ich extrem seltsam.

Wenn ich das aus der mein nehmen und für die ISR anpasse macht er genau 
das was ich will.

das ist aber nicht sinn der Sache.

MFG Matthias

[#ifndef F_CPU
#define F_CPU 1000000UL
#endif
#include <avr/io.h>
#include <util/delay.h>
#include <stdint.h> 
#include <avr/interrupt.h>

const uint8_t bib[] = {95,12,155,158,204,214,215,28,223,222};
  
uint8_t sekunden = 0;
uint8_t alte_sekunden = 0;
uint16_t milisekunden = 0;  
uint8_t counter = 0;  


void Latch_Serialout(volatile uint8_t* Port ,uint8_t SER,uint8_t SCK,uint8_t DATEN);
void Latch_Setoutput(volatile uint8_t* Port,uint8_t RCK);
uint8_t Int_to_bcd(uint8_t Zahl ,uint8_t Pointflag);




int main(void)
{
  sei(); //Interrupts erlaube
  DDRD  = 0xFF;
  PORTD = (0<<PD0) | (0<<PD1)| (0<<PD2);
  
  DDRB |= (1<<PB1); // Port OC1A mit angeschlossener LED als Ausgang
  TCCR1A = (1<<WGM10) | (1<<COM1A1); // PWM, phase correct, 8 bit.
  TCCR1B = (0<<CS12) | (0<<CS11) | (1<<CS10); // Prescaler 1 = Enable counter 1
  OCR1A = 200; // Duty cycle 50% (Anm. ob 128 oder 127 bitte prüfen)
  
  
  //DDRB |= (1<<PB2); // Port OC1B mit angeschlossener LED als Ausgang 
  //TCCR1A |= (1<<COM1B1); //Starte zweites PWM
  //OCR1B = 127;
  
  TCCR0 = (0<<CS02) | (1<<CS01) | (0<<CS00); // Prescaler 8 = Enable counter 0
  TIMSK |= (1<<TOIE0); //Interrupt ennabelen
  TCNT0 = 133; //Timer Preloaden
  

  
    
  while(1)
  {
    if(alte_sekunden != sekunden)
    {
      alte_sekunden = sekunden;
        Latch_Serialout(&PORTD,(1<<PD0),(1<<PD1),Int_to_bcd(counter,0));
        Latch_Setoutput(&PORTD,(1<<PD2));
        Latch_Serialout(&PORTD,(1<<PD0),(1<<PD1),0);
        Latch_Setoutput(&PORTD,(1<<PD2));  
      counter++;    
      if (counter > 9)
      {
        counter = 0;
      }
    }          
    //for (uint8_t x = 0;x <= 254;x++)
    //{
    //    OCR1A --;
    //  _delay_ms(500);
    //}
    //_delay_ms(1000);    
    //for (uint8_t x = 0;x <= 254;x++)
    //{
    //  OCR1A ++;
    //  _delay_ms(500);
    //}
    //_delay_ms(100);

    
    
  }
}

uint8_t Int_to_bcd(uint8_t Zahl, uint8_t Pointflag)
{
  if ((Zahl > 9) || (Zahl < 0))
  {
    return 0;
  }
  if (Pointflag > 0)
  {
    return bib[Zahl] + 32;
  }
  return bib[Zahl];   
}  

void Latch_Serialout(volatile uint8_t* Port ,uint8_t SER,uint8_t SCK,uint8_t DATEN)
// Die Funktion gibt ein Datenbyte an ein Latch (Schieberegister) weiter.
// SER pin 14 am Latch
// SCK pin 11 am Latch
// VORISCHT! Datenbyte würd umgedreht! Q0 = Q7 , Q1 = Q6 ..... usw
// Beispielaufruf Latch_Serialout(&PORTD,(1<<PD0),(1<<PD1),255);
{
  *Port &= ~SCK; //Serial clock ausmachen
  for (uint8_t x = 0;x < 8;x++) // Schleife für 8 bit = 1 Byte
  {
    if ((DATEN & 1) == 1){ // Wenn DATEN und Bitmaske (1) gleich sind
      *Port |= SER; //Setze SER Port High
    }
    *Port |= SCK; //SCK High setzen damit das Bit an Ser übernommen wird
    *Port &= ~SCK; //SCK Low setzen damit nichts mehr übernommen wird
    *Port &= ~SER; //SER Low Setzen da er schon übernommen wurde
    DATEN = (DATEN >> 1); // Schiebe die Daten um 1 nach rechts für die nächste stelle
  }
}


void Latch_Setoutput(volatile uint8_t* Port,uint8_t RCK)
// Die Funktion gibt ein Übernahme Signal an ein Latch (Schieberegister) weiter.
// RCK pin 12 am Latch
// Beispielaufruf Latch_Setoutput(&PORTD,(1<<PD2));
{
  //*Port &= ~RCK; // RCK auf 0 Setzen nur aktivieren falls es probleme gibt
  *Port |= RCK; // RCK auf High setzen damit die bits ins Output Register kommen
  *Port &= ~RCK; // RCk wieder low setzen
}

ISR (TIMER0_OVF_vect) //Timer 0 Überlauf Funktion
{
  TCNT0 = 133; // eigendlich 133
  PORTD ^= (1<<PD4); //toggeln
  milisekunden ++ ;
  if (milisekunden > 999)
  {
    sekunden++;
    milisekunden = 0;
  }
  //if  (PIND & (1<<PIND4))
  //{
  //  Latch_Serialout(&PORTD,(1<<PD0),(1<<PD1),255);  
  //}
  //else
  //{
  //  Latch_Serialout(&PORTD,(1<<PD0),(1<<PD1),0);
  //}
  //Latch_Setoutput(&PORTD,(1<<PD2));
}

[Timmo H. (masterfx)]

 volatile uint8_t sekunden = 0;

und die Millisekunden kannst du auch als Static in der isr machen

[Matthias T. (matthias_199)]

Das wars ^^
Aber nachts kommt mann da nimmer drauf und wenn man in c aufm pc coded 
benutzt man volatile so gut wie garnicht.


das mit dem static milisekunden ist ein guter tipp hab ich vergessen das 
mann da so schöner lösen kann.

aber dann kann ich doch auch aus

const uint8_t bib[] = {95,12,155,158,204,214,215,28,223,222};

in meine Int_to_bcd packen

static const uint8_t bib[] = {95,12,155,158,204,214,215,28,223,222};

Damit se nicht im head der File rumgammelt und später verloren geht.

MFG Matthias

[(prx) A. K. (prx)]

Matthias T. schrieb:
> wenn man in c aufm pc coded benutzt man volatile so gut wie garnicht.

Doch. Wenn man multithreaded programmiert.

[Matthias T. (matthias_199)]

A. K. schrieb:
> Doch. Wenn man multithreaded programmiert.

Jo hast recht aber bis jetz ist mir persöhnlich keine Anwendung 
untergekommen, und wird dir als normaler mensch wohl nicht, das es nötig 
wäre.
außer du hast vor cryengine nachzuschreiben oder ein Treiber.
aber wie gesagt wer macht das also normalo^^

[(prx) A. K. (prx)]

Matthias T. schrieb:
> und wird dir als normaler mensch wohl nicht, das es nötig wäre.

Dann bin ich offensichtlich kein normaler Mensch.

> außer du hast vor cryengine nachzuschreiben oder ein Treiber.
> aber wie gesagt wer macht das also normalo^^

Programmierung mit mehreren Threads setzt nicht voraus, dass die 
Hardware auch mehrere Threads parallel ausführen kann. Das hat nicht nur 
mit Performance zu tun, sondern auch mit Ablaufstrukturen. Ich habe auch 
auf einem AVR schon mit einem RTOS-Kernel gearbeitet.