Re: [PATCH V5 1/2] virtio: i2c: No need to have separate read-write buffers

From
Arnd Bergmann <>
Date
2021-10-12T13:58:31+00:00
ID
CAK8P3a1CAx0F=01MeVBs8+A0ur1BKCtC3T6Ead0=+
Thread
Re: [PATCH V5 1/2] virtio: i2c: No need to have separate read-write buffers
On Tue, Oct 12, 2021 at 1:23 PM Viresh Kumar <> wrote:
>
> The virtio I2C protocol allows to contain multiple read-write requests
> in a single I2C transaction using the VIRTIO_I2C_FLAGS_FAIL_NEXT flag,
> where each request contains a header, buffer and status.
>
> There is no need to pass both read and write buffers in a single
> request, as we have a better way of combining requests into a single I2C
> transaction. Moreover, this also limits the transactions to two buffers,
> one for read operation and one for write. By using
> VIRTIO_I2C_FLAGS_FAIL_NEXT, we don't have any such limits.
>
> Remove support for multiple buffers within a single request.
>
> Fixes: https://github.com/oasis-tcs/virtio-spec/issues/112
> Signed-off-by: Viresh Kumar <>

Thanks for splitting up the two patches, I think this is much clearer now.

Reviewed-by: Arnd Bergmann <>