From nobody Sat Feb 15 09:09:01 2025 Received: from mail-ed1-f68.google.com (mail-ed1-f68.google.com [209.85.208.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 137F2230993 for ; Tue, 10 Dec 2024 08:36:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.68 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733819775; cv=none; b=hyw6uMWmKKNBShE0bFrnVcb+qYOY2RJ+Lg7S2c6MjQlA5zvcyISPgE/JnZPxlv9XcatJvxxRJgJqqLL4I9IJaNUA7fShHCP6STO6f9S6Z6IuTL7UTnUz0rQLTuJJl0d/Fa7NrB3xXVKazAPxcx7fNnR7B2hyvOUzWLN39S7ljd0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733819775; c=relaxed/simple; bh=uow1N9VBwkTBx4xQA1Z6vJux5tMwXu6x8kqawBjy95c=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=rh5W2YUFLpwxQQlbEv/hUnQVBkLF1EbXsKfN0RPPSWVTG9MaEmCMOfvuEFs4NnXPoY52DUkvpNiumVi3NRyh1qM809N358OYgONCJ1JpX4gTGavxqqZtxQDHImQi3392ms3OXIEDFghpDJYzksHyXbh6KC23o0Qb7+sbKEDG7YY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=Ipsr5u3v; arc=none smtp.client-ip=209.85.208.68 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="Ipsr5u3v" Received: by mail-ed1-f68.google.com with SMTP id 4fb4d7f45d1cf-5cfa1ec3b94so6895293a12.2 for ; Tue, 10 Dec 2024 00:36:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1733819771; x=1734424571; darn=vger.kernel.org; h=cc:to:message-id:content-transfer-encoding:mime-version:subject :date:from:from:to:cc:subject:date:message-id:reply-to; bh=58qniorQWaWIqV9FaGTtR2lysYb+BUrgZIhD2vCySDU=; b=Ipsr5u3vdMSA/9OewO5sB482khT0ViYj8uN3rdBAs60buAf/jfMYnqjP6GJu8ur/jQ CSjXNndMVw3PUrWwrulaj1kCZi/efHN15phhQrq45238BAFK8dpNvLwnkiBTdaBFssb/ QpIYEYXncPM4BD+9bV+k1A6zdDuvJjmmaHBWvPIrAs6lN54cK0Spj4wNUmajlzrgEzy4 ZHAPP6lAVJk/zqLNQ+yEBzneonMoKrYvIRtFWTKpWpEoJPKdA9b3H27mwoFdIPnAw9AM rHaP7GftWWnymqgVvINHqw4zmpJjrfqDbFKcMbl/66+SbiaWL14q/GiOJrd12NzMO/RW Cfgg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1733819771; x=1734424571; h=cc:to:message-id:content-transfer-encoding:mime-version:subject :date:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=58qniorQWaWIqV9FaGTtR2lysYb+BUrgZIhD2vCySDU=; b=xVe5mPgk/9Pe21kG2u3hKWfyjZ2/vF+x/r7Wy3BCl04uJnoUFf6AIraJ8K0geFVN7m YTLzO7Qwyvq9NvB6kZKaNk08WSUXX/J5tbVp5LXpnRxjvxQyTyRinoWjt8YEMR+As8yg ZTOEGYGfEXoR3efzXMcWcOM77BFIl/oqeUNZzSQMmLGkxVb5CqYGcuZq1dCZuHUJGVoF oHNy4/o+9DoSNYuvbF+IR4qfn3GxXHKEP8YWp9X8QyiVquaoF7D97AKFM/FdjrN/eFlM 3Pqcb9RIYSu1fBIpV6DT1t4SaQBMQg+94sB14WhvLRj53kzZsbFiAmesyoxa+ETbEXFC HrEg== X-Forwarded-Encrypted: i=1; AJvYcCUg63ilt7B4e8UKroe6zcQ1pA+zucJLHSgNV3J97qZ2gxXoJADy+xexomP4H0nny9jsI2ZHsd577rdF3QM=@vger.kernel.org X-Gm-Message-State: AOJu0YztXZ/HAR6MqKqc2Srmg7S+7x9A5wq6vUGSoBV3Zq5F6Mqgag3a Xs1OrfUP9+Jpk+U8HDVuxpG8Hbh2rmceKFajYZv/uzoYuF6EwTcO6UnzT1m95jY= X-Gm-Gg: ASbGncuURJIdvZs5RL+9Jsb+lEwyiZwhqJJDqbLron9SnaKXWtjE5Ww/KTqm2pZYhoc OQzzmdWGnuhwzmTe/RfZ7Kt/dx7evldtZJxMDCpivlsrLDzb9aA+W0yNIGkUA8xrI69vDHQfKCU Q3x4JpLw8m07AauJss6nGDoozuAvEE3h5Oa+85/DXTB2NZ8GdXoHAbJSiEp2BDI/oK8Qudhaesd m0OBIHcNboXoXnU1Xug3BEN7DgA+KojcilRbXp1UJecPhI0FUr8k2/kuPpKzuHeF0I= X-Google-Smtp-Source: AGHT+IGQPNbWbY7Am19yEaRzmYLXWQmVwscaAezJ1evdHhKgXkg1An9ckZEBx5n9PUbBDaZlE+eghw== X-Received: by 2002:a17:907:774e:b0:aa6:79fa:b486 with SMTP id a640c23a62f3a-aa679fabfb7mr667984566b.49.1733819771260; Tue, 10 Dec 2024 00:36:11 -0800 (PST) Received: from [127.0.0.2] ([2a02:2454:ff21:ef40:6709:8d26:5167:3c1d]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-aa66c30e99csm438827866b.160.2024.12.10.00.36.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Dec 2024 00:36:10 -0800 (PST) From: Stephan Gerhold Date: Tue, 10 Dec 2024 09:36:01 +0100 Subject: [PATCH v2] arm64: dts: qcom: x1e80100-pmics: Enable all SMB2360 separately Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20241210-x1e80100-disable-smb2360-v2-1-2449be2eca29@linaro.org> X-B4-Tracking: v=1; b=H4sIAHD9V2cC/3WNSw6CQBBEr0J67ZjuBvm48h6GxQANdIKMmTEEQ 7i7I65dvkrVqw2CeJUA12QDL4sGdXMEPiXQjnYexGgXGRg5I+LKrCQlEqLpNNhmEhMeDac5GsZ WWsoveexBnD+99Loe6nsdedTwcv59PC30TX9SxvS/dCFDpsQq46boi7Szt0ln693Z+QHqfd8/v qH3pcEAAAA= X-Change-ID: 20241129-x1e80100-disable-smb2360-20cec1656411 To: Bjorn Andersson , Konrad Dybcio Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Abel Vesa , Johan Hovold , "Aiqun Yu (Maria)" X-Mailer: b4 0.14.2 At the moment, x1e80100-pmics.dtsi enables two of the SMB2360 PMICs by default and leaves the other two disabled. The third one was originally also enabled by default, but then disabled in commit a237b8da413c ("arm64: dts: qcom: x1e80100: Disable SMB2360_2 by default"). This is inconsistent and confusing. Some laptops will even need SMB2360_1 disabled by default if they just have a single USB-C port. Make this consistent by keeping all SMB2360 disabled in x1e80100-pmics.dtsi and enable them separately for all boards where needed. That way it is always clear which ones are available and avoids accidentally trying to read/write from missing chips when some of the PMICs are not present. Signed-off-by: Stephan Gerhold Reviewed-by: Abel Vesa Reviewed-by: Johan Hovold --- Changes in v2: - Clarify commit message, there are actually two SMB2360 disabled by default (3rd and 4th) and not just the third (Aiqun Yu (Maria)) - Link to v1: https://lore.kernel.org/r/20241203-x1e80100-disable-smb2360-v= 1-1-80942b7f73da@linaro.org --- arch/arm64/boot/dts/qcom/x1e001de-devkit.dts | 8 ++++++++ arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dts | 8 ++++++++ arch/arm64/boot/dts/qcom/x1e80100-asus-vivobook-s15.dts | 8 ++++++++ arch/arm64/boot/dts/qcom/x1e80100-crd.dts | 8 ++++++++ arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts | 8 ++++++++ arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts | 8 ++++++++ arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi | 8 ++++++++ arch/arm64/boot/dts/qcom/x1e80100-pmics.dtsi | 4 ++++ arch/arm64/boot/dts/qcom/x1e80100-qcp.dts | 8 ++++++++ 9 files changed, 68 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts b/arch/arm64/boot= /dts/qcom/x1e001de-devkit.dts index c9db6298d528ed505bae08e91bf4da02faef7d76..c3ec0bb2c42dfed9f5e9f56f188= 74b4ae6c0632c 100644 --- a/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts +++ b/arch/arm64/boot/dts/qcom/x1e001de-devkit.dts @@ -1062,11 +1062,19 @@ &remoteproc_cdsp { status =3D "okay"; }; =20 +&smb2360_0 { + status =3D "okay"; +}; + &smb2360_0_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l2b_3p0>; }; =20 +&smb2360_1 { + status =3D "okay"; +}; + &smb2360_1_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l14b_3p0>; diff --git a/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dts b/a= rch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dts index 975550139e1024420ed335a2a46e4d54df7ee423..4097d2677285757ce67eef76d8d= 17669c7130001 100644 --- a/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dts +++ b/arch/arm64/boot/dts/qcom/x1e78100-lenovo-thinkpad-t14s.dts @@ -635,11 +635,19 @@ &remoteproc_cdsp { status =3D "okay"; }; =20 +&smb2360_0 { + status =3D "okay"; +}; + &smb2360_0_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l2b_3p0>; }; =20 +&smb2360_1 { + status =3D "okay"; +}; + &smb2360_1_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l14b_3p0>; diff --git a/arch/arm64/boot/dts/qcom/x1e80100-asus-vivobook-s15.dts b/arch= /arm64/boot/dts/qcom/x1e80100-asus-vivobook-s15.dts index f25991b887de3fca0092c5f81c881c5d8bd71aac..10f140ed08f47f2cd96e953d4ad= 3c75f3aad002a 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-asus-vivobook-s15.dts +++ b/arch/arm64/boot/dts/qcom/x1e80100-asus-vivobook-s15.dts @@ -501,11 +501,19 @@ &remoteproc_cdsp { status =3D "okay"; }; =20 +&smb2360_0 { + status =3D "okay"; +}; + &smb2360_0_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l2b_3p0>; }; =20 +&smb2360_1 { + status =3D "okay"; +}; + &smb2360_1_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l14b_3p0>; diff --git a/arch/arm64/boot/dts/qcom/x1e80100-crd.dts b/arch/arm64/boot/dt= s/qcom/x1e80100-crd.dts index 39f9d9cdc10d8e79824b72288e2529536144fa9e..81c519e690f325f2d4d36c4a990= 61649da65ec87 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-crd.dts +++ b/arch/arm64/boot/dts/qcom/x1e80100-crd.dts @@ -932,11 +932,19 @@ &remoteproc_cdsp { status =3D "okay"; }; =20 +&smb2360_0 { + status =3D "okay"; +}; + &smb2360_0_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l2b_3p0>; }; =20 +&smb2360_1 { + status =3D "okay"; +}; + &smb2360_1_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l14b_3p0>; diff --git a/arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts b/arch/a= rm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts index b112092fbb9fd955adca1ae8a76294c776fa2d1e..288e818961670be0d25b696730e= 7186a8d61b56e 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts +++ b/arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts @@ -905,11 +905,19 @@ &remoteproc_cdsp { status =3D "okay"; }; =20 +&smb2360_0 { + status =3D "okay"; +}; + &smb2360_0_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l2b_3p0>; }; =20 +&smb2360_1 { + status =3D "okay"; +}; + &smb2360_1_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l14b_3p0>; diff --git a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts b/arc= h/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts index ca5a808f2c7df66a861a933df407fd4bdaea3fe1..3d7e0230dc0381a2c2a3b0c5b76= 6f5b854777937 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts +++ b/arch/arm64/boot/dts/qcom/x1e80100-lenovo-yoga-slim7x.dts @@ -717,11 +717,19 @@ &remoteproc_cdsp { status =3D "okay"; }; =20 +&smb2360_0 { + status =3D "okay"; +}; + &smb2360_0_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l2b_3p0>; }; =20 +&smb2360_1 { + status =3D "okay"; +}; + &smb2360_1_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l14b_3p0>; diff --git a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi b/arc= h/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi index 2236095023a135d8fb1baaede111a34be54d160c..af8459d38f210fb61536814577e= 468e6dfc4bb22 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi +++ b/arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi @@ -1112,11 +1112,19 @@ &remoteproc_cdsp { status =3D "okay"; }; =20 +&smb2360_0 { + status =3D "okay"; +}; + &smb2360_0_eusb2_repeater { vdd18-supply =3D <&vreg_l3d>; vdd3-supply =3D <&vreg_l2b>; }; =20 +&smb2360_1 { + status =3D "okay"; +}; + &smb2360_1_eusb2_repeater { vdd18-supply =3D <&vreg_l3d>; vdd3-supply =3D <&vreg_l14b>; diff --git a/arch/arm64/boot/dts/qcom/x1e80100-pmics.dtsi b/arch/arm64/boot= /dts/qcom/x1e80100-pmics.dtsi index 5b54ee79f048e3208cbcd6f91e0cec073420fe63..d7a2a2b8fc6c30bdb10df81eac7= d92306998838f 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-pmics.dtsi +++ b/arch/arm64/boot/dts/qcom/x1e80100-pmics.dtsi @@ -491,6 +491,8 @@ smb2360_0: pmic@7 { #address-cells =3D <1>; #size-cells =3D <0>; =20 + status =3D "disabled"; + smb2360_0_eusb2_repeater: phy@fd00 { compatible =3D "qcom,smb2360-eusb2-repeater"; reg =3D <0xfd00>; @@ -504,6 +506,8 @@ smb2360_1: pmic@a { #address-cells =3D <1>; #size-cells =3D <0>; =20 + status =3D "disabled"; + smb2360_1_eusb2_repeater: phy@fd00 { compatible =3D "qcom,smb2360-eusb2-repeater"; reg =3D <0xfd00>; diff --git a/arch/arm64/boot/dts/qcom/x1e80100-qcp.dts b/arch/arm64/boot/dt= s/qcom/x1e80100-qcp.dts index 5ef030c60abe2998d093ee60a6754a90cd5aaf72..ffd28fd8059895ec345f4ee8fe6= a2c37e7989747 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100-qcp.dts +++ b/arch/arm64/boot/dts/qcom/x1e80100-qcp.dts @@ -731,11 +731,19 @@ &remoteproc_cdsp { status =3D "okay"; }; =20 +&smb2360_0 { + status =3D "okay"; +}; + &smb2360_0_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l2b_3p0>; }; =20 +&smb2360_1 { + status =3D "okay"; +}; + &smb2360_1_eusb2_repeater { vdd18-supply =3D <&vreg_l3d_1p8>; vdd3-supply =3D <&vreg_l14b_3p0>; --- base-commit: f2b086fc9f039773445d2606dc65dc091ec1830f change-id: 20241129-x1e80100-disable-smb2360-20cec1656411 Best regards, --=20 Stephan Gerhold