All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: rockchip: correct vqmmc voltage for rk3399 platforms
@ 2017-10-24  9:15 Klaus Goger
  2017-10-24  9:33 ` Heiko Stuebner
  0 siblings, 1 reply; 5+ messages in thread
From: Klaus Goger @ 2017-10-24  9:15 UTC (permalink / raw)
  To: stable; +Cc: Shawn Lin, Heiko Stuebner, Klaus Goger

From: Shawn Lin <shawn.lin@rock-chips.com>

The vcc_sd or vcc_sdio used for IO voltage for sdmmc and sdio
interface on rk3399 platform have a limitation that it can't be
larger than 3.0v, otherwise it has a potential risk for the chip.
Correct all of them.

Fixes: 171582e00db1 ("arm64: dts: rockchip: add support for firefly-rk3399 board")
Cc: stable@vger.kernel.org
Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Signed-off-by: Klaus Goger <klaus.goger@theobroma-systems.com>
---

 arch/arm64/boot/dts/rockchip/rk3399-firefly.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
index ba1d9810ad1e..c27242a7d5e7 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
@@ -370,10 +370,10 @@
 				regulator-always-on;
 				regulator-boot-on;
 				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
+				regulator-max-microvolt = <3000000>;
 				regulator-state-mem {
 					regulator-on-in-suspend;
-					regulator-suspend-microvolt = <3300000>;
+					regulator-suspend-microvolt = <3000000>;
 				};
 			};
 
-- 
2.11.0

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

* Re: [PATCH] arm64: dts: rockchip: correct vqmmc voltage for rk3399 platforms
  2017-10-24  9:15 [PATCH] arm64: dts: rockchip: correct vqmmc voltage for rk3399 platforms Klaus Goger
@ 2017-10-24  9:33 ` Heiko Stuebner
  2017-10-24  9:56   ` Greg KH
  0 siblings, 1 reply; 5+ messages in thread
From: Heiko Stuebner @ 2017-10-24  9:33 UTC (permalink / raw)
  To: Klaus Goger; +Cc: stable, Shawn Lin

Am Dienstag, 24. Oktober 2017, 11:15:06 CEST schrieb Klaus Goger:
> From: Shawn Lin <shawn.lin@rock-chips.com>
> 
> The vcc_sd or vcc_sdio used for IO voltage for sdmmc and sdio
> interface on rk3399 platform have a limitation that it can't be
> larger than 3.0v, otherwise it has a potential risk for the chip.
> Correct all of them.
> 
> Fixes: 171582e00db1 ("arm64: dts: rockchip: add support for firefly-rk3399 board")
> Cc: stable@vger.kernel.org
> Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> Signed-off-by: Klaus Goger <klaus.goger@theobroma-systems.com>
> ---

Backported for compatiblity with 4.13.
Commit-Id of the original on mainline is
b31ce3041787b61f2dad39d2dcda5c4a81d10e2b


Heiko

>  arch/arm64/boot/dts/rockchip/rk3399-firefly.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
> index ba1d9810ad1e..c27242a7d5e7 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
> @@ -370,10 +370,10 @@
>  				regulator-always-on;
>  				regulator-boot-on;
>  				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <3300000>;
> +				regulator-max-microvolt = <3000000>;
>  				regulator-state-mem {
>  					regulator-on-in-suspend;
> -					regulator-suspend-microvolt = <3300000>;
> +					regulator-suspend-microvolt = <3000000>;
>  				};
>  			};
>  
> 

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

* Re: [PATCH] arm64: dts: rockchip: correct vqmmc voltage for rk3399 platforms
  2017-10-24  9:33 ` Heiko Stuebner
@ 2017-10-24  9:56   ` Greg KH
  0 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2017-10-24  9:56 UTC (permalink / raw)
  To: Heiko Stuebner; +Cc: Klaus Goger, stable, Shawn Lin

On Tue, Oct 24, 2017 at 11:33:27AM +0200, Heiko Stuebner wrote:
> Am Dienstag, 24. Oktober 2017, 11:15:06 CEST schrieb Klaus Goger:
> > From: Shawn Lin <shawn.lin@rock-chips.com>
> > 
> > The vcc_sd or vcc_sdio used for IO voltage for sdmmc and sdio
> > interface on rk3399 platform have a limitation that it can't be
> > larger than 3.0v, otherwise it has a potential risk for the chip.
> > Correct all of them.
> > 
> > Fixes: 171582e00db1 ("arm64: dts: rockchip: add support for firefly-rk3399 board")
> > Cc: stable@vger.kernel.org
> > Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
> > Signed-off-by: Heiko Stuebner <heiko@sntech.de>
> > Signed-off-by: Klaus Goger <klaus.goger@theobroma-systems.com>
> > ---
> 
> Backported for compatiblity with 4.13.
> Commit-Id of the original on mainline is
> b31ce3041787b61f2dad39d2dcda5c4a81d10e2b

Thanks for the backport, now queued up.

greg k-h

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

* Re: [PATCH] arm64: dts: rockchip: correct vqmmc voltage for rk3399 platforms
       [not found] ` <1508211484-210856-1-git-send-email-shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
@ 2017-10-17 17:50   ` Heiko Stuebner
  0 siblings, 0 replies; 5+ messages in thread
From: Heiko Stuebner @ 2017-10-17 17:50 UTC (permalink / raw)
  To: Shawn Lin
  Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, David Wu, Elaine Zhang

Am Dienstag, 17. Oktober 2017, 11:38:04 CEST schrieb Shawn Lin:
> The vcc_sd or vcc_sdio used for IO voltage for sdmmc and sdio
> interface on rk3399 platform have a limitation that it can't be
> larger than 3.0v, otherwise it has a potential risk for the chip.
> Correct all of them.
> 
> Signed-off-by: Shawn Lin <shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>

applied as fix for 4.14 with Klaus' tested tag.


Thanks
Heiko

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

* [PATCH] arm64: dts: rockchip: correct vqmmc voltage for rk3399 platforms
@ 2017-10-17  3:38 Shawn Lin
       [not found] ` <1508211484-210856-1-git-send-email-shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Shawn Lin @ 2017-10-17  3:38 UTC (permalink / raw)
  To: Heiko Stuebner
  Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, David Wu,
	Elaine Zhang, Shawn Lin

The vcc_sd or vcc_sdio used for IO voltage for sdmmc and sdio
interface on rk3399 platform have a limitation that it can't be
larger than 3.0v, otherwise it has a potential risk for the chip.
Correct all of them.

Signed-off-by: Shawn Lin <shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
---

 arch/arm64/boot/dts/rockchip/rk3399-firefly.dts   | 4 ++--
 arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi     | 4 ++--
 arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
index da373dd..4f28628 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
@@ -382,10 +382,10 @@
 				regulator-always-on;
 				regulator-boot-on;
 				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
+				regulator-max-microvolt = <3000000>;
 				regulator-state-mem {
 					regulator-on-in-suspend;
-					regulator-suspend-microvolt = <3300000>;
+					regulator-suspend-microvolt = <3000000>;
 				};
 			};
 
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi
index 53ff3d1..910628d 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi
@@ -325,12 +325,12 @@
 			vcc_sd: LDO_REG4 {
 				regulator-name = "vcc_sd";
 				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
+				regulator-max-microvolt = <3000000>;
 				regulator-always-on;
 				regulator-boot-on;
 				regulator-state-mem {
 					regulator-on-in-suspend;
-					regulator-suspend-microvolt = <3300000>;
+					regulator-suspend-microvolt = <3000000>;
 				};
 			};
 
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi
index 6c30bb0..0f873c8 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi
@@ -315,10 +315,10 @@
 				regulator-always-on;
 				regulator-boot-on;
 				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
+				regulator-max-microvolt = <3000000>;
 				regulator-state-mem {
 					regulator-on-in-suspend;
-					regulator-suspend-microvolt = <3300000>;
+					regulator-suspend-microvolt = <3000000>;
 				};
 			};
 
-- 
1.9.1

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

end of thread, other threads:[~2017-10-24  9:56 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-24  9:15 [PATCH] arm64: dts: rockchip: correct vqmmc voltage for rk3399 platforms Klaus Goger
2017-10-24  9:33 ` Heiko Stuebner
2017-10-24  9:56   ` Greg KH
  -- strict thread matches above, loose matches on Subject: below --
2017-10-17  3:38 Shawn Lin
     [not found] ` <1508211484-210856-1-git-send-email-shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2017-10-17 17:50   ` 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.