Codefrage für WS2801 RGB-Lichterkette

[Blinker (Gast)]

Hallo zusammen,

da nächste Woche bei uns in der Stadt ein Lichterfest ist habe ich vor 
eine Lichterkette mit veränderbaren Farben zu bauen.
Dazu habe ich eine Lichterkette mit 50 RGB LEDS und WS2801 Controller.
Zum testen habe ich ein CT-COM-auf-LAN Board also ein ATmega8535 mit 
einem 14.7456 Mhz Quarz. Der Code soll erstmal alle LEDs einschalten und 
dann wieder aus in einer schleife um zu sehen ob er funktioniert - so 
der Plan. Das macht er auch irgendwie Blinken alle Leds im Sekunden Takt 
nur alle mit einer anderen Farbe und die Farbe wechselt auch mit jedem 
Blinken.
Woran kann das liegen, an dem _delay_ms?

Ich habe folgenden Code verwendet:
(Quelle:Beispielcode aus der Codesammlung)

#define F_CPU 14745600UL /* Quarz mit 14.7456 Mhz */
#include <avr/io.h>
#include <util/delay.h>
#define  CLOCK_DDR  DDRA
#define  CLOCK_PORT  PORTA
#define  CLOCK_PIN  PA3

#define  DATA_DDR  DDRA
#define  DATA_PORT  PORTA
#define  DATA_PIN  PA2
void ws2801_init()
{
  // Init Clock Pin
  CLOCK_DDR   |= (1<<CLOCK_PIN);
  CLOCK_PORT   &= ~(1<<CLOCK_PIN);
  
  // Init Data Pin
  DATA_DDR   |= (1<<DATA_PIN);
  DATA_PORT   &= ~(1<<DATA_PIN);
}
void long_delay(uint16_t ms) {
  for(; ms>0; ms--) _delay_ms(1);
}
void ws2801_sendRGB(uint8_t* rgb, uint16_t num)
{
  // Start transmission
  for(uint8_t i=0; i<(num*3); i++) // 3 Byte per LED
  {
    for(uint8_t bit=0x80; bit; bit >>= 1)
    {
      if(rgb[i] & bit)
      DATA_PORT |= (1<<DATA_PIN);
      else
      DATA_PORT &= ~(1<<DATA_PIN);

      CLOCK_PORT |= (1<<CLOCK_PIN);
      CLOCK_PORT &= ~(1<<CLOCK_PIN);
    }
  }
  
  // Send 1ms both pins low
  DATA_PORT   &= ~(1<<DATA_PIN);
  CLOCK_PORT   &= ~(1<<CLOCK_PIN);
  _delay_ms(1);
}

int main()
{
  ws2801_init();
  while(1){
  uint8_t color[3];
  color[0] = 255;
  color[1] = 255;
  color[2] = 255;

  ws2801_sendRGB(color, 50);
  long_delay(1000);
  uint8_t color2[3];
  color2[0] = 0;
  color2[1] = 0;
  color2[2] = 0;

  ws2801_sendRGB(color2, 50);
  long_delay(1000);

}

  return 0;
}

[Marc V. (Firma: Vescomp) (logarithmus)]

Blinker schrieb:
>       CLOCK_PORT |= (1<<CLOCK_PIN);
>       CLOCK_PORT &= ~(1<<CLOCK_PIN);
>     }
>   }
>
>   // Send 1ms both pins low
>   DATA_PORT   &= ~(1<<DATA_PIN);
>   CLOCK_PORT   &= ~(1<<CLOCK_PIN);
>   _delay_ms(1);

 Vielleicht mal zur Sicherheit andersherum probieren ?
        CLOCK_PORT &= ~(1<<CLOCK_PIN);
    //****** kurzes Delay hier (paar CPU-Takte) ?
        CLOCK_PORT |= (1<<CLOCK_PIN);
    //****** und hier ?
 // Und dann beide auf Null
        DATA_PORT   &= ~(1<<DATA_PIN);
        CLOCK_PORT   &= ~(1<<CLOCK_PIN);
 // Nächster durchlauf
     }
 // Nächstes Byte
   }
 // Send 1ms both pins low
  DATA_PORT   &= ~(1<<DATA_PIN);
   CLOCK_PORT   &= ~(1<<CLOCK_PIN);
   _delay_ms(1);

 Auch wenn es unnötig erscheint, man sollte beim Fehler lieber
 etwas zweimal machen als einmal zu wenig. Erst mal zum Laufen
 bringen, dann kann die Geschwindigkeit erhöht werden.

[Blinker (Gast)]

Hallo  Marc,

Das wars!

Vielen Dank für den Tipp.
Das Beispiel war von jemanden der nur eine LED angesteuert hat da ist es 
egal.