rk_i2c bug fix

[br]

Summary

o Support for transfers > 32 bytes
o Fix slave address setting

[manu]

sys/arm64/rockchip/rk_i2c.c

535

Mhm, that should be | LSB I think.

[br]

sys/arm64/rockchip/rk_i2c.c

535

yes, but does not matter, since controller sets this bit internally depending on mode

[manu]

sys/arm64/rockchip/rk_i2c.c

363

That will not work if the transfer is not for a multiple of 32.

[br]

sys/arm64/rockchip/rk_i2c.c

363

why?

[manu]

sys/arm64/rockchip/rk_i2c.c

363

If I understand correctly if you try to do a transfer with a len of 48 bytes the first 32 bytes will be transmitted first (as it's the max length that this controller can support) but on the second transfer instead of setting TXCOUNT to 16 we will set it to 32 as you check sc->msg->len and not taking into account what have been transmitted already.

[br]

o Consider amount of bytes already transferred
o Add a small delay (not sure why it is needed, maybe another bug)

[br]

fix transfer_len for reading as well. I think eliminate a need of delay

[br]

sys/arm64/rockchip/rk_i2c.c

363

You are right, thanks!
I think driver works for me just fine now

[br]

restore err = 0