From: Luis C. <lfc...@lf...> - 2005-03-27 20:55:35
Attachments:
lcd4linux-i2c.gz
|
Hi! (This email is mainly directed to Paul and Michael) during this small Easter break, I finnaly managed to take a deep look into the changes Paul has made to the HD44780 driver. Here is a small patch to the drv_HD44780.c file, and two new files that abstract all I2C relevant code from the main driver. To make this work at all, the ./configure scripts need to know about these new files. I'm not an expert with Makefiles, so Michael please help me with this one :)... Unpack the attached file, apply the diff and copy the 'drv_generic_i2c.[h,c]'. This patchset is completely untested, as I don't currently have any I2C devices available for testing. Paul, about the SIGNAL stuff, they are defined in the config file. For example, my PCF based I2C used the same connections as the 4bit driver, Martin has made. This is the corresponding config entry: Display HD44780-I2C { Driver 'HD44780' Model 'WRAP1C-PCF8574' Bus 'i2c' Port '/dev/i2c-0' Device '70' Bits '4' Size '20x4' asc255bug 0 Icons 1 Wire { RW 'DB5' RS 'DB4' ENABLE 'DB6' GPO 'GND' } } Basicly it is here where you say what is connected to where, although I'm not sure if it will work if it is wired differently then it is here... Take a good shot at it and let me know. Luis Correia, ashamed because it could not be tested... :( |
From: Michael R. <re...@eu...> - 2005-03-28 06:31:00
|
Hi Luis, > during this small Easter break, I finnaly managed to take a deep look > into the changes Paul has made to the HD44780 driver. > Here is a small patch to the drv_HD44780.c file, and two new files that > abstract all I2C relevant code from the main driver. Well, I tried to apply your patch, but there's a problem: First, please name a tar archive .tar.gz (and not .gz only). Took me some time to find out that it's not a diff :-) Second, the file drv_generic_i2c.c is missing, but the drv_generic_i2c.c has been added twice to the tar. Please send me the drv_generic_i2c.c as soon as possible, so I can finish the patch... after all, your patch looks fine! bye, Michael -- Michael Reinelt <re...@eu...> http://home.pages.at/reinelt GPG-Key 0xDF13BA50 ICQ #288386781 |
From: Luis C. <lfc...@lf...> - 2005-03-28 06:50:37
Attachments:
lcd4linux-i2c.tar.gz
|
Michael Reinelt wrote: >Hi Luis, > > > >>during this small Easter break, I finnaly managed to take a deep look >>into the changes Paul has made to the HD44780 driver. >>Here is a small patch to the drv_HD44780.c file, and two new files that >>abstract all I2C relevant code from the main driver. >> >> > >Well, I tried to apply your patch, but there's a problem: > >First, please name a tar archive .tar.gz (and not .gz only). Took me >some time to find out that it's not a diff :-) > >Second, the file drv_generic_i2c.c is missing, but the drv_generic_i2c.c > has been added twice to the tar. > >Please send me the drv_generic_i2c.c as soon as possible, so I can >finish the patch... > >after all, your patch looks fine! > >bye, Michael > > > Hi! I told you i'm not very keen with this patch thighy... Here is the correct tar.gz file with all files in it! I forgot to tell you that the diff was created against last night CVS at sourceforge. Luis Correia |
From: Michael R. <re...@eu...> - 2005-03-28 07:08:26
|
Hi Luis, > I told you i'm not very keen with this patch thighy... Here is the > correct tar.gz file with all files in it! > I forgot to tell you that the diff was created against last night CVS a= t > sourceforge. Thanks for the tar. Looks much better now :-) Anyway, I cannot compile it.I get tons of compiler errors, all resulting from the including of linux/i2c.h. Seems to me that this file is not intended for userland inclusion :-) here are some of the errors: /usr/include/linux/jiffies.h:16: error: Fehler beim Parsen before "jiffies_64" /usr/include/linux/jiffies.h:20: error: Fehler beim Parsen before "get_jiffies_64" /usr/include/linux/bitmap.h:15: error: `BITS_PER_LONG' undeclared (first use in this function) /usr/include/linux/bitmap.h: In Funktion =BBbitmap_full=AB: /usr/include/linux/bitmap.h:29: error: `BITS_PER_LONG' undeclared (first use in this function) /usr/include/linux/bitmap.h: In Funktion =BBbitmap_equal=AB: /usr/include/linux/bitmap.h:44: error: `BITS_PER_LONG' undeclared (first use in this function) /usr/include/linux/bitmap.h: In Funktion =BBbitmap_shift_right=AB: /usr/include/linux/bitmap.h:85: error: `__shr_tmp' undeclared (first use in this function) /usr/include/linux/bitmap.h: In Funktion =BBbitmap_shift_left=AB: /usr/include/linux/bitmap.h:98: error: `__shl_tmp' undeclared (first use in this function) /usr/include/linux/bitmap.h:144: error: `BITS_PER_LONG' undeclared (first use in this function) /usr/include/linux/cpumask.h:15: error: variable-size type declared outside of any function /usr/include/linux/cpumask.h: In Funktion =BBnext_online_cpu=AB: /usr/include/linux/cpumask.h:56: error: structure has no member named `va= l' .... and so on. Does it compile cleanly in your environment? If so, why? bye, Michael --=20 Michael Reinelt <re...@eu...> http://home.pages.at/reinelt GPG-Key 0xDF13BA50 ICQ #288386781 |