mbox series

[v6,0/4] ov08x40: Enable use of ov08x40 on Qualcomm X1E80100 CRD

Message ID 20241010-b4-master-24-11-25-ov08x40-v6-0-cf966e34e685@linaro.org (mailing list archive)
Headers show
Series ov08x40: Enable use of ov08x40 on Qualcomm X1E80100 CRD | expand

Message

Bryan O'Donoghue Oct. 10, 2024, 12:33 p.m. UTC
Changes in v6:
- As I was applying a suggested fix from V5 of this series to other yaml
  schema in media/i2c it seemed to me that there were a number of
  properties that would be affected by converting properites: endpoint:
  additionalProperties: false to unevaluatedProperites: false
  I pinged Laurent, Sakari, Rob and Krsysztof about that leading to:

  link-frequencies: true is considered a valid hardware description.
  
  We don't want everything in /schemas/media/video-interfaces.yaml
  to be valid for each of the port{} descriptions for sensors so:

  a) use additionalProperties: false listing valid properties directly.
  b) Fixup various schema to list the valid properties in line with above.

- Convert unevaluatedProperites: false to additionalProperties: false
  Laurent

- This isn't exactly what Krsysztof gave his RB for so, I've omitted that

- Add remote-endpoint: true to port{} - required as result of additionalProperties: false

- I'll still follow up this series with a more general fix in-line with the
  above as TBH I just copy/pasted from some upstream yaml so it looks like
  a more general remediation is warranted, saving someone else from
  repeating my mistake.

- Add remote-endpoint: true to port{} - required as result of additionalProperties: false

- Use schemas/media/video-interface-devices.yaml# with unevaluatedProperites: false
  So that rotation and orientation are valid - bod

- Link to v5: https://lore.kernel.org/r/20241005-b4-master-24-11-25-ov08x40-v5-0-5f1eb2e11036@linaro.org

Changes in v5:
- Fixes smatch CI splat
- Link to v4: https://lore.kernel.org/r/20241003-b4-master-24-11-25-ov08x40-v4-0-7ee2c45fdc8c@linaro.org

Changes in v4:
- Drops link-frequencies from properties: as discussed here:
  https://lore.kernel.org/r/Zv6STSKeNNlT83ux@kekkonen.localdomain
- Link to v3: https://lore.kernel.org/r/20241002-b4-master-24-11-25-ov08x40-v3-0-483bcdcf8886@linaro.org

Changes in v3:
- Drops assigned-clock-* from description retains in example - Sakari,
  Krzysztof
- Updates example fake clock names to ov08x40_* instead of copy/paste
  ov9282_clk -> ov08x40_clk, ov9282_clk_parent -> ov08x40_clk_parent - bod
- Link to v2: https://lore.kernel.org/r/20241001-b4-master-24-11-25-ov08x40-v2-0-e478976b20c1@linaro.org

Changes in v2:
- Drops "-" in ovti,ov08x40.yaml after description: - Rob
- Adds ":" after first line of description text - Rob
- dts -> DT in commit log - Rob
- Removes dependency on 'xvclk' as a name in yaml
  and driver - Sakari
- Uses assigned-clock, assigned-clock-parents and assigned-clock-rates -
  Sakari
- Drops clock-frequency - Sakarai, Krzysztof
- Drops dovdd-supply, avdd-supply, dvdd-supply and reset-gpios
  as required, its perfectly possible not to have the reset GPIO or the
  power rails under control of the SoC. - bod

- Link to v1: https://lore.kernel.org/r/20240926-b4-master-24-11-25-ov08x40-v1-0-e4d5fbd3b58a@linaro.org

V1:
This series brings fixes and updates to ov08x40 which allows for use of
this sensor on the Qualcomm x1e80100 CRD but also on any other dts based
system.

Firstly there's a fix for the pseudo burst mode code that was added in
8f667d202384 ("media: ov08x40: Reduce start streaming time"). Not every I2C
controller can handle an arbitrary sized write, this is the case on
Qualcomm CAMSS/CCI I2C sensor interfaces which limit the transaction size
and communicate this limit via I2C quirks. A simple fix to optionally break
up the large submitted burst into chunks not exceeding adapter->quirk size
fixes.

Secondly then is addition of a yaml description for the ov08x40 and
extension of the driver to support OF probe and powering on of the power
rails from the driver instead of from ACPI.

Once done the sensor works without further modification on the Qualcomm
x1e80100 CRD.

Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
Bryan O'Donoghue (4):
      media: ov08x40: Fix burst write sequence
      media: dt-bindings: Add OmniVision OV08X40
      media: ov08x40: Rename ext_clk to xvclk
      media: ov08x40: Add OF probe support

 .../bindings/media/i2c/ovti,ov08x40.yaml           | 120 ++++++++++++++
 drivers/media/i2c/ov08x40.c                        | 181 ++++++++++++++++++---
 2 files changed, 277 insertions(+), 24 deletions(-)
---
base-commit: 2b7275670032a98cba266bd1b8905f755b3e650f
change-id: 20240926-b4-master-24-11-25-ov08x40-c6f477aaa6a4

Best regards,