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
