PWM an PB1(OC1A) nicht veränderbar

[Kenia (Gast)]

Hallo habe folgendes Prgrammm gestestet
an PB1(OC1A) soll der Ausgang sein
an PB0+PB2 hängen Taster nach GND.

Am Ausgang kommt zwar ein ca. 13kHz Signal aber ich kann keine Regelung 
vornehmen wo hängts.....?

Alles was mir auffällt kann nur OCR1A nutzen da compiler bei OC1A 
meckert.

hier der code :

#include <avr/io.h>
#include <avr/interrupt.h>
#define F_CPU 8000000UL
#include <util/delay.h>



int main()
{
  // OC1A => PB1 Ausgang
  DDRB |= (1<<PORTB1);  //Ausgang   DDRB &= ~(1<<PORTB0);  //  Eingang Tater auf GND
  DDRB &= ~(1<<PORTB2);  //  Eingang Taster auf GND
  PORTB |= (1<<PORTB0) | (1<<PORTB2); // an S1 und S3 Taster PUL UP Widerstände
    
  // PWM Konfig
  TCCR1A |= (1<<WGM10); //fast PWM 8-Bit
  TCCR1B |= (1<<WGM12);
  TCCR1B |= (1<<CS12);  // Prescaler 256
  TCCR1A |= (1<<COM1A1);  //nichtinvertierender PWM Modus
  OCR1A = 0; //LED aus;
  sei();
  
  while(1)   
  {  // ******* Tatster 1 (LED heller) ************
    if (!(PINB & (PINB0)))    
    {
      if(OCR1A= 255)  // wenn Maxwert noch nicht erreicht 
      OCR1A++;    // verbreitere die Impulsbreite
      _delay_ms(15);
    }
    
    //******* TASTER 2 (LED dunkler) *************
    if (!(PINB & (PINB2)))
    {
      if(OCR1A > 0)  // wenn Minimalwert noch nicht erreicht
      OCR1A--;    // verbreitere die Impulsbreite
      _delay_ms(15);
    }
  }   
}

[Dietrich L. (dietrichl)]

Kenia schrieb:
> sei();

Wozu das? Hast Du einen Interrupt? Dann fehlt die ISR.

Um welchen µC geht es eigentlich?

Gruß Dietrich

[Kenia (Gast)]

Atmega 168PA

[Karl H. (kbuchegg)]

> if(OCR1A= 255)

Das ist kein Vergleich sondern eine Zuweisung.
Ausserdem bezweifle ich, dass du an dieser Stelle einen Vergleich auf 
Gleichheit wolltest.

[Karl H. (kbuchegg)]

>    if (!(PINB & (PINB0)))

    if (!(PINB & (1<<PINB0)))


Das steckt doch alles voller Primtivfehler.
Dürfte ich vorschlagen, dass du dir selber mal eine Strategie 
zurechtlegst, um Fehler zu suchen und zu finden anstatt einfach 
dahinzutippen und wenns nicht geht, dann sofort das Forum zu befragen?

Fehler zu machen ist ok. Das gehört dazu.
Aber genauso gehört es dazu, dass man lernt seine eigenen Fehler zu 
finden. Und das lernt man nicht ohne Übung oder wenn einem die anderen 
alles vorbeten.
Beides sind Fehler, die du eigentlich längst selbst hättest finden 
müssen.

[Kenia (Gast)]

Manchmal ist es eben so dass man die Kleinigkeiten nicht mehr sieht nach 
langer Prorammtests habe noch andere Tests laufen, das war nur ein Teil.


ABER SUPER DANKE TROTZDEM dafür ist doch auch ein Forum da oder ?

[Kenia (Gast)]

Karl Heinz Buchegger schrieb:
>>    if (!(PINB & (PINB0)))
>
>     if (!(PINB & (1<<PINB0)))
>
>
> Das steckt doch alles voller Primtivfehler.


Hab ich korrigiert , keine Funktion !!!!

An was könnte es denn liegen ???

[Karl H. (kbuchegg)]

Was ist mit dem

> if(OCR1A= 255)

wie hast du ihn korrigiert?


(ehe ich mir die Arbeit antue und die Timer Konfiguration kontrolliere)

[Kenia (Gast)]

Was soll damit sein

[Karl H. (kbuchegg)]

Ich zitiere

> Das ist kein Vergleich sondern eine Zuweisung.
> Ausserdem bezweifle ich, dass du an dieser Stelle einen Vergleich auf
> Gleichheit wolltest.

[Kenia (Gast)]

OK gewollt war

if(OCR1A < 255)  // wenn Maxwert noch nicht erreicht

funktioniert immer noch nicht!!!

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Kenia schrieb:
> if(OCR1A= 255)  // wenn Maxwert noch nicht erreicht

Zumal das ja auch noch falsch ist, wenn man mal den Komentar ernster 
nimmt als die Programmzeile :-)

Ich denke mal, da soll eigentlich

 if (OCR1A < 255)  // wenn Maxwert noch nicht erreicht

stehen. Karl Heinz hat ja schon alles gesagt. Compilerwarnungen sind 
auch dazu da, sie zu lesen:
Warning: probably unwanted assignment...

Kenia schrieb:
> Karl Heinz Buchegger schrieb:
>>>    if (!(PINB & (PINB0)))
>>
>>     if (!(PINB & (1<<PINB0)))
>>
>>
>> Das steckt doch alles voller Primtivfehler.
>
>
> Hab ich korrigiert , keine Funktion !!!!

Das andere für PINB2 auch?

[Kenia (Gast)]

Alles korrigiert, geht aber immer noch nicht sorry

#include <avr/io.h>
#include <avr/interrupt.h>
#define F_CPU 8000000UL
#include <util/delay.h>



int main()
{
  // OCA1 => PB1 Ausgang
  DDRB |= (1<<OCR1A);    //Ausgang
  DDRB &= ~(1<<PORTB0);  //  Eingang Tater auf GND
  DDRB &= ~(1<<PORTB2);  //  Eingang Taster auf GND
  PORTB |= (1<<PORTB0) | (1<<PORTB2); // an S1 und S3 Taster PUL UP 
Widerstände

  // PWM Konfig
  TCCR1A |= (1<<WGM10); //fast PWM 8-Bit
  TCCR1B |= (1<<WGM12);
  TCCR1B |= (1<<CS12);  // Prescaler 256
  TCCR1A |= (1<<COM1A1);  //nichtinvertierender PWM Modus
  OCR1A = 0; //LED aus;
//  sei();

  while(1)
  {  // ******* Tatster 1 (LED heller) ************
    if (!(PINB & (1<<PINB0)))
    {
      if(OCR1A < 255)  // wenn Maxwert noch nicht erreicht
      OCR1A++;    // verbreitere die Impulsbreite
      _delay_ms(15);
    }

    //******* TASTER 2 (LED dunkler) *************
    if (!(PINB & (1<<PINB2)))
    {
      if(OCR1A > 0)  // wenn Minimalwert noch nicht erreicht
      OCR1A--;    // verbreitere die Impulsbreite
      _delay_ms(15);
    }
  }
}

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Kenia schrieb:
> DDRB |= (1<<OCR1A);    //Ausgang

OCR1A ist ein Register und kein Pin.

[Falk B. (falk)]

@  Kenia (Gast)

Eher so.

>  // OCA1 => PB1 Ausgang
>  DDRB |= (1<<PB1);    //Ausgang

[Kenia (Gast)]

SUUUPER vielen Dank jetzt gehts dank den PROFIS

[Kenia (Gast)]

Hier die finale PWM - Taster Version für jeden