AVR-GCC warning: return type defaults to 'int' + ISR_NAKED

[Wolfgang (Gast)]

Hallo,

ich versuche, eine enge und schnelle Interrupt Routine zu bauen - nur 
einen Wert auf ein IO-Register nageln.

#include <avr/interrupt.h>
ISR(TCC4_CCB_vect)
{
       VPORT2.DIR = portd_dir_mpx_drive;
}

Paßt soweit, erzeugt aber Standardprolog und -epilog (push Status, push 
R1 , ...).

Baue ich das als ISR_NAKED (dann mit inline asm)

ISR(TCC4_OVF_vect, ISR_NAKED) 
{
    asm volatile(
        "push   r24"  ); 
    asm volatile(
        "out    %0, %1"      "\n\t"
        :
        : "i" (_SFR_IO_ADDR(VPORT2.DIR)), "r" (portd_dir_mpx_drive) 
        :               // no clobber here
        );
    asm volatile(
        "pop  r24"         "\n\t"
        "reti"               "\n\t"
        );
}

bekomme ich zwar den gewünschte knappen Code (Status register muß ich ja 
nicht pushen, lds und out verändert da nichts), aber auch die Warnung:
Warning    type of '__vector_12' defaults to 'int' [-Wimplicit-int]

Wie bekomme ich diese warning weg?

Viele Grüße Wolfgang

[Jörg W. (dl8dtl) (Moderator)]

Hmm, ich habe deinen Code mal mit

#include <avr/io.h>
#include <avr/interrupt.h>

uint8_t portd_dir_mpx_drive;

ergänzt und mit -Os -mmcu=atxmega32e5 -Wall -Wextra compiliert – ohne 
jegliche Warnungen und mit dem gewünschten Ergebnis.

Davon abgesehen, wäre es hier nicht einfacher, das gleich in eine 
.S-Datei auszulagern, wenn du die ISR sowieso letztlich komplett als 
Assemblercode hinlegst?

[Wolfgang (Gast)]

Hallo,

mein Fehler. Ich hatte die ursprüngliche ISR (halt ohne das Keywort ISR) 
gleich dahinter nochmal stehen, und eigentlich hat der Compiler dort 
gemeckert.

Richtig, es ist ein xmega e5, ürsprünglich wollte ich das Thema mit 
Comparechannels lösen. Nur hat Atmel/Microchip den Prozessor kastriert - 
Auf Port D gibt es keinen Timer für die Ports 0..3. Okay, dann halt mit 
DMA - grml, dieser EDMA (E soll enhanced bedeutet) is eher ein RDMA 
(reduced): Trigger des DMA geht im Peripheral-Mode nur aus ausgewählten, 
wenigen Quellen und im Standardmode sind es nur 2 Channels. Also 
Interrupt. Zu viel Jitter, zu lange Totzeit.

Dann quasi der letzte Rettungsanker: asm. Wegen der paar Zeile wollte 
ich jetzt kein extra Modul bauen, das ins make/git packen usw.

Danke nochmal

Wolfgang

[Johann L. (gjlayde)]

Wolfgang schrieb:
> erzeugt aber Standardprolog und -epilog (push Status, push R1 , ...).

Schon mal eine aktuelle Compilerversion versucht (v8 oder neuer)?

#include <avr/io.h>
#include <avr/interrupt.h>

volatile uint8_t portd_dir_mpx_drive;

ISR (TCC4_CCB_vect)
{
    VPORT2.DIR = portd_dir_mpx_drive;
}

erzeugt etwa (-Os -mmcu=atxmega32e5):

000000ce <__vector_15>:
  ce:  8f 93         push  r24
  d0:  80 91 00 20   lds  r24, 0x2000
  d4:  88 bb         out  0x18, r24
  d6:  8f 91         pop  r24
  d8:  18 95         reti

Wolfgang schrieb:
> Dann quasi der letzte Rettungsanker: asm.

Dann aber korrekt, zum Beispiel so (dass dein Code wie gewünscht 
übersetzt wird bedeutet nicht, dass die Quelle korrekt ist :-))

ISR (TCC4_OVF_vect, ISR_NAKED) 
{
    asm volatile (
        "push  r24"         "\n\t" 
        "lds   r24, %1"     "\n\t"
        "out   %i0, r24"    "\n\t"
        "pop   r24"         "\n\t"
        "reti"
        :
        : "n" (&VPORT2.DIR), "i" (&portd_dir_mpx_drive)
        : "r24", "memory");