ATMEGA1284P TIMER3 FAST PWM

[Elektroboon (Gast)]

Hi,

ich versuche vergeblich den Timer3 zu verwenden.

Timer1 klappt ganz super mit:

TCCR1A = (1<<COM1A1) + (1<<COM1B1) + (1<<WGM11);
TCCR1B = (1<<WGM13) + (1<<WGM12) + (1<<CS10); // prescaler = 1 (none)
ICR1 = 15999;
OCR1A = 3999;
OCR1B = 11999;
DDRD |= (1 << DDD4) | (1 << DDD5);


Timer3 hingegen mit:

TCCR3A = (1<<COM3A1) + (1<<COM3B1) + (1<<WGM31);
TCCR3B = (1<<WGM33) + (1<<WGM32) + (1<<CS30); // prescaler = 1 (none)
ICR3 = 15999;
OCR3A = 3999;
OCR3B = 11999;
DDRD |= (1 << DDB6) | (1 << DDB7); //PB6 und 7 als AUSGANG

scheint nicht zu funktionieren.

Hat jemand ne Idee?

Danke schonmal.

[Chris K. (Gast)]

Du solltest auch DDRB beschrieben, wenn du du PB6 und PB7 verwenden 
willst.

[Elektroboon (Gast)]

OMG wie doof. Logisch. Danke.

[Elektroboon (Gast)]

Ich habe nun leider ein Problem bei der ISR. Mein uC läuft mit 20MHz.

Bei 1000ms soll hochgezählt werden. Ohne die PWM-Timer1/3 Inits 
funktioniert es mit "==". Mit PWM geht es nur sporadisch mit "==" und 
mit ">=" wird es sehr ungenau.

Kann mir jemand erklären warum die Variable über 1000 läuft und wie ich 
das beheben kann?

Beim PWM würde mir eine geringere Auflösung reichen falls dies helfen 
würde... jedoch benötige ich die 4 Kanäle.

void encode_init( void )
{
  int8_t new;

  //set pins as input
  ROTDDR &= ~((1<<ROTPA)|(1<<ROTPB)|(1<<ROTPBUTTON));
  //enable interrnal pullups;
  ROTPORT |= (1<<ROTPA)|(1<<ROTPB)|(1<<ROTPBUTTON);


  new = 0;
  if( PHASE_A ) new = 3;
  if( PHASE_B ) new ^= 1;       // convert gray to binary
  last = new;                   // power on state
  enc_delta = 0;
  TCCR0B = (1<<CS02);   //  prescaler 256
  TCCR0A = (1<<WGM01); //   CTC Modus
  OCR0A = 78; //78=1ms
  TIMSK0 |= 1<<OCIE0A;
}

TCCR1A = (1<<COM1A1) + (1<<COM1B1) + (1<<WGM11);
TCCR1B = (1<<WGM13) + (1<<WGM12) + (1<<CS10); // prescaler = 1 (none)
ICR1 = 16000;
OCR1A = 0; //R 
OCR1B = 0; //G
DDRD |= (1 << DDD4) | (1 << DDD5); //PD4 und 5 als AUSGANG


TCCR3A = (1<<COM3A1) + (1<<COM3B1) + (1<<WGM31);
TCCR3B = (1<<WGM33) + (1<<WGM32) + (1<<CS30); // prescaler = 1 (none)
ICR3 = 16000;
OCR3A = 0; //B
OCR3B = 4000; //0-10V Output
DDRB |= (1 << DDB6) | (1 << DDB7); //PB6 und 7 als AUSGANG

ISR( TIMER0_COMPA_vect )            // 1ms for manual movement
{
  int8_t new, diff;

  new = 0;
  if( PHASE_A ) new = 3;
  if( PHASE_B ) new ^= 1;          // convert gray to binary
  diff = last - new;               // difference last - new
  if( diff & 1 ) 
  {                 // bit 0 = value (1)
      last = new;                    // store new as next last
      enc_delta += (diff & 2) - 1;   // bit 1 = direction (+/-)
  }
  ms++;//Addiere +1 zu MilliSekunden
  if (ms == 1000)    //-------------------------------HIER!!!!!!! Bei == überläuft die Variable bei >= wird's ungenau!!!!
  {
    ms = 0;
    ss++;//Addiere +1 zu Sekunden
    lcd_burned_ss++;
    if (ss >= 60)
    {
      ss = 0;
      mm++;//Addiere +1 zu Minuten
      if (mm >= 60)
      {
        mm = 0;
        hh++;//Addiere +1 zu Stunden
        if (hh >= 24)
        {
          day++;
          if (day >= 8)
          {
            day=1;
          }
          hh = 0;
        }
      }
    }
  }
}

Danke vorab.

[Oliver S. (oliverso)]

Fehler in Zeile 42 (die du nicht zeigst).

Oliver

[Falk B. (falk)]

Elektroboon schrieb:
> Ich habe nun leider ein Problem bei der ISR. Mein uC läuft mit 20MHz.
>
> Bei 1000ms soll hochgezählt werden. Ohne die PWM-Timer1/3 Inits
> funktioniert es mit "==". Mit PWM geht es nur sporadisch mit "==" und
> mit ">=" wird es sehr ungenau.

Wie stellst du das fest?

> Kann mir jemand erklären warum die Variable über 1000 läuft

Das kann sie mit DEM Programm nicht.

> Beim PWM würde mir eine geringere Auflösung reichen falls dies helfen
> würde... jedoch benötige ich die 4 Kanäle.

Hilft nicht.

Zeige VOLLSTÄNDIGEN, ORIGINALEN Quelltext, keine losen Schnipsel!
Siehe Netiquette.

[Elektroboon (Gast)]

Das stelle ich fest indem ich die Sekunden im Display mit einer Uhr 
vergleiche und nach ca. 30s sehe wie sie langsam voneinander abweichen.

Den kompletten Code mit allen includes zu Posten würde den Rahmen 
sprengen.

[Peter D. (peda)]

Elektroboon schrieb:
> Den kompletten Code mit allen includes zu Posten würde den Rahmen
> sprengen.

Wow, Dein Code ist also über 10MB groß?
Ist aber auch kein Problem, dann zippe das komplette Projekt, plain Text 
läßt sich gut komprimieren.