SoftPWM als funktion läuft nicht

[Tobias N. (silberkristall)]

Hallo,

ich wollte die hier im Tutorial befindliche SoftPWM in einer funktion 
laufen lassen, allerdings geht dann garnichts mehr.

So läuft sie:

int main(void)
{  
 
  uint8_t pwm_soll = 100; // gewünschter Dimmerwert 0..100
  uint8_t pwm_phase = 0; // Laufwert der Schleife 0..100
 
  // LED + Widerstand mit PB0 und +5V verbunden
  // PB0 o-----|<-----###------o Vcc 5V
  DDRB |= (1<<PB1); // Pin PB0 an Port B als Ausgang
  // LED ist bereits an
 
  while(1)
  {
    if(pwm_soll == pwm_phase)
    {
      PORTB &= ~(1<<PB1); // active low LED aus
    }
    pwm_phase++;
    if(pwm_phase == 100)
    {
      pwm_phase = 0;
      PORTB |= (1<<PB1); // active low LED an
    }
  }
}

Jetzt dachte ich mir ich erstelle mir eine funktion namens dim() die 
dann den dimwert also pwm_soll enthält, also z.b. dim(20);

int dim(int dimwert)
{

  uint8_t pwm_soll = dimwert; // gewünschter Dimmerwert 0..100
  uint8_t pwm_phase = 0; // Laufwert der Schleife 0..100

if(pwm_soll == pwm_phase)
    {
      PORTB &= ~(1<<PB1); // active low LED aus
    }
    pwm_phase++;
    if(pwm_phase == 100)
    {
      pwm_phase = 0;
      PORTB |= (1<<PB1); // active low LED an
    }

}

int main(void)
{  
 
  
  // LED + Widerstand mit PB0 und +5V verbunden
  // PB0 o-----|<-----###------o Vcc 5V
  DDRB |= (1<<PB1); // Pin PB0 an Port B als Ausgang
  // LED ist bereits an
 
  while(1)
  {
    dim(20);
  }
}

Allerdings leuchtet hier die LED nur garnicht mehr. Komisch ist auch, 
das wenn ich bei dem ersten Code beispiel an das ende der while schleife 
noch was einfüge die LED dann auch nicht mehr leutet, also ich verstehe 
das hier mal rein garnichtmehr. Lasse ich den Code und sonst nichts 
weiteres in einer while schleife laufen oder schreibe ich vor dem pwm 
code was in die while geht alles.

Schreibe ich nach dem pwm code was in die while dann geht der ganze code 
nicht mehr.

Ich hoffe man versteht was ich meine und man kann mir weiter helfen.

Gruß

Tobias

[Eumel (Gast)]

Wieso steht in den Kommentaren über all PB0 und im Code PB1?
Gewöhn dir mal gewissenhaftes Arbeiten an und liefer nicht so einen 
Pfusch ab.

[Oliver J. (skriptkiddy)]

Tobias N. schrieb:
> uint8_t pwm_phase = 0; // Laufwert der Schleife 0..100

Diese variable musst du als static deklarieren. Sonst wird sie bei jedem 
Funktionsaufruf neu auf dem Stack erzeugt und mit Null beschreiben.

Gruß Oliver

[Michael (Gast)]

Wie soll die PWM laufen, wenn du pwm_phase immer wieder auf 0 setzt?

[Tobias N. (silberkristall)]

Hey, danke jetzt läufts, aber mit einem kleinen Fehler, ich weiss aber 
nicht warum,

hier mal der Code:

#include <avr/io.h>
static uint8_t pwm_phase = 100;

int dim(int dimwert, char pwmport)
{
  int pwm_soll = dimwert;

  if(pwm_soll == pwm_phase)
    {
      PORTB &= ~(1<<pwmport);
    }
    pwm_phase++;
    if(pwm_phase == 100)
    {
      pwm_phase = 0;
      PORTB |= (1<<pwmport);
    }    
  
  return 1;
}

int aufdimmen(int dimtime, char pwmport)
{
  int whiletime=0;
  int dimmingtime=0;
  int dimval=0;
  
  while(1)
  {
    dim(dimval, pwmport);
    dimmingtime++;
    if (dimmingtime == dimtime)
      {
        whiletime++;
        dimval++;
        dimmingtime = 0;
      }
    if (whiletime == 100) { return 1; break; }
  }
}

int abdimmen(int dimtime, char pwmport)
{
  int whiletime=0;
  int dimmingtime=0;
  int dimval=100;
  
  while(1)
  {
    dim(dimval, pwmport);
    dimmingtime++;
    if (dimmingtime == dimtime)
      {
        whiletime++;
        dimval--;
        dimmingtime = 0;
      }
    if (whiletime == 100) { return 2; break; }
  }
}

int main(void)
{  

 DDRB |= (1<<PB1);

int ledstatus = 1;

  while(1)
  {
  
  if (ledstatus == 2)
    {
      ledstatus = aufdimmen(1000, PB1);
    }
    else
    {
      ledstatus = abdimmen(1000, PB1);
    }
  }
}

Also das aufdimmen funktioniert perfekt, beim abdimmen, etwa bei der 
hälfte des dimmvorgangs blitzt die LED einmal kurz auf. Der Code des 
auf-, sowie abdimmens ist ja gleich deswegen verstehe ich das nicht. Es 
gibt auch sicherlich eine elegantere Lösung aber ich bin froh das sie so 
läuft wie es soll, bis auf diesen kleinen Fehler.

Danke euch.