[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4f722e53-011f-4176-b6af-080522165007@kernel.org>
Date: Wed, 22 May 2024 16:16:51 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Péter Ujfalusi <peter.ujfalusi@...il.com>,
Mighty <bavishimithil@...il.com>
Cc: Liam Girdwood <lgirdwood@...il.com>, Mark Brown <broonie@...nel.org>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Lopez Cruz <misael.lopez@...com>,
linux-sound@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5] ASoC: dt-bindings: omap-mcpdm: Convert to DT schema
On 22/05/2024 15:56, Péter Ujfalusi wrote:
>> +
>> +required:
>> + - compatible
>> + - reg
>> + - interrupts
>> + - ti,hwmods
>> + - clocks
>> + - clock-names
>> +
>> +additionalProperties: false
>> +
>> +examples:
>> + - |
>> + #include <dt-bindings/interrupt-controller/arm-gic.h>
>> + pdm@...32000 {
>
> The original label and name is preferred to be used.
Label is not used here.
About node name, not:
Node names should be generic. See also an explanation and list of
examples (not exhaustive) in DT specification:
https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation
>
>> + compatible = "ti,omap4-mcpdm";
>> + reg = <0x40132000 0x7f>, /* MPU private access */
>> + <0x49032000 0x7f>; /* L3 Interconnect */
>> + interrupts = <GIC_SPI 112 IRQ_TYPE_LEVEL_HIGH>;
>> + interrupt-parent = <&gic>;
>> + ti,hwmods = "mcpdm";
>> + clocks = <&twl6040>;
>> + clock-names = "pdmclk";
>
> The clocks cannot be added at the time when the node is defined, it is
> board specific. This way you imply that it is OK to have it in main dtsi
> file. It is not.
Wait, what? That's example and pretty standard. Example should be
complete. This is not an exceptional binding.
Best regards,
Krzysztof
Powered by blists - more mailing lists