devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 0/2] media: i2c: isl7998x: Add driver for Intersil ISL7998x
@ 2021-06-02 13:29 Michael Tretter
  2021-06-02 13:29 ` [PATCH v2 1/2] media: dt-bindings: Add Intersil ISL79987 DT bindings Michael Tretter
  2021-06-02 13:29 ` [PATCH v2 2/2] media: i2c: isl7998x: Add driver for Intersil ISL7998x Michael Tretter
  0 siblings, 2 replies; 4+ messages in thread
From: Michael Tretter @ 2021-06-02 13:29 UTC (permalink / raw)
  To: linux-media, devicetree, Marek Vasut
  Cc: Rob Herring, Mauro Carvalho Chehab, Sakari Ailus, kernel,
	Michael Tretter

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

-- 
2.29.2


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-06-10 17:57 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-06-02 13:29 [PATCH v2 0/2] media: i2c: isl7998x: Add driver for Intersil ISL7998x Michael Tretter
2021-06-02 13:29 ` [PATCH v2 1/2] media: dt-bindings: Add Intersil ISL79987 DT bindings Michael Tretter
2021-06-10 17:57   ` Rob Herring
2021-06-02 13:29 ` [PATCH v2 2/2] media: i2c: isl7998x: Add driver for Intersil ISL7998x Michael Tretter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).