Flashstring für Menu in struct

[Sam .. (sam1994)]

Hi

Ich versuche gerade OO Menüstrukturen für mein 2x16 Lcd zu erstellen. 
Die Strukturen sehen wie folgt aus:

#define MENU_TYPE_INT 0
#define MENU_TYPE_STR 1
#define MENU_TYPE_MENU 2

typedef union 
{
    char* str;
    uint8_t uint;
    void* menu;   
}Data;

typedef struct 
{
    char* name;
    uint8_t type;
    uint8_t current;
    uint8_t len;
    Data value[];
}MenuEntry;

typedef struct 
{
    void* prev; 
    uint8_t current; 
    uint8_t len;
    MenuEntry* entry[];
}Menu;

Dazu habe ich eine Testmenü erstellt:

const char test[] PROGMEM = "Test";
const char on[]   PROGMEM = "An";
const char off[]  PROGMEM = "Aus";

const MenuEntry entry1 PROGMEM = {test, MENU_TYPE_STR, 0, 2, {{on}, {off}}};  Z.7 1-3. Warnung
const Menu optionen PROGMEM = {0, 0, 1, {&entry1}};Z.8 4. Warnung

Code für das ausführen:

void do_menu(Menu m)
{
    lcd_clear();
    lcd_setcursor(0,1);
    write_entry(1, m.entry[0]);
}

void write_entry(uint8_t i, MenuEntry* m)
{
    char c;
    lcd_putc('0' + i);
    lcd_putc(' ');
    uint8_t len = 12;                   //Länge des Lcds - "i [...]: "
                                        //          16      -2     -2 = 12
                                        //Len ist wichtig um Data später rechtsbündig darzustellen
    char* p = m->name;                  //char* name: Zeiger auf array im Flash
    while(c = pgm_read_byte(p++))       //nächtes Byte lesen und Zeiger um 1 erhöhen Z.26 5. Warnung
    {
        len--;
        lcd_putc(c);
    }
    lcd_putc(':');
    lcd_putc(' ');
    //Der rest ist zurzeit auskommentiert und daher unwichtig, da schon das nicht funktioniert.
    //[...]
}

Problem ist aber, wenn ich den einen Menüeintrag auf dem Lcd ausgeben 
will, erscheint nur Murks. Die "1 " erscheint noch, dann kommen aber ein 
wildes Sammelsurium.

Ich vermute das der µC irgendwo liest wo er nicht lesen soll. Kurzzeitig 
wird der volle Block auf dem ganzen Display angezeigt. Er liest also 
erstmal länger 0xFF im leeren Flash.
Ich habe mal die HexDatei nach 0x00 durchsucht, aber nichts gefunden. Er 
könnte höchstens im Bootloader zum stoppen kommen. Sonst fällt mir keine 
Stelle ein, wo noch 0x00 sein könnte.

Der Compiler zeigt ein paar Warnungen an. Ich wüsste aber nicht was ich 
da casten sollte:

7  [Warning] initialization discards qualifiers from pointer target type 
7  [Warning] initialization discards qualifiers from pointer target type
7  [Warning] initialization discards qualifiers from pointer target type
8  [Warning] initialization discards qualifiers from pointer target type 
26 [Warning] suggest parentheses around assignment used as truth value

[Falk B. (falk)]

@  Samuel K. (sam1994)

>void do_menu(Menu m)
>{
>    lcd_clear();
>    lcd_setcursor(0,1);
>    write_entry(1, m.entry[0]);

Hier gibt es AFAIK schon Probleme, weil eben dein Menu im PROGMEM liegt, 
Zugriff geht nur über pgm_read_byte etc.

>    char* p = m->name;                  //char* name: Zeiger auf array im >Flash

char* ist ein Zeiger auf den RAM. Für Flash gibt es extra PGM_P, siehe 
doku der libc. Aber auch hier gibt es wahrscheinlich ein 
Dereferenzierungsproblem.

Ne Lösung hab ich aber nicht :-0

MFG
Falk

[Sam .. (sam1994)]

Falk Brunner schrieb:
>>void do_menu(Menu m)
>>{
>>    lcd_clear();
>>    lcd_setcursor(0,1);
>>    write_entry(1, m.entry[0]);
>
> Hier gibt es AFAIK schon Probleme, weil eben dein Menu im PROGMEM liegt,
> Zugriff geht nur über pgm_read_byte etc.

Danke, das hab ich glatt vergessen zu lesen. Mal schauen ob das hilft.

>>    char* p = m->name;                  //char* name: Zeiger auf array im >Flash
>
> char* ist ein Zeiger auf den RAM. Für Flash gibt es extra PGM_P, siehe
> doku der libc. Aber auch hier gibt es wahrscheinlich ein
> Dereferenzierungsproblem.

Das wusste ich nicht genau und hab dann einfach einen char* Zeiger 
genommen. Solange ich den über pgm_read_byte aufrufe dürfte das doch 
trotzdem passen, oder nicht?


Ein paar 0x00 sind übrigens doch in der Hex-Datei. Ich dachte der 
Hexreader unterstützt .hex Dateien. Das er die nicht unterstützt merkte 
ich daran, dass die Code angeblich 4,6KB groß sei.

[Stefan E. (sternst)]

Falk Brunner schrieb:
> char* ist ein Zeiger auf den RAM. Für Flash gibt es extra PGM_P,

Zeiger ist Zeiger. PGM_P ist in erster Linie was fürs Auge.

So auf die Schnelle:

void do_menu(Menu *m)
...
    write_entry(1,(MenuEntry*)pgm_read_word(&(m->entry[0])));

und

char* p = (char*)pgm_read_word(&(m->name));

PS:
Und was die Warnungen angeht: da fehlen diverse "const" in den 
Struct-Definitionen.

[Sam .. (sam1994)]

Ich hab den Code ein bisschen umgeändert und kam mit dem auf 6 falsche 
Zeichen zwischen "1 " und "Test:"

void do_menu(Menu m)
{
    lcd_clear();
    lcd_setcursor(0,1);
    write_entry(1, (MenuEntry*) pgm_read_word(m.entry[0]));
}

void write_entry(uint8_t i, MenuEntry* m)
{
    char c;
    lcd_putc('0' + i);
    lcd_putc(' ');
    uint8_t len = 12;                   //Länge des Lcds - "i [...]: "
                                        //          16      -2     -2 = 12
                                        //Len ist wichtig um Data später rechtsbündig darzustellen
    char* p = pgm_read_word(m->name);                  //char* name: Zeiger auf array im Flash
    while(c = pgm_read_byte(p++))       //nächtes Byte lesen und Zeiger um 1 erhöhen
    {
        len--;
        lcd_putc(c);
    }
    lcd_putc(':');
    lcd_putc(' ');
}

@Stefan
Deine Vorschläge machen es auch besser, allerdings erscheint das Test: 
in der 2. Zeile. Also sind >40 falsche Zeichen dazwischen.

Entry ist übrigens schon ein Pointer, siehe Definition. Da muss das "&" 
dann weg.

Irgendwie glaub ich grad, dass Assembler hier unkomplizierter wäre, da 
man nicht auf so ein Pointer pgm Zeug aufpassen muss.

Stefan Ernst schrieb:
> PS:
> Und was die Warnungen angeht: da fehlen diverse "const" in den
> Struct-Definitionen.

Hmm, ich hab alle PROGMEM mit einem const ausgestattet. Google meint 
auch was mit const, aber ich wüsste nicht wohin damit.

Nochmal langsam:
write_entry(1, *);
Hier muss man einen Zeiger MenuEntry* übergeben.
m.entry[0] ist der Zeiger auf den Flash
man liest also pgm_read_byte(m.entry[0])
dann bekommt man das 1. Byte vom MenuEntry
davon den Zeiger ergibt die Startadresse:

write_entry(1,&((MenuEntry) pgm_read_byte(m.entry[0])));

Das funktioniert aber nicht:

14 conversion to non-scalar type requested

Edit: Danke euch allen. Ich hab das Menu* Zeiger in Stefans Code 
übersehen. Es klappt wenn man den nutzt.

[Sam .. (sam1994)]

Ich hatte eine falsche Vorstellung der Flashsyntax.
Wenn man eine Konstante als PROGMEM deklariert ist sie immer ein Zeiger. 
Den man auch erst auslesen muss. Erst danach hat sie den "deklarierten" 
Wert.

PS: Der Rest des Programms funktioniert jetzt auch. Danke nochmals.


Was noch offen bleibt wäre wie ich die Warnungen entferne. Wäre nett 
wenn mir jemand kurz sagt wo noch ein const hinmuss.
Und warum man Menu als Zeiger übergeben muss.

Was mich noch wundert ist, dass obwohl

33 [Warning] passing argument 1 of 'do_menu' discards qualifiers from pointer target type

diese Warnung ausgespuckt wird, es trotzdem funktioniert.
In do_menu gebe ich &optionen an.

[Stefan E. (sternst)]

Samuel K. schrieb:
> Wenn man eine Konstante als PROGMEM deklariert ist sie immer ein Zeiger.

Nein, es ist genau das, als was man es deklariert hat. Aber die 
Funktionen zum Auslesen dieses "etwas" aus dem Flash brauchen natürlich 
die Adresse von diesem "etwas", daher immer das & bei den 
Funktionsargumenten.

Samuel K. schrieb:
> Wäre nett
> wenn mir jemand kurz sagt wo noch ein const hinmuss.

Na überall dort, wo du dann auch "const-Sachen" einträgst. Beispiel: 
wenn die Menü-Einträge alle im Flash liegen, dann sollte entry in 
Menu doch wohl ein Array aus "const MenuEntry*" sein.

[Sam .. (sam1994)]

So, das ganze funktioniert bisher ganz gut. Allerdings will die Union 
nicht so richtig:

#define MENU_TYPE_INT 0
#define MENU_TYPE_STR 1
#define MENU_TYPE_MENU 2

typedef union 
{
    char* str;
    uint8_t uint;
    void* menu;   
}Data;

typedef struct 
{
    char* name;
    uint8_t type;
    uint8_t* current;
    uint8_t len;
    Data value[];
}MenuEntry;

typedef struct 
{
    uint8_t len;
    MenuEntry* entry[];
}Menu;

/**************************Menü - Texte**************************/

uint8_t Vchannels = 0;

const char channels[] PROGMEM = "Kanäle";

//Kanäle
const uint8_t _1 PROGMEM = 1;
const uint8_t _2 PROGMEM = 2;
const uint8_t _3 PROGMEM = 3;
const uint8_t _4 PROGMEM = 4;

const MenuEntry Echannels PROGMEM = {channels, MENU_TYPE_INT, &Vchannels, 4, 
    {{.uint=_1},{.uint=_2},{.uint=_3},{.uint=_4}}};

Aber der Compiler sagt je 4x:

54 (near initialization for 'Echannels.value[x].uint') 
54 initializer element is not constant

Initialisiere ich die Union falsch?


Die anderen Warnungen kommmen übirgens immer noch, obwohl ich vor jedem 
Menu und Entry ein const stehen habe.

[Stefan E. (sternst)]

Samuel K. schrieb:
> Initialisiere ich die Union falsch?

Bei MenuEntry.len schreibst du bei der Initialisierung doch auch direkt 
eine 4 rein. Warum glaubst du, bei der Union die Zahlen vorher irgendwie 
im Flash ablegen zu müssen?

Samuel K. schrieb:
> Die anderen Warnungen kommmen übirgens immer noch, obwohl ich vor jedem
> Menu und Entry ein const stehen habe.

1) Und warum sehe ich in dem geposteten Code davon nichts?

2) Du hast ja auch noch mehr Konstanten im Flash, z.B. die Strings.

[Sam (Gast)]

Stefan Ernst schrieb:
> Samuel K. schrieb:
>> Initialisiere ich die Union falsch?
>
> Bei MenuEntry.len schreibst du bei der Initialisierung doch auch direkt
> eine 4 rein. Warum glaubst du, bei der Union die Zahlen vorher irgendwie
> im Flash ablegen zu müssen?
Stimmt daran hab ich irgendwie nicht gedacht.

> Samuel K. schrieb:
>> Die anderen Warnungen kommmen übirgens immer noch, obwohl ich vor jedem
>> Menu und Entry ein const stehen habe.
>
> 1) Und warum sehe ich in dem geposteten Code davon nichts?
Die Warnungen kommen bei den MenuEntry und Menu Definitionen. Die sind 
aber schon const. Kann es sein, dass man bei manchen Angaben für die 
Struktur ein (const) davor schreiben muss?

> 2) Du hast ja auch noch mehr Konstanten im Flash, z.B. die Strings.

Bei den Strings steht doch immer ein const davor. Verstehe ich dich da 
irgendwie falsch?

[Stefan E. (sternst)]

Sam schrieb:
> Kann es sein, dass man bei manchen Angaben für die
> Struktur ein (const) davor schreiben muss?

Natürlich. Ich dachte eigentlich, das längst geschrieben zu haben:

Stefan Ernst schrieb:
> Beispiel:
> wenn die Menü-Einträge alle im Flash liegen, dann sollte entry in
> Menu doch wohl ein Array aus "const MenuEntry*" sein.

[Falk B. (falk)]

Das const hat auf die Sache mit dem Flash keinerlei Einfluss beim AVR 
GCC.

[Stefan E. (sternst)]

Falk Brunner schrieb:
> Das const hat auf die Sache mit dem Flash keinerlei Einfluss beim AVR
> GCC.

Hat ja auch keiner behauptet. Es geht um das Entfernen von Warnungen.