Problem mit PROGMEM, structure und string kombination

[Stefan F. (stefan1987)]

Hallo zusammen,

ich habe ein Problem mit meinem Menü. Ursprünglich war ein Menüpunkt in 
einer Structure gespeichert (Text als String, Funktion die aufgerufen 
wird, int der den aktiven Punkt indiziert) Um Platz im RAM zu sparen 
wollte ich die Strings die jeweils ausgegeben werden sollen in den 
Flashspeicher verlagern. Das hat auch soweit geklappt. Wenn ich zum 
Beispiel einen solchen String wieder ausgeben wollte konnte ich das mit

    char buffer[10];
    for (unsigned char i = 0; i <4; i++)
    {
        strcpy_P(buffer, (PGM_P)pgm_read_word(&(MainMenu[i])));
        LCD_STRING (buffer,0,i);
    }

machen. Wenn ich nun allerdings versuche dieses Konstrukt in die 
Menupunkt Ausgabe zu integrieren kommt garnichts auf dem LCD an.
Ich habe mal den gesamten SourceCode mit angehängt, wobei die relevanten 
Files wohl PrintMenu.c und MainMenu.c sind. Der Aufruf geschieht in 
Moodlight204112.c.
Ich wäre sehr dankbar wenn mir jemand sagen könnte was ich falsch mache 
bzw falsch verstanden habe.
Vielen Dank schonmal.

Grüße Stefan

[Uwe (de0508)]

Hallo,

Stefan Friedrich schrieb:
> pgm_read_word(&(MainMenu[i]))

wenn Du diese Frage beantworten kannst, bist Du ein Stück weiter:

wie viele Bytes werden pro Aufruf gelesen ?

[Stefan F. (stefan1987)]

Danke schonmal für die schnelle Antwort.
Also wenn ich das richtig verstanden hab wird hier nur die Adresse des 
Strings weitergegebn. Und gelesen wird hier ein Word also 2 Byte.
Soviel ich weiß sollte das auch passen da die Adresse 16 Bit hat.
Und wenn ich diese Funktion nutze funktioniert das auch. Also:

char string_1[] PROGMEM = "Program";
char string_2[] PROGMEM = "Options";
char string_3[] PROGMEM = "Time";
char string_4[] PROGMEM = "Help";


PGM_P MainMen[] PROGMEM =
{
    string_1,
    string_2,
    string_3,
    string_4
};

void main(void)
{
    char buffer[10];

    for (unsigned char i = 0; i <4; i++)
    {
        strcpy_P(buffer, (PGM_P)pgm_read_word(&(MainMen[i])));

       LCD_STRING (buffer,0,i);
    }
    return;
}

Das hier funktioniert einwandtfrei.
Also denk ich mal das ich irgendwas falsch gemacht hab bei der DrawMenu 
oder DoMenu Funktion.

[Karl H. (kbuchegg)]

Im übrigen ist es nicht besonders schlau, wenn du zuerst den kompletten 
String ins SRAM kopierst um dann eine vorhandene Ausgabefunktion 
aufzurufen. Es ist wesentlich vernünftiger, wenn du dir eine 2-te String 
Ausgabefunktion baust, die den String Zeichen für Zeichen eben nicht aus 
dem SRAM sondern aus dem Flash holt.

Aber überarbeite erst mal deine LCD_STRING Funktion!

void LCD_STRING (char *s,int x, int line)
  {
    switch (line)
    {
    
    case 0:
    {
      LCD_GOTO_XY(x,line);
      while (*s)
      {
        LCD_DATA1 (*s);
        s++;
      }
      break;
    }
    case 1:
    {
      LCD_GOTO_XY(x,line);
      while (*s)
      {
        LCD_DATA1 (*s);
        s++;
      }
      break;
    }    
    case 2:
    {
      LCD_GOTO_XY(x,line);
      while (*s)
      {
        LCD_DATA2 (*s);
        s++;
      }
      break;
    }
    case 3:
      LCD_GOTO_XY(x,line);
      while (*s)
      {
        LCD_DATA2 (*s);
        s++;
      }
      break;
    }    
  }

Die Funktion ist mehr als ungeschickt gemacht und viel zu lang, für das 
was sie kann.

Merke: Bau dir Basisfunktionen, die du gut benutzen kannst und viele 
Probleme auf höherer Softwareebene sind plötzlich keine Probleme mehr.

[Karl H. (kbuchegg)]

Ooops

Geh das mal mit einem Wert für MenuSize von 4 durch

void DrawMenu( uint8_t MenuSize, struct MenuEntry Menu[],PGM_P Strings[] )     // Display the Menu Entries
{
  char buffer[25];
  uint8_t i=0;
  LCD_CLEAR();
  
  if (MenuSize <4)                  // less then 4 entries in this menu ?
  {
***** nicht der Fall           ....
  }
  
  if (MenuSize >3 && MenuSize <8)      // more then 4 and less then 8 entries ?
  {

***** passt: MenuSize > 3

  LCD_CLEAR();
  for( i = 4; i < (MenuSize); ++i )            // loop until last entry reached
    {

***** Wie oft wird diese Schleife ausgeführt, wenn MenuSize einen Wert von 4 hat?

...

PS:
Warum hast du eigentlich den String Pointer nicht gleich in dein struct 
MenuEntry mit aufgenommen? Da mit einem zusätzlichen Pointer Array zu 
hantieren ist nur eine zusätzliche Fehlerquelle mehr.

[Stefan F. (stefan1987)]

Das war das Problem. Danke für die schnellen Antworten.
Ich hab den Source Code nochmal angehängt falls jemand sowas brauchen 
kann.
Ausserdem habe ich auch versucht die Stringpointer in die Structure mit 
reinzupacken allerdings hat das nicht funktioniert.
Hab ich den Absatz falsch verstanden ? Oder nur falsch umgesezt ?

->PS:
->Warum hast du eigentlich den String Pointer nicht gleich in dein 
struct
->MenuEntry mit aufgenommen? Da mit einem zusätzlichen Pointer Array zu
->hantieren ist nur eine zusätzliche Fehlerquelle mehr.

Der neue Source ist auch im Anhang. Wäre super wenn jemand nochmal 
drüber schauen könnte.
Vielen Dank schonmal.