LCD nicht nur für einen Port in C

[Hans M. (gast4711)]

Habe zwar die LCD-Routine aus dem AVR-GCC-Tutorial mit Erfolg getestet:
Lcd.c:

#include <avr/io.h>
#include "lcd-routines.h"
int main(void){
  lcd_string("Hello World!");
  while(1);
  return 0;
}

Leider habe ich aber eine vorgegebene Hardware, an der RS und EN nicht 
an PD4 bzw. PD5 liegen, sondern
#define LCD_RS        PB7
#define LCD_EN        PB6
Diese Umdefinitionen reichen allein natürlich nicht aus, um das Programm 
wieder zum Laufen zu bringen.
Habe auch keine Ahnung vom AVR (Kenne nur 8051, wo man mit sbit RS=P1^2; 
und dann RS=1; einzelne Portleitungen setzen kann) und weiß nicht wie 
das bei Atmel geht, geschweige denn, was so komplexe Sequenzen wie
LCD_DDR = LCD_DDR | 0x0F | (1<<LCD_RS) | (1<<LCD_EN);
genau machen.

Vielleicht findet sich ja hier ein Profi, der sich herausgefordert fühlt 
das zu realisieren (oder hat so etwas schon einmal für verschiedene 
Portpins gemacht)?

Wäre sehr, sehr dankbar, wenn er es hier posten könnte !!!


------------------------------------------------------------------------ 
------------------------------------------------
Hier nochmal alle Quelltexte:
lcd-routines.h:

void lcd_data(unsigned char temp1);
void lcd_string(char *data);
void lcd_command(unsigned char temp1);
void lcd_enable(void);
void lcd_init(void);
void lcd_home(void);
void lcd_clear(void);
void set_cursor(uint8_t x, uint8_t y);
#define F_CPU 8000000
#define CLEAR_DISPLAY 0x01
#define CURSOR_HOME   0x02
#define LCD_PORT      PORTD
#define LCD_DDR       DDRD
#define LCD_RS        PD4
#define LCD_EN        PD5

lcd-routines.c:

 
#include <avr/io.h>
#include "lcd-routines.h"
#include <util/delay.h>
void lcd_data(unsigned char temp1)
{
   unsigned char temp2 = temp1;
  LCD_PORT |= (1<<LCD_RS);        // RS auf 1 setzen
  temp1 = temp1 >> 4;
  temp1 = temp1 & 0x0F;
  LCD_PORT &= 0xF0;
  LCD_PORT |= temp1;               // setzen
  lcd_enable();
  temp2 = temp2 & 0x0F;
  LCD_PORT &= 0xF0;
  LCD_PORT |= temp2;               // setzen
  lcd_enable();
  _delay_us(42);
}
void lcd_command(unsigned char temp1)
{
  unsigned char temp2 = temp1;
  LCD_PORT &= ~(1<<LCD_RS);        // RS auf 0 setzen
  temp1 = temp1 >> 4;              // oberes Nibble holen
  temp1 = temp1 & 0x0F;            // maskieren
  LCD_PORT &= 0xF0;
  LCD_PORT |= temp1;               // setzen
  lcd_enable();
  temp2 = temp2 & 0x0F;            // unteres Nibble holen und maskieren
  LCD_PORT &= 0xF0;
  LCD_PORT |= temp2;               // setzen
  lcd_enable();
 _delay_us(42);
}
void lcd_enable(void)
{
  LCD_PORT |= (1<<LCD_EN);
  _delay_us(1);                   // kurze Pause
  LCD_PORT &= ~(1<<LCD_EN);
}
void lcd_init(void)
{
  LCD_DDR = LCD_DDR | 0x0F | (1<<LCD_RS) | (1<<LCD_EN);   // Port auf Ausgang schalten
  _delay_ms(15);
  LCD_PORT &= 0xF0;
  LCD_PORT |= 0x03;            
  LCD_PORT &= ~(1<<LCD_RS);      // RS auf 0
  lcd_enable();
  _delay_ms(5);
  lcd_enable();
  _delay_ms(1);
  lcd_enable();
  _delay_ms(1);
  LCD_PORT &= 0xF0;
  LCD_PORT |= 0x02;
  lcd_enable();
  _delay_ms(1);
  lcd_command(0x28);
  lcd_command(0x0C); 
  lcd_command(0x06);
  lcd_clear();
}
void lcd_clear(void)
{
  lcd_command(CLEAR_DISPLAY);
  _delay_ms(5);
}
void lcd_home(void)
{
  lcd_command(CURSOR_HOME);
  _delay_ms(5);
}
void set_cursor(uint8_t x, uint8_t y)
{
  uint8_t tmp;
  switch (y) {
    case 1: tmp=0x80+0x00+x; break;    // 1. Zeile
    case 2: tmp=0x80+0x40+x; break;    // 2. Zeile
    case 3: tmp=0x80+0x10+x; break;    // 3. Zeile
    case 4: tmp=0x80+0x50+x; break;    // 4. Zeile
  }
  lcd_command(tmp);
}
void lcd_string(char *data)
{
  while(*data) {
     lcd_data(*data);
     data++;
  }
}

[Karl H. (kbuchegg)]

Hans Mustermann wrote:
> Habe zwar die LCD-Routine aus dem AVR-GCC-Tutorial mit Erfolg getestet:
> Lcd.c:
>

> #include <avr/io.h>
> #include "lcd-routines.h"
> int main(void){
>   lcd_string("Hello World!");
>   while(1);
>   return 0;
> }
>

>
> Leider habe ich aber eine vorgegebene Hardware, an der RS und EN nicht
> an PD4 bzw. PD5 liegen, sondern
> #define LCD_RS        PB7
> #define LCD_EN        PB6

Was ist mit den restlichen Datenleitungen? Liegen die ebenfalls
am Port B? Welche Pins?

Wenn die Datenleitungen ebenfalls am PORTB, Pins PB0 bis PB4 liegen,
dann reichen folgende Änderungen aus:


> #define LCD_PORT      PORTB
> #define LCD_DDR       DDRB
> #define LCD_RS        PB7
> #define LCD_EN        PB6

und das LCD sollte am PortB laufen.

[roboter (Gast)]

Pin #-LCD  Bezeichnung-LCD  Pin-µC
1  Vss  GND
2  Vcc  5V
3  Vee  GND oder Poti (siehe oben)
4  RS  PD4 am AVR
5  RW  GND
6  E  PD5 am AVR
7  DB0  offen
8  DB1  offen
9  DB2  offen
10  DB3  offen
11  DB4  PD0 am AVR
12  DB5  PD1 am AVR
13  DB6  PD2 am AVR
14  DB7  PD3 am AVR

das oben steht in der beschreibung. also musste db4 bis db7 an 
port0-port3 anschliessen.

[Stefan B. (stefan)]

Wenn die Datenleitungen wie im Tutorial an PORTD liegen und nur die 
Steuerleitungen wie von dir angegeben an PORTB, dann könntest du so 
vorgehen

#define LCD_PORT      PORTD
#define LCD_DDR       DDRD
#define LCD_RS        PD4
#define LCD_EN        PD5

ersetzt du durch

#define LCD_CTRL_PORT      PORTB
#define LCD_CTRL_DDR       DDRB
#define LCD_RS             PB7
#define LCD_EN             PB6 

#define LCD_DATA_PORT      PORTD
#define LCD_DATA_DDR       DDRD

In jedem Statement (Zeile) mit LCD_RS oder LCD_EN ersetzt du LCD_PORT 
durch LCD_CTRL_PORT.

Und in jeder anderen Zeile mit LCD_PORT ersetzt du LCD_PORT durch 
LCD_DATA_PORT

Den Abschnitt

  // Port auf Ausgang schalten
  LCD_DDR = LCD_DDR | 0x0F | (1<<LCD_RS) | (1<<LCD_EN);

ersetzt du durch

  // Ports auf Ausgang schalten
  LCD_DATA_DDR = LCD_DATA_DDR | 0x0F;
  LCD_CTRL_DDR = LCD_CTRL_DDR | (1<<LCD_RS) | (1<<LCD_EN);

[Hans M. (gast4711)]

kbuchegg wrote:
>Wenn die Datenleitungen ebenfalls am PORTB, Pins PB0 bis PB4 liegen,
>dann reichen folgende Änderungen aus: ...
Für so eine einfache Portsubstitution hätte ich dann doch das Forum 
nicht bemüht :)

stefan wrote:
>...

Habe jetzt wie beschrieben
#define LCD_CTRL_PORT      PORTB
#define LCD_CTRL_DDR       DDRB
#define LCD_RS             PB7
#define LCD_EN             PB6
#define LCD_DATA_PORT      PORTD
#define LCD_DATA_DDR       DDRD
und in LCD_RS- bzw LCD_EN-Zeilen LCD_PORT durch LCD_CTRL_PORT ersetzt
und in jeder anderen Zeil LCD_PORT durch LCD_DATA_PORT ersetzt und
es klappt tatsächlich!

Weiß nur noch nicht ganz sicher, ob alle andern, "freien" Pins 
(PD4...PD7 und PB0...PB5) noch uneingeschränkt anderweitige Aufgaben 
erfüllen können, ohne von LCD-Routinen beeinflußt zu werden.

Denn für einen AVR-Neuling sind im Gegensatz zu RS=1 bei Intel so Arten 
von “Schiebeoperationen“ zum Portsetzen wie
PORTB |= (1<<PB0);
doch etwas kryptisch (muß mich bei Zeiten mal da einarbeiten; auch ADW 
und Interruptsteuerung...).

Dank‘ Dir auf jeden Fall Stefan für die Hilfe!

[Karl H. (kbuchegg)]

Hans Mustermann wrote:
> kbuchegg wrote:
>>Wenn die Datenleitungen ebenfalls am PORTB, Pins PB0 bis PB4 liegen,
>>dann reichen folgende Änderungen aus: ...
> Für so eine einfache Portsubstitution hätte ich dann doch das Forum
> nicht bemüht :)

Dann solltest du dir zumindest vormerken, das nächste
vollständige Angaben in einer Anfrage zu machen :-)

[klaus (Gast)]

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

da drin ist die lcd-routine ein bisschen beschissen erstellt worden.
der lernende fragt sich , wo werden die pins0-pins3 in den routinen 
herausgestellt. es ist kein bezug da.

also, wenn was dargestellt werder soll, ein  bisschen mehr licht ins 
dunkel, dann hört auch die diskussion auf.

[Stefan B. (stefan)]

klaus wrote:
> http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial
>
> da drin ist die lcd-routine ein bisschen beschissen erstellt worden.

Ein "bisschen beschissen" gibt es nicht. Du solltest dir überlegen, 
welche Motivation du einem Artikelautor mit einem solchen Statement 
gibst, für dich und andere das Tutorial zu verbessern. Nein, ich bin 
nicht der Autor. Ja, ich finde, der Autor hat gute Arbeit geleistet.

> der lernende fragt sich , wo werden die pins0-pins3 in den routinen
> herausgestellt. es ist kein bezug da.
>
> also, wenn was dargestellt werder soll, ein  bisschen mehr licht ins
> dunkel, dann hört auch die diskussion auf.

Ich habe versucht aus deinem Text deinen Wunsch heraus zu klauben. Wenn 
es darum geht anzuzeigen, wo die Leitungen DB4 bis DB8 des LCD am AVR 
angeschlossen sind, dann steht das im Tutorialtext eindeutig drin. In 
dem Sourcecode steht es nicht eindeutig drin. Die Pinbelegung habe ich 
mit einem Kommentar im Quelltext ergänzt. Ich hoffe, das ist auch im 
Sinn des Tutorialautors.

[Karl H. (kbuchegg)]

klaus wrote:
> http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial
>
> da drin ist die lcd-routine ein bisschen beschissen erstellt worden.
> der lernende fragt sich , wo werden die pins0-pins3 in den routinen
> herausgestellt. es ist kein bezug da.

Es wird auch davon ausgegangen, dass 'der Lernende' zumindest
über Grundkenntnisse in C verfügt, bevor er sich an ein LCD wagt.

Für jemanden mit Grundkenntnissen, der auch bereit ist
den Code den er da abtippt zu verstehen, ist es absolut kein
Problem im veröffentlichten Code die entsprechenden Stellen
zu finden, zu verstehen und zu ändern.

[Peter D. (peda)]

Hans Mustermann wrote:
> Habe auch keine Ahnung vom AVR (Kenne nur 8051, wo man mit sbit RS=P1^2;
> und dann RS=1; einzelne Portleitungen setzen kann) und weiß nicht wie
> das bei Atmel geht

Eigentlich ganz genau so, wie beim 8051, nur daß man noch die 
Direction-Bits setzen muß.

Die Bitzugriffe kann man mit einem Macro (SBIT) definieren.

Anbei mal ein Beispiel für 2*40 LCD im 4Bit-Mode mit jedem beliebigen 
Pin.


Peter

[roboter (Gast)]

...Für jemanden mit Grundkenntnissen, der auch bereit ist
den Code den er da abtippt zu verstehen, ist es absolut kein
Problem im veröffentlichten Code die entsprechenden Stellen
zu finden, zu verstehen und zu ändern....

da sehe ich in einem tut anders, da gehört aufklärung rein, auch im 
code.

[roboter (Gast)]

....Anbei mal ein Beispiel für 2*40 LCD im 4Bit-Mode mit jedem 
beliebigen
Pin.......


dieses beispiel ist auch für anfänger wunderbar geschrieben, klasse.

mfg

[bastle (Gast)]

typedef unsigned char  u8;
typedef   signed char  s8;
typedef unsigned short u16;
typedef   signed short s16;
typedef unsigned long  u32;
typedef   signed long  s32;

welchen vorzug hat diese schreibweise? text zu sparen? u8 statt uint8_t 
!

[Karl H. (kbuchegg)]

roboter wrote:
> ...Für jemanden mit Grundkenntnissen, der auch bereit ist
> den Code den er da abtippt zu verstehen, ist es absolut kein
> Problem im veröffentlichten Code die entsprechenden Stellen
> zu finden, zu verstehen und zu ändern....
>
> da sehe ich in einem tut anders, da gehört aufklärung rein, auch im
> code.


Ist ganz einfach.
Wenn du denkst, dass diese Stellen nicht gut genug einen
Sachverhalt beschreiben, dann ändere sie. Genau aus diesem
Grund ist das ja ein Wiki.

[Spitti (Gast)]

Warum wird eigentlich bei dir der C code So Farbig angezeigt?
Bei mir geht das garnicht!

Das beispiel von Peter Danneger klappt super!
Damit habe ich vorhin zum ersten mal ein Display angesteuert!

[Chris W. (verleihnix85)]

Ich hab mal den Code verändert.

"3 mal w".verleihnix.890m.com/index.php?p=1_16

der nimmt meine URL sonst nicht an

die lcd-routines.h

// Ansteuerung eines HD44780 kompatiblen LCD im 4-Bit-Interfacemodus
// http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial
//

void lcd_string(char *data);

void lcd_enable(void);
void lcd_init(void);
void lcd_home(void);
void lcd_clear(void);
void set_cursor(uint8_t x, uint8_t y);

// Hier die verwendete Taktfrequenz in Hz eintragen, wichtig!
 
#define F_CPU 8000000
 
// LCD Befehle
 
#define CLEAR_DISPLAY 0x01
#define CURSOR_HOME   0x02
 
// Pinbelegung für das LCD, an verwendete Pins anpassen
 
#define LCD_PORT_4    PORTD
#define LCD_DDR_4     DDRD
#define LCD_D4      PD1

#define LCD_PORT_5    PORTC
#define LCD_DDR_5     DDRC
#define LCD_D5      PC1

#define LCD_PORT_6    PORTC
#define LCD_DDR_6     DDRC
#define LCD_D6      PC2

#define LCD_PORT_7    PORTC
#define LCD_DDR_7     DDRC
#define LCD_D7      PC3

#define LCD_RS_PORT    PORTC
#define LCD_RS_DDR    DDRC
#define LCD_RS        PC5

#define LCD_EN1_PORT  PORTB
#define LCD_EN1_DDR    DDRB
#define LCD_EN1       PB0

die lcd-routines.c

// Ansteuerung eines HD44780 kompatiblen LCD im 4-Bit-Interfacemodus
// http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial
//
// Die Pinbelegung ist über defines in lcd-routines.h einstellbar
 
#include <avr/io.h>
#include "lcd-routines.h"
#include <util/delay.h>
 
 
// sendet ein Datenbyte an das LCD
 
void lcd_data(unsigned char temp1)
{
   unsigned char temp2 = temp1;
 
   LCD_RS_PORT |= (1<<LCD_RS);        // RS auf 1 setzen
 
   temp1 = temp1 >> 4;
   temp1 = temp1 & 0x0F;
   
   LCD_PORT_4 &= ~(1<<LCD_D4);
   LCD_PORT_5 &= ~(1<<LCD_D5);
   LCD_PORT_6 &= ~(1<<LCD_D6);
   LCD_PORT_7 &= ~(1<<LCD_D7);
 
   if(temp1 & 0x01) LCD_PORT_4 |=(1<<LCD_D4) ;// setzen
   if(temp1 & 0x02) LCD_PORT_5 |=(1<<LCD_D5) ;
   if(temp1 & 0x04) LCD_PORT_6 |=(1<<LCD_D6) ;
   if(temp1 & 0x08) LCD_PORT_7 |=(1<<LCD_D7) ;
   lcd_enable();
   
   
 
   temp2 = temp2 & 0x0F;
   
   LCD_PORT_4 &= ~(1<<LCD_D4);
   LCD_PORT_5 &= ~(1<<LCD_D5);
   LCD_PORT_6 &= ~(1<<LCD_D6);
   LCD_PORT_7 &= ~(1<<LCD_D7); 
   
   if(temp2 & 0x01) LCD_PORT_4 |=(1<<LCD_D4) ;// setzen
   if(temp2 & 0x02) LCD_PORT_5 |=(1<<LCD_D5) ;
   if(temp2 & 0x04) LCD_PORT_6 |=(1<<LCD_D6) ;
   if(temp2 & 0x08) LCD_PORT_7 |=(1<<LCD_D7) ;
   lcd_enable();

   
   _delay_us(42);
}
 
// sendet einen Befehl an das LCD
 
void lcd_command(unsigned char temp1)
{
   unsigned char temp2 = temp1;
 
   LCD_RS_PORT &= ~(1<<LCD_RS);        // RS auf 0 setzen
 
   temp1 = temp1 >> 4;              // oberes Nibble holen
   temp1 = temp1 & 0x0F;
   
   LCD_PORT_4 &= ~(1<<LCD_D4);
   LCD_PORT_5 &= ~(1<<LCD_D5);
   LCD_PORT_6 &= ~(1<<LCD_D6);
   LCD_PORT_7 &= ~(1<<LCD_D7);            // maskieren
   
   if(temp1 & 0x01) LCD_PORT_4 |=(1<<LCD_D4) ;// setzen
   if(temp1 & 0x02) LCD_PORT_5 |=(1<<LCD_D5) ;
   if(temp1 & 0x04) LCD_PORT_6 |=(1<<LCD_D6) ;
   if(temp1 & 0x08) LCD_PORT_7 |=(1<<LCD_D7) ;
   lcd_enable();
 
   temp2 = temp2 & 0x0F;
   
   LCD_PORT_4 &= ~(1<<LCD_D4);
   LCD_PORT_5 &= ~(1<<LCD_D5);
   LCD_PORT_6 &= ~(1<<LCD_D6);
   LCD_PORT_7 &= ~(1<<LCD_D7);        // unteres Nibble holen und maskieren
   
   if(temp2 & 0x01) LCD_PORT_4 |=(1<<LCD_D4) ;// setzen
   if(temp2 & 0x02) LCD_PORT_5 |=(1<<LCD_D5) ;
   if(temp2 & 0x04) LCD_PORT_6 |=(1<<LCD_D6) ;
   if(temp2 & 0x08) LCD_PORT_7 |=(1<<LCD_D7) ;
   lcd_enable();
   
   _delay_us(42);
}
 
// erzeugt den Enable-Puls

void lcd_enable(void)
{
   // Bei Problemen ggf. Pause gemäß Datenblatt des LCD Controllers einfügen
   
   // http://www.mikrocontroller.net/topic/81974#685882
   LCD_EN1_PORT |= (1<<LCD_EN1);
  
 
    _delay_us(1);                   // kurze Pause
   // Bei Problemen ggf. Pause gemäß Datenblatt des LCD Controllers verlängern
   // http://www.mikrocontroller.net/topic/80900
   LCD_EN1_PORT &= ~(1<<LCD_EN1);
   
   
  
} 
// Initialisierung: 
// Muss ganz am Anfang des Programms aufgerufen werden.
 
void lcd_init(void)
{
   // Ports auf Ausgang schalten
   LCD_DDR_4 |=(1<<LCD_D4);
   LCD_DDR_5 |=(1<<LCD_D5);
   LCD_DDR_6 |=(1<<LCD_D6);
   LCD_DDR_7 |=(1<<LCD_D7);
   
   LCD_EN1_DDR |= (1<<LCD_EN1);
 
   LCD_RS_DDR  |= (1<<LCD_RS);
   // muss 3mal hintereinander gesendet werden zur Initialisierung
   
   _delay_ms(15);
   
   LCD_PORT_4 |=  (1<<LCD_D4);
   LCD_PORT_5 |=  (1<<LCD_D5);
   LCD_PORT_6 &= ~(1<<LCD_D6);
   LCD_PORT_7 &= ~(1<<LCD_D7);  
  
   LCD_RS_PORT &= ~(1<<LCD_RS);      // RS auf 0
   lcd_enable();
 
   _delay_ms(5);
   lcd_enable();
 
   _delay_ms(1);
   lcd_enable();
   _delay_ms(1);
 
   // 4 Bit Modus aktivieren
 
   
   
   LCD_PORT_4 &=  ~(1<<LCD_D4);
   LCD_PORT_5 |=   (1<<LCD_D5);
   LCD_PORT_6 &=  ~(1<<LCD_D6);
   LCD_PORT_7 &=  ~(1<<LCD_D7);  
   
   lcd_enable();
   _delay_ms(1);
 
   // 4Bit / 2 Zeilen / 5x7
   lcd_command(0x28);
    
   // Display ein / Cursor aus / kein Blinken
   lcd_command(0x0C); 
 
   // inkrement / kein Scrollen
   lcd_command(0x06);
 
   lcd_clear();
}
 
// Sendet den Befehl zur Löschung des Displays
 
void lcd_clear(void)
{
   lcd_command(CLEAR_DISPLAY);
   _delay_ms(5);
}
 
// Sendet den Befehl: Cursor Home
 
void lcd_home(void)
{
   lcd_command(CURSOR_HOME);
   _delay_ms(5);
}
 
// setzt den Cursor in Zeile y (1..4) Spalte x (0..15)
 
void set_cursor(uint8_t x, uint8_t y)
{
  uint8_t tmp;
 
  switch (y) {
    case 1: tmp=0x80+0x00+x;  break;    // 1. Zeile
    case 2: tmp=0x80+0x40+x;  break;    // 2. Zeile
    case 3: tmp=0x80+0x10+x;  break;    // 3. Zeile
    case 4: tmp=0x80+0x50+x;  break;    // 4. Zeile
  }
  lcd_command(tmp);
}
 
// Schreibt einen String auf das LCD
 
void lcd_string(char *data)
{
    while(*data) {
        lcd_data(*data);
        data++;
    }
}

falls es Fehler gibt bitte melden
Ich hoffe ich konnte helfen

[Sebastian Z. (hometronix)]

Genial, genau nach so einer angepassten LCD Routine habe ich die letzten 
Tage gesucht und nun funktioniert endlich mein LCD an dem AVR NET I/O 
von Pollin, vielen Dank an Chris ;-)

Ich verwende PortD und Pin0 sowie Pin1 sind schon belegt vom seriellen 
Anschluss, somit musste ich die Daten Pins ab Port2 bis Port5 verwenden 
und irgendwie habe ich es vorher mit der Maskierung nie hinbekommen. RS 
habe ich an PD6 und EN an PD7, RW konnte ich zum Glück auf GND legen 
denn der PortD ist nun voll belegt.

Das LCD funktioniert jetzt jedenfalls problemlos. Was mich nur etwas 
verwundert, ich bekomme folgende Warnung beim Make All:

lcd-routines.c:185: warning: 'tmp' may be used uninitialized in this 
function

In der Zeile 185 steht --> uint8_t tmp;

Der Befehl set_cursor funktioniert bei mir trotzdem ohne Probleme.

[Hanns W. (hannsw)]

Sebastian Z. wrote:
>.......
> Das LCD funktioniert jetzt jedenfalls problemlos. Was mich nur etwas
> verwundert, ich bekomme folgende Warnung beim Make All:
>
> lcd-routines.c:185: warning: 'tmp' may be used uninitialized in this
> function
>
> In der Zeile 185 steht --> uint8_t tmp;

  Ja schon, aber was ist mit tmp, wenn "y" < 1 oder > 4 ist?
  dann wäre tmp nicht initialisiert, und das prog stürzt ab.

>
> Der Befehl set_cursor funktioniert bei mir trotzdem ohne Probleme.

weil Du die Function sicher immer nur mit 1 < y < 5 aufrufst.

Gruß Hanns

[Sebastian Z. (hometronix)]

Hanns Weil wrote:
>
> weil Du die Function sicher immer nur mit 1 < y < 5 aufrufst.
>
> Gruß Hanns

Ja, stimmt. Ich verwende für y in meinen "set_cursor" Befehlen immer nur 
1 oder 2, da ich ein zweizeiliges LCD besitze.

Sollte ich dann lieber tmp am Anfang standardmäßig auf irgendeinen Wert 
setzen? Prinzipiell sollte es bei mir ja keinen Programmabsturz geben, 
da meine Befehle korrekt mit Zeile 1 oder Zeile 2 verwendet werden.

[Hanns W. (hannsw)]

Sebastian Z. wrote:
...
> Sollte ich dann lieber tmp am Anfang standardmäßig auf irgendeinen Wert
> setzen? Prinzipiell sollte es bei mir ja keinen Programmabsturz geben,
> da meine Befehle korrekt mit Zeile 1 oder Zeile 2 verwendet werden.

Im Prinzip ja, wenn Du "sauber" arbeiten möchtest.Oder Du lässt es so, 
und schaltest die Warnungen ab.

Zum "sauber" arbeiten gehörte auch im switch ein default-statement und 
dann keine ANzeige:
z.b. so, wobei du Dich wunderst, warum nichts angezeigt wird:

void set_cursor(uint8_t x, uint8_t y)
{
  uint8_t tmp;
  // vermeide Compliler-Meldung
  tmp = 0; 

  switch (y) {
    case 1: tmp=0x80+0x00+x;  break;    // 1. Zeile
    case 2: tmp=0x80+0x40+x;  break;    // 2. Zeile
    case 3: tmp=0x80+0x10+x;  break;    // 3. Zeile
    case 4: tmp=0x80+0x50+x;  break;    // 4. Zeile
    // falsche Parameter übergaben:
    default:{
              //kannst DU ne Warnung ausgeben und notieren? 
              // log-fle oder so? auf alle Fälle zurück aus der Funktion
              return;
            } 
         
  }
  // hier hätten wir dann die korrekten Parameter,
  // und können den cursor setzen
  lcd_command(tmp);
}

Wobei Du im Fehler-Falle dann weiter oben in Deinem Programm - nach dem 
Aufruf - Probleme mit der eigentlichen Textanzeige bekommst, wenn der 
Cursor nicht gesetzt werden konnte.

Daher kannst Du folgendes machen:
aus der void machst Du ne boolean:

boolean set_cursor(uint8_t x, uint8_t y)
{
  uint8_t tmp;
   // ...
    // falsche Parameter übergaben im switch
    default:return false;         
  }
  // hier hätten wir dann die korrekten Parameter,
  // und können den cursor setzen
  lcd_command(tmp);
  return true;
}

und im Hauptprogramm

   // .. Cursor fehlertolerant setzen
{
   if ( set_cursor( x,  y) ) {
       // zeige Daten an
   }else {
       // zeige Fehler an
   }

Gruß Hanns

[Simon K. (simon)]

Chris W. wrote:
Die Warnung kriegst du ganz einfach weg:

 void set_cursor(uint8_t x, uint8_t y)
 {
   uint8_t tmp;
 
   switch (y) {
     case 1: tmp=0x80+0x00+x;  break;    // 1. Zeile
     case 2: tmp=0x80+0x40+x;  break;    // 2. Zeile
     case 3: tmp=0x80+0x10+x;  break;    // 3. Zeile
     case 4: tmp=0x80+0x50+x;  break;    // 4. Zeile
     default:
       return;
   }
   lcd_command(tmp);
 }

[Sebastian Z. (hometronix)]

Vielen Dank Euch beiden, ich habe mich dann doch für die letztere 
Möglichkeit entschieden, da es am einfachsten und schnellsten ging.

Nun bekomme ich keine Warnung mehr und etwas anderes als 1 oder 2 werde 
ich bei diesem zweizeiligen LCD sowieso nicht verwenden.

Grüße
Sebastian

[Hanns W. (hannsw)]

Simon K. wrote:
> Chris W. wrote:
> Die Warnung kriegst du ganz einfach weg:
>
>

>  void set_cursor(uint8_t x, uint8_t y)
>  {
>    uint8_t tmp;
> 
>    switch (y) {
>      case 1: tmp=0x80+0x00+x;  break;    // 1. Zeile
>      case 2: tmp=0x80+0x40+x;  break;    // 2. Zeile
>      case 3: tmp=0x80+0x10+x;  break;    // 3. Zeile
>      case 4: tmp=0x80+0x50+x;  break;    // 4. Zeile
>      default:
>        return;
>    }
>    lcd_command(tmp);
>  }
>

Simon, hast Du es mal getestet?
Ich meine im default Falle ist tmp immer noch nicht mit einem Wert 
versehen, und das meckert der Kompiler doch an!

Sebastian, ist mit Simons Variante die Fehlermeldung tatsächlich weg?


Gruß Hanns

[Sebastian Z. (hometronix)]

Hanns Weil wrote:
> Sebastian, ist mit Simons Variante die Fehlermeldung tatsächlich weg?
>
>
> Gruß Hanns

Ja, ich bekomme nun wirklich keine Warnung mehr angezeigt!

Grüße
Sebastian

[Simon K. (simon)]

Hanns Weil wrote:
> Simon, hast Du es mal getestet?
> Ich meine im default Falle ist tmp immer noch nicht mit einem Wert
> versehen, und das meckert der Kompiler doch an!

Ja, aber andererseits wird die (uninitialisierte) Variable auch nicht 
mehr verwendet, wenn man die Funktion per return beendet ;)

Compiler sind verdammt schlaue Leute! :D

[Hanns W. (hannsw)]

Aber um das letzte Wort zu behalten:
Wir sind uns einig, dass zum "ordentlichen" Programmieren das Init einer 
Variablen gehört, oder??

Gruß Hanns

[Peter D. (peda)]

Sebastian Z. wrote:
> Genial, genau nach so einer angepassten LCD Routine habe ich die letzten
> Tage gesucht und nun funktioniert endlich mein LCD an dem AVR NET I/O
> von Pollin, vielen Dank an Chris ;-)


Ich wundere mich bloß, warum es keinem auffällt, das sehr viel Code 
verschwendet wird.
Z.B. taucht dieser Abschnitt 4-mal auf:

   temp1 = temp1 & 0x0F;
   
   LCD_PORT_4 &= ~(1<<LCD_D4);
   LCD_PORT_5 &= ~(1<<LCD_D5);
   LCD_PORT_6 &= ~(1<<LCD_D6);
   LCD_PORT_7 &= ~(1<<LCD_D7);            // maskieren
   
   if(temp1 & 0x01) LCD_PORT_4 |=(1<<LCD_D4) ;// setzen
   if(temp1 & 0x02) LCD_PORT_5 |=(1<<LCD_D5) ;
   if(temp1 & 0x04) LCD_PORT_6 |=(1<<LCD_D6) ;
   if(temp1 & 0x08) LCD_PORT_7 |=(1<<LCD_D7) ;
   lcd_enable();

Ehe ich etwas zweimal hinschreibe, mache ich daraus ne Unterfunktion.
Das ist zum einen Schreibfaulheit und zum anderen Zuverlässigkeit.
Was nur einmal dasteht, kann auch nur einmal falsch sein (muß nur einmal 
korrigiert werden).

Es ist eine beliebte Fehlerquelle, in Copy&Paste Codemonstern etwas 
nicht an allen Stellen zu korrigieren und sich dann zu wundern, warum 
der Fehler noch da ist.


Mit Unterfunktionen wird dann so ein LCD-Treiber gleich viel schlanker 
und durchschaubarer:

Beitrag "Re: LCD nicht nur für einen Port in C"


Zum Thema Default-Zweig:
Ich würde als Default die 1.Zeile setzen.
Return halte ich für die schlechteste Lösung, da man damit Fehler 
verschleiert, d.h sich die Fehlersuche nur selber schwer macht.


Peter

[Hanns W. (hannsw)]

Peter Dannegger wrote:
------
> Es ist eine beliebte Fehlerquelle, in Copy&Paste Codemonstern etwas
> nicht an allen Stellen zu korrigieren und sich dann zu wundern, warum
> der Fehler noch da ist.
>
>
> Mit Unterfunktionen wird dann so ein LCD-Treiber gleich viel schlanker
> und durchschaubarer:
>

Völlige Zustimmung

> Zum Thema Default-Zweig:
> Ich würde als Default die 1.Zeile setzen.

> Return halte ich für die schlechteste Lösung, da man damit Fehler
> verschleiert, d.h sich die Fehlersuche nur selber schwer macht.

Peter, wenn Du die 1.Zeile als default benutzt, verschleierst Du 
ebenfalls Fehler, indem Du fehlerhafte Parameter einem fehlerfreien 
gleichsetzt!

Ich bin nach wie vor der Meinung, daß ein

default: return false;

einer boolean funktion, mit auswertung des Rückgabewertes in der 
aufrufenden func ( z.B Anzeige in Zeile 1 blinkend) des sicherere Weg 
ist.

Klar ist mir, daß in diesem Fall sicherlich keine Zeile außerhalb des 
Displays aufgerufen werden wird; aber, wie Du selber sagst, gehört 
mehrfach zu benutzender Code in eine eine Subroutine. Wenn diese sich 
später einmal verselbständigt, und void bleibt, wirst Du irgenwann mal - 
bei Deiner Lösung - nicht mehr wissen, ob  nun ein Fehler auftrat oder 
nicht.
Aber vielleicht bin ich da zu pingelig

Gruß Hanns

[Martin (Gast)]

Hallo,

ich habe mit disem Code gerade mein LCD am laufen zu bekommen :-) bzw. 
Teilerweise :-(
habe ein 4x20 LCD und wenn ich mit dem gesamte Code compiliere und 
flashe kommen nur die erste 2 Zeile mit meine String

"hallo1ghhghjgjghgjkk" (20 Zeile)

wie bekomme ich die 3 und 4 Zeile ??

hier steht:


   // 4Bit  2 Zeilen  5x7
   lcd_command(0x28);

wie ist dann für die 4 Zeile? habe rum probiert aber leider nichts !

Gruß
Martin

[Pete K. (pete77)]

4 zeilige Displays haben meistens zwei Controller mit 2 Enable 
Leitungen.

Such mal nach Wintek 2704. Das ist so ein Dislay und hierfür gibt es 
auch schon fertigen Code. (Ist ein Pollin Display)

[Lothar S. (magic33)]

Ich hab das Problem das ich an einem Port bin nur ein durcheinander mit 
den Pins habe

kann mir da mal jemand das auseinanderdividieren
ich bring das LCD nicht ans laufen.
Pinnbelegung
// LCD_RS =PA3 , LCD_EN = PA1 ; DB4= PA0 DB5= PA2 DB6 = PA4 DB7=PA6 == 
LCD_AD =0x55 LCD_ADA = 0xAA

// Ansteuerung eines HD44780 kompatiblen LCD im 4-Bit-Interfacemodus
// http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial
//
// Die Pinbelegung ist über defines in lcd-routines.h einstellbar
 
#include <avr/io.h>
#include "lcd-routines.h"
#include <avr/delay.h>

// LCD_RS =PA3 , LCD_EN = PA1 ; DB4= A0 DB5= A2 DB6 = PA4 DB7=PA6 == LCD_AD =0x55 LCD_ADA = 0xAA
 
// sendet ein Datenbyte an das LCD
 
void lcd_data(unsigned char temp1)
{
   unsigned char temp2 = temp1;
 
   LCD_PORT |= (1<<LCD_RS);        // RS auf 1 setzen
 
   temp1 = temp1 >> 4;
   temp1 = temp1 & LCD_AD;
   LCD_PORT &= LCD_ADA;
   LCD_PORT |= temp1;               // setzen
   lcd_enable();
 
   temp2 = temp2 & LCD_AD;
   LCD_PORT &= LCD_ADA;
   LCD_PORT |= temp2;               // setzen
   lcd_enable();
   
   _delay_us(42);
}
 
// sendet einen Befehl an das LCD
 
void lcd_command(unsigned char temp1)
{
   unsigned char temp2 = temp1;
 
   LCD_PORT &= ~(1<<LCD_RS);        // RS auf 0 setzen
 
   temp1 = temp1 >> 4;              // oberes Nibble holen
   temp1 = temp1 & LCD_AD;            // maskieren
   LCD_PORT &= LCD_ADA;
   LCD_PORT |= temp1;               // setzen
   lcd_enable();
 
   temp2 = temp2 & LCD_AD;            // unteres Nibble holen und maskieren
   LCD_PORT &= LCD_ADA;
   LCD_PORT |= temp2;               // setzen
   lcd_enable();
   
   _delay_us(42);
}

[Michael U. (amiga)]

Hallo,

4x20 haben üblicherweise nur einen Controller, 80 Zeichen ist die 
Ramgröße der 44780 und kompatiblen.

Initialisiert als 2-zeilig, die 1 und 3. Zeile sowie die 2. und 4. Zeile 
sind dann von den Adressen direkt hintereinander.

Erst bei mehr als 80 Zeichen (z.B. das Pollin 4x27) ist ein zweiter 
Controller nötig.

Gruß aus Berlin
Michael

[Lothar S. (magic33)]

Es geht nicht um die Adresse der speicherstellen sondern um die 
Portbelegung.

ich wollte nicht das grosse programm verwenden das jeden pin des LCD an 
einem anderen port hat

[Peter D. (peda)]

Lothar Sammel wrote:
> ich wollte nicht das grosse programm verwenden das jeden pin des LCD an
> einem anderen port hat


http://www.mikrocontroller.net/attachment/30300/lcd_drv.zip

Was ist denn daran groß?


Peter

[dummy (Gast)]

>ich wollte nicht das grosse programm verwenden das jeden pin des LCD an
>einem anderen port hat

Das wirst du bei deiner verpfuschten Portbelegung aber müssen.

[magic (Gast)]

ok danke werd ich wohl so machen müssen

win reinschiften der temp in die portD adresse
nach dem motto
PortD.0=temp1.0 geht nicht?

[dummy (Gast)]

>win reinschiften der temp in die portD adresse
>nach dem motto
>PortD.0=temp1.0 geht nicht?

Jetzt auch noch mit Strukturen arbeiten?
Wenn du wenigstens die Datenleitungen für
das LCD in einer Reihe am Port angeschlossen hättest,
dann könnte man das Beispiel aus dem Tutorial
relativ einfach anpassen. Du musstest sie aber
unbedingt alle auseinanderreissen. Nimm Peters
Code. Das was man tun müsste um deine Pinbelegung
gerade zu biegen wird auch nicht kleiner.