On Sun, Dec 22, 2019 at 5:39 AM Thierry Reding <thierry.reding@xxxxxxxxx> wrote: > > From: Thierry Reding <treding@xxxxxxxxxx> > > Convert the device tree bindings for the Tegra124 EMC controller to the > DT schema format using json-schema. While at it, clean up the binding a > little bit by removing any mention of how RAM code and clock frequency > are represented in unit-addresses (which they aren't) and by adding the > EMC clock without which the EMC controller can't change the frequency at > which the external memory is clocked. While this is technically an ABI > break (the clock was not required before), this should be fine because > there isn't much that the EMC driver can do without access to the EMC > clock. > > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > --- > .../nvidia,tegra124-emc.txt | 374 ------------- > .../nvidia,tegra124-emc.yaml | 528 ++++++++++++++++++ > 2 files changed, 528 insertions(+), 374 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.txt > create mode 100644 Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.yaml The example fails 'make dt_binding_check': /home/rob/proj/git/linux-dt/.build-arm64/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.example.dt.yaml: memory-controller@70019000: '#reset-cells' is a required property /home/rob/proj/git/linux-dt/.build-arm64/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.example.dt.yaml: external-memory-controller@7001b000: emc-timings-0:timing-0: 'nvidia,emc-mrs-wait-cnt' is a required property /home/rob/proj/git/linux-dt/.build-arm64/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.example.dt.yaml: external-memory-controller@7001b000: emc-timings-0:timing-0: 'nvidia,emc-xm2dqspadctrl2' is a required property /home/rob/proj/git/linux-dt/.build-arm64/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.example.dt.yaml: external-memory-controller@7001b000: emc-timings-0:timing-0: 'nvidia,emc-zcal-interval' is a required property Rob