Warnung bei globaler PROGMEM-Deklaration: Uninitialized variable

[xfr (Gast)]

Ich habe in meinem Programm ein globales Array mit Funktionspointern, 
das im Flashspeicher liegen soll. Das Programm funktioniert wie es soll, 
allerdings bekomme ich beim Compilieren eine Warnung (siehe unten). Hier 
erstmal die relevanten Codeteile:

Damit ich von verschiedenen Programmteilen auf das Array zugreifen kann, 
habe ich es in einem Headerfile als extern deklariert:

// vtimer_config.h

#define VTIMER_CLIENT_NUM 2

// vtimer.h

#include "vtimer_config.h"

// Callback function prototype
typedef void (*vtimer_callback_t)(void);

// Callback function array (must be defined by user)
extern const PROGMEM vtimer_callback_t vtimer_callback[VTIMER_CLIENT_NUM];

Die Definition des Arrays erfolgt im Hauptprogramm:

// main.c

#include "vtimer.h"

void client_callback_1(void)
{
    leds_toggle(LED_GREEN);
}

void client_callback_2(void)
{
    leds_toggle(LED_RED);
}

const PROGMEM vtimer_callback_t vtimer_callback[VTIMER_CLIENT_NUM] = {
    client_callback_1,
    client_callback_2,
};

In vtimer.c wird in einer ISR auf das globale Array zugegriffen, um die 
Callback-Funktion aufzurufen:

// vtimer.c

#include "vtimer.h"

ISR(RTC_COMP_vect)
{
    // [...]
    ((vtimer_callback_t) pgm_read_word(&vtimer_callback[i]))();
    // [...]
}

Es funktioniert auch alles so weit. Allerdings bekomme ich beim 
Compilieren von vtimer.c folgende Compilerwarnung:

In function '__vector_11':
[Pfad]\vtimer.h(20,40): uninitialized variable 'vtimer_callback' put into program memory area [-Wuninitialized]

Hat das etwas zu bedeuten? Wird das Array wirklich nochmal in vtimer.c 
angelegt? Das Programm funktioniert wie gesagt, verwendet wird also 
offensichtlich schon das Array aus main.c. Soll ich die Warnung 
ignorieren? Bzw. gibt es eine Möglichkeit, sie loszuwerden?

Ich benutzte AVR-Studio 6.0 mit dem AVR-GCC 4.6.2, der standardmäßig 
dabei ist.

Danke schonmal!

[Johann L. (gjlayde)]

xfr schrieb:
> uninitialized variable 'vtimer_callback' put into
> program memory area [-Wuninitialized]
>
> Hat das etwas zu bedeuten?

Es bedeutet, daß du folgende Optionen hast:

• Mit der Warnung leben

• -Wno-uninitialized (oder eine Obermenge davon) deaktivieren.

• Auf eine avr-gcc Version oder Distribution umsteigen, die einen
  Fix für PR51756 hat, z.B. 4.6.3 oder neuer.

[xfr (Gast)]

Alles klar, vielen Dank!

Dann war ja tatsächlich mal der Compiler Schuld ... :)

Die Warnung verschwindet übrigens auch, wenn man in der Deklaration das 
"PROGMEM" weglässt. Allerdings ist das ja eine essentielle Information 
für denjenigen, der das Array definieren soll, also keine Lösung.

[Johann L. (gjlayde)]

xfr schrieb:

> Dann war ja tatsächlich mal der Compiler Schuld ... :)

Nein, der Compiler ist unschuldig. Ich war Schuld.

> Die Warnung verschwindet übrigens auch, wenn man in der Deklaration das
> "PROGMEM" weglässt. Allerdings ist das ja eine essentielle Information
> für denjenigen, der das Array definieren soll, also keine Lösung.

Es genügt wenn das progmem Attribut an der Definition steht.

[xfr (Gast)]

Johann L. schrieb:
> Es genügt wenn das progmem Attribut an der Definition steht.

Aus Sicht des Compilers ja, zur Selbstdokumentation allerdings nicht. 
Wenn sich jemand nur die Header-Datei ansieht, kann er ja nicht ahnen, 
dass das Array im Flash abgelegt werden soll (bzw. ist, wenn man es 
andersrum verwenden würde). Man könnte es natürlich entsprechend 
kommentieren, bleibt aber unsauber.

Naja, habe noch mal grundsätzlich drüber nachgedacht und es jetzt anders 
gelöst: Das Array kommt mit static-Attribut in die vtimer_config.h und 
wird nur in vtimer.c eingebunden. Dann stehen auch die Anzahl der 
Einträge und die Einträge selbst an einer Stelle.

Nachteil ist, dass ich in vtimer_config.h jetzt die Header zu allen 
Funktionen einbinden muss, auf die ich verweise. Die wären in main.c 
halt schon da. Außerdem kann ich in in main.c nicht mehr Funktionen ohne 
Header-Datei definieren und als Callbacks benutzten. Aber gut, man kann 
nicht alles haben ...

Danke nochmal für den Hinweis. :)

[Karl H. (kbuchegg)]

xfr schrieb:
> Johann L. schrieb:
>> Es genügt wenn das progmem Attribut an der Definition steht.
>
> Aus Sicht des Compilers ja, zur Selbstdokumentation allerdings nicht.
> Wenn sich jemand nur die Header-Datei ansieht, kann er ja nicht ahnen,
> dass das Array im Flash abgelegt werden soll (bzw. ist, wenn man es
> andersrum verwenden würde). Man könnte es natürlich entsprechend
> kommentieren, bleibt aber unsauber.

Die Krise an dieser Stelle ist, dass das PROGMEM hier beim extern im 
Grunde auch nichts anderes als eine Dokumentation ist.

Ob es da ist oder nicht, spielt keinen Walzer. Weder wirst du vom 
Compiler gezwungen, da irgendwas spezielles damit zu machen und du wirst 
auch nicht vom Compiler gezwungen hier

((vtimer_callback_t) pgm_read_word(&vtimer_callback[i]))();

einen pgm_read_word anzuwenden, anstatt den Funktionsaufruf direkt über 
das Array zu machen.

So gesehen: ob du einen Kommentar machst, oder ob du da PROGMEM 
hinschreibst, ist Jacke wie Hose. OK, PROGMEM sieht ein wenig 
offizieller aus.

[xfr (Gast)]

Stimmt schon, mehr als ein Hinweis ist es nicht. Wenn das PROGMEM in der 
Deklaration steht, könnte man sie halt einfach kopieren und zu einer 
Definition machen, ohne das PROGMEM zu vergessen. Das beste ist aber 
wahrscheinlich, die ganzen PROGMEM-Geschichten möglichst lokal zu halten 
und nicht nach außen zu zeigen.

Falls es jemanden interessiert, hab es jetzt so gemacht:

// vtimer_config.h

// Callback function type
typedef void (*vtimer_callback_t)(void);

// Number of callback functions
#define VTIMER_CALLBACK_NUM 2

// Callback functions array initializer
#define VTIMER_CALLBACK_ARRAY { \
  client_callback_1, \
  client_callback_2, \
}

// vtimer.h

#include "vtimer_config.h"

// vtimer.c

#include "vtimer.h"

// Callback functions array
static const PROGMEM vtimer_callback_t vtimer_callback[VTIMER_CALLBACK_NUM] = VTIMER_CALLBACK_ARRAY;

Gewinnt sicher auch keinen Schönheitspreis. Aber damit befindet sich 
immerhin alles, was mit PROGMEM zu tun hat, nur noch in vtimer.c und 
kann bei Bedarf auch geändert werden, ohne die Konfigurations-Dateien 
der Anwendungen anfassen zu müssen.

[Karl H. (kbuchegg)]

xfr schrieb:

> Gewinnt sicher auch keinen Schönheitspreis.

Find ich jetzt ehrlich gesagt gar nicht soooo wild. Man muss halt im 
Hinterkopf halten, dass es da ein potentielles Problem gibt, welches du 
so 'entschärfst'. Von daher finde ich das sogar gar keine schlechte 
Lösung.