<div dir="ltr"><div dir="ltr">Hi Andy,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 7 Mar 2022 at 11:44, Andy Shevchenko <<a href="mailto:andriy.shevchenko@linux.intel.com">andriy.shevchenko@linux.intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sun, Mar 06, 2022 at 03:33:20PM +0200, Tomer Maimon wrote:<br>
> On Thu, 3 Mar 2022 at 16:11, Andy Shevchenko <<br>
> <a href="mailto:andriy.shevchenko@linux.intel.com" target="_blank">andriy.shevchenko@linux.intel.com</a>> wrote:<br>
> > On Thu, Mar 03, 2022 at 02:35:58PM +0200, Tali Perry wrote:<br>
> > > > On Thu, Mar 3, 2022 at 12:45 PM Andy Shevchenko <<br>
> > <a href="mailto:andriy.shevchenko@linux.intel.com" target="_blank">andriy.shevchenko@linux.intel.com</a>> wrote:<br>
<br>
...<br>
<br>
> > But hold on and read set of questions below.<br>
> ><br>
> > Previously it was a fixed field with the NPCM_I2CTXF_STS_TX_BYTES mask<br>
> > applied,<br>
> > right? From above I have got that FIFO is growing twice. Is it correct?<br>
> <br>
> What do you mean by growing twice? TX and RX?<br>
<br>
I meant from 16 bytes to 32 bytes.<br></blockquote><div>Yes, the FIFO grow to 32 byte </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> > Does the LSB stay at the same offset? What is the meaning of the MSB in 32<br>
> > byte<br>
> > case? If it's reserved then why not to always use 32 byte approach?<br>
> <br>
> Yes, the LSB stays in the same place, and bit 5 is reserved in the NPCM7XX<br>
> SoC.<br>
> Unfortunately, the I2C test failed when we tried to use the 32 bytes<br>
> approach at NPCM7XX Soc, this is why we added NPCM_I2CTXF_STS_TX_BYTES and<br>
> NPCM_I2C_STSRXF_RX_BYTES to the data structure.<br>
> <br>
> The device tree data structure pass data for each specific device, so I<br>
> don't understand why not use device tree data for supporting the I2C<br>
> specific device? this is not the case here?<br>
<br>
Basically we use compatible strings for that, but in any case if something<br>
can be autodetected from hardware, it's better to use autodetection.</blockquote><div>Thanks for the clarification, in our case we don't autodetect from the hardware...</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
-- <br>
With Best Regards,<br>
Andy Shevchenko<br>
<br>
<br></blockquote><div>We will send a new version soon, highly appreciate your help!</div><div><br></div><div>Best regards,</div><div><br></div><div>Tomer</div><div><br></div><div> </div></div></div>