linux-kernel - Re: [PATCH v2 1/1] arm64: dts: qcom: x1e80100-pmics: Disable pm8010 by default

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: <CAMcHhXouMRKEu+1hK-XYeuHa9RCXsEOR=ho0vEVCXewTsWFrtw@mail.gmail.com>
Date: Thu, 17 Jul 2025 12:20:04 +0200
From: Aleksandrs Vinarskis <alex.vinarskis@...il.com>
To: Bjorn Andersson <andersson@...nel.org>, Konrad Dybcio <konrad.dybcio@....qualcomm.com>, 
	linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Cc: Konrad Dybcio <konradybcio@...nel.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, laurentiu.tudor1@...l.com, 
	abel.vesa@...aro.org, bryan.odonoghue@...aro.org, 
	jens.glathe@...schoolsolutions.biz, stable@...r.kernel.org, 
	Johan Hovold <johan+linaro@...nel.org>
Subject: Re: [PATCH v2 1/1] arm64: dts: qcom: x1e80100-pmics: Disable pm8010
 by default

On Tue, 1 Jul 2025 at 20:36, Aleksandrs Vinarskis
<alex.vinarskis@...il.com> wrote:
>
> pm8010 is a camera specific PMIC, and may not be present on some
> devices. These may instead use a dedicated vreg for this purpose (Dell
> XPS 9345, Dell Inspiron..) or use USB webcam instead of a MIPI one
> alltogether (Lenovo Thinbook 16, Lenovo Yoga..).
>
> Disable pm8010 by default, let platforms that actually have one onboard
> enable it instead.
>
> Cc: <stable@...r.kernel.org>
> Fixes: 2559e61e7ef4 ("arm64: dts: qcom: x1e80100-pmics: Add the missing PMICs")
> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
> Reviewed-by: Johan Hovold <johan+linaro@...nel.org>
> Reviewed-by: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
> Signed-off-by: Aleksandrs Vinarskis <alex.vinarskis@...il.com>
> ---

Hi,

I've noticed this was reviewed some time ago, but was not included in
for upcoming 6.17. Perhaps it was forgotten?

Thanks in advance,
Alex

>  arch/arm64/boot/dts/qcom/x1e80100-pmics.dtsi | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/x1e80100-pmics.dtsi b/arch/arm64/boot/dts/qcom/x1e80100-pmics.dtsi
> index e3888bc143a0..621890ada153 100644
> --- a/arch/arm64/boot/dts/qcom/x1e80100-pmics.dtsi
> +++ b/arch/arm64/boot/dts/qcom/x1e80100-pmics.dtsi
> @@ -475,6 +475,8 @@ pm8010: pmic@c {
>                 #address-cells = <1>;
>                 #size-cells = <0>;
>
> +               status = "disabled";
> +
>                 pm8010_temp_alarm: temp-alarm@...0 {
>                         compatible = "qcom,spmi-temp-alarm";
>                         reg = <0x2400>;
> --
> 2.48.1
>