Re: [PATCH 0/2] v4l2-cci: little-endian registers

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 11/1/23 12:17, Alexander Stein wrote:
> Hi,
> 
> after the discussions at [1] and [2], this series adds proper support for
> little-endian encoded registers.
> Patch 1 adds the infrastructure and patch 2 fixes imx290 driver.
> As v6.6 was released with imx290 broken, both should be added to stable. But
> I'm not sure if adding a Fixes tag to Patch 1 is correct, because it adds a
> new feature patch 2 requires. Any suggestion is welcomed.

I usually just add the Fixes: + Cc: stable@ tags to both patches in cases
like these. Please add the tags to both patches for v2 (with cci_read()
also fixed).

Regards,

Hans




> [1] https://lore.kernel.org/linux-media/20231030173637.GA2977515@gnbcxd0016.gnb.st.com/
> [2] https://lore.kernel.org/linux-media/ZUIuNDTJAN_fz3q6@kekkonen.localdomain/
> 
> Alexander Stein (2):
>   media: v4l2-cci: Add support for little-endian encoded registers
>   media: i2c: imx290: Properly encode registers as little-endian
> 
>  drivers/media/i2c/imx290.c         | 42 +++++++++++++++---------------
>  drivers/media/v4l2-core/v4l2-cci.c | 22 +++++++++++++---
>  include/media/v4l2-cci.h           |  5 ++++
>  3 files changed, 44 insertions(+), 25 deletions(-)
>