linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH V2] ARM: dts: omap36xx: Remove turbo mode for 1GHz variants
@ 2021-01-09 17:01 Adam Ford
  2021-01-26 11:20 ` Tony Lindgren
  2021-10-01  7:54 ` Andreas Kemnade
  0 siblings, 2 replies; 6+ messages in thread
From: Adam Ford @ 2021-01-09 17:01 UTC (permalink / raw)
  To: linux-omap
  Cc: hns, aford, Adam Ford, Benoît Cousson, Tony Lindgren,
	Rob Herring, devicetree, linux-kernel

Previously, the 1GHz variants were marked as a turbo,
because that variant has reduced thermal operating range.

Now that the thermal throttling is in place, it should be
safe to remove the turbo-mode from the 1GHz variants, because
the CPU will automatically slow if the thermal limit is reached.

Signed-off-by: Adam Ford <aford173@gmail.com>
---
V2:  The orignal patch had the wrong file added. Add the omap36xx.dtsi

diff --git a/arch/arm/boot/dts/omap36xx.dtsi b/arch/arm/boot/dts/omap36xx.dtsi
index 05fe5ed127b0..20844dbc002e 100644
--- a/arch/arm/boot/dts/omap36xx.dtsi
+++ b/arch/arm/boot/dts/omap36xx.dtsi
@@ -72,7 +72,6 @@ opp1g-1000000000 {
 					 <1375000 1375000 1375000>;
 			/* only on am/dm37x with speed-binned bit set */
 			opp-supported-hw = <0xffffffff 2>;
-			turbo-mode;
 		};
 	};
 
-- 
2.25.1


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

* Re: [PATCH V2] ARM: dts: omap36xx: Remove turbo mode for 1GHz variants
  2021-01-09 17:01 [PATCH V2] ARM: dts: omap36xx: Remove turbo mode for 1GHz variants Adam Ford
@ 2021-01-26 11:20 ` Tony Lindgren
  2021-10-01  7:54 ` Andreas Kemnade
  1 sibling, 0 replies; 6+ messages in thread
From: Tony Lindgren @ 2021-01-26 11:20 UTC (permalink / raw)
  To: Adam Ford
  Cc: linux-omap, hns, aford, Benoît Cousson, Rob Herring,
	devicetree, linux-kernel

* Adam Ford <aford173@gmail.com> [210109 19:01]:
> Previously, the 1GHz variants were marked as a turbo,
> because that variant has reduced thermal operating range.
> 
> Now that the thermal throttling is in place, it should be
> safe to remove the turbo-mode from the 1GHz variants, because
> the CPU will automatically slow if the thermal limit is reached.

Thanks applying into omap-for-v5.12/dt.

Tony

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

* Re: [PATCH V2] ARM: dts: omap36xx: Remove turbo mode for 1GHz variants
  2021-01-09 17:01 [PATCH V2] ARM: dts: omap36xx: Remove turbo mode for 1GHz variants Adam Ford
  2021-01-26 11:20 ` Tony Lindgren
@ 2021-10-01  7:54 ` Andreas Kemnade
  2021-10-01  9:59   ` H. Nikolaus Schaller
  1 sibling, 1 reply; 6+ messages in thread
From: Andreas Kemnade @ 2021-10-01  7:54 UTC (permalink / raw)
  To: Adam Ford
  Cc: linux-omap, hns, aford, Benoît Cousson, Tony Lindgren,
	Rob Herring, devicetree, linux-kernel

On Sat,  9 Jan 2021 11:01:03 -0600
Adam Ford <aford173@gmail.com> wrote:

> Previously, the 1GHz variants were marked as a turbo,
> because that variant has reduced thermal operating range.
> 
> Now that the thermal throttling is in place, it should be
> safe to remove the turbo-mode from the 1GHz variants, because
> the CPU will automatically slow if the thermal limit is reached.
> 
> Signed-off-by: Adam Ford <aford173@gmail.com>
> ---
> V2:  The orignal patch had the wrong file added. Add the omap36xx.dtsi
> 
hmm, I somehow expected that there is a revert of this thing going
through. But now, the turbo-mode is still missing and I understood the
revert is only in Nikolaus' trees. The 1Ghz mode was working for some
time but does not anymore. Is it just me or do others also have the
same problems?

Regards,
Andreas

> diff --git a/arch/arm/boot/dts/omap36xx.dtsi b/arch/arm/boot/dts/omap36xx.dtsi
> index 05fe5ed127b0..20844dbc002e 100644
> --- a/arch/arm/boot/dts/omap36xx.dtsi
> +++ b/arch/arm/boot/dts/omap36xx.dtsi
> @@ -72,7 +72,6 @@ opp1g-1000000000 {
>  					 <1375000 1375000 1375000>;
>  			/* only on am/dm37x with speed-binned bit set */
>  			opp-supported-hw = <0xffffffff 2>;
> -			turbo-mode;
>  		};
>  	};
>  


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

* Re: [PATCH V2] ARM: dts: omap36xx: Remove turbo mode for 1GHz variants
  2021-10-01  7:54 ` Andreas Kemnade
@ 2021-10-01  9:59   ` H. Nikolaus Schaller
  2021-10-02 12:12     ` Adam Ford
  0 siblings, 1 reply; 6+ messages in thread
From: H. Nikolaus Schaller @ 2021-10-01  9:59 UTC (permalink / raw)
  To: Andreas Kemnade
  Cc: Adam Ford, linux-omap, aford, Benoît Cousson, Tony Lindgren,
	Rob Herring, devicetree, linux-kernel



> Am 01.10.2021 um 09:54 schrieb Andreas Kemnade <andreas@kemnade.info>:
> 
> On Sat,  9 Jan 2021 11:01:03 -0600
> Adam Ford <aford173@gmail.com> wrote:
> 
>> Previously, the 1GHz variants were marked as a turbo,
>> because that variant has reduced thermal operating range.
>> 
>> Now that the thermal throttling is in place, it should be
>> safe to remove the turbo-mode from the 1GHz variants, because
>> the CPU will automatically slow if the thermal limit is reached.
>> 
>> Signed-off-by: Adam Ford <aford173@gmail.com>
>> ---
>> V2:  The orignal patch had the wrong file added. Add the omap36xx.dtsi
>> 
> hmm, I somehow expected that there is a revert of this thing going
> through. But now, the turbo-mode is still missing

tagging by turbo-mode means the OPP is *disabled* by default and
needs to be enabled actively.

> and I understood the
> revert is only in Nikolaus' trees.

It is just a revert for the gta04a5 because I think it is the only board
which is affected (maybe it would need SmartReflex in operation to
fine tune the OPPs compared to the generic table). Therefore I have
a patch which adds turbo-mode to the gta04a5.dts

> The 1Ghz mode was working for some
> time but does not anymore. Is it just me or do others also have the
> same problems?

That would be interesting to know.

BR,
NIkolaus

> 
> 
>> diff --git a/arch/arm/boot/dts/omap36xx.dtsi b/arch/arm/boot/dts/omap36xx.dtsi
>> index 05fe5ed127b0..20844dbc002e 100644
>> --- a/arch/arm/boot/dts/omap36xx.dtsi
>> +++ b/arch/arm/boot/dts/omap36xx.dtsi
>> @@ -72,7 +72,6 @@ opp1g-1000000000 {
>> 					 <1375000 1375000 1375000>;
>> 			/* only on am/dm37x with speed-binned bit set */
>> 			opp-supported-hw = <0xffffffff 2>;
>> -			turbo-mode;
>> 		};
>> 	};
>> 
> 


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

* Re: [PATCH V2] ARM: dts: omap36xx: Remove turbo mode for 1GHz variants
  2021-10-01  9:59   ` H. Nikolaus Schaller
@ 2021-10-02 12:12     ` Adam Ford
  2021-10-07 18:00       ` Andreas Kemnade
  0 siblings, 1 reply; 6+ messages in thread
From: Adam Ford @ 2021-10-02 12:12 UTC (permalink / raw)
  To: H. Nikolaus Schaller
  Cc: Andreas Kemnade, Linux-OMAP, Adam Ford-BE, Benoît Cousson,
	Tony Lindgren, Rob Herring, devicetree,
	Linux Kernel Mailing List

On Fri, Oct 1, 2021 at 4:59 AM H. Nikolaus Schaller <hns@goldelico.com> wrote:
>
>
>
> > Am 01.10.2021 um 09:54 schrieb Andreas Kemnade <andreas@kemnade.info>:
> >
> > On Sat,  9 Jan 2021 11:01:03 -0600
> > Adam Ford <aford173@gmail.com> wrote:
> >
> >> Previously, the 1GHz variants were marked as a turbo,
> >> because that variant has reduced thermal operating range.
> >>
> >> Now that the thermal throttling is in place, it should be
> >> safe to remove the turbo-mode from the 1GHz variants, because
> >> the CPU will automatically slow if the thermal limit is reached.
> >>
> >> Signed-off-by: Adam Ford <aford173@gmail.com>
> >> ---
> >> V2:  The orignal patch had the wrong file added. Add the omap36xx.dtsi
> >>
> > hmm, I somehow expected that there is a revert of this thing going
> > through. But now, the turbo-mode is still missing
>
> tagging by turbo-mode means the OPP is *disabled* by default and
> needs to be enabled actively.
>
> > and I understood the
> > revert is only in Nikolaus' trees.
>
> It is just a revert for the gta04a5 because I think it is the only board
> which is affected (maybe it would need SmartReflex in operation to
> fine tune the OPPs compared to the generic table). Therefore I have
> a patch which adds turbo-mode to the gta04a5.dts
>
> > The 1Ghz mode was working for some
> > time but does not anymore. Is it just me or do others also have the
> > same problems?
>
> That would be interesting to know.
>

What version are you seeing the break?  It's working for me on 5.15.y branch

# cat /sys/devices/system/cpu/cpufreq/policy0/scaling_available_frequencies
300000 600000 800000 1000000
# uname -a
Linux buildroot 5.14.9-00260-g70248e7b378b-dirty #1 SMP Sat Oct 2
06:56:02 CDT 2021 armv7l GNU/Linux
#

adam


> BR,
> NIkolaus
>
> >
> >
> >> diff --git a/arch/arm/boot/dts/omap36xx.dtsi b/arch/arm/boot/dts/omap36xx.dtsi
> >> index 05fe5ed127b0..20844dbc002e 100644
> >> --- a/arch/arm/boot/dts/omap36xx.dtsi
> >> +++ b/arch/arm/boot/dts/omap36xx.dtsi
> >> @@ -72,7 +72,6 @@ opp1g-1000000000 {
> >>                                       <1375000 1375000 1375000>;
> >>                      /* only on am/dm37x with speed-binned bit set */
> >>                      opp-supported-hw = <0xffffffff 2>;
> >> -                    turbo-mode;
> >>              };
> >>      };
> >>
> >
>

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

* Re: [PATCH V2] ARM: dts: omap36xx: Remove turbo mode for 1GHz variants
  2021-10-02 12:12     ` Adam Ford
@ 2021-10-07 18:00       ` Andreas Kemnade
  0 siblings, 0 replies; 6+ messages in thread
From: Andreas Kemnade @ 2021-10-07 18:00 UTC (permalink / raw)
  To: Adam Ford
  Cc: H. Nikolaus Schaller, Linux-OMAP, Adam Ford-BE,
	Benoît Cousson, Tony Lindgren, Rob Herring, devicetree,
	Linux Kernel Mailing List

On Sat, 2 Oct 2021 07:12:05 -0500
Adam Ford <aford173@gmail.com> wrote:

> On Fri, Oct 1, 2021 at 4:59 AM H. Nikolaus Schaller <hns@goldelico.com> wrote:
> >
> >
> >  
> > > Am 01.10.2021 um 09:54 schrieb Andreas Kemnade <andreas@kemnade.info>:
> > >
> > > On Sat,  9 Jan 2021 11:01:03 -0600
> > > Adam Ford <aford173@gmail.com> wrote:
> > >  
> > >> Previously, the 1GHz variants were marked as a turbo,
> > >> because that variant has reduced thermal operating range.
> > >>
> > >> Now that the thermal throttling is in place, it should be
> > >> safe to remove the turbo-mode from the 1GHz variants, because
> > >> the CPU will automatically slow if the thermal limit is reached.
> > >>
> > >> Signed-off-by: Adam Ford <aford173@gmail.com>
> > >> ---
> > >> V2:  The orignal patch had the wrong file added. Add the omap36xx.dtsi
> > >>  
> > > hmm, I somehow expected that there is a revert of this thing going
> > > through. But now, the turbo-mode is still missing  
> >
> > tagging by turbo-mode means the OPP is *disabled* by default and
> > needs to be enabled actively.
> >  
> > > and I understood the
> > > revert is only in Nikolaus' trees.  
> >
> > It is just a revert for the gta04a5 because I think it is the only board
> > which is affected (maybe it would need SmartReflex in operation to
> > fine tune the OPPs compared to the generic table). Therefore I have
> > a patch which adds turbo-mode to the gta04a5.dts
> >  
> > > The 1Ghz mode was working for some
> > > time but does not anymore. Is it just me or do others also have the
> > > same problems?  
> >
> > That would be interesting to know.
> >  
> 
> What version are you seeing the break?  It's working for me on 5.15.y branch
> 
> # cat /sys/devices/system/cpu/cpufreq/policy0/scaling_available_frequencies
> 300000 600000 800000 1000000
> # uname -a
> Linux buildroot 5.14.9-00260-g70248e7b378b-dirty #1 SMP Sat Oct 2
> 06:56:02 CDT 2021 armv7l GNU/Linux
> #
> 

I see this one: I remember there was also a discussion about proper
handling of that but I do not remember the final result:

Author: H. Nikolaus Schaller <hns@goldelico.com>
Date:   Mon Dec 2 22:10:55 2019 +0100

    Revert "ARM: OMAP2+: Fix warnings with broken omap2_set_init_voltage()"
    
    This reverts commit cf395f7ddb9ebc6b2d28d83b53d18aa4e7c19701.
    
    This patch is in conflict with 1GHz OPP.




commit ae44b701fb6afb2be62defff590531e3b0632772
Author: H. Nikolaus Schaller <hns@goldelico.com>
Date:   Sat Apr 10 16:59:53 2021 +0200

    ARM: DTS: gta04a5: disable 1GHz OPP again because it is not reliable
    
    To enable on a running system:
    
    echo 1 >/sys/devices/system/cpu/cpufreq/boost
    
    The symptom can be spurious hangs and virtual paging error,
    kernel panics and strange things.

on top of 5.13-rcX in Nikolaus trees.

Regards,
Andreas

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

end of thread, other threads:[~2021-10-07 18:00 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-09 17:01 [PATCH V2] ARM: dts: omap36xx: Remove turbo mode for 1GHz variants Adam Ford
2021-01-26 11:20 ` Tony Lindgren
2021-10-01  7:54 ` Andreas Kemnade
2021-10-01  9:59   ` H. Nikolaus Schaller
2021-10-02 12:12     ` Adam Ford
2021-10-07 18:00       ` Andreas Kemnade

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).