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

[Heiko Schocher]

Hello Fedor,

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?

Thanks!

bye,
Heiko

-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de