Problem mit Schleife für LED-Fading

[Marcel (Gast)]

Hallo liebe Gemeinde,

ich hab ein klitzekleines Problem mit einer Routine für ein LED-Fading 
und hoffe, mir kann hier jemand einen Rat dazu geben.

Hier der Ausschnitt:

#define nBtnModeTimeout 60      // Timeout-Wert für das 'key_press'-Ereignis
#define nFadingTime 200          // Ein-/Ausblendzeit(in ~ms)
#define DIMM_TIMEOUT 10       // Zählerschritte(keine s!), bevor von 100% wieder heruntergedimmt wird

int16_t bRelaisSignal = 0;
int32_t nRelaisCount = 0;
uint8_t nBtnMode = 0;            // Programm-Modus des Tasters
uint8_t nBtnModeCount = 0;       // Zähler für das 'key_press'-Ereignis
int16_t nDimmCount = 0;         // Zähler für 
uint8_t bDimmDirection = 1;      // Dimmungrichtung(1:EIN;0:AUS)
uint8_t bFadingSignal = 0;
uint8_t nFadingCount = 0;        // Zähler für die Ein-/Ausblendfunktion(Fading)
uint8_t nSavedPWM = 255;          // Variable zur Speicherung des letzten PWM- bzw. Helligkeitswertes

ISR(TIMER1_OVF_vect)
{
  /***************************************************************************************************/
  if(nBtnMode != 0) // sobald irgendein Programm-Modus(s. Tasterabfragen) ausgewählt wurde,...
  {
    nBtnModeCount++; // ... Start des Zählers für die 'key'-Events(Timeout)

    if(KEY_PIN & (1<<KEY1))
    {
      if(nBtnModeCount > nBtnModeTimeout && nBtnModeCount < nBtnModeTimeout*3)
      {
        switch(nBtnMode)
        {
          /*---------------------------------------------------------------------------------*/
          case(1): dataCAN1 = 50;
               dataCAN2 = 0;
               dataCAN3 = 0;
               break;
          /*---------------------------------------------------------------------------------*/
          case(2): dataCAN1 = 0;
               dataCAN2 = 0;

               if(dataCAN3 < 2) //(!(PINB & (1<<PINB1)))
                 bFadingSignal = 1; // Signal:FadeIN
               else
                 bFadingSignal = 2; // Signal:FadeOUT
               break;
          /*---------------------------------------------------------------------------------*/
        }
        sendCAN();       // Senden der neuen PWM-Werte über das CAN-Netzwerk
        nBtnMode = 0;      // Zurücksetzung des Programm-Modus
        nBtnModeCount = 0; // Zurücksetzung des Zählers
      }
      if(nBtnModeCount >= nBtnModeTimeout*3)
      {
        nBtnMode = 0;      // Zurücksetzung des Programm-Modus
        nBtnModeCount = 0; // Zurücksetzung des Zählers
      }
    }
  }
  /***************************************************************************************************/
  if(bFadingSignal != 0)
  {
    nFadingCount++;
    
    if(nFadingCount <= nFadingTime)
    {
      if(bFadingSignal == 1) // bFadingSignal:1=FadeIN,2=FadeOUT
        // da 'nFadingCount*(nSavedPWM/nFadingTime)' nur int-Werte verarbeiten kann, wird
        // hier durch Benutzung einer temporären größeren 32Bit-Variable UND der
        // LEFT-Shift-Operation in einem 'Vielfachen' der Ausgangswerte weitergerechnet
        dataCAN3 = (nFadingCount*(((int32_t)nSavedPWM << 16)/nFadingTime)) >> 16;
      else
        // mit 'nSavedPWM-Zählerfaktor(s.o.)' wird hier der Zähler rückwärts ausgewertet
        dataCAN3 = nSavedPWM-((nFadingCount*(((int32_t)nSavedPWM << 16)/nFadingTime)) >> 16);

      sendCAN();
    }
    else
    {
      bFadingSignal = 0;
      nFadingCount = 0;
      
      USART_SEND_BYTE(dataCAN3);
      //dataCAN3 = 0;
    }
  }
}

Das Problem liegt eigentlich nur darin, dass die Routine zwar bestens 
arbeitet, aber der Fading-Endwert jeweils immer um eins höher bzw. 
niedriger als der gespeicherte Soll-Wert ist. Also bei FadeIN=254 statt 
255 und bei FadeOUT=1 statt 0.
Ich komme einfach nicht auf den Fehler:(.

Liebe Grüße

[Hartmut W. (killozap)]

Die Variable nFadingCount wird am Anfang der Fading-Auswertung 
inkrementiert. Sie ist also immer mindestens 1.

 Ich vermute, dass wenn man das Inkrement erst nach der Auswertung 
durchführt das Problem behoben ist.

[Marcel (Gast)]

Hallo Hartmut,

danke für die schnelle Antwort. Leider geht es aber trotzdem nicht. Sie 
bleibt weiterhin bei 1 stehen.:(

Lg

[Marcel (Gast)]

Ps.:

Hab's jetzt so gelöst, dass ich jeweils einfach den fehlenden Einer-Wert 
mit in die Gleichungen reinrechne:

dataCAN3 = ((nFadingCount*(((int32_t)nSavedPWM3 << 16)/nFadingTime)) >> 
16)+1;

dataCAN3 = (nSavedPWM3-((nFadingCount*(((int32_t)nSavedPWM3 << 
16)/nFadingTime)) >> 16))-1;

Liebe Grüße