kann globale variable nicht im interrrupt beschreiben

[LittleBoy]

Hallo Leute,
Ich will für den Nibobee ein Odometriesystem schreiben und stoße dabei auf ein kleines Problem:
Ich kann eine globale Countervariable nicht korrekt beschreiben.

// mit und ohne static, sowie auch ohne volatile negativ getestet
/*static*/ volatile unsigned  int odometry_count_r=0;
/*static*/ volatile unsigned  int odometry_count_l=0;

/* wenn etwas überläuft, wird das ausgeführt */
// mit und ohne volatile getestet
/*volatile*/ void ( * odometry_l_overflow)(void);
/*volatile*/ void ( * odometry_r_overflow)(void);


ISR(INT0_vect)
{
// u.U. muss man das nicht atomisch machen
//      sei();
        odometry_count_l++;
        #ifdef __TEST
        set_led(LED_L_Y,1);
        _delay_ms(20);
        set_led(LED_L_Y,0);
        #endif
        // eigentlich 0, da dann ein ueberlauf da ist, 3 zum testen.
        if(odometry_count_l==3)
        {
                odometry_l_overflow();
        }
        reti();
}
// analog dazu vect1

void _init_odometrie(void)
{
        GICR|= 1<<INT0;
        GICR|= 1<<INT1;
        MCUCR|= 1<<ISC01 | 1<<ISC00;
        MCUCR|= 1<<ISC11 | 1<<ISC10;
        sei();
        #ifdef __TEST
        // lauffeuer laesst leds der reihe nach blinken.
        odometry_l_overflow=lauffeuer;
        odometry_r_overflow=lauffeuer;
        #else
        odometry_l_overflow=nop;
        odometry_r_overflow=nop;
        #endif

        #ifdef __TEST
        _init_led();
        #endif
}

void nop(void)
{
        asm("nop");
}

Aber es wird nie odometry_l_overflow ausgeführt. Eigentlich sollte auch der Speicher nicht korrumpiert sein
und der Funktonspointer funktioniert auch. Die ganzen Led-Sachen sind nur da um zu sehen wann der Interrupt ausgelößt wurde.

Danke im Vorraus
LittleBoy

[shedepe]

Also ein paar Sachen:
1. Warum rufst du in einem Interrupt sei() auf.
2. reti() brauchst du wenn du mit einem Atmega arbeitest nicht. -> Hier wird sogar gesagt (http://cboard.cprogramming.com/c-pro...-function.html) dass das Programm dann crashen kann.
Siehe dazu auch: http://www.nongnu.org/avr-libc/user-...e31f9a6d10533b

Ich vermute mal, dass es funktioniert wenn du reti() weglässt.

[LittleBoy]

Ja ok das reti() hat alles kaput gemacht.
Danke!
btw:
das sei() ist da, damit ich lange Prozeduren nicht atomisch ausführen kann.
Das wird dann per Variable ein und aus geschaltet.

[shedepe]

Das sei bewirkt in dem Interrupt absolut gar nichts. Wenn das System in den Interrupt gesprungen ist, dann sind die interrupts bereits aktiv.

[LittleBoy]

tja jetzt funktioniert es immer noch nicht so ganz:
ich habe zwei variablen eingefügt, die wie ein timer compare interrupt funktionieren sollen,
allerdings kann ich die nicht immer erfolgreich beschreiben:

static volatile unsigned int odometry_count_r=0;
static volatile unsigned int odometry_count_l=0;
// auch static getestet
volatile unsigned int odometry_comp_r=0;
volatile unsigned int odometry_comp_l=0;

/*wie gesagt compare */
void ( * odometry_l_comp)(void);
void ( * odometry_r_comp)(void);


ISR(INT0_vect)
{
        odometry_count_l++;
        #ifdef __TEST
        set_led(LED_L_Y,1);
        _delay_ms(20);
        set_led(LED_L_Y,0);
        #endif
        
        if(odometry_count_l==odometry_comp_l)
        {
                odometry_l_comp();
        }
}
void _init_odometrie(void)
{
        if(add_mod(_MOD_ODO));
        {
                GICR|= 1<<INT0;
                GICR|= 1<<INT1;
                MCUCR|= 1<<ISC01 | 1<<ISC00;
                MCUCR|= 1<<ISC11 | 1<<ISC10;
                sei();
                #ifdef __TEST
                odometry_l_comp=lauffeuer;
                odometry_r_comp=lauffeuer;
                #else
                odometry_l_comp=nop;
                odometry_r_comp=nop;
                #endif
// das wuerde funktionieren
//              odometry_comp_l=5;
//              odometry_comp_r=5;


         }
}

// aus odtest.c

int main(void)
{
// funktioniert nicht.
        odometry_comp_l=5;
        odometry_comp_r=5;
//auch andersherum getestet
        _init_odometrie();
        while(1);
}

irgendwie kommt mir das komisch vor.
Danke in Vorraus
LittleBoy

[seite5]

hi,
hast Du die beiden Variablen im Modul odtest.c mit extern deklariert ?
mfg
Achim

[LittleBoy]

nein,
da ich die Variablen in odometrie.h definiert habe und in odotest.c mit `#include"odometrie.h"`
eingebunden habe, genauso wie in
odometrie.c, wo die Funktion `_init_odometrie()` drin ist.
Das ist das was mich ja so wundert.
lg
LittleBoy

[Wsk8]

Eine Variable im Header kann gar nicht funktionieren. Da würde ein normaler, funktionierender Compiler sofort melden, dass "multiple definitions" vorhanden sind und abbrechen.

Pack die Variablen als static ins source-file und implementiere Funktionen zum ändern.

inline void IncrOdoCount();
inline int GetOdoCount();
etc...

Dann hast du eine funktionierende, saubere und Design-konforme Lösung. Globale Variablen sind sch*****!!

mfg