Re: [PATCH v6] USB: serial: ftdi_sio: Add MTP NVM support

2018-07-16 Thread Johan Hovold
On Sun, Jul 15, 2018 at 10:30:39PM +0200, Loic Poulain wrote: > Hi Johan, > > Thanks for the review. > > On 10 July 2018 at 11:19, Johan Hovold wrote: > > The CBUS configuration is stored in just a couple of words at a known > > offset and could easily be retrieved without depending on the

Re: [PATCH v6] USB: serial: ftdi_sio: Add MTP NVM support

2018-07-15 Thread Loic Poulain
Hi Johan, Thanks for the review. On 10 July 2018 at 11:19, Johan Hovold wrote: > Hi, > > I finally found some time to dig into the ftdi eeprom handling. > > On Tue, Jun 26, 2018 at 02:54:48PM +0200, Loic Poulain wrote: >> Most of FTDI's devices have an EEPROM which records FTDI devices >>

Re: [PATCH v6] USB: serial: ftdi_sio: Add MTP NVM support

2018-07-10 Thread Johan Hovold
Hi, I finally found some time to dig into the ftdi eeprom handling. On Tue, Jun 26, 2018 at 02:54:48PM +0200, Loic Poulain wrote: > Most of FTDI's devices have an EEPROM which records FTDI devices > configuration setting (e.g. the VID, PID, I/O config...) and user > data. For example, FT232R and

Re: [PATCH v6] USB: serial: ftdi_sio: Add MTP NVM support

2018-07-09 Thread Loic Poulain
Hi Johan, On 29 June 2018 at 21:52, Ajay Gupta wrote: > Hi Loic. > Thanks for fixing comments. Looks good to me now. > > On 6/26/18, Loic Poulain wrote: >> Most of FTDI's devices have an EEPROM which records FTDI devices >> configuration setting (e.g. the VID, PID, I/O config...) and user >>

Re: [PATCH v6] USB: serial: ftdi_sio: Add MTP NVM support

2018-06-29 Thread Ajay Gupta
Hi Loic. Thanks for fixing comments. Looks good to me now. On 6/26/18, Loic Poulain wrote: > Most of FTDI's devices have an EEPROM which records FTDI devices > configuration setting (e.g. the VID, PID, I/O config...) and user > data. For example, FT232R and FTX chips have 128-byte and 2048-byte

[PATCH v6] USB: serial: ftdi_sio: Add MTP NVM support

2018-06-26 Thread Loic Poulain
Most of FTDI's devices have an EEPROM which records FTDI devices configuration setting (e.g. the VID, PID, I/O config...) and user data. For example, FT232R and FTX chips have 128-byte and 2048-byte internal EEPROM respectively. This patch adds support for FTDI EEPROM read/write via USB control