[RFC PATCH] ASoC: dt-bindings: adi,adau17x1: Convert to DT schema
Convert the binding document for adi,adau17x1 from txt to yaml so one could validate dt-entries correctly and any future additions can go into yaml format. Add address and size cells to example to prevent errors regarding reg format.
Signed-off-by: Vijaya Anand sunrockers8@gmail.com Cc: Daniel Baluta daniel.baluta@nxp.com --- .../bindings/sound/adi,adau17x1.txt | 32 ------------ .../bindings/sound/adi,adau17x1.yaml | 52 +++++++++++++++++++ 2 files changed, 52 insertions(+), 32 deletions(-) delete mode 100644 Documentation/devicetree/bindings/sound/adi,adau17x1.txt create mode 100644 Documentation/devicetree/bindings/sound/adi,adau17x1.yaml
diff --git a/Documentation/devicetree/bindings/sound/adi,adau17x1.txt b/Documentation/devicetree/bindings/sound/adi,adau17x1.txt deleted file mode 100644 index 1447dec28125..000000000000 --- a/Documentation/devicetree/bindings/sound/adi,adau17x1.txt +++ /dev/null @@ -1,32 +0,0 @@ -Analog Devices ADAU1361/ADAU1461/ADAU1761/ADAU1961/ADAU1381/ADAU1781 - -Required properties: - - - compatible: Should contain one of the following: - "adi,adau1361" - "adi,adau1461" - "adi,adau1761" - "adi,adau1961" - "adi,adau1381" - "adi,adau1781" - - - reg: The i2c address. Value depends on the state of ADDR0 - and ADDR1, as wired in hardware. - -Optional properties: - - clock-names: If provided must be "mclk". - - clocks: phandle + clock-specifiers for the clock that provides - the audio master clock for the device. - -Examples: -#include <dt-bindings/sound/adau17x1.h> - - i2c_bus { - adau1361@38 { - compatible = "adi,adau1761"; - reg = <0x38>; - - clock-names = "mclk"; - clocks = <&audio_clock>; - }; - }; diff --git a/Documentation/devicetree/bindings/sound/adi,adau17x1.yaml b/Documentation/devicetree/bindings/sound/adi,adau17x1.yaml new file mode 100644 index 000000000000..9c1dbf2b4e70 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/adi,adau17x1.yaml @@ -0,0 +1,52 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/adi,adau17x1.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Analog Devices ADAU1361/ADAU1461/ADAU1761/ADAU1961/ADAU1381/ADAU1781 + +maintainers: + - Lars-Peter Clausen lars@metafoo.de + +properties: + compatible: + enum: + - adi,adau1361 + - adi,adau1461 + - adi,adau1761 + - adi,adau1961 + - adi,adau1381 + - adi,adau1781 + + reg: + maxItems: 1 + description: The i2c address. Value depends on the state of ADDR0 and ADDR1, as wired in hardware. + + clock-names: + const: mclk + + clocks: + maxItems: 1 + description: + phandle + clock-specifiers for the clock that provides the audio master + clock for the device. + +required: + - compatible + - reg + +unevaluatedProperties: false + +examples: + - | + i2c_bus { + #address-cells = <1>; + #size-cells = <0>; + adau1361@38 { + compatible = "adi,adau1761"; + reg = <0x38>; + clock-names = "mclk"; + clocks = <&audio_clock>; + }; + };
On 11/03/2023 21:03, Vijaya Anand wrote:
Convert the binding document for adi,adau17x1 from txt to yaml so one could validate dt-entries correctly and any future additions can go into yaml format. Add address and size cells to example to prevent errors regarding reg format.
Signed-off-by: Vijaya Anand sunrockers8@gmail.com Cc: Daniel Baluta daniel.baluta@nxp.com
Thank you for your patch. There is something to discuss/improve.
-Examples: -#include <dt-bindings/sound/adau17x1.h>
- i2c_bus {
adau1361@38 {
compatible = "adi,adau1761";
reg = <0x38>;
clock-names = "mclk";
clocks = <&audio_clock>;
};
- };
diff --git a/Documentation/devicetree/bindings/sound/adi,adau17x1.yaml b/Documentation/devicetree/bindings/sound/adi,adau17x1.yaml new file mode 100644 index 000000000000..9c1dbf2b4e70 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/adi,adau17x1.yaml @@ -0,0 +1,52 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/adi,adau17x1.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml#
+title: Analog Devices ADAU1361/ADAU1461/ADAU1761/ADAU1961/ADAU1381/ADAU1781
But what is this? Codec? Amplifier?
+maintainers:
- Lars-Peter Clausen lars@metafoo.de
+properties:
- compatible:
- enum:
- adi,adau1361
- adi,adau1461
- adi,adau1761
- adi,adau1961
- adi,adau1381
- adi,adau1781
Please put them in some order, e.g. alphabetical.
- reg:
- maxItems: 1
- description: The i2c address. Value depends on the state of ADDR0 and ADDR1, as wired in hardware.
This exceeds 80 characters (see coding style).
- clock-names:
- const: mclk
- clocks:
- maxItems: 1
- description:
phandle + clock-specifiers for the clock that provides the audio master
clock for the device.
Drop "phandle + clock-specifiers for the " and make it simpler, just items with description:
clocks items: - description: .......
+required:
- compatible
- reg
+unevaluatedProperties: false
Instead: additionalProperties: false Unless you plan to reference dai-common or component-common
+examples:
- |
- i2c_bus {
i2c
#address-cells = <1>;
#size-cells = <0>;
adau1361@38 {
Node names should be generic. https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetre...
compatible = "adi,adau1761";
reg = <0x38>;
clock-names = "mclk";
clocks = <&audio_clock>;
Best regards, Krzysztof
participants (1)
-
Krzysztof Kozlowski
-
Vijaya Anand