mbox series

[v3,0/2] v4l2-cci: little-endian registers

Message ID 20231102095048.3222110-1-alexander.stein@ew.tq-group.com
Headers show
Series v4l2-cci: little-endian registers | expand

Message

Alexander Stein Nov. 2, 2023, 9:50 a.m. UTC 
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.

I just focues on fixing the endianess issue. Any future cleanup as already
discussed is discarded for now.

Changes in v3:
* Explicitly name IMX290 as an example sensor using little-endian registers
* Move CCI_REG_LE to first location on register definition
* Use unsigned numbers for registers width definition
* Added Laurent's and Hans' R-b

Changes in v2:
* Add proper Fixes/CC tags to both patches
* Add little-endian support for cci_read()

Best regards,
Alexander

[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 | 44 ++++++++++++++++++++++++------
 include/media/v4l2-cci.h           |  5 ++++
 3 files changed, 62 insertions(+), 29 deletions(-)