[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <53d44689-798e-4b5f-a0f1-8a39bea2f19b@quicinc.com>
Date: Fri, 6 Dec 2024 20:15:35 +0530
From: Janaki Ramaiah Thota <quic_janathot@...cinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
CC: Liam Girdwood <lgirdwood@...il.com>, Mark Brown <broonie@...nel.org>,
"Rob
Herring" <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
"Conor
Dooley" <conor+dt@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
"Konrad Dybcio" <konradybcio@...nel.org>,
Marcel Holtmann
<marcel@...tmann.org>,
"Luiz Augusto von Dentz" <luiz.dentz@...il.com>,
Bartosz Golaszewski <brgl@...ev.pl>, <quic_mohamull@...cinc.com>,
<quic_hbandi@...cinc.com>, <quic_anubhavg@...cinc.com>,
Bartosz Golaszewski
<bartosz.golaszewski@...aro.org>,
<linux-arm-msm@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<devicetree@...r.kernel.org>, <linux-bluetooth@...r.kernel.org>,
<linux-pm@...r.kernel.org>
Subject: Re: [PATCH v4 2/4] arm64: dts: qcom: qcs6490-rb3gen: add and enable
BT node
On 12/5/2024 4:29 AM, Dmitry Baryshkov wrote:
> On Wed, Dec 04, 2024 at 06:47:04PM +0530, Janaki Ramaiah Thota wrote:
>> Add a node for the PMU module of the WCN6750 present on the
>> qcs6490-rb3gen board and assign its power outputs to the Bluetooth
>> module.
>>
>> Signed-off-by: Janaki Ramaiah Thota <quic_janathot@...cinc.com>
>> ---
>> arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts | 165 ++++++++++++++++++-
>> 1 file changed, 164 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
>> index 27695bd54220..07650648214e 100644
>> --- a/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
>> +++ b/arch/arm64/boot/dts/qcom/qcs6490-rb3gen2.dts
>> @@ -1,6 +1,6 @@
>> // SPDX-License-Identifier: BSD-3-Clause
>> /*
>> - * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved.
>> + * Copyright (c) 2023-2024 Qualcomm Innovation Center, Inc. All rights reserved.
>> */
>>
>> /dts-v1/;
>> @@ -33,6 +33,7 @@
>>
>> aliases {
>> serial0 = &uart5;
>> + serial1 = &uart7;
>> };
>>
>> chosen {
>> @@ -217,6 +218,63 @@
>> regulator-min-microvolt = <3700000>;
>> regulator-max-microvolt = <3700000>;
>> };
>> +
>> + wcn6750-pmu {
>> + compatible = "qcom,wcn6750-pmu";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&bt_en>;
>> + vddaon-supply = <&vreg_s7b_0p972>;
>> + vddasd-supply = <&vreg_l11c_2p8>;
>> + vddpmu-supply = <&vreg_s7b_0p972>;
>> + vddrfa0p8-supply = <&vreg_s7b_0p972>;
>> + vddrfa1p2-supply = <&vreg_s8b_1p272>;
>> + vddrfa1p7-supply = <&vreg_s1b_1p872>;
>> + vddrfa2p2-supply = <&vreg_s1c_2p19>;
>> +
>> + bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
>
> Doesn't WCN6750 also have SW_CTRL and wifi-enable pins?
>
For Bluetooth, these pins are not needed. We have verified Bluetooth
functionality, and it is working fine.
Thanks,
Janakiram
Powered by blists - more mailing lists