ATmega16 Problem

Dein Programm stürzt laufend ab, weil in TIMSK = 
(1<<OCIE1B)|(1<<OCIE1A); der Interrupt OCIE1B ohne die passende 
ISR(TIMER1_COMPB_vect) angeschaltet wird.

Zwei Dinge fallen zunächst auf:

1

        TIMSK=(1<<OCIE1B)|(1<<OCIE1A);

Wenn Du beide Interrupts freigibst, müssen auch beide 
Interruptroutinen vorhanden sein, sonst hast Du einen garantierten 
Aufhänger gebaut.  Es existiert in Deinem Code aber nur eine.

1

void xwait_10ms ()

2

{

3

int i=0;

4

   for(i=0;i<2000;i++);

5

}

Die Schleife wird wegoptimiert, da i nicht volatile ist.  Es ergibt sich 
keine Verzögerung von 10 ms, sondern von 0.  Nimm die Delay-Routinen der 
avr-libc.

Weiter habe ich nicht geschaut.

Die LED an PA0 soll also alle 1 s blinken. Das wäre wenn der Atmega16 
mit 11059000Hz läuft.

Wenn der AVR allerdings nur mit 1 MHz läuft (Werkseinstellung) ist die 
Blinkzeit ca. 11 s.

Hast du mal nach der Codekorrektur so lange die laufende Schaltung 
beobachtet? Funktioniert die Ansteuerung der LED im statischen Fall (um 
Fehler im Aufbau auszuschliessen)?

Tobias Neumann schrieb:

> ja also ich habe einen 11,059 Mhz Quarz laufen der den Takt angibt..

Quarz allein in die Schaltung bauen reicht nicht. Schau dir den Artikel 
AVR Fuses an.

> also ich habe die Schaltung konstant an und die LED geht wirklich nie an
> und aus sie leuchtet zu 100% konstant.

Wie lange beobachtet?

LED von Anfang an vermeintlich dauerleuchtend passt zu einer /active 
high/ angeschlossenen LED und obigem Code inkl. Taktratenproblem. Ist 
deine LED active high angeschlossen?

Tobias Neumann schrieb:
> Also ich hab den Quartz an XTAL1 und XTAL2 angeschlossen und der Quartz
> hat einen Takt von 11,059 Mhz, heißt das nicht das der Atmega16 dann mit
> dieser Taktung läuft? Entschuldige.. beschäftige mich erst seit einer
> Woche mit Microcontrollern :)

Reicht nicht siehe oben.

> Also ich habe es mind. 2 Min Beobachtet und an ist es seit Stunden.

Der Hinweis auf die notwendige Codeänderung kam doch erst vor ein paar 
Minuten. Hast du den Atmega16 noch nicht neu programmiert? Dann stürzt 
er weiterhin laufend ab.

Dann überprüf als erstes, ob die ISR überhaupt aufgerufen wird.

Leg die Lampenmanipulation im Hauptprogramm still und schalte in der ISR 
einfach nur eine Lampe ein (oder als Gegentest aus). Lass dein Programm 
laufen und sieh nach, ob das tatsächlich passiert. Dann weißt du schon 
mal, ob die ISR überhaupt aufgerufen wird oder nicht.

Zum Thema Quarz

Dieses Testprogramm

1

#define F_CPU 11059000

2

3

#include <avr/io.h>

4

#include <utils/delay.h>

5

6

int main()

7

{

8

  DDRA = 0xFF;

9

10

  while( 1 ) {

11

    PORTA = 0xFF;

12

    _delay_ms( 1000 );

13

    PORTA = 0x00;

14

    _delay_ms( 1000 );

15

  }

16

}

lässt deine Lampe blinken. Die Frage ist nur wie schnell?
Ist die LED 1 Sekunde an / 1 Sekunde aus, dann läuft dein µC mit 11Mhz 
und der Quarz ist aktiv.
Ist die LED 11 Sekunden an / 11 Sekunden aus, dann läuft dein µC mit 
1Mhz und der QUarz ist nicht aktiv.

Tobias Neumann schrieb:
> @ Karl Heinz
>
> Habe das Test Programm aufgespielt und die LEDs gehen etwa alle 11
> sekunden aus bzw an! Was soll ich nun tun? :)

Deinen µC so umstellen, dass er den Quarz auch benutzt

http://www.mikrocontroller.net/articles/AVR_Fuses#Taktquellen_Fuse_Einstellung

Tobias Neumann schrieb:
> @ Jo das ich meinen Quartz nicht benutze ist eine sehr hilfreiche
> Erkentniss, danke! Aber letztendlich löst es nicht mein Problem das die
> ISR nicht aufgerufen wird.. woran könnte das liegen?.. ich vermute
> schwer das es was mit dem übergebenen Argument von ISR zu tun hat!

? Ich denke, die LED blinkt jetzt..dann muss doch die ISR aufgerufen 
werden...

Tobias Neumann schrieb:
> @ Jo das ich meinen Quartz nicht benutze ist eine sehr hilfreiche
> Erkentniss, danke! Aber letztendlich löst es nicht mein Problem das die
> ISR nicht aufgerufen wird.. woran könnte das liegen?.. ich vermute
> schwer das es was mit dem übergebenen Argument von ISR zu tun hat!

Deine Vermutung ist falsch. An eine ISR werden keine Argumente 
übergeben.

Aber speck doch dein Programm mal etwas ab. In diesem langen 
Programmwust findet man doch nichts mehr

1

#include <avr/io.h>

2

#include <avr/interrupt.h>

3

#include <avr/wdt.h>

4

5

6

volatile uint8_t z;

7

volatile uint8_t stop = 1;

8

9

ISR(TIMER1_COMPA_vect)

10

{

11

  z++;

12

  if( z >= 40 )

13

  {

14

    stop ^= 1;

15

    z = 0;

16

  }

17

}

18

19

int main()

20

{

21

  DDRA = 0xFF;

22

23

  TIMSK  = (1<<OCIE1A);

24

  OCR1A  = 2500;

25

  TCCR1B = (1<<CS10)/*|(1<<CS12)*/|(1<<WGM12);

26

27

  sei();

28

29

  while(1) {

30

    if ( stop == 1 )

31

    {

32

      PORTA = 0x00;

33

    }

34

    else

35

    {

36

      PORTA = 0xFF;

37

    }

38

  }

39

}

Damit müsstest du schon ein Blinken sehen können

Tobias Neumann schrieb:
> Hm wenn ich dies versuche auf meinen MC zu brennen

Wenn du was auf deinen MC brennst.

Das zuletzt gepostete Programm kann es nicht sein, denn

> led.c:6:1: warning: "F_CPU" redefined
> <command line>:1:1: warning: this is the location of the previous
> definition

weder habe ich eine F_CPU Definition im Programmtext, noch ....

> warning: #warning "Compiler optimizations disabled; functions from
> <util/delay.h> won't work as designed"

wird <util/delay.h> includiert

Für mich ist nicht offensichtlich, was da schief läuft.

Häng vielleicht das led2.hex an deine nächste Frage an, damit man es 
Disassemblieren (Linksammlung: Disassembler) oder Simulieren kann.

Hast du selbst die Möglichkeit per (AVR-Simulation) das Programm im 
Einzelschritt zu debuggen?

Hier Dein Quellcode und die im Simulator des AVR Studio funktionierende 
HEX-Datei für Atmega16 @ 11.0592 MHz. Bei 1 MHz sind die Blinkzeiten 
natürlich nicht 1 s sondern 11.0592 s.