Re: [PATCH 1/2] dt-bindings: media: i2c: Add OmniVision OV13858

[Sakari Ailus]

Hi George,

On Thu, Oct 15, 2026 at 10:48:19PM +0100, George Neacsu via B4 Relay wrote:
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov13858.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov13858.yaml
> new file mode 100644
> index 0000000..a1cbdb1
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov13858.yaml
> @@ -0,0 +1,113 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/media/i2c/ovti,ov13858.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: OmniVision OV13858 CMOS Sensor
> +
> +maintainers:
> +  - George Neacsu <geocausa@xxxxxxxxx>
> +
> +description:
> +  The OmniVision OV13858 is a 13 megapixel CMOS image sensor with an I2C
> +  control interface. It outputs 10-bit Bayer data over a four-lane MIPI
> +  CSI-2 D-PHY interface and takes a 19.2 MHz input clock.

I wouldn't mention the input clock frequency here...

> +
> +allOf:
> +  - $ref: /schemas/media/video-interface-devices.yaml#
> +
> +properties:
> +  compatible:
> +    const: ovti,ov13858
> +
> +  reg:
> +    maxItems: 1
> +
> +  clocks:
> +    description: Input clock (19.2 MHz)

...because it's also documented here.

The frequency range is from 6 to 27 MHz.

-- 
Regards,

Sakari Ailus