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@xxxxxxxxxxxxx> > Signed-off-by: Rob Herring <robh@xxxxxxxxxx> > --- > 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!
