diff mbox series

arm64: dts: rockchip: fix rk3399-puma vcc5v0-host gpio

Message ID 20200604091239.424318-1-heiko@sntech.de (mailing list archive)
State Mainlined
Commit 7a7184f6cfa9279f1a1c10a1845d247d7fad54ff
Headers show
Series arm64: dts: rockchip: fix rk3399-puma vcc5v0-host gpio | expand

Commit Message

Heiko Stübner June 4, 2020, 9:12 a.m. UTC 
From: Heiko Stuebner <heiko.stuebner@theobroma-systems.com>

The puma vcc5v0_host regulator node currently uses opposite active-values
for the enable pin. The gpio-declaration uses active-high while the
separate enable-active-low property marks the pin as active low.

While on the kernel side this works ok, other DT users may get
confused - as seen with uboot right now.

So bring this in line and make both properties match, similar to the
gmac fix.

Fixes: 2c66fc34e945 ("arm64: dts: rockchip: add RK3399-Q7 (Puma) SoM")
Signed-off-by: Heiko Stuebner <heiko.stuebner@theobroma-systems.com>
---
 arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Heiko Stübner June 17, 2020, 8:37 a.m. UTC | #1 
On Thu, 4 Jun 2020 11:12:39 +0200, Heiko Stuebner wrote:
> The puma vcc5v0_host regulator node currently uses opposite active-values
> for the enable pin. The gpio-declaration uses active-high while the
> separate enable-active-low property marks the pin as active low.
> 
> While on the kernel side this works ok, other DT users may get
> confused - as seen with uboot right now.
> 
> [...]

Applied, thanks!

[1/1] arm64: dts: rockchip: fix rk3399-puma vcc5v0-host gpio
      commit: 7a7184f6cfa9279f1a1c10a1845d247d7fad54ff

Best regards,
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi
index 531520e771e7..72c06abd27ea 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi
@@ -101,7 +101,7 @@  vcc3v3_sys: vcc3v3-sys {
 
 	vcc5v0_host: vcc5v0-host-regulator {
 		compatible = "regulator-fixed";
-		gpio = <&gpio4 RK_PA3 GPIO_ACTIVE_HIGH>;
+		gpio = <&gpio4 RK_PA3 GPIO_ACTIVE_LOW>;
 		enable-active-low;
 		pinctrl-names = "default";
 		pinctrl-0 = <&vcc5v0_host_en>;