Problem mit LCD Display

[FX (Gast)]

Hey,

Ich habe ein Problem mit meinem LCD Display.
Orientieren tue ich mich an folgendem Tutorial:
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/LCD-Ansteuerung

Schreibe ich ein Init und dann einen String ist alles wunderbar.
Mache ich ein Init, Text, gefolgt von einem clear und wieder ein Text, 
dann sind das irgendwelche anderen Zeichen... Koreanisch, chinesisch, 
ich weiß es nicht.

Ich möchte und kann aber auch nicht immer wieder den ganzen Init 
durchlaufen lassen, da dort ziemlich viele delays enthalten sind.
Jemand eine Idee?

[Hubert G. (hubertg)]

Du könntest z.B. den Code hier hereinstellen, dann kann man sich das 
ansehen und nicht nur raten.
Du kannst aber auch den Code von Peter Fleury probieren.

[FX (Gast)]

Den Code sieht man ja bei dem Link den ich gepostet habe.

Laufen tut es wie folgt:

lcd_init();
lcd_string(distance);

Das Problem dabei ist, dass jedes mal ein erneuter Init blöd ist. Daher 
würde ich gerne zu begin der main() einmal ein init machen und dann nur 
noch den clear Befehl aufrufen:

lcd_clear();
lcd_string(distance);

Nur dann kommen eben immer nur chinesische Zeichen.

Ich copy/paste einfach mal die init und clear Funktion in den Beitrag:

void lcd_clear( void )
{
  lcd_command( LCD_CLEAR_DISPLAY );
  _delay_ms( LCD_CLEAR_DISPLAY_MS );
}

void lcd_init( void )
{
  // verwendete Pins auf Ausgang schalten
  uint8_t pins = (0x0F << LCD_DB) |           // 4 Datenleitungen
  (1<<LCD_RS) |                // R/S Leitung
  (1<<LCD_EN);                 // Enable Leitung
  LCD_DDR |= pins;
  
  // initial alle Ausgänge auf Null
  LCD_PORT &= ~pins;
  
  // warten auf die Bereitschaft des LCD
  _delay_ms( LCD_BOOTUP_MS );
  
  // Soft-Reset muss 3mal hintereinander gesendet werden zur Initialisierung
  lcd_out( LCD_SOFT_RESET );
  _delay_ms( LCD_SOFT_RESET_MS1 );
  
  lcd_enable();
  _delay_ms( LCD_SOFT_RESET_MS2 );
  
  lcd_enable();
  _delay_ms( LCD_SOFT_RESET_MS3 );
  
  // 4-bit Modus aktivieren
  lcd_out( LCD_SET_FUNCTION |
  LCD_FUNCTION_4BIT );
  _delay_ms( LCD_SET_4BITMODE_MS );
  
  // 4-bit Modus / 2 Zeilen / 5x7
  lcd_command( LCD_SET_FUNCTION |
  LCD_FUNCTION_4BIT |
  LCD_FUNCTION_2LINE |
  LCD_FUNCTION_5X7 );
  
  // Display ein / Cursor aus / Blinken aus
  lcd_command( LCD_SET_DISPLAY |
  LCD_DISPLAY_ON |
  LCD_CURSOR_OFF |
  LCD_BLINKING_OFF);
  
  // Cursor inkrement / kein Scrollen
  lcd_command( LCD_SET_ENTRY |
  LCD_ENTRY_INCREASE |
  LCD_ENTRY_NOSHIFT );
  
  lcd_clear();
}

[Spess53 (Gast)]

Hi

Ich benutze zwar kein C, aber das Problem dürfte lcd_string(distance); 
sein
Wie ist distance definiert?

MfG Spess

[FX (Gast)]

Stimmt, wenn ich einfach nur "Test" nehme geht es auch mit clear.

So wird distance erstellt:
(Also in diesem Fall würde ich lcd_string(dist); ausgeben)

int16_t dist = distances / SRF02_WERT_MITTELWERTANZAHL;
distances = 0;
                
char dist[] = "Distance: ";
char unit[] = " cm";
char Buffer[5];
                
itoa(x, Buffer, 10);
strcat(dist, Buffer);
strcat(dist, " cm");

[0815 (Gast)]

LC Display nicht LC Disply Display  LCD D;)

[Spess53 (Gast)]

Hi

>distances = 0;

lcd_string verlangt aber einen Zeiger auf einen String.

MfG Spess

[FX (Gast)]

Hab das problem gefunden :(

Ich hatte zum Debugging von einer anderen Sache (da noch LED's) auf 
PortD 0xFF geschickt... nur an PortD ist jetzt das LCD und das hat dem 
LCD wohl nicht gefallen! :D

Aber danke für die Hilfe :)

[Karl H. (kbuchegg)]

FX schrieb:

>

int16_t dist = distances / SRF02_WERT_MITTELWERTANZAHL;
> distances = 0;
> 
> char dist[] = "Distance: ";
> char unit[] = " cm";
> char Buffer[5];
> 
> itoa(x, Buffer, 10);
> strcat(dist, Buffer);
> strcat(dist, " cm");
>

Und jetzt gehst du gleich mal her und machst das Array dist größer.
Der Compiler hat dir das Array so dimensioniert, dass "Distance: " da 
gerade noch reingeht. Du kannst da nicht noch hinten einige Dinge drann 
hängen. Dein Array ist dafür nicht groß genug!

D.h. können tust du technisch gesehen schon. Nur bügelst du dann im 
Speicher Dinge über den Haufen, die nicht mehr zum Array gehören. Dem 
strcat ist das egal. Der beschreibt den Speicher einfach ohne Rücksicht 
auf Verluste.

[Blackbird (Gast)]

>> Laufen tut es wie folgt:lcd_init();
>> lcd_string(distance);

"distance" wird auch in seinem anderen Posting nicht deklariert, sondern 
nur verwendet und zugewiesen.

"dist[]" ist was Anderes als "distance".

Wir wissen also immer noch nicht, von welchem Type "distance" ist.

Aber der Fehler lag ja wieder ganz woanders.

Blackbird