16x2 LCD nur eine Zeile mit Bitte den Code zu prüfen

[Tim (Gast)]

Hallo zusammen,

warum zeigt das 16x2 Display nur eine, nämlich bei der oberen Zeilen die 
Buchstaben an?

#define F_CPU  4000000UL
#include <avr/delay.h>

#include <avr/io.h>


/*LCD function declarations */
void LCD_send_command(unsigned char cmnd);
void LCD_send_data(unsigned char data);
void LCD_init();
void LCD_goto(unsigned char y, unsigned char x);
void LCD_print(char *string);
void LCD_blink();
void LCD_scroll(unsigned char direction);

#define LCD_DATA_PORT PORTC
#define LCD_DATA_DDR  DDRC
#define LCD_DATA_PIN  PINC

#define LCD_CNTRL_PORT  PORTD
#define LCD_CNTRL_DDR  DDRD
#define LCD_CNTRL_PIN  PIND

#define LCD_RS_PIN    2
#define LCD_RW_PIN    3
#define LCD_ENABLE_PIN  4

int main(void)
{
  unsigned char i;
  
  LCD_init();
  LCD_goto(1,2);
  LCD_print("AVR TUTORIALS");
  LCD_goto(2,1);
  LCD_print("WELCOMES YOU");
  
    while(1)
    {/*
         for(i=0;i<2;i++)
         LCD_blink();
         
         for(i=0;i<2;i++)
         LCD_scroll(0);
         
         for(i=0;i<4;i++)
         LCD_scroll(1);
         
         for(i=0;i<2;i++)
         LCD_scroll(0);*/
    }
}

/* This function sends a command 'cmnd' to the LCD module*/
void LCD_send_command(unsigned char cmnd)
{
  LCD_DATA_PORT = cmnd;
  LCD_CNTRL_PORT &= ~(1<<LCD_RW_PIN);
  LCD_CNTRL_PORT &= ~(1<<LCD_RS_PIN);
  
  LCD_CNTRL_PORT |= (1<<LCD_ENABLE_PIN);
  _delay_us(2);
  LCD_CNTRL_PORT &= ~(1<<LCD_ENABLE_PIN);
  _delay_us(100);
}

/* This function sends the data 'data' to the LCD module*/
void LCD_send_data(unsigned char data)
{
  LCD_DATA_PORT = data;
  LCD_CNTRL_PORT &= ~(1<<LCD_RW_PIN);
  LCD_CNTRL_PORT |= (1<<LCD_RS_PIN);
  
  LCD_CNTRL_PORT |= (1<<LCD_ENABLE_PIN);
  _delay_us(2);
  LCD_CNTRL_PORT &= ~(1<<LCD_ENABLE_PIN);
  _delay_us(100);
}

void LCD_init()
{
  LCD_CNTRL_DDR = 0xFF;
  LCD_CNTRL_PORT = 0x00;
  LCD_DATA_DDR = 0xFF;
  LCD_DATA_PORT = 0x00;
  
  _delay_ms(10);
  LCD_send_command(0x38);
  LCD_send_command(0x0C);
  LCD_send_command(0x01);
  _delay_ms(10);
  LCD_send_command(0x06);
}

/* This function moves the cursor the line y column x on the LCD module*/
void LCD_goto(unsigned char y, unsigned char x)
{
  unsigned char firstAddress[] = {0x80,0xC0,0x94,0xD4};
  
  LCD_send_command(firstAddress[y-1] + x-1);
  _delay_ms(10);
}

void LCD_print(char *string)
{
  unsigned char i;
  
  while(string[i]!=0)
  {
    LCD_send_data(string[i]);
    i++;
  }
}

void LCD_blink()
{
  LCD_send_command(0x08);
  _delay_ms(250);
  LCD_send_command(0x0C);
  _delay_ms(250);
}

void LCD_scroll(unsigned char direction)
{
  if(direction == 0)
  LCD_send_command(0x18);
  else
  LCD_send_command(0x1C);
  
  _delay_ms(500);
}

Kann mir jemand sagen wo der Fehler ist?

Dank

Tim

[Karl H. (kbuchegg)]

Tim schrieb:

> warum zeigt das 16x2 Display nur eine, nämlich bei der oberen Zeilen die
> Buchstaben an?

Weil diese Schreibweise

  LCD_send_command(0x38);
  LCD_send_command(0x0C);
  LCD_send_command(0x01);

scheisse ist.
Jetzt heisst es wieder Datenblatt bzw. die LCD Kommandos studieren um 
rauszufinden welches das Kommando ist, mit dem eigentlich dem LCD 
mitgeteilt wird, dass es auf 2 Zeilen schalten soll und um rauszufinden 
ob das passiert ist oder nicht.

[Tim (Gast)]

Darf ich fragen, wie die gute Programmierung aussieht?

speziell nur mal die 3 Zeilen.

Danke

[Karl H. (kbuchegg)]

So zum Beispiel

...
 
    // 4-bit Modus / 2 Zeilen / 5x7
    lcd_command( LCD_SET_FUNCTION |
                 LCD_FUNCTION_4BIT |
                 LCD_FUNCTION_2LINE |
                 LCD_FUNCTION_5X7 );
...

Quelle:
http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/LCD-Ansteuerung

Kann natürlich sein, dass du 2 Zeilen aktivierst und ganz einfach nur 
die Konstanten für den Zeilenversatz in der Cursorpositionierung falsch 
sind. Kann alles sein. Aber ohne die Bits in

  LCD_send_command(0x38);
  LCD_send_command(0x0C);
  LCD_send_command(0x01);

in ihre Bedeutung aufzudröseln, werden wir es nie erfahren.

[John (Gast)]

Karl Heinz schrieb:
> Kann alles sein. Aber ohne die Bits in
>   LCD_send_command(0x38);
>   LCD_send_command(0x0C);
>   LCD_send_command(0x01);
> in ihre Bedeutung aufzudröseln, werden wir es nie erfahren.

Man könnte auch einfach die HEX-Werte in das Hex-Feld im LCD Simulator 
eingeben. Und schon wird in der Tip-Box angezeigt was der Befehl macht. 
Für Befehle muss R/W und RS low sein (Grundeinstellung vom Simulator).
Nach zwei Klicks auf die Schaltfläche unter den E (-> Enabel high -> 
Enable low), wird der Befehl im Simulator ausgeführt.

LCD Simulator:
http://www.dinceraydin.com/djlcdsim/djlcdsim.html

Gruß
John