[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <DA72DKCKVX7T.269HYJZNIABOB@linaro.org>
Date: Tue, 27 May 2025 16:55:59 +0100
From: "Alexey Klimov" <alexey.klimov@...aro.org>
To: "Konrad Dybcio" <konrad.dybcio@....qualcomm.com>, "Srinivas Kandagatla"
<srini@...nel.org>, "Mark Brown" <broonie@...nel.org>,
<linux-sound@...r.kernel.org>
Cc: "Liam Girdwood" <lgirdwood@...il.com>, "Rob Herring" <robh@...nel.org>,
"Krzysztof Kozlowski" <krzk+dt@...nel.org>, "Krzysztof Kozlowski"
<krzysztof.kozlowski@...aro.org>, "Conor Dooley" <conor+dt@...nel.org>,
"Bjorn Andersson" <andersson@...nel.org>, "Dmitry Baryshkov"
<lumag@...nel.org>, "Konrad Dybcio" <konradybcio@...nel.org>, "Jaroslav
Kysela" <perex@...ex.cz>, "Takashi Iwai" <tiwai@...e.com>,
<linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <linux-gpio@...r.kernel.org>
Subject: Re: [PATCH v3 10/12] arm64: dts: qcom: qrb4210-rb2: enable wsa881x
amplifier
On Thu May 22, 2025 at 7:13 PM BST, Konrad Dybcio wrote:
> On 5/22/25 7:41 PM, Alexey Klimov wrote:
>> One WSA881X amplifier is connected on QRB4210 RB2 board
>> hence only mono speaker is supported. This amplifier is set
>> to work in analog mode only. Also add required powerdown
>> pin/gpio.
>>
>> Cc: Srinivas Kandagatla <srinivas.kandagatla@...aro.org>
>> Signed-off-by: Alexey Klimov <alexey.klimov@...aro.org>
>> ---
>> arch/arm64/boot/dts/qcom/qrb4210-rb2.dts | 26 ++++++++++++++++++++++++++
>> 1 file changed, 26 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qrb4210-rb2.dts b/arch/arm64/boot/dts/qcom/qrb4210-rb2.dts
>> index 6bce63720cfffd8e0e619937fb1f365cbbbcb283..4b878e585227ee6b3b362108be96aad99acba21d 100644
>> --- a/arch/arm64/boot/dts/qcom/qrb4210-rb2.dts
>> +++ b/arch/arm64/boot/dts/qcom/qrb4210-rb2.dts
>> @@ -270,6 +270,24 @@ zap-shader {
>> };
>> };
>>
>> +&i2c1 {
>> + clock-frequency = <400000>;
>> + status = "okay";
>> +
>> + wsa881x: amplifier@f {
>> + compatible = "qcom,wsa8815";
>> + reg = <0x0f>;
>> + pinctrl-0 = <&wsa_en_active>;
>> + pinctrl-names = "default";
>> + clocks = <&q6afecc LPASS_CLK_ID_MCLK_2 LPASS_CLK_ATTRIBUTE_COUPLE_NO>;
>> + powerdown-gpios = <&lpass_tlmm 16 GPIO_ACTIVE_LOW>;
>> + mclk-gpios = <&lpass_tlmm 18 GPIO_ACTIVE_HIGH>;
>> + sound-name-prefix = "SpkrMono";
>> + #sound-dai-cells = <0>;
>> + #thermal-sensor-cells = <0>;
>> + };
>> +};
>> +
>> &i2c2_gpio {
>> clock-frequency = <400000>;
>> status = "okay";
>> @@ -736,6 +754,14 @@ wcd_reset_n: wcd-reset-n-state {
>> drive-strength = <16>;
>> output-high;
>> };
>> +
>> + wsa_en_active: wsa-en-active-state {
>> + pins = "gpio106";
>
> Are there two separate enable pins? Or is the powerdown-gpio something
> else?
No, should be only one. I think 106 on tlmm is wired into 16 on lpass tlmm.
We need to assign gpio function to such pins, aren't we?
Best regards,
Alexey
Powered by blists - more mailing lists