[PATCHv2 3/3] i2c: imx_lpi2c: Support read transfers longer than 256 bytes

[Fedor Ross]

On Wed, Aug 7, 2024 at 12:50 PM Heiko Schocher <hs at denx.de> wrote:

Hello Heiko,

> On 30.07.24 17:25, fedorross at gmail.com wrote:
> > From: Fedor Ross <fedor.ross at ifm.com>
> >
> > The TXFIFO register of LPI2C only has one byte length, and if the length
> > of the data that needs to be read exceeds 256 bytes, it needs to be
> > written to TXFIFO multiple times.
> >
> > Signed-off-by: Fedor Ross <fedor.ross at ifm.com>
> > ---
> > Cc: Heiko Schocher <hs at denx.de>
> > Cc: Tom Rini <trini at konsulko.com>
> > Cc: Marek Vasut <marex at denx.de>
> > ---
> >
> > Changes in v2:
> > - Add additional '---' below SoB line to separate Cc list. This way it
> >    does not get included in the commit message when the patch is applied.
> > - Prefix 'CHUNK_DATA' macro with 'LPI2C_' to namespace it.
> > - Do bit manipulation in writel() function directly and drop the
> >    'val = assignment' .
> >
> >   drivers/i2c/imx_lpi2c.c | 71 ++++++++++++++++++++++++++---------------
> >   1 file changed, 46 insertions(+), 25 deletions(-)
>
> Patch breaks world build, see for example
>
> https://dev.azure.com/hs0298/hs/_build/results?buildId=114&view=logs&j=9261107f-b16b-528d-034d-a5b366785bfc&t=aa057498-3634-5fad-2ef8-82ed58702aca&l=1197
>
> Find the whole pipeline (with other patches too...)
>
> https://dev.azure.com/hs0298/hs/_build/results?buildId=114&view=results
>
> Could you take a look please?

Done.
Sorry for the inconvenience.

[...]

Best regards,
Fedor