m328 pwm + timer

[grundschüler (Gast)]

ich versuche mich mit pwm + isr:

static void pwm_timer_init(void)
{
    ICR1 = 0xFF; // set TOP to 16bit
 
    // set none-inverting mode
    TCCR1A |= (1 << COM1A1)|(1 << COM1B1);

    // set Fast PWM mode using ICR1 as TOP
    TCCR1A |= (1 << WGM11);
    TCCR1B |= (1 << WGM12)|(1 << WGM13);
    

    TCCR0A |= (1 << COM0A1)|(1 << COM0B1);
    // set none-inverting mode
    TCCR0A |= (1 << WGM01) | (1 << WGM00);
    // set fast PWM Mode



    TCCR2A |= (1 << COM2A1)|(1 << COM2B1);
    // set none-inverting mode
    TCCR2A |= (1 << WGM21) | (1 << WGM20);
    // set fast PWM Mode
 
 
TCCR0B |= ti_prescaler_8;//1
TCCR2B |= ti_prescaler_8;//1
TCCR1B |= ti_prescaler_8;//1

    OCR1A = 60;
    OCR1B = 60;
  OCR0A = 60;
   OCR0B = 60;
  OCR2A = 60;
  OCR2B = 60;
  
 
  DDRB |= (1 << 3); DDRD |= (1 << 3);
  DDRB |= (1 << 1) | (1 << 2);
  DDRD |= (1 << 5) | (1 << 6);
  
  // Enable Timer1 capture event interrupt.
 // TIFR0 = _BV(ICF1) | _BV(OCF1B) | _BV(OCF1A) | _BV(TOV1) ;
//  TIMSK0 = 
//  TCCR0A |= (1 << WGM01);

  TIMSK0 |= (1 << OCIE0A);    //Set the ISR COMPA vect
 //  TIMSK0 |= (1 << TOIE0);    //Set the ISR ovf vect
 // Enable Timer1 capture event interrupt.

//  TIFR1 = (1<<ICF1) | (1<<OCF1B) | (1<<OCF1A) | (1<<TOV1) ;
//  TIMSK1 = (1<<ICIE1);

sei();
}

v u16 tim;

ISR(TIMER0_COMPA_vect)
{
tim++;
if(tim>10){
  tim=0;
  led1_tog;
  }
}

pwm funktioniert, die isr wird aber nur einmal angesprungen.

Was ist falsch?

[Einer K. (Gast)]

Warum ist tim nicht volatile?

[grundschüler (Gast)]

Arduino F. schrieb:
> Warum ist tim nicht volatile?

ist es:
#define v volatile

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Arduino F. schrieb:
> Warum ist tim nicht volatile?

grundschüler schrieb:
> v u16 tim;

Solls wohl sein, aber der Rest der Welt weiss nicht, ob v wirklich für 
volatile steht. Da tim aber ausserhalb der ISR nicht genutzt wird, kann 
man besser eine static innerhalb der ISR nehmen, damit entsteht die 
Verwirrung gar nicht.
Lustige Sachen passieren bei sowas, wenn noch irgendwo anders ein 'v' 
benutzt wird... Da es die Lesbarkeit auch nicht erhöht, halte ich 
persönlich solche defines für Blödsinn.
Wirklich sinnvoll wären defines für die PWM Ausgänge, stattdessen steht 
da:
  DDRB |= (1 << 3); DDRD |= (1 << 3);
  DDRB |= (1 << 1) | (1 << 2);
  DDRD |= (1 << 5) | (1 << 6);

Das ist nun wirklich Quak.

[grundschüler (Gast)]

dämlicher fehler:

mit

if(tim>1000){
  led1_tog;
  tim=0;
  }

blinkt die led.