On Tue Apr 22 13:20:52 2025 +0100, Umang Jain wrote: > CCI_REG_HNUM should be using CCI_REG16_LE() instead of CCI_REG8() > as HNUM spans from 0x302e[0:7] to 0x302f[0:3]. > > Signed-off-by: Umang Jain <[email protected]> > Signed-off-by: Kieran Bingham <[email protected]> > Fixes: 8f0926dba799 ("media: imx335: Use V4L2 CCI for accessing sensor > registers") > Cc: [email protected] > Signed-off-by: Sakari Ailus <[email protected]> > Signed-off-by: Hans Verkuil <[email protected]>
Patch committed. Thanks, Hans Verkuil drivers/media/i2c/imx335.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- diff --git a/drivers/media/i2c/imx335.c b/drivers/media/i2c/imx335.c index 0beb80b8c458..d400a019f6b3 100644 --- a/drivers/media/i2c/imx335.c +++ b/drivers/media/i2c/imx335.c @@ -31,7 +31,7 @@ #define IMX335_REG_CPWAIT_TIME CCI_REG8(0x300d) #define IMX335_REG_WINMODE CCI_REG8(0x3018) #define IMX335_REG_HTRIMMING_START CCI_REG16_LE(0x302c) -#define IMX335_REG_HNUM CCI_REG8(0x302e) +#define IMX335_REG_HNUM CCI_REG16_LE(0x302e) /* Lines per frame */ #define IMX335_REG_VMAX CCI_REG24_LE(0x3030)
