Problem mit der Ansteuerung LCD (HD44780)

[Juergen B. (jbaben)]

Hallo,

ich habe ein Problem bei der Anzeige von Text auf einem LC-Display 
(HD44780, 2x8).
Es soll folgender Text ausgegeben werden: Zeile 1: "CIRCUIT ", Zeile 2: 
"DIGEST  ".
Der Text wird aber nicht richtig angezeigt.
Z.B. nach einem Reset: "CCCCCCCC" und dann weiter Zeichen auf beiden 
Zeilen.
Meine Hardware:
Mikrocontroller: AT89C51RC2 mit Quarz 10MHz
LCD: HD44780 mit 2x8 Zeichen.
Übertragung mit Flip (Microchip, ATMEL)

Ich habe schon mehrere Beispiele ausprobiert (auch Assembler), aber 
sobald ich mehrere Zeichen ausgebe werden diese mehrfach ausgegeben.
Hier mein Programm Code (Keil C).

// Program for LCD Interfacing with 8051 Microcontroller (AT89S52) 
// https://circuitdigest.com/microcontroller-projects/lcd-interfacing-with-8051-microcontroller-89s52

#include<reg51.h>
#define display_port P2      //Data pins connected to port 2 on microcontroller
sbit rs = P0^0;  // RS pin connected to pin 1 of port 0
sbit rw = P0^2;  // RW pin connected to pin 3 of port 0
sbit e =  P0^1;  // E pin connected to pin 2 of port 0

void msdelay(unsigned int time)  // Function for creating delay in milliseconds.
{
    unsigned i,j ;
    for(i=0;i<time;i++)    
    for(j=0;j<1275;j++);
}
void lcd_cmd(unsigned char command)  //Function to send command instruction to LCD
{
    display_port = command;
    rs= 0;
    rw=0;
    e=1;
    msdelay(5);
    e=0;
}

void lcd_data(unsigned char disp_data)  //Function to send display data to LCD
{
    display_port = disp_data;
    rs= 1;
    rw=0;
    e=1;
    msdelay(5);
    e=0;
}

 void lcd_init()    //Function to prepare the LCD  and get it ready
{
    lcd_cmd(0x38);  // for using 2 lines and 5X7 matrix of LCD
    msdelay(10);
    lcd_cmd(0x38);
    msdelay(10);
    lcd_cmd(0x38);
    msdelay(10); 
    lcd_cmd(0x0C);  // turn display ON, cursor blinking 0x0F
    msdelay(10);
    lcd_cmd(0x01);  //clear screen 0x01
    msdelay(10);
    lcd_cmd(0x02);  // cursor home
    msdelay(10);
    lcd_cmd(0x80);  // bring cursor to position 1 of line 1
    msdelay(10);
}
void main()
{
    unsigned char a1[8]="CIRCUIT ";    //string of 8 characters with a null terminator.
    unsigned char a2[8]="DIGEST  "; 
    int l=0;
    lcd_init();
    while(a1[l] != '\0') // searching the null terminator in the sentence
    {
        lcd_data(a1[l]);
        l++;
        msdelay(50);
    }
    l=0;
    lcd_cmd(0xC0);        // bring cursor to position 1 of line 2
    msdelay(10);
     while(a2[l] != '\0') // searching the null terminator in the sentence
    {
        lcd_data(a2[l]);
        l++;
        msdelay(50);
    }
}

Bei dem Beispiel wird als Mikrocontroller AT89S52 angegeben ich werwende 
aber AT89C51.
Ist das ein Problem ?

Woran könnte mein Problem liegen ?

MfG
Juergen B.

[Mr. Claudius (Gast)]

P0 ist open-drain, fehlen vielleicht externe Pullup-Widerstände?

[Heinz (Gast)]

Juergen B. schrieb:
> unsigned char a1[8]="CIRCUIT ";    //string of 8 characters with a null
> terminator.
>     unsigned char a2[8]="DIGEST  ";

Mit den Leerzeichen am Ende sind Deine Strings schon 8 Zeichen lang und 
der Platz für den "Null Terminator" fehlt. Entweder Leerzeichen 
wegmachen oder Strings längermachen.

[Juergen B. (jbaben)]

Hallo,

vielen Dank für die schnellen Antworten.
Der Hinweis auf die fehlenden Pullup-Widerstände hat geholfen.
Ich habe die drei Ausgänge (RS, E, RW) mit einem Widerstand 10K gegen 
+5V verbunden, jetzt funktioniert die Textausgabe.
Den Hinweis auf den Port 0 habe ich wohl im Datenblatt vom AT89C51RC2 
übersehen.

MfG

Juergen B.

[Astro (Gast)]

Den Hinweis von Heinz, daß deine Arrays zu klein sind, solltest du aber 
nicht auch noch übersehen.

[Juergen B. (jbaben)]

Hallo,

habe ich auch nicht.

MfG

Juergen B.