Thread (23 messages) 23 messages, 4 authors, 2020-09-19

Re: [PATCH v3 12/14] dt-bindings: mtd: gpmi-nand: Fix matching of clocks on different SoCs

From: Rob Herring <robh+dt@kernel.org>
Date: 2020-09-08 16:52:37
Also in: linux-clk, linux-devicetree, linux-pm, linux-pwm, linux-serial, linux-watchdog, lkml

On Mon, Sep 7, 2020 at 12:10 AM Krzysztof Kozlowski [off-list ref] wrote:
On Fri, Sep 04, 2020 at 04:36:39PM -0600, Rob Herring wrote:
quoted
On Fri, Sep 4, 2020 at 9:25 AM Krzysztof Kozlowski [off-list ref] wrote:
quoted
Driver requires different amount of clocks for different SoCs.  Describe
these requirements properly to fix dtbs_check warnings like:

    arch/arm64/boot/dts/freescale/imx8mm-beacon-kit.dt.yaml: nand-controller@33002000: clock-names:1: 'gpmi_apb' was expected

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

---

Changes since v1:
1. Do not require order of clocks (use pattern).
To the extent that you can, you should fix the order in dts files
first. If we just adjust the schemas to match the dts files, then
what's the point?
The DTSes do not have mixed order of clocks between each other, as fair
as I remember. It was fix after Sasha Hauer comment that order is not
necessarily good.

We have the clock-names property, why enforcing the order?
Because DT/OpenFirmware has always had a defined order for property
values. '*-names' is just extra information.

Rob

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help