Ulrich Radig´s DCF für Atmega2560 an INT2 statt INT0

[lufthaken (Gast)]

hallo zusammen,

Die DCF Uhr lief lange zeit echt super mit dem code vom Uli.
ich habe es an INT0 am Atmega2560 angeschlossen.
Nun möchte ich zusätzlich einen DS1307 über den TWI Bus nutzen, TWI 
liegt am Atmega an PD0 und PD1 liegt. Problematisch dabei ist nun das 
PD0 gleich INT0 ist.

Atmega2560 Datenblatt auszüge:
  Bit    7     6     5     4     3     2     1      0
(0x69) ISC31 ISC30 ISC21 ISC20 ISC11 ISC10 ISC01 ISC00 EICRA

...

ISCn1 ISCn0 Description
0       0   The low level of INTn generates an interrupt request.
0       1   Any edge of INTn generates asynchronously an interrupt request.
1       0   The falling edge of INTn generates asynchronously an interrupt request.
1       1   The rising edge of INTn generates asynchronously an interrupt request.

eigentlich kein ding, einfach dcf signal an INT2(PD2) hängen und 
folgende zeilen ändern;

//für Atmega 2560 Int0:
//Interrupt an dem das DCF77 Modul hängt hier INT0
#define DCF77_INT_ENABLE()  enable_external_int (1<<INT0);
#define DCF77_INT      SIG_INTERRUPT0

#define INT0_CONTROL    EICRA
#define INT0_FALLING_EDGE  0x02
#define INT0_RISING_EDGE  0x03

//für Atmega2560 INT2:
//Interrupt an dem das DCF77 Modul hängt hier INT2(PD2)
#define DCF77_INT_ENABLE()  EIMSK |= (1<<INT2);
#define DCF77_INT      SIG_INTERRUPT2
#define INT2_CONTROL    EICRA
#define INT2_FALLING_EDGE  0x32
#define INT2_RISING_EDGE  0x48

//############################################################################
//DCF77 Modul empfängt Träger 
SIGNAL (DCF77_INT)
//############################################################################
{ 
   //Auswertung der Pulseweite 
   if (INT2_CONTROL == INT2_RISING_EDGE)
      {
      flags.dcf_rx ^= 1;
      //Secunden Hilfs Counter berechnen // SYSCLK defined in USART.H
      h_ss = h_ss + TCNT1 - (65535 - (F_CPU / 1024));
      //Zurücksetzen des Timers
      TCNT1 = 65535 - (F_CPU / 1024);
      //ist eine Secunde verstrichen // SYSCLK defined in USART.H
      if (h_ss > (F_CPU / 1024 / 100 * 90)) //90% von 1Sekunde
         {
         //Addiere +1 zu Sekunden
         Add_one_Second();
         //Zurücksetzen des Hilfs Counters
         h_ss = 0;
         };
      //Nächster Interrupt wird ausgelöst bei abfallender Flanke
      INT2_CONTROL = INT2_FALLING_EDGE;
      }
   else
      {
      //Auslesen der Pulsweite von ansteigender Flanke zu abfallender Flanke
      unsigned int pulse_wide = TCNT1;
      //Zurücksetzen des Timers
      TCNT1 = 65535 - (F_CPU / 1024);
      //Secunden Hilfs Counter berechnen
      h_ss = h_ss + pulse_wide - (65535 - (F_CPU / 1024));
      //Parity speichern
      //beginn von Bereich P1/P2/P3
      if (rx_bit_counter ==  21 || rx_bit_counter ==  29 || rx_bit_counter ==  36) 
         {
         flags.parity_err = 0;
         };
      //Speichern von P1
      if (rx_bit_counter ==  28) {flags.parity_P1 = flags.parity_err;};
      //Speichern von P2
      if (rx_bit_counter ==  35) {flags.parity_P2 = flags.parity_err;};
      //Speichern von P3
      if (rx_bit_counter ==  58) {flags.parity_P3 = flags.parity_err;};
      //Überprüfen ob eine 0 oder eine 1 empfangen wurde
      //0 = 100ms
      //1 = 200ms
      //Abfrage größer als 150ms (15% von 1Sekund also 150ms)   
      if (pulse_wide > (65535 - (F_CPU / 1024)/100*85))
         {
         //Schreiben einer 1 im dcf_rx_buffer an der Bitstelle rx_bit_counter
         dcf_rx_buffer = dcf_rx_buffer | ((unsigned long long) 1 << rx_bit_counter);
         //Toggel Hilfs Parity
         flags.parity_err = flags.parity_err ^ 1;
         }
      //Nächster Interrupt wird ausgelöst bei ansteigender Flanke
      INT2_CONTROL = INT2_RISING_EDGE;
      //RX Bit Counter wird um 1 incrementiert
      rx_bit_counter++;
      }
};

dcf liegt nun an 'PD2 und nicht an PD0 aber es geht noch nicht.
hab ich was übersehen?

[Stefan E. (sternst)]

#define INT2_FALLING_EDGE  0x32
#define INT2_RISING_EDGE  0x48

Nö.

Quizfrage: was bedeutet das 0x vor den Zahlen?

[lufthaken (Gast)]

Ui, ja hex zahl und ich schreib da die dez zahl rein, ürgh...

habs geändert und es ging irgendwie immernochnich auf INT2.
dann hab ich das ganze auf INT3 versucht, das ging sofort.
seltsam mit INT2(PD2) muss im übrigen code nochn fehler sein oder der 
pin is doof. -.-

[Karl H. (kbuchegg)]

Das hier

   if (INT2_CONTROL == INT2_RISING_EDGE)

ist auch verbesserungswürdig.

Wenn da noch andere Flags im EICRA benutzt werden, dann ist hier Ärger 
vorprogrammiert.

[Karl H. (kbuchegg)]

lufthaken schrieb:
> Ui, ja hex zahl und ich schreib da die dez zahl rein, ürgh...

Schreibs so

#define INT2_FALLING_EDGE  (1<<ISC20)
#define INT2_RISING_EDGE   ((1<<ISC20) | (1<<ISC21))

dann hast du das Problem nicht.

[Karl H. (kbuchegg)]

Karl Heinz Buchegger schrieb:
> Das hier
>

>    if (INT2_CONTROL == INT2_RISING_EDGE)
>

> ist auch verbesserungswürdig.
>
> Wenn da noch andere Flags im EICRA benutzt werden, dann ist hier Ärger
> vorprogrammiert.

Genauso wie hier

     INT2_CONTROL = INT2_FALLING_EDGE;

bzw.

      INT2_CONTROL = INT2_RISING_EDGE;

Das muss ziemlich alter Code sein. SIGNAL ist schon lange aus der Mode 
und der Code ist von der Registerbenutzung her nicht sehr sauber 
geschrieben.

Da würde ich erst mal Hand anlegen und das ganze auf eine saubere 
Bitoperation setzen die nicht davon ausgeht, dass das hier der einzige 
benutzte externe Interrupt ist.

[Karl H. (kbuchegg)]

lufthaken schrieb:

> habs geändert und es ging irgendwie immernochnich auf INT2.
> dann hab ich das ganze auf INT3 versucht, das ging sofort.
> seltsam mit INT2(PD2) muss im übrigen code nochn fehler sein oder der
> pin is doof. -.-

Pin auf Eingang gestellt?
Pullup eingeschaltet (falls notwendig)?

Gib dir in die ISR eine Ausgabe rein, die dir anzeigt ob die ISR 
aufgerufen wird (LED toggeln). Dann weiß man schon mal mehr.

(Die Initialisierung ebenfalls auf 'wilde' Registerkonstrukte 
durchsehen!)