Soft PWM in C für PIC optimieren

[Gast (Gast)]

Kann man hier noch optimieren? Wird mit PICC-Lite compiliert und soll 
bei 4 MHz mit 80*256 Hz laufen..

Irgendwie schafft es aber nur 60 Hz obwohl die Simulation 
durchschnittlich nur 33 Zyklen (=33 µs * 256 -> 118 Hz) braucht.

Oder liegt es am toggle:  60 = 118 / 2 ?

void interrupt isr(void)
{
  if(T0IF)
  {
    TMR0 = RELOAD;
    gTick++;
    if(gTick == 0)
    {
      if(gRed) 
        RED = ON;
      if(gGreen)
        GREEN = ON;
      if(gBlue)
        BLUE = ON;
    }
    else
    {
      if(gTick > gRed)
        RED = OFF;
      if(gTick > gGreen)
        GREEN = OFF;
      if(gTick > gBlue)
        BLUE = OFF;
    }
    if(gDelay > 0) gDelay--;
    T0IF = 0;
  }
}

[C. H. (_ch_)]

Hallo?

jetzt wäre noch interessant:
- Timer-prescaler?
- Interrupt-Post-Scaler?
- Auflösung gRed, gGreen, gBlue

Gruß
Christian

[Chris (Gast)]

80*256 Hz  sollten kein Problem darstellen, aber hier fehlen die ganzen 
Typen, die Initialisatzion des TMR0 incl Prescaler (OPTION).

Hier ein paar kleinigkeiten, wegen Optimierung.

void interrupt isr(void)
{
  if(T0IF)
  {
    TMR0 = RELOAD;
    if(!++gTick)
    {
 // Wenn RED ein flag (bit) ist, wäre  RED=gRed; effizienter.
     if(gRed)
        RED = ON;
      if(gGreen)
        GREEN = ON;
      if(gBlue)
        BLUE = ON;
    }
    else
    {
      if(gTick > gRed)
        RED = OFF;
      if(gTick > gGreen)
        GREEN = OFF;
      if(gTick > gBlue)
        BLUE = OFF;
    }
// wenn gDelay ein unsigned ist, dann ist if(gDelay) gDelay--; 
effizienter
    if(gDelay > 0) gDelay--;
    T0IF = 0;
  }
}

[Gast (Gast)]

Auflösung 8 Bit, kein Prescaler.

[Gast (Gast)]

Hier die Variablen

unsigned char gRed = 0;
unsigned char gGreen = 0;
unsigned char gBlue = 0;
volatile unsigned char gTick = 0;
volatile unsigned char gDelay = 0;

> if(!++gTick)
Ok.

> if(gDelay > 0)
Das hätte ich sehen müssen ;-)

> Wenn RED ein flag (bit) ist, wäre  RED=gRed; effizienter.
Ist ein GPIO bit

static volatile bit  GPIO0  @ (unsigned)&GPIO*8+0;

aber das funktioniert nicht (es wird nie gesetzt).

Die beiden anderen Änderungen wirken sich nicht auf die Codegröße aus, 
aber vielleicht spart es ja irgendwelche Spünge..

Danke soweit.
Fehlt noch eine Tastenentprellung und ein 2 kHz Rechteck.. :-)

[Gast (Gast)]

Da ist irgendwie eine Zeile verschwunden:

Die beiden Sachen ändern nichts an der Codegröße,

[Chris (Gast)]

> Wenn RED ein flag (bit) ist, wäre  RED=gRed; effizienter.
Ist ein GPIO bitstatic volatile bit  GPIO0  @ (unsigned)&GPIO*8+0;

aber das funktioniert nicht (es wird nie gesetzt).

In diesem Falle ist RED eine bitvariable, also
 REG=gRed ist besser.

Poste mal den ganzen code, denn auch wenn der Code doppels so groß wäre,
müsstest du die Frequenz locker schaffen.