Re: [Owfs-developers] DS2408 / HD44780 LCD_H regression

2016-04-07 Thread Johan Ström
Great! :) On 07/04/16 20:33, Nico Bouthoorn wrote: > I've pulled the latest master, in a fresh install.It is working! > > thanks, > > Nico > > Johan Ström wrote: >> Since this HW is out there in unknown number of deployments, it makes more >> sense to add a SW fix. I added a verify_mask which

Re: [Owfs-developers] DS2408 / HD44780 LCD_H regression

2016-04-07 Thread Nico Bouthoorn
I've pulled the latest master, in a fresh install.It is working! thanks, Nico Johan Ström wrote: > Since this HW is out there in unknown number of deployments, it makes more > sense to add a SW fix. I added a verify_mask which ignores the button > pull-ups on read-back. > > Nico: Can you p

Re: [Owfs-developers] DS2408 / HD44780 LCD_H regression

2016-04-06 Thread Johan Ström
Since this HW is out there in unknown number of deployments, it makes more sense to add a SW fix. I added a verify_mask which ignores the button pull-ups on read-back. Nico: Can you pull latest master and try that? If 'git show HEAD' shows your revert at the top, you must first do 'git reset -

Re: [Owfs-developers] DS2408 / HD44780 LCD_H regression

2016-04-03 Thread Johan Ström
Hello again, I've gotten some helpful off-list response from Der Tiger, and to sum it up. The hardware Tiger uses (Hobbyboard) is electrically identical to the on Nico uses (https://fstab.nl/en/lcd_owfs), except that the Hobbyboard schematic has 3 pushbuttons between P0..P3 and GND, with 10nF

Re: [Owfs-developers] DS2408 / HD44780 LCD_H regression

2016-03-31 Thread Jan Kandziora
Am 31.03.2016 um 23:31 schrieb Johan Ström: > Great, now we have the exact source of the problem at least :) > > I'm wondering if that code is actually right (with or without > patch).. Not sure what LCD_COMMAND_ATTENTION is intended to do? And > shouldn't LCD_COMMAND_4_BIT be written with NIBBLE_

Re: [Owfs-developers] DS2408 / HD44780 LCD_H regression (was: Louis Swart LCD issues)

2016-03-31 Thread Johan Ström
Great, now we have the exact source of the problem at least :) I'm wondering if that code is actually right (with or without patch).. Not sure what LCD_COMMAND_ATTENTION is intended to do? And shouldn't LCD_COMMAND_4_BIT be written with NIBBLE_CTRL? Anyone on the list who uses these/wrote the