Soft-PWM Version 3: Probleme beim Anpassen des Programms

[Andi M. (rootsquash)]

Hallo.

Ich versuche mich gerade am Soft-PWM-Beispiel (Version 3).

/******************************************************************/
// nur zum testen, in der Anwendung entfernen
/*
// Test values
volatile uint8_t tmp;
const uint8_t t1[8]={255, 40, 3, 17, 150, 99, 5, 9};
const uint8_t t2[8]={27, 40, 3, 0, 150, 99, 5, 9};
const uint8_t t3[8]={27, 40, 3, 17, 3, 99, 3, 0};
const uint8_t t4[8]={0, 0, 0, 0, 0, 0, 0, 0};
const uint8_t t5[8]={9, 1, 1, 1, 1, 1, 1, 1};
const uint8_t t6[8]={33, 33, 33, 33, 33, 33, 33, 33};
const uint8_t t7[8]={0, 0, 0, 0, 0, 0, 0, 88};
 
 
// Messung der Interruptdauer
    tmp =1;
    tmp =2;
    tmp =3;
 
// Debug 
 
    memcpy(pwm_setting, t1, 8);
    pwm_update();
 
    memcpy(pwm_setting, t2, 8);
    pwm_update();
 
    memcpy(pwm_setting, t3, 8);
    pwm_update();
 
    memcpy(pwm_setting, t4, 8);
    pwm_update();
 
    memcpy(pwm_setting, t5, 8);
    pwm_update();
    
    memcpy(pwm_setting, t6, 8);
    pwm_update();
    
    memcpy(pwm_setting, t7, 8);
    pwm_update();
*/
/******************************************************************/

setzt doch nur einmal zu Beginn die Ausgänge nacheinander auf die 
eingetragenen Werte.

Wenn ich jetzt irgendwelche zeitlichen Verläufe einprogrammieren will, 
z.B. einen einfachen Wechsel nach 2 Sekunden, sollte doch sowas helfen:

// globale Variablen

volatile uint8_t foo; pwm_durchlaeufe;    // ----Neue Variablen

 void countpwm(void) {        // ----Neue Funktion
   if (pwm_durchlaeufe == 200) {
     pwm_durchlaeufe = 0;
     if (foo == 1) {
       memcpy(pwm_setting, t6, 8);
       foo=0;
       pwm_update();
     }
     else {
       memcpy(pwm_setting, t5, 8);
       foo=1;
       pwm_update();
     }
   }
   else {
     pwm_durchlaeufe++;
   }
 }

int main(viod) {
.
.
.
const uint8_t t5[8]={0, 0, 0, 0, 0, 12, 200, 0};
const uint8_t t6[8]={0, 0, 0, 0, 0, 200, 12, 0};
memcpy(pwm_setting, t5, 8);    // Neue Startbedingungen
pwm_update();
foo = 0; pwm_durchlaeufe = 0;

Aber nach 2 Sekunden wirds einfach dunkel.
Sieht jemand woran das liegt?
Komplette main.c ist im Anhang.
Falls relevant:
Wenn ich die Werte für die PWM in den Startbedingungen ändere passiert 
genau was ich erwarte.
µC ist ein ATMega8 auf dem Pollin Eva-Board 2.0.1, Takt ist per Fuses 
auf 8 MHz gestellt.

Vielen Dank schomal.

[Andi M. (rootsquash)]

Ich vermute dass ich pwm_update(); nicht in der ISR aufrufen sollte, 
aber auch wenn ich einfach ein Flag setze und dann in der while-Schleife 
das Flag zurücksetze und pwm_update aufrufe klappt es nicht.

Vorsichtshalber habe ich mal den/die/das(?) Makefile angehangen.

[Andi M. (rootsquash)]

Da man ja nicht editieren kann, hier ein weiterer Post mit einem Update.
Keine Compiler-Warnings mehr, gleiches Problem.

[Andi M. (rootsquash)]

Schon wieder zu spät zum Editieren :-/

Habs hinbekommen. Ich habe keine Ahnung was noch falsch war, aber jetzt 
gehts.

Jetzt muss ich das noch auf mehr Kanälen machen und schnellere und 
kompliziertere Abläufe hinkriegen, mal gucken ob ich die Ideen aus 
http://www.mikrocontroller.net/articles/LED-Fading umsetzen kann.

[Andi M. (rootsquash)]

Hallo.

Ich habe mal weiter an dem Programm gestrickt und wieder ein Problem 
gefunden, diesmal beim Anpassen des Beispielprogramms: 
http://www.mikrocontroller.net/articles/Soft-PWM für mehr Kanäle (16 
statt 8).

main.c ist im Anhang.

Das Problem ist, dass die beiden LEDs an PD5 und PD6 (Atmel EVA Board 
2.0.1 von Pollin) jetzt zwar augenscheinlich ein Blinkmuster 
wiederholen, dieses aber nicht den vorgegebenen Werten entspricht. Es 
sieht eher nach wildem Blinken aus.
Mit 8 Kanälen funktioniert es

Ich würde mich wirklich über Hilfe freuen.

Grüße, Andi

[Andi M. (rootsquash)]

Aktueller Quelltext im Anhang.

#define PWM_PORT      PORTB              // Port für PWM
#define PWM_PORT2     PORTD              // Port für PWM
#define PWM_DDR       DDRB               // Datenrichtungsregister für PWM
#define PWM_DDR2      DDRD               // Datenrichtungsregister für PWM

funktioniert augenscheinlich,

#define PWM_PORT2      PORTB              // Port für PWM
#define PWM_PORT     PORTD              // Port für PWM
#define PWM_DDR2       DDRB               // Datenrichtungsregister für PWM
#define PWM_DDR      DDRD               // Datenrichtungsregister für PWM

blinkt wild.

Vorkommen im Quelltext:

        PWM_PORT = (uint8_t)(tmp & 0x00FF);                         // Ports setzen zu Begin der PWM
        PWM_PORT2 = (uint8_t)((tmp >> 8) & 0x00FF);                   // zusätzliche PWM-Ports hier setzen

        PWM_PORT &= (uint8_t)(tmp & 0x00FF);                        // Ports löschen
        PWM_PORT2 &= (uint8_t)((tmp >> 8) & 0x00FF);                   // zusätzliche PWM-Ports hier setzen

  PWM_DDR = 0xFF;  
  PWM_DDR2 = 0xFF;

Liegt es am Bit-Shift? Wenn ich das rausnehme blinkt er (oh Wunder) auch 
mit den sonst funktionierenden Definitionen.
Oder schreibt er irgendeinen Mist in die Variablen?
Ist der Code den ich da gestrickt habe so langsam dass er sich 
verheddert? (mit 30 FPS geht es auch nicht)
Passiert irgenwas komisches beim Sortieren wenn mehrere der Werte gleich 
sind? Eigentlich ist dafür ja Aufräum-Code vorhanden...

[Andi M. (rootsquash)]

uint16_t  pwm_setting[PWM_CHANNELS];           // Einstellungen für die einzelnen PWM-Kanäle
uint16_t  pwm_setting_tmp[PWM_CHANNELS+1];     // Einstellungen der PWM Werte, sortiert
                                              // ändern auf uint16_t für mehr als 8 Bit Auflösung

das sollten uint8_t sein, macht aber auch keinen Unterschied.

Ich verstehe jetzt einfach nicht, warum es nicht klappt:
- mit 8 Kanälen klappt es
- die Variablen deren Breite man ändern muss sind entsprechend 
beschriftet
- die Breite der betroffenen Variablen ist geändert
- die Stellen an denen man die weiteren Ports/Kanäle definieren/setzen 
muss sind beschriftet.
- der weitere Port/die weiteren Kanäle wurden hinzugefügt:

Andi M. schrieb:

>

#define PWM_PORT      PORTB              // Port für PWM
> #define PWM_PORT2     PORTD              // Port für PWM
> #define PWM_DDR       DDRB               // Datenrichtungsregister für
> PWM
> #define PWM_DDR2      DDRD               // Datenrichtungsregister für
> PWM

>
>

>         PWM_PORT = (uint8_t)(tmp & 0x00FF);                         //
> Ports setzen zu Begin der PWM
>         PWM_PORT2 = (uint8_t)((tmp >> 8) & 0x00FF);                   //
> zusätzliche PWM-Ports hier setzen
>

>

>         PWM_PORT &= (uint8_t)(tmp & 0x00FF);                        //
> Ports löschen
>         PWM_PORT2 &= (uint8_t)((tmp >> 8) & 0x00FF);
> // zusätzliche PWM-Ports hier setzen
>

>

>   PWM_DDR = 0xFF;
>   PWM_DDR2 = 0xFF;
>

Wenn ich PWM_PORT2 auf die LEDs ausgebe siehts OK aus, wenn ich PWM_PORT 
auf die LEDs ausgeben lasse blinken sie in einem Muster das dem 
Erwartungswert nicht ganz unähnlich ist.

Was kann denn da kaputt sein?