[2/2] ARM: dts: exynos: Fix G3D power domain supply on Arndale Octa boards
diff mbox series

Message ID 20200228145502.8316-2-m.szyprowski@samsung.com
State Mainlined
Commit 604e8b79c8864a308a459689c7054a508d52841f
Headers show
Series
  • [1/2] ARM: dts: exynos: Fix G3D power domain supply on Odroid XU3/XU4/HC1 boards
Related show

Commit Message

Marek Szyprowski Feb. 28, 2020, 2:55 p.m. UTC
G3D power domain in Exynos5420 SoC is supplied from PVDD_G3DS_1V0 and
PVDD_G3D_1V0. Besides the main GPU MALI module it also contains the power
domain control logic and clocks. Turning the power supplies off causes
the power domain to fail to operate properly if GPU drivers are loaded as
modules. GPU should use PVDD_G3D_1V0 supply mainly to control the DVFS.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
---
 arch/arm/boot/dts/exynos5420-arndale-octa.dts | 2 ++
 1 file changed, 2 insertions(+)

Comments

Krzysztof Kozlowski Feb. 29, 2020, 2:41 p.m. UTC | #1
On Fri, Feb 28, 2020 at 03:55:02PM +0100, Marek Szyprowski wrote:
> G3D power domain in Exynos5420 SoC is supplied from PVDD_G3DS_1V0 and
> PVDD_G3D_1V0. Besides the main GPU MALI module it also contains the power
> domain control logic and clocks. Turning the power supplies off causes
> the power domain to fail to operate properly if GPU drivers are loaded as
> modules. GPU should use PVDD_G3D_1V0 supply mainly to control the DVFS.
> 
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> ---
>  arch/arm/boot/dts/exynos5420-arndale-octa.dts | 2 ++

Thanks, applied.

Best regards,
Krzysztof

Patch
diff mbox series

diff --git a/arch/arm/boot/dts/exynos5420-arndale-octa.dts b/arch/arm/boot/dts/exynos5420-arndale-octa.dts
index ee28d30f5476..e9a09dd0a49b 100644
--- a/arch/arm/boot/dts/exynos5420-arndale-octa.dts
+++ b/arch/arm/boot/dts/exynos5420-arndale-octa.dts
@@ -584,6 +584,7 @@ 
 				regulator-name = "PVDD_G3DS_1V0";
 				regulator-min-microvolt = <800000>;
 				regulator-max-microvolt = <1100000>;
+				regulator-always-on;
 
 				regulator-state-mem {
 					regulator-on-in-suspend;
@@ -697,6 +698,7 @@ 
 				regulator-name = "PVDD_G3D_1V0";
 				regulator-min-microvolt = <800000>;
 				regulator-max-microvolt = <1400000>;
+				regulator-always-on;
 
 				regulator-state-mem {
 					regulator-off-in-suspend;