linux-kernel - Re: [PATCH v3] ASoC: dt-bindings: realtek,alc5623: convert to DT schema

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20250807000356.GA2072715-robh@kernel.org>
Date: Wed, 6 Aug 2025 19:03:56 -0500
From: Rob Herring <robh@...nel.org>
To: Mahdi Khosravi <mmk1776@...il.com>
Cc: devicetree@...r.kernel.org, Liam Girdwood <lgirdwood@...il.com>,
	Mark Brown <broonie@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>, linux-sound@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] ASoC: dt-bindings: realtek,alc5623: convert to DT
 schema

On Wed, Aug 06, 2025 at 04:18:18PM +0300, Mahdi Khosravi wrote:
> I converted the alc5623 audio codec binding from text to DT schema.

Convert the alc5623...

> This is my first try and I used make dt_binding_check & make dtbs_check to verify
> without getting any errors.

Drop this. Passing is assumed. Actually, 'make dtbs_check' warnings are 
fine if you think the .dts has a mistake.

> 
> Signed-off-by: Mahdi Khosravi <mmk1776@...il.com>
> ---
> Changes in v3:
> - Drop allOf, just use $ref for uint32
> - Remove stray '>' in descriptions
> - Fix subject to "to DT schema"
> 
> Changes in v2:
> - Add dai-common ref
> - Switch add-ctrl/jack-det-ctrl to allOf uint32
> - Use unevaluatedProperties
> - Fix example compatible
> ---
>  .../devicetree/bindings/sound/alc5623.txt     | 25 ---------
>  .../bindings/sound/realtek,alc5623.yaml       | 52 +++++++++++++++++++
>  2 files changed, 52 insertions(+), 25 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/sound/alc5623.txt
>  create mode 100644 Documentation/devicetree/bindings/sound/realtek,alc5623.yaml
> 
> diff --git a/Documentation/devicetree/bindings/sound/alc5623.txt b/Documentation/devicetree/bindings/sound/alc5623.txt
> deleted file mode 100644
> index 26c86c98d671..000000000000
> --- a/Documentation/devicetree/bindings/sound/alc5623.txt
> +++ /dev/null
> @@ -1,25 +0,0 @@
> -ALC5621/ALC5622/ALC5623 audio Codec
> -
> -Required properties:
> -
> - - compatible:	"realtek,alc5623"
> - - reg:		the I2C address of the device.
> -
> -Optional properties:
> -
> - - add-ctrl:	  Default register value for Reg-40h, Additional Control
> -		  Register. If absent or has the value of 0, the
> -		  register is untouched.
> -
> - - jack-det-ctrl: Default register value for Reg-5Ah, Jack Detect
> -		  Control Register. If absent or has value 0, the
> -		  register is untouched.
> -
> -Example:
> -
> -	alc5621: alc5621@1a {
> -		compatible = "alc5621";
> -		reg = <0x1a>;
> -		add-ctrl = <0x3700>;
> -		jack-det-ctrl = <0x4810>;
> -	};
> diff --git a/Documentation/devicetree/bindings/sound/realtek,alc5623.yaml b/Documentation/devicetree/bindings/sound/realtek,alc5623.yaml
> new file mode 100644
> index 000000000000..2a389ca95b0d
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/sound/realtek,alc5623.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/realtek,alc5623.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: ALC5621/ALC5622/ALC5623 Audio Codec
> +
> +maintainers:
> +  - Mahdi Khosravi <mmk1776@...il.com>
> +
> +allOf:
> +  - $ref: dai-common.yaml#
> +
> +properties:
> +  compatible:
> +    const: realtek,alc5623

What about alc5621/22? At least 21 is in use:

arch/arm/boot/dts/marvell/kirkwood-t5325.dts:                           compatible = "realtek,alc5621";

> +
> +  reg:
> +    maxItems: 1
> +
> +  add-ctrl:
> +    description:
> +      Default register value for Reg-40h, Additional Control Register.
> +      If absent or zero, the register is left untouched.
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +
> +  jack-det-ctrl:
> +    description:
> +      Default register value for Reg-5Ah, Jack Detect Control Register.
> +      If absent or zero, the register is left untouched.
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
> +examples:
> +  - |
> +    i2c {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        codec@1a {
> +            compatible = "realtek,alc5623";
> +            reg = <0x1a>;
> +            add-ctrl = <0x3700>;
> +            jack-det-ctrl = <0x4810>;
> +        };
> +    };
> -- 
> 2.50.1
>