Re: [PATCH 5/5] arm64: dts: qcom: use generic node name for CS35L41 speaker

[Konrad Dybcio]

On 24.12.2022 16:42, Krzysztof Kozlowski wrote:
> Node names should be generic so use consistently speaker-amp for CS35L41
> speaker amplifier.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>

Konrad
>  arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi    | 4 ++--
>  arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami.dtsi | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi b/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi
> index 09a31f707639..25c3e02f224b 100644
> --- a/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi
> @@ -454,7 +454,7 @@ &i2c2 {
>  	status = "okay";
>  	clock-frequency = <1000000>;
>  
> -	cs35l41_l: cs35l41@40 {
> +	cs35l41_l: speaker-amp@40 {
>  		compatible = "cirrus,cs35l41";
>  		reg = <0x40>;
>  		interrupt-parent = <&tlmm>;
> @@ -469,7 +469,7 @@ cs35l41_l: cs35l41@40 {
>  		#sound-dai-cells = <1>;
>  	};
>  
> -	cs35l41_r: cs35l41@41 {
> +	cs35l41_r: speaker-amp@41 {
>  		compatible = "cirrus,cs35l41";
>  		reg = <0x41>;
>  		interrupt-parent = <&tlmm>;
> diff --git a/arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami.dtsi b/arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami.dtsi
> index 1f2d660f8f86..4862fd69413e 100644
> --- a/arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8350-sony-xperia-sagami.dtsi
> @@ -461,7 +461,7 @@ &i2c11 {
>  	status = "okay";
>  	clock-frequency = <1000000>;
>  
> -	cs35l41_l: cs35l41@40 {
> +	cs35l41_l: speaker-amp@40 {
>  		compatible = "cirrus,cs35l41";
>  		reg = <0x40>;
>  		interrupt-parent = <&tlmm>;
> @@ -476,7 +476,7 @@ cs35l41_l: cs35l41@40 {
>  		#sound-dai-cells = <1>;
>  	};
>  
> -	cs35l41_r: cs35l41@41 {
> +	cs35l41_r: speaker-amp@41 {
>  		compatible = "cirrus,cs35l41";
>  		reg = <0x41>;
>  		interrupt-parent = <&tlmm>;