Problem mit PWM für RGB Streifen Atmega644

[Jan H. (cod1ac)]

Ich habe jetzt schon mehrere Tutorials für AVRs durchgearbeitet und 
wollte mich nun mal an eigenen Code wagen. Leider funktioniert mein 
Programm nicht ich vermute es ist ein simpler Fehler den ich aber leider 
nicht finden kann. es wäre schön wenn mir jemand helfen kann.

hier mein Programm:

#include <avr/io.h>
#include <util/delay.h>
#include <math.h>
#include <avr/pgmspace.h>

uint16_t pwmtable[32]  PROGMEM = {0, 1, 2, 2, 2, 3, 3, 4, 5, 6, 7, 8, 10, 11,
                                    13, 16, 19, 23, 27, 32, 38, 45, 54, 64, 76,
                                    91, 108, 128, 152, 181, 215, 255};


int main(void)
{
  DDRD = 0xFF; //Alles auf Ausgang
  DDRB = 0xFF; //Alles auf Ausgang
  PORTD = 0x00; 
  PORTD = 0x00;
  DDRA = 0x00;
  PORTA = 0x00;
  
  uint8_t valuered = 0;
  uint8_t valueblue = 0;
  uint8_t valuegreen = 0;
  uint8_t adcvalue = 0;
  uint8_t step = 1;
  uint8_t stepsize = 1;
  
  // PWM 0
  // 8-bit PWM, Phase correct
  // 8 bits give us 256 levels of brightness
  TCCR0A |= (1<<WGM00);

  // Clear OC0A/OC0B on Compare Match
  // Set OC0A/OC0B at BOTTOM (non-inverting mode)
  TCCR0A |= (1<<COM0A1);

  // Set prescaler to 8
  // 1 MHz / 8*256 = ~490 Hz PWM frequency
  TCCR0B |= (1<<CS01);
  

  // PWM 1

  // 8-bit PWM, Phase correct
  // 8 bits give us 256 levels of brightness
  TCCR1A |= (1<<WGM10);

  // Clear OC0A/OC0B on Compare Match
  // Set OC0A/OC0B at BOTTOM (non-inverting mode)
  TCCR1A |= (1<<COM1A1);

  // Set prescaler to 8
  // 1 MHz / 8*256 = ~490 Hz PWM frequency
  TCCR1B |= (1<<CS11);

  while(1)
  {

    if(adcvalue = 191)adcvalue=0;
    adcvalue++;
    
    if(adcvalue<32){
      valuered = 32;
      valueblue = 0;
      valuegreen = adcvalue;
    }
    if(31<adcvalue<64){
      valuered = 32 - adcvalue;
      valueblue = 0;
      valuegreen = 32;
    }
    if(63<adcvalue<96){
      valuered = 0;
      valueblue = adcvalue;
      valuegreen = 32;
    }
    if(95<adcvalue<128){
      valuered = 0;
      valueblue = 32;
      valuegreen = 32 - adcvalue;
    }
    if(127<adcvalue<160){
      valuered = adcvalue;
      valueblue = 32;
      valuegreen = 0;
    }
    if(159<adcvalue<192){
      valuered = 32;
      valueblue = 32 - adcvalue;
      valuegreen = 0;
    }
    
    OCR1A = pwmtable[valuered];
    OCR1B = pwmtable[valuegreen];
    OCR0A = pwmtable[valueblue];    
    _delay_ms(40);
  }

  return 0;
}

[holger (Gast)]

>Leider funktioniert mein
>Programm nicht ich vermute es ist ein simpler Fehler den ich aber leider
>nicht finden kann. es wäre schön wenn mir jemand helfen kann.

Wenn du mal sagen würdest was dein Problem ist, dann könnte
man vieleicht auch eine Antwort darauf geben.

[Jan H. (cod1ac)]

Ich möchte eine dreikanal hardware PWM realisieren für einen RGB strip 
für jede farbe ein Kanal. Aktuell möchte ich einfach nur alle farben 
einmal durchlaufen lassen. Dafür die vielen if Abfragen. Später möchte 
ich mit einem Poti über den ADC den Wert adcvalue ändern um mit dem Poti 
die farbe manuell einstellen können. aktuell durchläuft der adcvalue den 
bereich von 0 bis 192 und fängt wieder von vorne an. Leider bekomme ich 
an den PWM ausgängen keine messbare Spannung bzw. meine test Leds 
leuchten nicht ich kann ausschließen das ein fehler bei der beschaltung 
vorliegt. Ich vermute, dass ich einfach Register falsch gesetzt habe.

[holger (Gast)]

if(adcvalue = 191)adcvalue=0;
    adcvalue++;

Man kann adcvalue auch einfacher dauerhaft auf 1 setzen;)

[holger (Gast)]

Und wer Warnings ignoriert hat es echt nicht besser verdient.

[holger (Gast)]

>      valueblue = 32 - adcvalue;

Was kommt eigentlich bei 32 - 160 raus;)

[holger (Gast)]

>uint16_t pwmtable[32]  PROGMEM = {0, 1, 2, 2, 2, 3, 3

    OCR1A = pwmtable[valuered];

So holt man keinen Wert aus PROGMEM.

>ich vermute es ist ein simpler Fehler

Nö, das ganze Programm ist ein kompletter Fehler.

[Jan H. (cod1ac)]

Alles klar ich leg am besten nochmal von vorne los danke für die Hilfe. 
Werde wohl erstmal möglichst einfach anfangen und das Programm dann 
erweitern.