Re: Re: [PATCH] FT232R and FT245R EEPROM user area size check

Hi Robin, On Thursday, 15. January 2015 22:59:21 Robin Haberkorn wrote: > On 15.01.2015 22:50, Robin Haberkorn wrote: > > ... > > > > Perhaps the other chip's user area space checks in libftdi should also > > be reviewed, based on what "FT Prog" or FT_EE_UASize() reports. > > ... > > I can conf

Re: Re: [PATCH] FT232R and FT245R EEPROM user area size check

Thomas Jarosch wrote: > Hi Robin, > > > > See doc/EEPROM-structure, and git commit a45ea32b4ffd. I believe the > > > existing code intentionally goes against what AN_121 describes, as > > > written in that doc. > > > > After some investigation, I think there's a misunderstanding. > > Yes, it app

Re: Re: [PATCH] FT232R and FT245R EEPROM user area size check

Hi Robin, > > See doc/EEPROM-structure, and git commit a45ea32b4ffd. I believe the > > existing code intentionally goes against what AN_121 describes, as > > written in that doc. > > After some investigation, I think there's a misunderstanding. > Yes, it appears as if 4 bytes after the serial nu

Re: [PATCH] FT232R and FT245R EEPROM user area size check

Hi again, On 15.01.2015 22:50, Robin Haberkorn wrote: > ... > Perhaps the other chip's user area space checks in libftdi should also be reviewed, based on what "FT Prog" or FT_EE_UASize() reports. ... > I can confirm the 44 character (88 byte) limit on FT230X using "FT Prog". So at least th

Re: [PATCH] FT232R and FT245R EEPROM user area size check

Hi Jim, On 15.01.2015 02:35, Jim Paris wrote: Hi Robin, See doc/EEPROM-structure, and git commit a45ea32b4ffd. I believe the existing code intentionally goes against what AN_121 describes, as written in that doc. After some investigation, I think there's a misunderstanding. Yes, it appears a

Re: [PATCH] FT232R and FT245R EEPROM user area size check

Hi Robin, See doc/EEPROM-structure, and git commit a45ea32b4ffd. I believe the existing code intentionally goes against what AN_121 describes, as written in that doc. Either way, the _230X is decribed in its datasheet sa having an identical EEPROM structure as the _R, so any patch that treats th

[PATCH] FT232R and FT245R EEPROM user area size check

Hello, I'm working on the libftdi Git head. I found that ftdi_eeprom_build's user area size check for type R chips is too restrictive. See the patch attached and http://www.ftdichip.com/Support/Documents/AppNotes/AN_121_FTDI_Device_EEPROM_User_Area_Usage.pdf Best regards, Robin -- libftdi -