1te PWM funktioniert nicht

[dennis g. (Firma: schule) (dnns)]

Nabend zusammen.
Arbeite z.z. an einer Temperatur Reglung.
PI Regler
2 PWM gesteuerte Ausgänge für Heizung und Lüftung.
Die Zweite PWM funktioniert.
Doch bei der Ersten PWM kommt nichts am Ausgang an.
Könntet ihr mir vielleicht helfen?

float pi_regler(float temp_sensor_value,float temperature)
{

DDRB = (1 << PB1 );        // PB1 als Ausgang für PWM
DDRB = (1 << PB2 );        // PB3 als Ausgang für PWM

//Einstellungen für WGM-Mode 5 (Fast PWM 8-Bit)
//Output Compare mode ist nicht invertierender Modus
TCCR1A = 0b10100001;                  //COM1A1|B1=1 - COM1A0|B0=0 - WGM11=0,WGM10=1
TCCR1B = 0b00001011;                 //WGM13=0 - WGM12=1 - Vorteiler=64

//TCCR2 = 0b01001000;

//TCCR1A = (1<<WGM10)|(1<<COM1A1);       //1<<WGM10 --> 8 Bit PWM //1<<COM1A1 Nicht invertierende PWM 
//TCCR1B = (1<<CS12) | (1<<CS10);       // Takt von CK / 1024


float iwert=0,pwert=0,e=0,y=0;                       //I-Anteil                       //P-anteil
float ki= 1.5;                             //integrations beiwert(verstärkung der itegration)                                //Regeldifferenz                                    //Stellgröße
float kp = 0.5;                            //Proportionalanteil 
float ta = 0.01;                           //Abtastzeit 10ms
float esum=0;

//temp_sensor_value = Adwandlung();
e = temperature - temp_sensor_value;       //aktuelle Regelabweichung bestimmen
if (e>50)                           //e Begrenzen auf +50°
{
e=50;
}
if (e<-50)                                  //e begrenzen auf -50°
{
e=-50;
}
pwert=(kp*e);                        //P-Anteil ausrechnen
iwert=(ki*ta*esum);                      //I-Anteil ausrechen
y=pwert+iwert;                        //Stellgröße ausrechnen
if ((y<50)&&(y>0))                      //Integration nur in diesen bereich ausrechen
{                                  //Wenn kein abweichnung da ist Inegration Einfrieren
esum=esum+e;                                //Integration I anteil
}
if ((y>-50)&&(y<0))                         //Wenn stellgröße negativ ist PRÜFEEENNNN!!!!!!!!!!!!!!
{
esum=esum-e;
}
if (y==0)                                   //wenn die stellgröße gleich 0 ist           
{
esum=0;                              //Setz Integration zurück 
}
if (e>0)                                    //wenn e größer 0 ist muss die heizung angehen
{
OCR1AL = y;                                  //vergleichwert für compare match y
//OCR2 = 0;                                 //wenn heizung an soll der lütfter aus sein
OCR1BL=0;
}
if (e<0)                                    //wenn e kleiner 0 ist muss die lüftung angehen
{
//OCR2 = -y ;                         //vergleichwert für compare match y
OCR1BL=-y;
OCR1AL = 0;                                 //wenn lüfter an soll die heizung aus sein
}

//
return (esum);
}

[Hartmut G. (Gast)]

dennis go schrieb:
> DDRB = (1 << PB1 );        // PB1 als Ausgang für PWM
> DDRB = (1 << PB2 );        // PB3 als Ausgang für PWM

Ich weiß zwar nicht, welchen AVR du verwendest, aber hier widerspricht 
sich Code und Kommentar. Richtiger Pin wirklich als Ausgang 
konfiguriert?

[dennis g. (Firma: schule) (dnns)]

Hartmut G. schrieb:
> dennis go schrieb:
>> DDRB = (1 << PB1 );        // PB1 als Ausgang für PWM
>> DDRB = (1 << PB2 );        // PB3 als Ausgang für PWM
>
> Ich weiß zwar nicht, welchen AVR du verwendest, aber hier widerspricht
> sich Code und Kommentar. Richtiger Pin wirklich als Ausgang
> konfiguriert?

Oha habe heute soviel hin und her probiert
PB2 ist konfiguriert

ATmega 8
avr gcc

[spess53 (Gast)]

Hi

>DDRB = (1 << PB1 );        // PB1 als Ausgang für PWM
>DDRB = (1 << PB2 );        // PB3 als Ausgang für PWM

Dir ist bewusst, das die zweite Zeile die Wirkung der ersten zunichte 
macht?

MfG Spess

[dennis g. (Firma: schule) (dnns)]

hmm nein ist es nicht.
also sollte ich PB3 nehmen ?

[Hartmut G. (Gast)]

dennis go schrieb:
> hmm nein ist es nicht.
> also sollte ich PB3 nehmen ?

Schau ins Datenblatt, welchem Pin der jeweilige Compare Match des Timers 
zugeordnet ist und setze den dann als Ausgang, und zwar wie spess53 
richtig erkannt hat:

Nicht so:
>DDRB = (1 << PB1 );        // PB1 als Ausgang für PWM
>DDRB = (1 << PB3 );        // PB3 als Ausgang für PWM

Sondern so:
>DDRB |= (1 << PB1 );        // PB1 als Ausgang für PWM
>DDRB |= (1 << PB3 );        // PB3 als Ausgang für PWM

Das | ist wichtig...

[Micha K. (Gast)]

Hier ist auch erklärt warum das so ist und wie es funktioniert:
http://www.mikrocontroller.net/articles/Bitmanipulation

[dennis g. (Firma: schule) (dnns)]

das wars vielen dank für die antwort

[Loonix (Gast)]

Hartmut G. schrieb:
> Nicht so:
>>DDRB = (1 << PB1 );        // PB1 als Ausgang für PWM
>>DDRB = (1 << PB3 );        // PB3 als Ausgang für PWM
>
> Sondern so:
>>DDRB |= (1 << PB1 );        // PB1 als Ausgang für PWM
>>DDRB |= (1 << PB3 );        // PB3 als Ausgang für PWM

...oder gleich so:

DDRB |= (1<<PB1)|(1<<PB3);

[Helfer (Gast)]

dennis go schrieb:
> das wars vielen dank für die antwort

Im Simulator sieht man sofort, dass die Datenrichtung für den ersten PWM 
nicht richtig gesetzt ist.

[dennis g. (Firma: schule) (dnns)]

Habe noch eine kleine frage.
Ich muss die Reglung auch Präsentieren.
Könntet ihr mir einen Programm empfehlen womit man die Regelkurve 
anzeigen kann!?

Danke

[Markus O. (pipimaxi)]

Matlab, Excel, OpenOffice, such dir was aus ;)