Re: [PATCH v3 7/9] arm64: dts: qcom: pm660l: add analog audio codec

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] arm64: dts: qcom: pm660l: sound-name-prefix violates analog
  codec DT schema
--

commit 2cdb7e07dbee3037de67d0c38e09df937ce70258
Author: Richard Acayan <mailingradian@gmail.com>

arm64: dts: qcom: pm660l: add analog audio codec

This commit adds the analog audio codec node to the PM660L PMIC device tree.

> diff --git a/arch/arm64/boot/dts/qcom/pm660l.dtsi b/arch/arm64/boot/dts/qcom/pm660l.dtsi
> index 3f8b9eafe1641..608c1c2120c96 100644
> --- a/arch/arm64/boot/dts/qcom/pm660l.dtsi
> +++ b/arch/arm64/boot/dts/qcom/pm660l.dtsi
> @@ -87,6 +87,43 @@ pm660l_wled: leds@d800 {
>  			status = "disabled";
>  		};
>  
> +		pm660l_codec: audio-codec@f000 {
> +			compatible = "qcom,pm660l-wcd-analog-codec",
> +				     "qcom,pm8953-wcd-analog-codec";
> +			reg = <0xf000>;
> +			interrupts = <0x3 0xf0 0x0 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf0 0x1 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf0 0x2 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf0 0x3 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf0 0x4 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf0 0x5 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf0 0x6 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf0 0x7 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf1 0x0 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf1 0x1 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf1 0x2 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf1 0x3 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf1 0x4 IRQ_TYPE_EDGE_BOTH>,
> +				     <0x3 0xf1 0x5 IRQ_TYPE_EDGE_BOTH>;
> +			interrupt-names = "cdc_spk_cnp_int",
> +					  "cdc_spk_clip_int",
> +					  "cdc_spk_ocp_int",
> +					  "mbhc_ins_rem_det1",
> +					  "mbhc_but_rel_det",
> +					  "mbhc_but_press_det",
> +					  "mbhc_ins_rem_det",
> +					  "mbhc_switch_int",
> +					  "cdc_ear_ocp_int",
> +					  "cdc_hphr_ocp_int",
> +					  "cdc_hphl_ocp_det",
> +					  "cdc_ear_cnp_int",
> +					  "cdc_hphr_cnp_int",
> +					  "cdc_hphl_cnp_int";
> +			sound-name-prefix = "Analog";

[Severity: Low]
Does sound-name-prefix here cause a make dtbs_check failure for boards
including pm660l.dtsi?

Documentation/devicetree/bindings/sound/qcom,pm8916-wcd-analog-codec.yaml
sets additionalProperties: false without listing sound-name-prefix or
referencing dai-common.yaml#.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261107022453.159341-1-mailingradian@gmail.com?part=7