[PATCH] ARM: dts: omap36xx: declare 1GHz OPP as turbo again

Andreas Kemnade posted 1 patch 1 month, 1 week ago
arch/arm/boot/dts/ti/omap/omap36xx.dtsi | 1 +
1 file changed, 1 insertion(+)
[PATCH] ARM: dts: omap36xx: declare 1GHz OPP as turbo again
Posted by Andreas Kemnade 1 month, 1 week ago
Operating stable without reduced chip life at 1Ghz needs several
technologies working: The technologies involve
- SmartReflex
- DVFS

As this cannot directly specified in the OPP table as dependecies in the
devicetree yet, use the turbo flag again to mark this OPP as something
special to have some kind of opt-in.

So revert commit
5f1bf7ae8481 ("ARM: dts: omap36xx: Remove turbo mode for 1GHz variants")

Practical reasoning:
At least the GTA04A5 (DM3730) has become unstable with that OPP enabled.
Furthermore nothing enforces the availability of said technologies,
even in the kernel configuration, so allow users to rather opt-in.

Cc: Stable@vger.kernel.org
Fixes: 5f1bf7ae8481 ("ARM: dts: omap36xx: Remove turbo mode for 1GHz variants")
Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
---
 arch/arm/boot/dts/ti/omap/omap36xx.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/ti/omap/omap36xx.dtsi b/arch/arm/boot/dts/ti/omap/omap36xx.dtsi
index c3d79ecd56e39..c217094b50abc 100644
--- a/arch/arm/boot/dts/ti/omap/omap36xx.dtsi
+++ b/arch/arm/boot/dts/ti/omap/omap36xx.dtsi
@@ -72,6 +72,7 @@ opp-1000000000 {
 					 <1375000 1375000 1375000>;
 			/* only on am/dm37x with speed-binned bit set */
 			opp-supported-hw = <0xffffffff 2>;
+			turbo-mode;
 		};
 	};
 
-- 
2.39.2
Re: [PATCH] ARM: dts: omap36xx: declare 1GHz OPP as turbo again
Posted by Kevin Hilman 3 weeks, 6 days ago
On Fri, 18 Oct 2024 23:47:27 +0200, Andreas Kemnade wrote:
> Operating stable without reduced chip life at 1Ghz needs several
> technologies working: The technologies involve
> - SmartReflex
> - DVFS
> 
> As this cannot directly specified in the OPP table as dependecies in the
> devicetree yet, use the turbo flag again to mark this OPP as something
> special to have some kind of opt-in.
> 
> [...]

Applied, thanks!

[1/1] ARM: dts: omap36xx: declare 1GHz OPP as turbo again
      commit: 96a64e9730c2c76cfa5c510583a0fbf40d62886b

Best regards,
-- 
Kevin Hilman <khilman@baylibre.com>