mbox series

[v2,0/2] media: i2c: isl7998x: Add driver for Intersil ISL7998x

Message ID 20210602132950.4167596-1-m.tretter@pengutronix.de (mailing list archive)
Headers show
Series media: i2c: isl7998x: Add driver for Intersil ISL7998x | expand

Message

Michael Tretter June 2, 2021, 1:29 p.m. UTC 
Hello,

This is a respin/v2 of the previous series for adding the isl7998x driver [0]
sent by Marek Vasut. I updated the patches according to the review comments of
this previous series.

The isl7998x is an analog video to MIPI CSI-2 or BT.656 converter. The dt
binding describes only the isl79987 chip, which supports MIPI CSI-2. The
driver could be extended to handle isl79988 (for BT.656), too, but this isn't
implemented.

As this is a v2, I left the patch author as is, but added myself as a
maintainer for the driver. Please tell me, if I should handle this differently
and/or further document my changes in the patches.

checkpatch.pl complains about the macros that are used for handling the
regmaps and paged register space in the driver. I don't have a good idea how
to handle this differently. Suggestions are welcome, if this is actually
problem.

Michael

[0] https://lore.kernel.org/linux-media/20190520201812.7937-1-marex@denx.de/

Changelog:

v2:

- convert dt binding to yaml
- change binding to ISL79987 only
- general driver cleanup
- convert driver to pm_runtime
- use ports in device tree for specifying inputs
- add reset gpio

Marek Vasut (2):
  media: dt-bindings: Add Intersil ISL79987 DT bindings
  media: i2c: isl7998x: Add driver for Intersil ISL7998x

 .../bindings/media/i2c/isil,isl79987.yaml     |  108 ++
 MAINTAINERS                                   |    8 +
 drivers/media/i2c/Kconfig                     |    9 +
 drivers/media/i2c/Makefile                    |    1 +
 drivers/media/i2c/isl7998x.c                  | 1421 +++++++++++++++++
 5 files changed, 1547 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/media/i2c/isil,isl79987.yaml
 create mode 100644 drivers/media/i2c/isl7998x.c