Re: [PATCH v3] ASoC: dt-bindings: ak4458: Convert to dtschema

On 27/03/2023 23:32, Saalim Quadri wrote:

Convert the AK4458 audio DAC bindings to DT schema.

Signed-off-by: Saalim Quadri danascape@gmail.com

Changes: V1 -> V2: Use the correct way for dsd-path property Drop ak4458 label form example V2 -> V3: ak4458 is the only one that does not support dsd-path, so we do not require to define an array

.../devicetree/bindings/sound/ak4458.txt | 28 -------- .../bindings/sound/asahi-kasei,ak4458.yaml | 65 +++++++++++++++++++ 2 files changed, 65 insertions(+), 28 deletions(-) delete mode 100644 Documentation/devicetree/bindings/sound/ak4458.txt create mode 100644 Documentation/devicetree/bindings/sound/asahi-kasei,ak4458.yaml

diff --git a/Documentation/devicetree/bindings/sound/ak4458.txt b/Documentation/devicetree/bindings/sound/ak4458.txt deleted file mode 100644 index 0416c14895d6..000000000000 --- a/Documentation/devicetree/bindings/sound/ak4458.txt +++ /dev/null @@ -1,28 +0,0 @@ -AK4458 audio DAC

-This device supports I2C mode.

-Required properties:

-- compatible : "asahi-kasei,ak4458" or "asahi-kasei,ak4497" -- reg : The I2C address of the device for I2C

-Optional properties: -- reset-gpios: A GPIO specifier for the power down & reset pin -- mute-gpios: A GPIO specifier for the soft mute pin -- AVDD-supply: Analog power supply -- DVDD-supply: Digital power supply -- dsd-path: Select DSD input pins for ak4497

•        0: select #16, #17, #19 pins
  

•        1: select #3, #4, #5 pins
  

-Example:

-&i2c {

• ak4458: dac@10 {

• compatible = "asahi-kasei,ak4458";
  

• reg = <0x10>;
  

• reset-gpios = <&gpio1 10 GPIO_ACTIVE_LOW>
  

• mute-gpios = <&gpio1 11 GPIO_ACTIVE_HIGH>
  

• };

-}; diff --git a/Documentation/devicetree/bindings/sound/asahi-kasei,ak4458.yaml b/Documentation/devicetree/bindings/sound/asahi-kasei,ak4458.yaml new file mode 100644 index 000000000000..3842e75d9921 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/asahi-kasei,ak4458.yaml @@ -0,0 +1,65 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/asahi-kasei,ak4458.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml#

+title: AK4458 audio DAC

+maintainers:

• • Shengjiu Wang shengjiu.wang@nxp.com

+properties:

• compatible:
• enum:

•  - asahi-kasei,ak4458
  

•  - asahi-kasei,ak4497
  

• reg:
• maxItems: 1
• avdd-supply:
• description: Analog power supply
• dvdd-supply:
• description: Digital power supply
• reset-gpios:
• maxItems: 1
• mute-gpios:
• maxItems: 1
• description:

•  GPIO used to mute all the outputs
  

What happened to dsd-path? Do not remove properties from the bindings without some valid reason.

+allOf:

• • if:

•  properties:
  

•    compatible:
  

•      contains:
  

•        const:
  

•          asahi-kasei,ak4458
  

This is one line - const: foo bar

• then:

•  properties:
  

•    dsd-path: false
  

Best regards, Krzysztof