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@xxxxxxxxxxxxxxxxxxxxx/ > [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(-) >
