dt-bindings: PCI: fu740-pci: fix missing clock-names
authorConor Dooley <conor.dooley@microchip.com>
Fri, 19 Aug 2022 23:14:10 +0000 (00:14 +0100)
committerLorenzo Pieralisi <lpieralisi@kernel.org>
Wed, 24 Aug 2022 11:38:26 +0000 (13:38 +0200)
commitb408fad61d34c765c3e01895286332af2d50402a
tree64c09eaff02f64a3b71c4472c9c16c2d6a2f746b
parent7f08e806a03e0453a0de27137b668d4de52fcd49
dt-bindings: PCI: fu740-pci: fix missing clock-names

The commit b92225b034c0 ("dt-bindings: PCI: designware: Fix
'unevaluatedProperties' warnings") removed the clock-names property as
a requirement and from the example as it triggered unevaluatedProperty
warnings. dtbs_check was not able to pick up on this at the time, but
now can:

arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dtb: pcie@e00000000: Unevaluated properties are not allowed ('clock-names' was unexpected)
        From schema: linux/Documentation/devicetree/bindings/pci/sifive,fu740-pcie.yaml

The property was already in use by the FU740 DTS and the clock must be
enabled. The Linux and FreeBSD drivers require the property to enable
the clocks correctly Re-add the property and its "clocks" dependency,
while making it required.

Link: https://lore.kernel.org/r/20220819231415.3860210-2-mail@conchuod.ie
Fixes: b92225b034c0 ("dt-bindings: PCI: designware: Fix 'unevaluatedProperties' warnings")
Fixes: 43cea116be0b ("dt-bindings: PCI: Add SiFive FU740 PCIe host controller")
Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Lorenzo Pieralisi <lpieralisi@kernel.org>
Reviewed-by: Rob Herring <robh@kernel.org>
Documentation/devicetree/bindings/pci/sifive,fu740-pcie.yaml