[PATCH v2 00/11] media: i2c: max9286: Small new features

From: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
To: linux-media@vger.kernel.org
Cc: linux-renesas-soc@vger.kernel.org,
	"Jacopo Mondi" <jacopo@jmondi.org>,
	"Kieran Bingham" <kieran.bingham@ideasonboard.com>,
	"Niklas Söderlund" <niklas.soderlund@ragnatech.se>,
	"Thomas Nizan" <tnizan@witekio.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	devicetree@vger.kernel.org, "Mark Brown" <broonie@kernel.org>,
	"Liam Girdwood" <lgirdwood@gmail.com>
Subject: [PATCH v2 00/11] media: i2c: max9286: Small new features
Date: Sat,  1 Jan 2022 20:27:55 +0200	[thread overview]
Message-ID: <20220101182806.19311-1-laurent.pinchart+renesas@ideasonboard.com> (raw)

Hello,

This small patch series adds a few new features to the max9286 driver:

- Support for per-port supplies (01/11 and 04/11)
- Remote I2C bus speed selection (02/11 and 09/11)
- GMSL bus width selection (03/11 and 10/11)
- Manual framesync operation (05/11)
- RAW12 support (06/11 and 07/11)

The remaining patches are small cleanups. Please see individual patches
for details.

I'm in two minds about the bus width selection. It would be possible to
query the information from the connected serializers at runtime, using
for instance the .g_mbus_config() subdev operation. On the other hand,
there's *lots* of GMSL-specific bus configuration options, which would
require a rework of .g_mbus_config() to pass a bus-specific structure.
We would then end up having many configuration parameters not specific
to video there (such as the I2C speed for instance). Furthermore, while
some parameters may be different between cameras (high-immunity mode,
for instance, can be configured per port), many need to be identical on
all ports. I'm not sure yet what the best way to address that without an
overcomplicated implementation would be (one option would be to get the
parameters from the first camera and simply ignore conflicting values
reported by other cameras). Thoughts are welcome.

Could someone with RDACM20 cameras test the series ? I'm a bit worried
about patch 11/11 in particular, there's a small risk of regression.

Laurent Pinchart (10):
  dt-bindings: media: i2c: max9286: Add support for per-port supplies
  dt-bindings: media: i2c: max9286: Add property to select I2C speed
  dt-bindings: media: i2c: max9286: Add property to select bus width
  media: i2c: max9286: Support manual framesync operation
  media: i2c: max9286: Rename MAX9286_DATATYPE_RAW11 to RAW12
  media: i2c: max9286: Support 12-bit raw bayer formats
  media: i2c: max9286: Define macros for all bits of register 0x15
  media: i2c: max9286: Configure remote I2C speed from device tree
  media: i2c: max9286: Configure bus width from device tree
  media: i2c: max9286: Select HS as data enable signal

Thomas Nizan (1):
  media: i2c: max9286: Add support for port regulators

 .../bindings/media/i2c/maxim,max9286.yaml     |  28 +-
 drivers/media/i2c/max9286.c                   | 425 +++++++++++++++---
 2 files changed, 387 insertions(+), 66 deletions(-)

base-commit: 68b9bcc8a534cd11fe55f8bc82f948aae7d81b3c
-- 
Regards,

Laurent Pinchart