Atmega48 - UART als Interrupt

Julian W. schrieb:
> ISR(USART_RX0_vect )
Der Vektor heisst USART_RX_vect. Compilerwarnungen beachten!

mfg.

Julian W. schrieb:

> funktioniert auch tadelos. Nur sobald jetzt ein Zeichen empfangen wird
> passiert nichts und der Atmega48 stürzt einfach ab!?

Wenn Interrupts im Spiel sind und der µC bei Auftreten eines INterrupts 
abschmiert, dann stimmt meistens der Name der ISR nicht.
D.h. da wurde an den Interrupt KEINE ISR gebunden. Und sobald dieser 
Fall eintritt, tritt der Default-Handler auf die Bühne, welcher .... den 
µC resettet.

Also: Überprüf den Namen deiner ISR

Okay, hab ich mal geändert... aber darüber hat der Compiler eigetnlich 
nie gemekert.
Einzigst was noch immer kommt:

main.c|67|warning: control reaches end of non-void function|

Julian W. schrieb:
> Okay, hab ich mal geändert... aber darüber hat der Compiler eigetnlich
> nie gemekert.
> Einzigst was noch immer kommt:
>
> main.c|67|warning: control reaches end of non-void function|

Schön.
UNd welches ist die Zeile 67?

Das hier:

1

ISR(USART_RX_vect)

2

{

3

    uint8_t tmp = UDR0;

4

    LightOn(LED_RESET);

5

}

Karl Heinz Buchegger schrieb:
> Julian W. schrieb:
>> Okay, hab ich mal geändert... aber darüber hat der Compiler eigetnlich
>> nie gemekert.
>> Einzigst was noch immer kommt:
>>
>> main.c|67|warning: control reaches end of non-void function|
>
> Schön.
> UNd welches ist die Zeile 67?

Das Ende der main wahrscheinlich. Die sollte int zurückgeben. Macht er 
aber nicht. Macht wohl nur return; oder gar nix.
Ist aber auch für das Problem hier nicht relevant.

gruß cyblord

../usart0.c:119:1: warning: 'USART_RX0_vect' appears to be a misspelled 
signal handler

Julian W. schrieb:
> aber darüber hat der Compiler eigetnlich nie gemekert.
Wie dem auch sei.

Karl Heinz Buchegger schrieb:
> Schön. UNd welches ist die Zeile 67?
Mach doch mal die Augen auf:

Julian W. schrieb:
> [...]

Julian W. schrieb:
> control reaches end of non-void function
Da wurde der "Point of no return" erreicht.

cyblord ---- schrieb:
> Ist aber auch für das Problem hier nicht relevant.
Ist dem GCC aber auch egal.

mfg.

Julian W. schrieb:
> Das hier:
>
>

1

> ISR(USART_RX_vect)

2

> {

3

>     uint8_t tmp = UDR0;

4

>     LightOn(LED_RESET);

5

> }

6

>

Womit wir wieder beim Punkt angelangt sind, dass der Name der ISR immer 
noch nicht stimmt.

Da ist deine Warnung, dass der Name nicht stimmt, denn eine ISR ist 
definitiv eine void Funktion.
Wenn der Compiler das nicht erkannt hat und auswirft
Control reaches end of non-void function
dann bedeutet das, dass er das als int Funktion ansieht. Woraus man 
schliessen kann, dass er das nicht als ISR anerkennt

cyblord ---- schrieb:
> Karl Heinz Buchegger schrieb:
>> Julian W. schrieb:
>>> Okay, hab ich mal geändert... aber darüber hat der Compiler eigetnlich
>>> nie gemekert.
>>> Einzigst was noch immer kommt:
>>>
>>> main.c|67|warning: control reaches end of non-void function|
>>
>> Schön.
>> UNd welches ist die Zeile 67?
>
> Das Ende der main wahrscheinlich. Die sollte int zurückgeben. Macht er
> aber nicht.

Muss er auch nicht.

Für main() gibt es eine Ausnahme :-)
main ist zwar eine int Funktion, aber wenn der Code selbst kein return 
enthält, muss sich der COmpiler ein
   return EXIT_SUCCESS;
dazudenken.

Frag mich nicht ....

ich weiß einfach nicht mehr weiter... hier der komplette Code fals es 
weiter hilft.

Hab das UART Signal mir mal mit dem Oszi angeschaut, das passt aber :(

Karl Heinz Buchegger schrieb:
>> ISR(USART_RX_vect)
Doch. Die stimmt jetzt.
Aber "LightOn(LED_RESET);" befindet sich noch immer zwischen den ersten 
beiden Punkten: [...]

mfg.

Thomas Eckmann schrieb:
> Aber "LightOn(LED_RESET);" befindet sich noch immer zwischen den ersten
> beiden Punkten: [...]

LightOn ist in der io.c und sieht wie folgt aus:

1

void LightOn(uint8_t led)

2

{

3

    PORTD &= ~(1<<led);

4

}

Waaahhhh... ich hab es....

es fehlte einfach nur

1

#include <avr/interrupt.h>

in der uart.h -.- Naja da war ich wohl sehr Betriebsbilnd. Doof das der 
Compiler nicht meckert.

Danke für die vielen und schnellen Antworten ;)

Thomas Eckmann schrieb:
> Karl Heinz Buchegger schrieb:
>>> ISR(USART_RX_vect)
> Doch. Die stimmt jetzt.
> Aber "LightOn(LED_RESET);" befindet sich noch immer zwischen den ersten
> beiden Punkten: [...]

Ich versteh nicht.

Die Fehlermeldung ist nicht typisch dafür.

Julian W. schrieb:
> Waaahhhh... ich hab es....
>
> es fehlte einfach nur
>

1

> #include <avr/interrupt.h>

2

>

>
> in der uart.h -.- Naja da war ich wohl sehr Betriebsbilnd. Doof das der
> Compiler nicht meckert.

Hat er doch :-)

Für den Compiler war das hier

1

ISR( UART_RXC_vect )

2

{

3

}

eine ganz normale C-Funktion.

OK. Der Compiler musste seinen Freiraum komplett ausnutzen, um daraus 
etwas Sinnvolles zu machen - in dem er alle unbekannten Datentypen als 
int annahm, aber ... das darf er.
Für ihn stand da

1

int ISR( int UART_RXC_vect )

2

{

3

  ...

4

}

und: diese Funktion hat keinen return, obwohl sie eine int-Funktion ist. 
UNd das hat er dir freundlicherweise mitgeteilt.

Erst durch den #include wurde dann aus "ISR" etwas ganz anders.

Karl Heinz Buchegger schrieb:
> Die Fehlermeldung ist nicht typisch dafür.
Doch wäre sie, wenn die Funktion nicht void wäre. Daß sie das ist, habe 
ich erst nachher gesehen.

Julian W. schrieb:
> es fehlte einfach nur#include <avr/interrupt.h>
Aber das erklärt den Fehler.

mfg.

Thomas Eckmann schrieb:
> Karl Heinz Buchegger schrieb:
>> Die Fehlermeldung ist nicht typisch dafür.
> Doch wäre sie, wenn die Funktion nicht void wäre. Daß sie das ist, habe
> ich erst nachher gesehen.

Ich hab dich immer noch nicht.
Die Fehlermeldung

control reaches end of non-void function|

an dieser Stelle

1

ISR(USART_RX_vect)

2

{

3

     uint8_t tmp = UDR0;

4

     LightOn(LED_RESET);

5

}     // <--------------

hat nichts mit der Funktion LightOn zu tun.

Aber egal.

Karl Heinz Buchegger schrieb:
> hat nichts mit der Funktion LightOn zu tun.
Der Fehler kam ja auch daher, daß der Compiler nicht wusste, was eine 
ISR ist. Deswegen gab es auch keine Warnung wegen dem misspelled 
handler.

Da ich aber weiß, was das ist, bin ich davon ausgegangen, daß dem 
Compiler das auch mitgeteilt wurde. Somit blieb aus der ISR heraus als 
nächste Funktion, die nichts zurück gibt die LightOn(...

Das ist eben das Resultat, wenn man aus Codefetzen, bei denen alles 
weggelassen wurde, woran es garantiert nicht liegt, versucht, sich etwas 
zusammen zu reimen.

mfg.