Code für eine Fußraumbeleuchtung im LKW

[Siro]

Hallo Live85
nimm mal das "return 0;" raus.
Das gehört da nicht rein.....
Ein Return brauchst Du nur in Unterfunktionen die vom Hauptprogramm oder woher auch immer aufgerufen werden.
Du hast ja nur eine einzige Schleife, wo soll er dann landen mit dem Return.

sorry, das return steht ja hinter der while Schleife, da stört es keinen, da kommt der code ja nie an.....
Da sind aber ein paar geschweifte Klammern zu viel, das hat mich verwirrt.
Du hast 2 mal { bei dem while und am Ende dann auch zwei mal } das tut nicht weh, ist aber unnötig und verwirrt.
Dein return am Ende darf dort stehen und hat oftmals auch seine Berechtigung. Das ist eigentlich für Programme zum Beispiel auf nem PC
da gibt man dann beim Verlassen des Programms eine Wert 0 zurück um zu signalisieren dass das Programm ordnungsgemäß
verlassen wurde oder auch ein Übergabewert für andere Programme (Der sogenannte Exitcode)
Dein Programm läuft aber ständig und sollte dort niemals ankommen.

Andreas hat recht, das b benutzt Du ja garnicht. Da liegt das Problem

Ein schönes Wochenende
Siro

[Bumbum]

nimm mal das "return 0;"

Das stimmt so nicht! main ist eine Funktion (vom Typ int) und in eine Funktion MUSS ein Return rein, sonst warnt der Compiler!
Man sollte nur dafür sorgen, dass der Ablauf dort nie hin kommt, was in einem sinnvoll strukturiertem Programm auch nicht der Fall sein wird.

[Siro]

Habe ich korrigiert Andreas, das hatte ich falsch interpretiert, ich dachte er springt aus der While Schleife.

[Live85]

Hallo, habe die Fehler beseitigt und jetzt geht alles wie gewollt, ich bin davon ausgegangen das OCR0A dann automatisch zu "b" wird da der wert "10" ist, habe dem ganzen wohl zuviel Logisches denken zugemutet und den absatz im AVR Buch vollkommen falsch interpretiert ...
Auf dem Ozi kann ich auch schön sehen wie sich das PWM Signal ändert....

Danke für die Tipps

  
 #include <avr/io.h>
 #define F_CPU 1200000UL  // 1,2 MHz
 #include <util/delay.h>



 int main (void)
 {
	 DDRB =(1<<PB0);                                                             // Ausgang PB0

	 TCCR0A=(1<<COM0A1) | (1<<WGM00) | (1<<WGM01);     // PWM Phase Korrigieren, OCR0A an TOP setzen
	 TCCR0B=_BV(CS01) ;                                                        // Prescaler 8

	 int a=5;                                                                            // Min Wert für Lichtstärke
	 int b=255                                                                         // Max Wert für Lichtstärke

         while (1) 
			 
{
		 OCR0A = a;                                                              // Startpunkt mit Min Lichtstärke
		                                 
		 if (PINB & (1<<PB3))                                                 // Prüfe auf High Signal an PB3
		
		 {
			if (a <255) a++;                                                // Wenn Wahr dann erhöhe in  5s auf den Wert "b"
			 _delay_ms(5);			 
		 }

		
		 else // (!(PINB & (1<<PB3)))                                     // Prüfe ob PB3 Low wenn Wahr zurück zu "a"
		 
		 {
			 if (a >10) a--;                                                    // Wenn solange der Wert Max gößer als Min dann a--
			 _delay_ms(5);	
		 }
}	 
	 return 0;

 }