Datenstruktur in Flash auslagern

[Thorsten H. (lordvader91)]

Hallo!

Ich würde gerne eine Datenstruktur vom RAM in den Flash-Speicher 
auslagern.
Das ganze sind die Daten für eine LED-Matrix.

Meine Datenstruktur besteht aus den folgenden Typen:

typedef union {
    const uint32_t const matrix[4];
    const uint8_t duration;   // duration * 10ms
} picture_t;


typedef struct {
    const picture_t const __flash * pictures;    // picture array
    const uint8_t pictureCnt;    // picture count in the array
    const uint8_t repeatCnt;         // how often is the picture array repeatet
} animation_t;

typedef struct {
    const animation_t** animations;
    const uint8_t animationCnt;
} animationSet_t;

Eine beispielhafte Initialisierung der Struktur:

/* alleAnAus
   Description:
     */
const __flash picture_t alleAnAusPictures[2] = {
    {{0xFFFFFF19, 0xFFFFFFFF, 0xFFFFFFFF, 0xFFFFFFFF}},
    {{0x19, 0x0, 0x0, 0x0}},
};
const __flash animation_t alleAnAus = {
   alleAnAusPictures,
    2,
    1
};

Mein Problem ist nun, dass ich nicht so recht weiß, wie ich den __flash 
Qualifier anzuwenden habe. Mit dem Code von oben funktioniert es leider 
nicht.
Der Compiler kompiliert zwar ohne Warnungen und die Daten werden auch im 
Flash abgelegt. Das Programm funktioniert dann allerdings nicht mehr.


Wäre super, wenn mit jemand auf die Sprünge helfen könnte.

Gruß
Thorsten

Edit:

Ganz vergessen: Das ganze läuft auf einem ATmega32 und wird mit avr-gcc 
Version 4.9.2 kompiliert.

[Sven S. (boldie)]

Also mein PC hat keinen Flash. Was für eine Architektur + Compiler hast 
du?

[Thorsten H. (lordvader91)]

Sorry, hab ich ganz vergessen. Hab den Beitrag editiert.
ATMega32 mit avr-gcc 4.9.2

[isidor (Gast)]

Sven S. schrieb:
> Was für eine Architektur + Compiler hast du?

Hat er doch geschrieben, kannst nich lesen ?

[Thorsten H. (lordvader91)]

isidor schrieb:
> Sven S. schrieb:
> Hat er doch geschrieben, kannst nich lesen ?

Na, aber erst im Nachhinein. ;-)

Das Problem hat sich gelöst. Ich habe vergessen für die Variablen in den 
Funktionen, die auf die Flash-Variablen zugreifen, __flash zu benutzen.
Jetzt funktioniert es.

[isidor (Gast)]

Thorsten H. schrieb:
> Jetzt funktioniert es.

Kannst bitte ein Minimalbeispiel listen, es würde mich
auch interessieren.

[Thorsten H. (lordvader91)]

Folgende Funktionen greifen auf die Variablen zu:

bool playAnimation(const __flash animation_t const * animation);
bool playAnimationSet(const __flash animationSet_t const * animationSet);
bool playPicture(const __flash picture_t const * picture);
uint32_t getColumn(const __flash picture_t const * picture, uint8_t col);

Dazu noch zwei globale Variablen, die auf die aktuellen Daten verweisen:

const __flash picture_t* currentPic = 0;
const __flash animation_t* currentAnimation = 0;

Dort fehlte überall das __flash.

[isidor (Gast)]

Danke sehr.

Irritierend finde ich bei Typdeklarationen dass "const"
zweimal vorkommt. Hab es nicht getestet, muss das so sein?

[Thorsten H. (lordvader91)]

Das erste const für die Variable, die sich nicht ändern soll und das 
zweite const für den Pointer, der auch nicht geändert werden soll.
So habe ich mir das gedacht. Korrigiert mich, wenn ich da einen 
Denkfehler habe.

[Hans (Gast)]

Wenn der Zeiger (sprich die Adresse) nicht geändert werden soll, müsste 
das const rechts vom "*" stehen. Allerdings ist das bei einem 
Funktionsargument recht sinnlos, weil die Funktion ja eine Kopie des 
Zeigers bekommt. Ob sie die ändert oder nicht spielt für den Aufrufer 
keine Rolle.

[Johann L. (gjlayde)]

Thorsten H. schrieb:
> Das erste const für die Variable, die sich nicht ändern soll und das
> zweite const für den Pointer, der auch nicht geändert werden soll.

No, die beiden const sind redundantz, d.h. eines kann weggelassen 
werden: Ob du schreibst "typ const" oder "const typ" ist Wurscht, ditto 
beliebig viele Kopien vor oder nach dem Typ.

Bei Zeigern spielt es eine Rolle, auf welcher Seite des "*" das const 
steht, aber bei deinem Code ist es an keiner Stelle ein typ*.

[Rolf M. (rmagnus)]

Sven S. schrieb:
> Also mein PC hat keinen Flash.

Echt? Von wo kommt dann das BIOS?

Johann L. schrieb:
> No, die beiden const sind redundantz, d.h. eines kann weggelassen
> werden: Ob du schreibst "typ const" oder "const typ" ist Wurscht, ditto
> beliebig viele Kopien vor oder nach dem Typ.
>
> Bei Zeigern spielt es eine Rolle, auf welcher Seite des "*" das const
> steht,

Naja, allgemein ausgedrückt: Das const wirkt immer auf das, was links 
davon steht. Einzige Ausnahme ist, wenn das const selbst ganz links 
steht. Dann wirkt es auf das, was rechts steht.
Also:

X const *       // nicht-konstanter Zeiger auf konstantes X
X * const       // konstanter Zeiger auf nicht-konstantes X
X const * const // konstanter Zeiger auf konstantes X
const X *       // Sonderfall, Effekt ist wie beim ersten Beispiel
const X * const // wie Fall 3
const X const * // redundantes const, beides bezieht sich auf das X

> aber bei deinem Code ist es an keiner Stelle ein typ*.

Nicht?

Thorsten H. schrieb:
> uint32_t getColumn(const __flash picture_t const * picture, uint8_t
> col);