linux-kernel - RE: [PATCH v2] arm64: dts: qcom: sc7280: setting the vddio supply for Bluetooth node on CRD

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: <SN4PR0201MB8725606835F55D90A4F50FDAE3309@SN4PR0201MB8725.namprd02.prod.outlook.com>
Date:   Fri, 11 Feb 2022 13:49:02 +0000
From:   "Sai Teja Aluvala (Temp) (QUIC)" <quic_saluvala@...cinc.com>
To:     "bjorn.andersson@...aro.org" <bjorn.andersson@...aro.org>,
        "Sai Teja Aluvala (Temp) (QUIC)" <quic_saluvala@...cinc.com>
CC:     "marcel@...tmann.org" <marcel@...tmann.org>,
        "johan.hedberg@...il.com" <johan.hedberg@...il.com>,
        "mka@...omium.org" <mka@...omium.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-bluetooth@...r.kernel.org" <linux-bluetooth@...r.kernel.org>,
        "Hemant Gupta (QUIC)" <quic_hemantg@...cinc.com>,
        "linux-arm-msm@...r.kernel.org" <linux-arm-msm@...r.kernel.org>,
        quic_bgodavar <quic_bgodavar@...cinc.com>,
        quic_rjliao <quic_rjliao@...cinc.com>,
        quic_hbandi <quic_hbandi@...cinc.com>,
        "abhishekpandit@...omium.org" <abhishekpandit@...omium.org>,
        "mcchou@...omium.org" <mcchou@...omium.org>
Subject: RE: [PATCH v2] arm64: dts: qcom: sc7280: setting the vddio supply for
 Bluetooth node on CRD



-----Original Message-----
From: Bjorn Andersson <bjorn.andersson@...aro.org> 
Sent: Friday, February 11, 2022 6:09 AM
To: Sai Teja Aluvala (Temp) (QUIC) <quic_saluvala@...cinc.com>
Cc: marcel@...tmann.org; johan.hedberg@...il.com; mka@...omium.org; linux-kernel@...r.kernel.org; linux-bluetooth@...r.kernel.org; Hemant Gupta (QUIC) <quic_hemantg@...cinc.com>; linux-arm-msm@...r.kernel.org; quic_bgodavar <quic_bgodavar@...cinc.com>; quic_rjliao <quic_rjliao@...cinc.com>; quic_hbandi <quic_hbandi@...cinc.com>; abhishekpandit@...omium.org; mcchou@...omium.org
Subject: Re: [PATCH v2] arm64: dts: qcom: sc7280: setting the vddio supply for Bluetooth node on CRD

On Tue 08 Feb 05:13 CST 2022, Sai Teja Aluvala wrote:

> setting the vddio supply for Bluetooth SoC WCN6750 node on SC7280 crd 
> board
> 

While this does better describe what the change does, as Matthias requested on v1, it fails to provide any information about why this done.

Is vddio simply not powered on the IDP? Or did someone forget to add it to the idp.dtsi?
[Sai]: Vddio regulator handle is different for IDP and IDP2. So in common dtsi file i.e. sc7280-idp.dtsi we haven't enabled vddio instead we have add the regulator handle in board specific dts file.
So for CRD too we are adding it in the sc7280-crd.dts file.


Regards,
Bjorn

> Signed-off-by: Sai Teja Aluvala <quic_saluvala@...cinc.com>
> ---
>  arch/arm64/boot/dts/qcom/sc7280-crd.dts | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-crd.dts 
> b/arch/arm64/boot/dts/qcom/sc7280-crd.dts
> index cd2755c..53ea3b4 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-crd.dts
> +++ b/arch/arm64/boot/dts/qcom/sc7280-crd.dts
> @@ -23,6 +23,10 @@
>  	};
>  };
>  
> +&bluetooth {
> +	vddio-supply = <&vreg_l18b_1p8>;
> +};
> +
>  ap_tp_i2c: &i2c0 {
>  	status = "okay";
>  	clock-frequency = <400000>;
> --
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc.
>