AVR-C: Funktionen aufrufen

[Mkolo (Gast)]

Hallo,

ich bin ein bisschen ratlos, folgende Beispiele
funktionieren bei meinem Attiny13 nicht und ich
weiss nicht warum:

void blub(){
PORTB= 1;
}

int main(){
DDRB = 1;
blub();
while(1){}
}

Soweit hab ichs runterreduziert...
Es fing damit aber an:

void blub(volatile uint8_t *prt){
*prt = 1;
}

int main(){
DDRB=1;
blub(&PORTB);

[...]
}

Verpack ich die Funktionen allerdings nochmal
in eine Funktion, also:

void foo(){
blub()
}

Und rufe foo() dann auf, dann funktioniert es...
Das verwundert mich, eine Eigenheit von avr-c?
Wie funktioniert das Ganze?


lg

[Karl H. (kbuchegg)]

Zeig den ganzen, compilierbaren Code. Nicht einen ähnlichen, nicht etwas 
von dem du denkst, dass er so ähnlich aussieht, sondern den den du 
tatsächlich benutzt.
Am geposteten ist erst mal nichts falsch (aber es fehlen zb die 
#include). Aber im echten Code könntest du zb die () Klammern beim 
Funktionsaufruf vergessen haben. So was passiert schon mal.

Am besten hängst du das C-File so wie es ist einfach als Attachment an.

[Mkolo (Gast)]

Ja, es verwundert mich auch sehr:

#include <avr/io.h>
#include <util/delay.h>


void foo(){
  PORTB = 1;
}

int main() {
  DDRB = 1;
  foo();
  while(1) {}

}

tut nicht. LED ist an PB0 angeschlossen.

#include <avr/io.h>
#include <util/delay.h>

void foo(){
  PORTB = 1;
}

void foo2(){
foo();
}

int main() {
  DDRB = 1;
  foo2();
  while(1) {}

}

hingegen funktioniert und es ist mir ein Raetsel warum!

[Mkolo (Gast)]

Mkolo schrieb:
> hingegen funktioniert und es ist mir ein Raetsel warum!

Ok, mir kam grade der Lichtblitz, dass ich ja einfach
ein Makefile aus dem Forum uebernommen hab.
Dort war die Optimierung fuer Dateigroesse drin.
Mit -O0 funktioniert auch das erste Beispiel.
Es ist mir dennoch fraglich, was der Compiler da
im Funktionsaufruf wegoptimiert.



lg

[Peter II (Gast)]

Mkolo schrieb:
> Es ist mir dennoch fraglich, was der Compiler da
> im Funktionsaufruf wegoptimiert.

erzeuge mal bitte für beide Versionen das lss file und lade es hoch.

[Yalu X. (yalu) (Moderator)]

Mkolo schrieb:
> tut nicht.

Wie äußert sich das?

Hast du das auf einem realen ATtiny13 getestet, indem du das Signal an
PB0 gemessen oder mit einer LED sichtbar gemacht hast?

Oder hast du das Programm nur im Simulator/Debugger laufen lassen? Da
können je nach verwendeter GCC- bzw. Debugger-Version und je nach
Debug-Option unerwartete Dinge passieren, da die Funktionen bei
eingeschalteter Optimierung geinlinet werden.

[Axel S. (a-za-z0-9)]

Mkolo schrieb:
> ich bin ein bisschen ratlos, folgende Beispiele
> funktionieren bei meinem Attiny13 nicht und ich
> weiss nicht warum

> void blub(){
> PORTB= 1;
> }
> 
> int main(){
> DDRB = 1;
> blub();
> while(1){}
> }

...

Mkolo schrieb:
> mir kam grade der Lichtblitz, dass ich ja einfach
> ein Makefile aus dem Forum uebernommen hab.
> Dort war die Optimierung fuer Dateigroesse drin.
> Mit -O0 funktioniert auch das erste Beispiel.

Hmm. Vielleicht hast du nur eine merkwürdige Auffassung davon was 
"funktionieren" für deinen Beispieplcode bedeutet. Denn in der Tat wird 
der Compiler mit aktivierter Optimierung den Funktionsaufruf einsparen 
und die Funktion "inlinen" - d.h. an Stelle des Funktionsaufrufs direkt 
den Rumpf der aufgerufenen Funktion in das Programm einfügen.

Für die Funktion des Programms - daß nämlich PB0 auf H gesetzt wird - 
ist diese Optimierung unerheblich. Es verhält sich nach außen genau 
gleich, unabhängig davon ob da intern ein Funktionsaufruf stattfindet 
oder nicht.

[Mkolo (Gast)]

Axel S. schrieb:
> Für die Funktion des Programms - daß nämlich PB0 auf H gesetzt wird -
> ist diese Optimierung unerheblich. Es verhält sich nach außen genau
> gleich, unabhängig davon ob da intern ein Funktionsaufruf stattfindet
> oder nicht.

Das Beispiel wurde mit einem realen Attiny getestet,
und PB0 war laut intakter LED nicht auf High.
Wenn der Compiler ne Funktion inline setzt ist das ja
eine Sache, meinetwegen kann er auch eine ungenutzte
Variable wegmachen, aber es wundert mich doch arg, dass
eine Optimierung die Funktionalitaet des Programms wesentlich
veraendert.



Im Anhang die gewuenschten lss Dateien(heute aber ungetestet).
Und nochmal der code (button.c):

#include <avr/io.h>
#include <util/delay.h>


void test(){
  PORTB = 1;
}


int main() {
  DDRB = 0b00000001;
  test();
  while (1) {}

}

lg und nochmals vielen Dank

ps: [alle@foo ~]$ avr-gcc --version
avr-gcc (GCC) 5.2.0

[Tassilo H. (tassilo_h)]

Bist du sicher, daß die letzte Datei mit Optimierung Os war? Sieht 
identisch zur ersten aus.

[Alexander S. (esko)]

Mkolo schrieb:
> button_unoptimiert.lss
> button_unoptimiert.lss
> button_optimiert_mit_s.lss

Alle drei Dateien sind identisch.

[Mkolo (Gast)]

Ich kann nicht leugnen, dass ihr recht habt,
ich haetts vorher mit diff ueberpruefen koennen.

Zusaetzlich hab ich jetzt optimierte und unoptimierte
Version real getestet... und beide funktionieren.

Gestern bin ich allerdings 2h an diesem Problem
verzweifelt, was jetzt einfach nicht da ist.

Ja ehm... ich fuerchte mein nicht reproduzierbarer
Fehler wird nun auf Inkompetenz meinerseits
zurueckgefuehrt. Damit muss ich dann wohl leben.

Aber damit schliesse ich und danke nochmals!