All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] arm64: dts: rockchip: Increase maximal SDIO voltage to 3.3 V
@ 2021-01-24 21:03 ` Uwe Kleine-König
  0 siblings, 0 replies; 6+ messages in thread
From: Uwe Kleine-König @ 2021-01-24 21:03 UTC (permalink / raw)
  To: Rob Herring, Heiko Stuebner; +Cc: devicetree, linux-rockchip, Aditya Prayoga

While comparing the vendor dts with the one added in v5.11-rc1 I noticed
that the vendor dts specified 3.3 V which matches the SDIO specification.

So fix the sdio regulator to provide (up to) 3.3 V.

Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
---
 arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts b/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
index 2a561be724b2..d6446c32b91e 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
@@ -251,11 +251,11 @@ vcc_sdio_s0: LDO_REG4 {
 				regulator-always-on;
 				regulator-boot-on;
 				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3000000>;
+				regulator-max-microvolt = <3300000>;
 
 				regulator-state-mem {
 					regulator-on-in-suspend;
-					regulator-suspend-microvolt = <3000000>;
+					regulator-suspend-microvolt = <3300000>;
 				};
 			};
 

base-commit: 19c329f6808995b142b3966301f217c831e7cf31
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 1/2] arm64: dts: rockchip: Increase maximal SDIO voltage to 3.3 V
@ 2021-01-24 21:03 ` Uwe Kleine-König
  0 siblings, 0 replies; 6+ messages in thread
From: Uwe Kleine-König @ 2021-01-24 21:03 UTC (permalink / raw)
  To: Rob Herring, Heiko Stuebner; +Cc: devicetree, Aditya Prayoga, linux-rockchip

While comparing the vendor dts with the one added in v5.11-rc1 I noticed
that the vendor dts specified 3.3 V which matches the SDIO specification.

So fix the sdio regulator to provide (up to) 3.3 V.

Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
---
 arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts b/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
index 2a561be724b2..d6446c32b91e 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
@@ -251,11 +251,11 @@ vcc_sdio_s0: LDO_REG4 {
 				regulator-always-on;
 				regulator-boot-on;
 				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3000000>;
+				regulator-max-microvolt = <3300000>;
 
 				regulator-state-mem {
 					regulator-on-in-suspend;
-					regulator-suspend-microvolt = <3000000>;
+					regulator-suspend-microvolt = <3300000>;
 				};
 			};
 

base-commit: 19c329f6808995b142b3966301f217c831e7cf31
-- 
2.29.2


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/2] arm64: dts: rockchip: Rely on SoC external pull up on pmic-int-l
  2021-01-24 21:03 ` Uwe Kleine-König
@ 2021-01-24 21:03   ` Uwe Kleine-König
  -1 siblings, 0 replies; 6+ messages in thread
From: Uwe Kleine-König @ 2021-01-24 21:03 UTC (permalink / raw)
  To: Rob Herring, Heiko Stuebner; +Cc: devicetree, linux-rockchip, Aditya Prayoga

According to the schematic there is an external pull up, so there is no
need to enable the internal one additionally. Using no pull up matches
the vendor device tree.

Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
---
 arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts b/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
index d6446c32b91e..1e880a0a3f69 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
@@ -331,7 +331,7 @@ sys_red_led_on: sys-red-led-on {
 
 	pmic {
 		pmic_int_l: pmic-int-l {
-			rockchip,pins = <0 RK_PB2 RK_FUNC_GPIO &pcfg_pull_up>;
+			rockchip,pins = <0 RK_PB2 RK_FUNC_GPIO &pcfg_pull_none>;
 		};
 	};
 
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/2] arm64: dts: rockchip: Rely on SoC external pull up on pmic-int-l
@ 2021-01-24 21:03   ` Uwe Kleine-König
  0 siblings, 0 replies; 6+ messages in thread
From: Uwe Kleine-König @ 2021-01-24 21:03 UTC (permalink / raw)
  To: Rob Herring, Heiko Stuebner; +Cc: devicetree, Aditya Prayoga, linux-rockchip

According to the schematic there is an external pull up, so there is no
need to enable the internal one additionally. Using no pull up matches
the vendor device tree.

Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
---
 arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts b/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
index d6446c32b91e..1e880a0a3f69 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts
@@ -331,7 +331,7 @@ sys_red_led_on: sys-red-led-on {
 
 	pmic {
 		pmic_int_l: pmic-int-l {
-			rockchip,pins = <0 RK_PB2 RK_FUNC_GPIO &pcfg_pull_up>;
+			rockchip,pins = <0 RK_PB2 RK_FUNC_GPIO &pcfg_pull_none>;
 		};
 	};
 
-- 
2.29.2


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] arm64: dts: rockchip: Increase maximal SDIO voltage to 3.3 V
  2021-01-24 21:03 ` Uwe Kleine-König
@ 2021-01-25 23:56   ` Heiko Stuebner
  -1 siblings, 0 replies; 6+ messages in thread
From: Heiko Stuebner @ 2021-01-25 23:56 UTC (permalink / raw)
  To: Uwe Kleine-König, Rob Herring
  Cc: Heiko Stuebner, Aditya Prayoga, linux-rockchip, devicetree

On Sun, 24 Jan 2021 22:03:27 +0100, Uwe Kleine-König wrote:
> While comparing the vendor dts with the one added in v5.11-rc1 I noticed
> that the vendor dts specified 3.3 V which matches the SDIO specification.
> 
> So fix the sdio regulator to provide (up to) 3.3 V.

Applied, thanks!

[1/2] arm64: dts: rockchip: Increase maximal SDIO voltage to 3.3 V
      commit: d134ac731feed91a81b447e0e63c1c35523fc574
[2/2] arm64: dts: rockchip: Rely on SoC external pull up on pmic-int-l
      commit: 19de5156d0029ccdf0153cee35fb8356b56ba9c6

Best regards,
-- 
Heiko Stuebner <heiko@sntech.de>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] arm64: dts: rockchip: Increase maximal SDIO voltage to 3.3 V
@ 2021-01-25 23:56   ` Heiko Stuebner
  0 siblings, 0 replies; 6+ messages in thread
From: Heiko Stuebner @ 2021-01-25 23:56 UTC (permalink / raw)
  To: Uwe Kleine-König, Rob Herring
  Cc: linux-rockchip, Aditya Prayoga, Heiko Stuebner, devicetree

On Sun, 24 Jan 2021 22:03:27 +0100, Uwe Kleine-König wrote:
> While comparing the vendor dts with the one added in v5.11-rc1 I noticed
> that the vendor dts specified 3.3 V which matches the SDIO specification.
> 
> So fix the sdio regulator to provide (up to) 3.3 V.

Applied, thanks!

[1/2] arm64: dts: rockchip: Increase maximal SDIO voltage to 3.3 V
      commit: d134ac731feed91a81b447e0e63c1c35523fc574
[2/2] arm64: dts: rockchip: Rely on SoC external pull up on pmic-int-l
      commit: 19de5156d0029ccdf0153cee35fb8356b56ba9c6

Best regards,
-- 
Heiko Stuebner <heiko@sntech.de>

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2021-01-26  0:08 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-24 21:03 [PATCH 1/2] arm64: dts: rockchip: Increase maximal SDIO voltage to 3.3 V Uwe Kleine-König
2021-01-24 21:03 ` Uwe Kleine-König
2021-01-24 21:03 ` [PATCH 2/2] arm64: dts: rockchip: Rely on SoC external pull up on pmic-int-l Uwe Kleine-König
2021-01-24 21:03   ` Uwe Kleine-König
2021-01-25 23:56 ` [PATCH 1/2] arm64: dts: rockchip: Increase maximal SDIO voltage to 3.3 V Heiko Stuebner
2021-01-25 23:56   ` Heiko Stuebner

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.