Hi, this is the 3rd series adding support for OV9281 which is quite similar to OV9282. This includes: * a small cleanup (Patch 1) * adding a new compatible (Patch 2 & 3) * adding support for regulators (Patch 4 & 5) * Fixing v4l2 subdev name depending on actual model name (Patch 6) * Add regmap support (Patch 7) Thanks for anyone doing review and giving a feedback. Here are the changes in v3: * Removed struct field documentation as well (Patch 1) * Dropped v2 Patch 6 (wrong approach) * Added new Patch 6 to set subdev name according to model parsed form compatible * Added new Patch 7 adding regmap support This is a preparation to solve the defunct auto-increment using regmap's 'use_single_read' (still WIP) Here are the changes in v2: * Added Krzysztof's a-b for Patch 2 & 4 * Added Daniele's a-b for Patch 1 & 3 * Removed additional error message in ov9282_power_off * Renamed function from ov9282_configure_regulators to ov9282_get_regulators * Cleaned-up reading ID registers The regulator support is based on the driver from Raspberry Pi downstream kernel [1], the ID register read fix as well. Please refer to [2] why this fix is required. I can confirm this is necessary by checking with a Logic analyzer on the i2c bus. Best regards, Alexander [1] https://github.com/raspberrypi/linux/blob/rpi-5.15.y/drivers/media/i2c/ov9281.c [2] https://github.com/raspberrypi/linux/commit/58deee7c917e1c3c5e37987c3a89ad19d791f58a Alexander Stein (7): media: i2c: ov9282: remove unused and unset i2c_client member media: dt-bindings: media: Add compatible for ov9281 media: i2c: ov9282: Add ov9281 compatible media: dt-bindings: media: ov9282: Add power supply properties media: i2c: ov9282: Add regulator support media: i2c: ov9282: Set v4l2 subdev name according to sensor model media: i2c: ov9282: Add regmap support .../bindings/media/i2c/ovti,ov9282.yaml | 14 ++- drivers/media/i2c/ov9282.c | 102 ++++++++++++------ 2 files changed, 84 insertions(+), 32 deletions(-) -- 2.25.1
