Re: [PATCH] arm64: dts: meson: remove CPU opps below 1GHz for G12B/SM1

[Christian Hewitt]

> On 10 Feb 2022, at 5:31 am, Kevin Hilman <khilman@xxxxxxxxxxxx> wrote:
> 
> Christian Hewitt <christianshewitt@xxxxxxxxx> writes:
> 
>> Amlogic G12B and SM1 devices experience CPU stalls and random board
>> wedges when the system idles and CPU cores clock down to lower opp
>> points. Recent vendor kernels include a change to remove 100-250MHz
>> (with no explanation) [0] but other downstream sources also remove
>> the 500/667MHz points (also with no explanation). Unless 100-667Mhz
>> opps are removed or the CPU governor forced to performance, stalls
>> are observed, so let's remove them an improve stability/uptime.
> 
> Just curious: what CPUfreq governor do you use by default for the
> LibreELEC kernel?

LE uses ondemand. One of the original clues on the problem us that the
issue isn’t seen in some of the retro-gaming forks on LE's codebase
which use the performance governor (and overclocks, etc.)

> Your patch greatly improves the stability I'm seeing, but doesn't quite
> elimitate it.
> 
> I'm testing suspend/resume in a loop on VIM3, and with schedutil
> (default) or ondemand, it eventually hangs.  With either powersave or
> performance it's stable.  
> 
> Kevin