Hi Thomas, On Sat, May 16, 2020 at 9:53 PM Yegor Yefremov <yegorsli...@googlemail.com> wrote: > > Hi Thomas, > > On Thu, May 14, 2020 at 11:21 PM Thomas Jarosch > <thomas.jaro...@intra2net.com> wrote: > > > > Hi Yegor, > > > > You wrote on Mon, Apr 13, 2020 at 11:32:47PM +0200: > > > I think the decoding of boolean bit fields all need to be changed > > > to "!!(bit test)", so the in memory value will truely be one or zero. > > > > > > Then we can apply changes like this everywhere: > > > > > > - if ( eeprom->high_current_b == HIGH_CURRENT_DRIVE) > > > + if (eeprom->high_current_b) > > > output[0x01] |= HIGH_CURRENT_DRIVE; > > > > I've bit the bullet and fixed all eeprom boolean bit fields on the branch > > > > origin/ftdi-eeprom-fix-bitflag-handling > > > > > > @Yegor: I would appreciate it if you could review it. > > I triple reviewed it on my side not to break anything, > > but may be I missed something *fingers crossed* > > So far it looks goods to me aside from a typo in the last commit log: > > "On TYPE_R 00.3 set mean D2XX, on other devices VCP" -> s/mean/means > > I'll try to make a test with FT-X chip and let you know my results.
The tests with FTX were successful. Regards, Yegor > > > > > > Additionally I've seen that the handling of the VCP driver field > > for TYPE_R chips was not inverted properly, see the extra commit. > > > > > > A colleague recently recommended a new code diff tool to me called "delta": > > https://github.com/dandavison/delta > > > > Here's how I run it: > > > > git diff origin/master..origin/ftdi-eeprom-fix-bitflag-handling \ > > |delta --dark --keep-plus-minus-markers > > > > > > It makes reading subtle code changes more obvious to me. > > Never heard about delta but I like its approach. Thanks. It would be > great if tig would have this functionality. > > Cheers, > Yegor > > > Cheers, > > Tom -- libftdi - see http://www.intra2net.com/en/developer/libftdi for details. To unsubscribe send a mail to libftdi+unsubscr...@developer.intra2net.com