ATmega328PB LED rgb PWM

[Fleur (Gast)]

Hallo,

kann mir vielleicht einer helfen?
Ich weiß einfach nicht, warum die grüne LED nicht angehen möchte.. 
Selbst wenn ich Sie in den ersten Case reinsetze.
Rot und blau funktionieren..
Hab ich was mit den Timern falsch gemacht?

#define F_CPU 16000000UL

#include <avr/io.h>
#include <avr/pgmspace.h>
#include <avr/interrupt.h>
#include "lookup.dat"

#define LED_RED_init()       DDRB |= (1<<DDB1)
#define RED()                OCR1A = w;     OCR1B = 0; OCR2A = 0;
             

#define LED_BLUE_init()      DDRB |= (1<<DDB3)
#define BLUE()               OCR1A = 0; OCR1B = 0; OCR2A = w;


#define LED_GREEN_init()     DDRB |= (1<<DDB2)
#define GREEN()              OCR1A = 0; OCR1B = w; OCR2A = 0;

#define WHITE_init()         DDRB |= ( (1<<DDB1) | (1<<DDB2) | (1<<DDB3) )
#define WHITE()              OCR1A = x; OCR1B = x; OCR2A = x;

#define BREAK()              OCR1A = 0; OCR1B = 0; OCR2A = 0;  


// Globale Variablen
volatile int intensitaet = 0;
volatile char farbe = 0;
volatile char w;
volatile char x; 

enum
{
    RED, BREAK1, GREEN, BREAK2, WHITE, BREAK3
} ;
//   0     1       2      3       4      5

void init_rgb(void)  
{
LED_BLUE_init();
LED_RED_init();
LED_GREEN_init();
WHITE_init();

}

void init_timer(void)
{       
    //Timer 0
    TCCR0B |= (1<<CS02) | (1<<CS00);    // Prescaler 1024
    TCCR0A |= (1<<WGM00);               // Phase Correct Mode 1
    TCCR0A |= (1<<COM0A1);              // Non-inverting Mode, CM0A1 --> 1
    OCR0A = 121;   
    
    //Timer 1
    TCCR1B |= (1<<CS12);                // Prescaler 256
    TCCR1A |= (1<<WGM10);               // Mode 1
    TCCR1A |= (1<<COM1A1) | (COM1B1);
    
    //Timer 2
    TCCR2B |= (1<<CS22) | (1<<CS21);    // Prescaler 256
    TCCR2A |= (1<<WGM20);               // Mode 1
    TCCR2A |= (1<<COM2A1);
}

ISR(TIMER0_COMPA_vect)
{
    intensitaet++;
}

void programm(void)
{  
  

       
    switch(farbe)           
    {           
        case RED: 
            RED ();           
            if(intensitaet==255)
            {
                intensitaet = 0;
                farbe = BREAK1;   
            }
            break;
            
        case BREAK1:
            BREAK();              
            if(intensitaet==255)
            {
                intensitaet = 0;
                farbe = GREEN;   
            }
            break;
            
        case GREEN:          
            GREEN();            
            if(intensitaet==255)
            {
                intensitaet = 0;
                farbe = BREAK2;   
            }
            break;
            
        case BREAK2:
            BREAK();           
            if(intensitaet==255)
            {              
                farbe = WHITE;   
            }
            break;
            
        case WHITE:          
            WHITE();            
            if(intensitaet==0)
            {          
                farbe = BREAK3;   
            }
            break;
            
        case BREAK3:
            BREAK();           
            if(intensitaet==255)
            {
                intensitaet = 0;
                farbe = RED;   
            }
            break;            
    }
}

int main(void)
{
    init_rgb();
    init_timer();
    sei();     
    TIMSK0 |= (1<<OCIE0A);
    intensitaet = 0;    
         
    while (1)
    {        
        programm();
      
        w = pgm_read_byte_near(lookup + intensitaet); 
        x = pgm_read_byte_near(lookup +255 -intensitaet);         
    }
}

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Fleur schrieb:
> Ich weiß einfach nicht, warum die grüne LED nicht angehen möchte..
Leuchtet die LED, wenn du den Pin statisch auf "EIN" setzt?

Das Programm an sich hat aber auch noch ein paar kleine Ecken, die 
einige Stunden Fehlersuche kosten können...

Denn es ist generell eine ganz üble Sache, einen Zähler von 
verschiedenen Stellen aus zu manipulieren:

volatile int intensitaet = 0;    -- warum eigentlich ein int, 
                                 -- wenn doch nur 8 Bit gezählt werden?

ISR(TIMER0_COMPA_vect)
{
    intensitaet++;                    -- Zähler wird geändert
}

void programm(void)
{  
    switch(farbe)           
    {           
        case RED: 
            RED ();           
            if(intensitaet==255)
            {   
                intensitaet = 0;     -- Zähler wird geändert

Und was passiert hier:

            if(intensitaet==255)

wenn es der Zähler "zufällig" schafft, durch einen Interrupt auf 256 zu 
kommen?

BTW: bitte fürderhin die [c] Tags selber einfügen!

[Fleur (Gast)]

Wenn ich den statisch auf ein setze, dann leuchtet es ja.
Habe ihn jetzt auf ein char gesetzt, hatte da nicht genau drüber 
nachgedacht.
also muss ich alle OCR Werte für blau rot und grün schon auf 255 setzen 
bei White, damit er dann runterzählt?

Zum Zähler, sollte ich in der ISR dann ein if einbauen für den Zähler?