selbstdefiniertes Zeichen wird am LCD nicht richtig ausgegeben

[Jan P. (jiss)]

Hallo,

ich habe mir bei einem LCD 8 Zeichen selbst definiert, welche auf den 
Adressen 0x00 bis 0x07 liegen.
Wenn ich mit lcd_write_char(0x00) z.b. das erste zeichen anzeigen lassen 
will, dann funktioniert dies ohne probleme.
wenn ich mit lcd_write_char(LCD_SYMBOL_CANCEL) das auszugebende Zeichen 
an das LCD senden will, wird dieses auch richtig angezeigt.

Nun möchte ich aber folgendes machen:

  menu_content[0][0].button.level_up.symbol    = LCD_SYMBOL_CANCEL;
  menu_content[0][0].button.back.symbol        = LCD_SYMBOL_UP;
  menu_content[0][0].button.forward.symbol     = LCD_SYMBOL_DOWN;
  menu_content[0][0].button.ok.symbol          = LCD_SYMBOL_OK;

wobei die Zeichen über defines wie folgt definiert sind:

  // self-defined symbols for LCD
  #define        LCD_SYMBOL_CANCEL    0b00000000      // symbol cancel
  #define        LCD_SYMBOL_LEFT      0b00000001      // symbol left
  #define        LCD_SYMBOL_RIGHT     0b00000010      // symbol right
  #define        LCD_SYMBOL_UP        0b00000011      // symbol up
  #define        LCD_SYMBOL_DOWN      0b00000100      // symbol down
  #define        LCD_SYMBOL_OK        0b00000101      // symbol ok
  #define        LCD_SYMBOL_SELECT    0b00000110      // symbol select
  #define        LCD_SYMBOL_LEVEL_UP  0b00000111      // symbol level up

Wenn ich nun mit lcd_write_char-Funktion das CANCEL-Symbol ausgeben 
lassen möchte, dann wird mir nur mist angezeigt (irgendwelche anderen 
ascii-zeichen):

lcd_write_char(menu_content[0][0].button.level_up.symbol);

als datentyp für "menu_content[0][0].button.level_up.symbol" habe ich 
sowohl char als auch int ausprobiert und es funktioniert bei beiden 
nicht.
mit char als 8-bit datentyp sollte es doch aber funktionieren???

das display wird im 4-bit modus betrieben.

[Falk B. (falk)]

Wahrscheinlich hast du beim Laden des CG-RAMs einen Bug drin. Lass man 
den ganzen Schnulli weg ud gibt einfach direkt die Zeichen 0-7 aus. Dann 
weißt du woran du bist.

MFG
Falk

[Jan P. (jiss)]

Falk Brunner schrieb:
> Wahrscheinlich hast du beim Laden des CG-RAMs einen Bug drin. Lass man
> den ganzen Schnulli weg ud gibt einfach direkt die Zeichen 0-7 aus. Dann
> weißt du woran du bist.
>
> MFG
> Falk

Wieso sollte da ein Bug sein? Wenn ich wie gesagt die Adresse direkt in 
lcd_write_char eintrage, dann funktioniert es doch.
hier liegt irgendwie ein problem beim datentyp vor, denn wenn ich.z.b. 
mittels

if (menu_content[0][0].button.level_up.symbol == LCD_SYMBOL_CANCEL)
{ ...
}
else
{
...
}

abfrage, was in der variablen steht, wird immer der else-zweig 
aufgerufen.

[Falk B. (falk)]

@Jan Peukert (jiss)

>Wieso sollte da ein Bug sein? Wenn ich wie gesagt die Adresse direkt in
>lcd_write_char eintrage, dann funktioniert es doch.

Das hast du aber scheinbar nur für Adresse 0 geprüft.

>Wenn ich mit lcd_write_char(0x00) z.b. das erste zeichen anzeigen lassen
>will, dann funktioniert dies ohne probleme.

>abfrage, was in der variablen steht, wird immer der else-zweig
>aufgerufen.

Kann es sein, dass du auf einem AVR programmierst und die Menus im FLash 
liegen? Dann muss man nämlich mit pgm_read_byte drauf zugreifen. Bzw. 
Zeig doch einfach mal deinen VOLLSTÄNDIGEN Quelltext als Anhang.

MFG
Falk

[Helfer (Gast)]

1) Wenn du in Zeile
> if (menu_content[0][0].button.level_up.symbol == LCD_SYMBOL_CANCEL)
einen Breakpoit im Debugger setzt und dir beim Auftreffen auf den 
Breakpoint den Inhalt der Variablen anzeigen lässt - was steht in der 
Variablen drin? Ich würde mir

2) Solltest du bei der Definition von 
menu_content[0][0].button.level_up.symbol unions verwendet haben, denk 
daran, dass Variablen mit unterschiedlichem Namen auf dem gleichen 
Speicherplatz liegen. Es ist dann hochwahrscheinlich nicht den 
erwarteten Inhalt zu finden.

[Walter S. (avatar)]

Jan Peukert schrieb:
> Wieso sollte da ein Bug sein? Wenn ich wie gesagt die Adresse direkt in
> lcd_write_char eintrage, dann funktioniert es doch.
> hier liegt irgendwie ein problem beim datentyp vor, denn wenn ich.z.b.
> mittels

wenn du das schon selber vermutest warum zeigst du verdammt noch Mal 
nicht wie du dieses ominöse Menu definiert hast

[Jan P. (jiss)]

Hier mal alle notwendigen Projektdateien von mir.
in der main() wird die Funktion menu_show_select_and_scroll(...) (siehe 
menu.c) aufgerufen und an diese mit den zuvor definierten Dimensionen 
dim_1 und dim_2 mitgeteilt, dass das MAIN-MENU dargestellt werden soll.
In der menu_init() werden die Menüstrukturen angelegt und dort auch die 
Symbole für die letzte Zeile, welche meine Button-Zeile werden soll, in 
der zuvor genannten Struktur angelegt.

Mit den beiden Funktionen lcd_init() und lcd_selfdefined_symbols() 
werden das Display initialisiert und die selbsterstellten Symbole in den 
CGRAM geladen. Die Symboladressen sind über meine Defines LCD_SYMBOL_XXX 
mit den Adressen 0x00 bis 0x07 festgelegt (in binärschreibweise)

[Walter (Gast)]

wie gross ist dein Speicherbedarf?
Allein das Menue schätze ich schon Mal auf ca. 1KByte

[Jan P. (jiss)]

Walter schrieb:
> wie gross ist dein Speicherbedarf?
> Allein das Menue schätze ich schon Mal auf ca. 1KByte

Program:   14538 bytes (22.2% Full)
(.text + .data + .bootloader)

Data:       1738 bytes (42.4% Full)
(.data + .bss + .noinit)

Bewegt sich immer um diese Zahlen herum. Also noch genügend Luft

[Jan P. (jiss)]

Ich habe gerade etwas seltsames rausgefunden, wo der Fehler anscheinend 
liegt. Nur kann ich mir den Fehler nicht erklären.
Ist vielleicht der Zugriff auf das übergebene Array innerhalb der 
Funktion fehlerhaft?

Wenn ich alles in der Funktion menu_init() hinter der Zeile

  menu_content[1][0].line_menu[2].scroll      = 0;

rauskommentiere, dann funktioniert die Übertragung des Symbols und die 
Anzeige des Menüpunkts main-menu.
Sobald jedoch die Zeile

  // button symbols
  menu_content[1][0].button.level_up.symbol = LCD_SYMBOL_LEVEL_UP;

hinzukommt ist es aus. Das Menü main-menu wird nicht mehr richtig 
angezeigt und es erscheint Code-Kauderwelsch auf dem LCD:
Schaut euch bitte mal an wie ich die der Datei config_and_constants.h 
den Datentyp für menu_content (als t_menu_content) festgelegt habe. Dort 
muss der Hund scheinbar begraben sein??!??

Hier nochmal der gesamte Inhalt der menu_init() bis zu dieser Stelle:

void menu_init(  t_menu_content     menu_content[MENU_ARRAY_DIM_1][MENU_ARRAY_DIM_2], 
        t_butt_state     *butt_state,
        t_config_pins    *config_pins
        )
{
  // variables & constants
  char    string[MENU_LINE_DIM];
  char    add_string[MENU_LINE_DIM];
  t_string  string_array;

  // program-code
  // fill menu_menu_content with menu structure

  // main-menu [0][0]
  menu_content[0][0].show_line_2          = 1;
  menu_content[0][0].show_line_3          = 2;
  menu_content[0][0].submenu_count        = 5;
  
  // title: main-menu
  strcpy(menu_content[0][0].line_title,      "<<<< MAIN-MENU >>>> ");
  
  // menu-lines
  strcpy(menu_content[0][0].line_menu[1].text,    "  conf. analog  pins");
  menu_content[0][0].line_menu[1].selected     = 0;
  menu_content[0][0].line_menu[1].scroll      = 0;

  strcpy(menu_content[0][0].line_menu[2].text,    "  conf. digital pins");
  menu_content[0][0].line_menu[2].selected     = 0;
  menu_content[0][0].line_menu[2].scroll      = 0;

  strcpy(menu_content[0][0].line_menu[3].text,    "  setup controller  ");
  menu_content[0][0].line_menu[3].selected     = 0;
  menu_content[0][0].line_menu[3].scroll      = 0;

  strcpy(menu_content[0][0].line_menu[4].text,    "  rev. information  ");
  menu_content[0][0].line_menu[4].selected     = 0;
  menu_content[0][0].line_menu[4].scroll      = 0;

  strcpy(menu_content[0][0].line_menu[5].text,    "--------------------");
  menu_content[0][0].line_menu[5].selected     = 0;
  menu_content[0][0].line_menu[5].scroll      = 0;

  // button symbols
  menu_content[0][0].button.level_up.symbol    = LCD_SYMBOL_CANCEL;
  menu_content[0][0].button.back.symbol      = LCD_SYMBOL_UP;
  menu_content[0][0].button.forward.symbol    = LCD_SYMBOL_DOWN;
  menu_content[0][0].button.ok.symbol        = LCD_SYMBOL_OK;
  // enable buttons
  menu_content[0][0].button.level_up.enabled    = 0;
  menu_content[0][0].button.back.enabled      = 1;
  menu_content[0][0].button.forward.enabled    = 1;
  menu_content[0][0].button.ok.enabled      = 1;






  // sub-menu configure analog pins [1][0]
  menu_content[1][0].show_line_2          = 1;
  menu_content[1][0].show_line_3          = 2;
  menu_content[1][0].submenu_count        = 6;

  // title: configure analog pins
  strcpy(menu_content[1][0].line_title,      " CONFIG ANALOG PINS ");

  // menu-lines
  strcpy(menu_content[1][0].line_menu[1].text,  "  1  2  3  4  5  6  ");
  menu_content[1][0].line_menu[1].selected     = 0;
  menu_content[1][0].line_menu[1].scroll      = 0;

  // check status and show symbols
  string_array = menu_get_analog_pin_conf_string(config_pins);
  strcpy(menu_content[1][0].line_menu[2].text,  string_array.str);
  menu_content[1][0].line_menu[2].selected     = 0;
  menu_content[1][0].line_menu[2].scroll      = 0;

  // button symbols
  menu_content[1][0].button.level_up.symbol    = LCD_SYMBOL_LEVEL_UP;
/*
 .
 .
 .
*/
}