[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <99a2f644-072f-4a81-9f22-73677637edff@oss.qualcomm.com>
Date: Mon, 8 Dec 2025 17:24:47 +0100
From: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
To: Krishna Kurapati <krishna.kurapati@....qualcomm.com>,
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>
Cc: linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: qcom: hamoa-iot-evk: Add vbus regulator
support for Type-A ports
On 12/8/25 9:51 AM, Krishna Kurapati wrote:
> The Multiport controller on Hamoa EVK platform is connected to Two Type-A
> ports. VBUS for each of these ports are provided by a TPS2559QWDRCTQ1
> regulator, controlled from PMIC GPIOs.
>
> Add the necessary regulators and GPIO configuration to power these.
>
> Signed-off-by: Krishna Kurapati <krishna.kurapati@....qualcomm.com>
> ---
> arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts | 37 ++++++++++++++++++++++
> 1 file changed, 37 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts b/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts
> index 36dd6599402b..665983ed6714 100644
> --- a/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts
> +++ b/arch/arm64/boot/dts/qcom/hamoa-iot-evk.dts
> @@ -6,6 +6,7 @@
> /dts-v1/;
>
> #include "hamoa-iot-som.dtsi"
> +#include <dt-bindings/pinctrl/qcom,pmic-gpio.h>
>
> / {
> model = "Qualcomm Technologies, Inc. Hamoa IoT EVK";
> @@ -167,6 +168,26 @@ pmic_glink_ss2_con_sbu_in: endpoint {
> };
> };
>
> + regulator-usb3-vbus {
> + compatible = "regulator-fixed";
> + regulator-name = "USB3_VBUS";
> + gpio = <&pm8550ve_9_gpios 4 GPIO_ACTIVE_HIGH>;
> + pinctrl-0 = <&usb3_en>;
> + pinctrl-names = "default";
> + enable-active-high;
> + regulator-always-on;
> + };
> +
> + regulator-usb6-vbus {
> + compatible = "regulator-fixed";
> + regulator-name = "USB6_VBUS";
> + gpio = <&pm8550ve_9_gpios 5 GPIO_ACTIVE_HIGH>;
> + pinctrl-0 = <&usb6_en>;
> + pinctrl-names = "default";
> + enable-active-high;
> + regulator-always-on;
> + };
> +
> vreg_edp_3p3: regulator-edp-3p3 {
I just noticed - 'regulator-e'dp-3p3 < 'regulator-u'sbX-vbus..
Could you please resend and retain my rb?
Konrad
Powered by blists - more mailing lists