[PATCH v2] media: dt-bindings: Merge OV5695 into OV5693 binding
Rob Herring
robh at kernel.org
Tue Sep 5 13:54:17 PDT 2023
On Thu, 17 Aug 2023 15:26:46 -0500, Rob Herring wrote:
> The OV5695 binding is almost the same as the OV5693 binding. The only
> difference is 'clock-names' is defined for OV5695. However, the lack of
> clock-names is an omission as the Linux OV5693 driver expects the same
> 'xvclk' clock name.
>
> 'link-frequencies' is required by OV5693, but not OV5695, so make that
> conditional. Really, this shouldn't vary by device, but we're stuck with
> the existing binding use.
>
> The rockchip-isp1 binding example is missing required properties, so it
> has to be updated as well.
>
> Reviewed-by: Conor Dooley <conor.dooley at microchip.com>
> Signed-off-by: Rob Herring <robh at kernel.org>
> ---
> v2:
> - Make link-frequencies conditionally required
> ---
> .../devicetree/bindings/media/i2c/ov5695.txt | 41 -------------------
> .../bindings/media/i2c/ovti,ov5693.yaml | 31 ++++++++++----
> .../bindings/media/rockchip-isp1.yaml | 1 +
> 3 files changed, 25 insertions(+), 48 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/media/i2c/ov5695.txt
>
Applied, thanks!
More information about the Linux-rockchip
mailing list