[PATCH 2/3] rockchip: i2c: move ACK comment where it applies
Heiko Schocher
hs at nabladev.com
Mon Nov 10 06:57:39 CET 2025
Hello Quentin,
On 07.11.25 12:39, Quentin Schulz wrote:
> From: Quentin Schulz <quentin.schulz at cherry.de>
>
> The I2C_CON_LASTACK is kind of a misnomer as setting it means sending a
> NACK as last byte acknowledge when the controller is in receive mode. It
> should therefore be used only when there's no more data to transfer
> after this.
>
> Move the comment in the proper if block.
>
> Sync the comment with the Linux kernel's while at it so it's more
> explicit.
>
> Fixes: 5deaa530280f ("rockchip: i2c: fix >32 byte reads")
> Signed-off-by: Quentin Schulz <quentin.schulz at cherry.de>
> ---
> drivers/i2c/rk_i2c.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
Reviewed-by: Heiko Schocher <hs at nabladev.com>
bye,
Heiko
--
Nabla Software Engineering
HRB 40522 Augsburg
Phone: +49 821 45592596
E-Mail: office at nabladev.com
Geschäftsführer : Stefano Babic
More information about the U-Boot
mailing list