LCD Routine klappt nicht

[Jürgen (Gast)]

Ich nutze immer wieder gerne mal die LCD Routine von MyAVR.
Normalerweise ist das Display in dieser Routine folgendermaßen 
angeschlossen:

Bit 4-7 - PortD Bit 4-7 ----> Neu PortD Bit 0-3
RS      - PortD Bit 2   ----> Neu PortD Bit 5
E       - PortD Bit 3   ----> Neu PortD Bit 4

Ich war eigentlich der Meinung ich hätte alles soweit angepasst aber bis 
auf jeweils einen schwarzen Balken in Zeile 1 und 2 kommt leider nichts.

Ich gehe davon aus das ich in der lcd_send Routine etwas falsch habe.
Ich habe im übrigen ein 4x20 Zeilen Display und betreibe es im 4 Bit 
Modus.

Hier mal die gesamten Funktionen.
Bedanke mich jetzt schonmal für jegliche Hilfe.

//========================================================
void lcd_send(char data)
{
  char rs=PORTD;
  rs&=20;        // aktuelles RS ermitteln
  
  char tmp=data;   // high teil
  tmp&=0xf0;
  tmp = tmp >> 4;
  tmp|=rs;
  
  PORTD=tmp;
  
  sbi(PORTD,4);  // schreiben
  cbi(PORTD,4);
  
  tmp=data;      // low teil
  tmp&=0x0f;
         tmp*=16;
  tmp|=rs;
  
  PORTD=tmp;
  
  sbi(PORTD,4);  // schreiben
  cbi(PORTD,4);
  waitMs(1);
}
//========================================================
void lcd_cmd(char cmd)
{
  cbi(PORTD,5);    
  lcd_send(cmd);  
}
//========================================================
void lcd_write(char text)
{
  sbi(PORTD,5);    
  lcd_send(text);
}
//========================================================
void lcd_write(const char* pText)
{
  while(pText[0]!=0)
  {
    lcd_write(pText[0]);
    pText++;
  }
}
//========================================================
void lcd_write(char* pText, int count)
{
  while(count!=0)
  {
    lcd_write(pText[0]);
    pText++;
    count--;
  }
}
//========================================================
void lcd_home()
{
  lcd_cmd(0x02);
  waitMs(2);      
}
//========================================================
void lcd_clear()
{
  lcd_cmd(0x01);
  waitMs(2);      
}
//========================================================
void lcd_on()
{
  lcd_cmd(0x0E);
}
//========================================================
void lcd_off()
{
  lcd_cmd(0x08);
}
//========================================================
void lcd_goto(int row, int col)
{
  row--;        
  row&=0x01;      
  row*=0x40;      
  col--;        
  col&=0x0f;      
  char tmp=row|col;
  tmp|=0x80;      
  lcd_cmd(tmp);    
}
//========================================================
void lcd_init()
{
  PORTD = 0b00000011;  // soft reset
  sbi(PORTD,4);
  cbi(PORTD,4);
  waitMs(5);
  PORTD = 0b00000011;  
  sbi(PORTD,4);
  cbi(PORTD,4);
  waitMs(1);
  PORTD = 0b00000011;  
  sbi(PORTD,4);
  cbi(PORTD,4);
  waitMs(5);
  
  PORTD=0b00000010; // 4 Bit aktivieren
  sbi(PORTD,4);
  cbi(PORTD,4);
  waitMs(5);

  lcd_cmd (0b00101000); // function set  
  lcd_cmd (0b00000110); // entry mode
  lcd_on();
  lcd_clear(); 
  
}
//========================================================

[Karl H. (kbuchegg)]

  sbi(PORTD,4);  // schreiben
  cbi(PORTD,4);
  
  tmp=data;      // low teil
  tmp&=0x0f;
         tmp*=16;
  tmp|=rs;

Wieso *16?
Dadurch verschiebst du ja die unteren 4 Bits in die höheren 4 Bits.

> ch war eigentlich der Meinung ich hätte alles soweit angepasst
> aber bis auf jeweils einen schwarzen Balken in Zeile 1 und 2 kommt
> leider nichts.

Normalerweise ein Indiz, dass die Intialisierung nicht klappt.

[mschoeldgen (Gast)]

Jürgen schrieb:
> PORTD=tmp;

Mit dem Befehl überschreibst du auch die Kontrollsignale, die müssen 
vorher ausmaskiert bzw. reinverODERert werden z.B.
rs = PIND & 0xf0;
PORTD = rs | tmp;

[Klaus (Gast)]

Tschuldigung!

mir schmerzen irgendwie die Augen!

Bekommst du keine Fehlermeldungen 3 mal lcd_write??


void lcd_write(char text)
void lcd_write(const char* pText)

void lcd_write(char* pText, int count)
{
  while(count!=0)
  {
    lcd_write(pText[0]);    Irgendwie sieht das nach Rekursiven aufruf 
aus.
    pText++;
    count--;
  }
}

In der Init_routine vermisse ich den Löschbefehl.
Auserdem werden die Comandos nicht im 4-Bit Modus gesendet.

Also ab hier:
  lcd_cmd (0b00101000); // function set
  lcd_cmd (0b00000110); // entry mode
  lcd_on();
  lcd_clear();

[Jürgen (Gast)]

Karl Heinz Buchegger schrieb:
> Wieso *16?
>
> Dadurch verschiebst du ja die unteren 4 Bits in die höheren 4 Bits.

Sorry stimmt das kam noch aus der originalen Routine.

Aber selbst ohne funktioniert es leider nicht.

Ich habe ehrlich gesagt auch das Gefühl, das das Display gar nicht 
wirklich bootet.

Vielleicht ist irgendetwas am Function set krumm.

[Klaus (Gast)]

Sorry!

void lcd_cmd(char cmd)
{
  cbi(PORTD,5);
  lcd_send(cmd);
}


übersehen!

[Karl H. (kbuchegg)]

Kannst du nicht das LCD erst mal temporär so verkabeln, dass du die 
Originalroutine benutzen kannst?

Dann weißt du wenigstens, ob es Timingprobleme gibt.

schwarze Balken deuten darauf hin, dass die Initialisierung schon nicht 
klappt. Und das ist meistens ein Timimgproblem.

[Jürgen (Gast)]

Also es war der Watchdog Timer an sodass gar nichts passierte.

Hab den jetzt soweit rausgenommen und das Display bootet zumindest 
schonmal. Die schwarzen Balken sind also weg.

Eine Ausgabe bekomme ich dennoch nicht hin ?!

[Jürgen (Gast)]

Ok zur allgemeinen Info.
Jetzt klappt alles. Ich musste nur noch rs&=20 gegen rs&=0b00100000 
tauschen

[Düsendieb (Gast)]

Jürgen schrieb:
> Ok zur allgemeinen Info.
> Jetzt klappt alles. Ich musste nur noch rs&=20 gegen rs&=0b00100000
> tauschen

Tja, 20 ist ungleich 0x20

[Karl H. (kbuchegg)]

Jürgen schrieb:
> Ok zur allgemeinen Info.
> Jetzt klappt alles. Ich musste nur noch rs&=20 gegen rs&=0b00100000
> tauschen

Argh. Und ich bin das ganze ein paar mal im Kopf duchgegangen und hab 
intuitiv 0x20 gelesen.

Ein Tipp.
Spar dir diese Zahlen! Schreib das ganze so:

#define RS_PIN     5
#define E_PIN      4


void lcd_send(char data)
{
  char rs=PORTD;
  rs &= (1<<RS_PIN);
  
  char tmp=data;   // high teil
  tmp&=0xf0;
  tmp = tmp >> 4;
  tmp|=rs;
  
  PORTD=tmp;
  
  sbi(PORTD,E_PIN);  // schreiben
  cbi(PORTD,E_PIN);
...

void lcd_cmd(char cmd)
{
  cbi(PORTD,RS_PIN);    
  lcd_send(cmd);  
}
....

und natürlich überall anders auch die konstanten Zahlen gegen Makros 
austauschen. Dann passieren dir nicht solche Flüchtigkeitsfehler und 
eine Anpassung an andere Pin-Nummern ist auch einfacher: einfach nur im 
#define die andere Zahl eintragen und den Rest macht der Compiler.
(Man kann das auch mit den Datenleitungen so hinkriegen, allerdings wird 
der Code dann ein wenig komplizierter)

Es liegt in deiner Hand, dir den Code so zu gestalten, dass du wenige 
bis keine Portierungsfehler machen kannst. Du musst es nur tun! Die 
Zeit, die du jetzt für Fehlersuche verbraten hast, ist ein Vielfaches 
dessen, was dir eine derartige Codeänderung zeitmässig gekostet hätte.