Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-22 Thread nnet
> Could I add your Tested-by line to patch series? Yes, by: Philip Soares Thank you for the patches!

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-21 Thread nnet
> Could you test if 1.155V voltage for L1 is stable on 1.2 GHz variant? ++#define MIN_VOLT_MV_FOR_L1_1200MHZ 1155 ... ++ if (avs_min_l1 > dvfs->avs[0]) ++ avs_min_l1 = dvfs->avs[0]; ++ ++ if (dvfs->avs[1] < avs_min_l1) ++ dvfs->av

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-16 Thread nnet
> Therefore I'm thinking if the correct way is instead to use L1 := L0 voltage > value for 1/1.2 GHz mode. This latest 04/10 works fine for me going 600MHz <-> 1.2GHz under with and without load.

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-15 Thread nnet
> Could you test following change instead of PATCH 04/10? I added here also > logic for 1.2 GHz variant with 1.132 V value another change is that > value for load L0 is not touched as it is stable. These changes to patch 04/10 worked going 600 MHz <-> 1.2 GHz , _but_ only with: ++#define MIN_VOLT

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-13 Thread nnet
On Sat, Feb 13, 2021, at 2:01 AM, Pali Rohár wrote: > On Thursday 11 February 2021 16:41:13 nnet wrote: > > On Thu, Feb 11, 2021, at 3:44 PM, Pali Rohár wrote: > > > On Thursday 11 February 2021 12:22:52 nnet wrote: > > > > On Thu, Feb 11, 2021, at 11:55 AM,

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-11 Thread nnet
On Thu, Feb 11, 2021, at 3:44 PM, Pali Rohár wrote: > On Thursday 11 February 2021 12:22:52 nnet wrote: > > On Thu, Feb 11, 2021, at 11:55 AM, Pali Rohár wrote: > > > On Wednesday 10 February 2021 11:08:59 nnet wrote: > > > > On Wed, Feb 10, 2021,

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-11 Thread nnet
On Thu, Feb 11, 2021, at 11:55 AM, Pali Rohár wrote: > On Wednesday 10 February 2021 11:08:59 nnet wrote: > > On Wed, Feb 10, 2021, at 10:03 AM, Pali Rohár wrote: > > > > > Hello! Could you please enable userspace governor during kern

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-10 Thread nnet
On Wed, Feb 10, 2021, at 10:03 AM, Pali Rohár wrote: > On Wednesday 10 February 2021 09:34:07 nnet wrote: > > On Wed, Feb 10, 2021, at 1:23 AM, Pali Rohár wrote: > > > On Tuesday 09 February 2021 18:07:41 nnet wrote: > > > > On Tue, Feb 9, 2021, at 5:51 PM, nnet

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-10 Thread nnet
On Wed, Feb 10, 2021, at 1:23 AM, Pali Rohár wrote: > On Tuesday 09 February 2021 18:07:41 nnet wrote: > > On Tue, Feb 9, 2021, at 5:51 PM, nnet wrote: > > > On Tue, Feb 9, 2021, at 5:31 PM, nnet wrote: > > > > On Tue, Feb 9, 2021, at 3:26 PM, Marek Behún wrote: >

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-09 Thread nnet
On Tue, Feb 9, 2021, at 5:51 PM, nnet wrote: > On Tue, Feb 9, 2021, at 5:31 PM, nnet wrote: > > On Tue, Feb 9, 2021, at 3:26 PM, Marek Behún wrote: > > > On Tue, 09 Feb 2021 15:16:45 -0800 > > > nnet wrote: > > > > > > > I've two of the

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-09 Thread nnet
On Tue, Feb 9, 2021, at 5:31 PM, nnet wrote: > On Tue, Feb 9, 2021, at 3:26 PM, Marek Behún wrote: > > On Tue, 09 Feb 2021 15:16:45 -0800 > > nnet wrote: > > > > > I've two of these and I've just swapped them (and re-pasted the heat > > > sinks

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-09 Thread nnet
On Tue, Feb 9, 2021, at 3:26 PM, Marek Behún wrote: > On Tue, 09 Feb 2021 15:16:45 -0800 > nnet wrote: > > > I've two of these and I've just swapped them (and re-pasted the heat sinks). > > > > The second one ran under load for awhile and now has frozen as

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-09 Thread nnet
eze. They fine with both those workloads @1000_800. Perhaps it's heat? Unfortunately I don't have any numbers on that ATM. On Tue, Feb 9, 2021, at 2:56 PM, Pali Rohár wrote: > On Tuesday 09 February 2021 14:52:56 nnet wrote: > > On Tue, Feb 9, 2021, at 2:42 PM, Pali Rohár wrote:

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-09 Thread nnet
On Tue, Feb 9, 2021, at 2:42 PM, Pali Rohár wrote: > On Tuesday 09 February 2021 13:45:08 nnet wrote: > > On Tue, Feb 9, 2021, at 1:33 PM, Pali Rohár wrote: > > > On Tuesday 09 February 2021 13:00:26 nnet wrote: > > > > > If you have other Armada 3720 board

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-09 Thread nnet
On Tue, Feb 9, 2021, at 1:33 PM, Pali Rohár wrote: > On Tuesday 09 February 2021 13:00:26 nnet wrote: > > > If you have other Armada 3720 boards (Espressobin v5/v7, uDPU, Devel > > > Board, ...) then it will be nice to do an additional tests and check if > > >

Re: [PATCH mvebu v2 00/10] Armada 37xx: Fix cpufreq changing base CPU speed to 800 MHz from 1000 MHz

2021-02-09 Thread nnet
> If you have other Armada 3720 boards (Espressobin v5/v7, uDPU, Devel Board, > ...) then it will be nice to do an additional tests and check if instability > issues are finally fixed. These patches applied to the 5.4.96 in OpenWrt (98d61b5) work fine so far on an Espressobin v7 AFAICT per chan