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

2021-02-22 Thread Philip Soares
Tested-by: Philip Soares On Mon, Feb 22, 2021, at 8:36 AM, nnet wrote: > > 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-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-22 Thread Pali Rohár
On Sunday 21 February 2021 19:17:40 nnet wrote: > > 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]; > ++ > ++

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) ++

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

2021-02-19 Thread Pali Rohár
On Tuesday 16 February 2021 08:27:10 nnet wrote: > > 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. Ok, thanks for testing! Just to note

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-16 Thread Pali Rohár
On Monday 15 February 2021 21:48:08 nnet wrote: > > 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

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

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

2021-02-14 Thread Pali Rohár
On Saturday 13 February 2021 10:30:19 nnet wrote: > 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

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, Pali Rohár wrote: > > > > > On

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

2021-02-13 Thread Pali Rohár
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, Pali Rohár wrote: > > > > On Wednesday 10 February 2021 11:08:59 nnet wrote: > > > > > On Wed, Feb

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, at 10:03 AM, 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-11 Thread Pali Rohár
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, at 10:03 AM, Pali Rohár wrote: > > > > > > Hello! Could you please enable userspace governor during kernel > >

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 kernel > > > > > compilation? > > > > > > > > > >

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

2021-02-11 Thread Pali Rohár
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 kernel > > > > compilation? > > > > > > > > CONFIG_CPU_FREQ_GOV_USERSPACE=y > > > > > > > > It can be activated via

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

2021-02-10 Thread Marek Behún
On Wed, 10 Feb 2021 11:08:59 -0800 nnet wrote: > => md d0012604 1; md d0012604 1; md d0012604 1 > d0012604: 2b417501 .uA+ > d0012604: 945b [... > d0012604: So this means that in OTP you

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 wrote: > > > > > On Tue, Feb 9,

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

2021-02-10 Thread Pali Rohár
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 wrote: > > > > On Tue, Feb 9, 2021, at 5:31 PM, nnet wrote: > > > > > On Tue, Feb 9, 2021, at

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: > > > > > On Tue, 09 Feb 2021

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

2021-02-10 Thread Pali Rohár
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: > > > > On Tue, 09 Feb 2021 15:16:45 -0800 > > > > nnet wrote: > > > > > > > > > I've two of

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

2021-02-09 Thread Marek Behún
On Tue, 09 Feb 2021 17:51:53 -0800 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 these and I've just swapped them (and re-pasted 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: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 these and I've just swapped them (and re-pasted the heat >

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). > > > > > > The second one ran under load for

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 well. > > > > Under a moderate load

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

2021-02-09 Thread Marek Behún
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 well. > > Under a moderate load `wget -O /dev/null ` @X00Mbits they are fine. > > Under a 1 min

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
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 well. Under a moderate load `wget -O /dev/null ` @X00Mbits they are fine. Under a 1 min speed test of load ~200Mbits routed WireGuard they freeze. They

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

2021-02-09 Thread Pali Rohár
On Tuesday 09 February 2021 14:52:56 nnet wrote: > 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

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 boards (Espressobin v5/v7, uDPU, Devel > > >

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

2021-02-09 Thread Pali Rohár
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 boards (Espressobin v5/v7, uDPU, Devel > > > > Board, ...) then it will be nice to do an additional

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 > > > instability issues are finally

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

2021-02-09 Thread Pali Rohár
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 > > instability issues are finally fixed. > > These patches applied to the 5.4.96 in OpenWrt

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

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

2021-02-03 Thread Anders Trier Olesen
Hi Marek & Pali On 1/14/21 1:40 PM, Pali Rohár wrote: Hello! The armada-37xx-cpufreq driver changes base CPU speed from 1000 MHz to 800 MHz on EspressoBIN and Turris MOX. The commit message in patch 2/10 explains why and how can this be discovered. That patch 2/10 led us to discover another

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

2021-02-01 Thread Tomasz Maciej Nowak
W dniu 14.01.2021 o 13:40, Pali Rohár pisze: > Hello! > > The armada-37xx-cpufreq driver changes base CPU speed from 1000 MHz to > 800 MHz on EspressoBIN and Turris MOX. The commit message in patch 2/10 > explains why and how can this be discovered. > > That patch 2/10 led us to discover another