Re: [PATCH v9 00/17] Consolidate and improve NVIDIA Tegra CPUIDLE driver(s)

From: Dmitry Osipenko
Date: Wed Feb 12 2020 - 19:38:19 EST


13.02.2020 02:51, Dmitry Osipenko ÐÐÑÐÑ:
> Hello,
>
> This series does the following:
>
> 1. Unifies Tegra20/30/114 drivers into a single driver and moves it out
> into common drivers/cpuidle/ directory.
>
> 2. Enables CPU cluster power-down idling state on Tegra30.
>
> In the end there is a quite nice clean up of the Tegra CPUIDLE drivers
> and of the Tegra's arch code in general. Please apply, thanks!
>
> !!!WARNING!!! This series was made on top of the cpufreq patches [1]. But it
> should be fine as long as Thierry Reding would pick up this and
> the cpufreq patchsets via the Tegra tree, otherwise there will
> one minor merge-conflict.
>
> [1] https://patchwork.ozlabs.org/project/linux-tegra/list/?series=158206
>
> Changelog:
>
> v9: - Added acks from Peter De Schrijver.
>
> - Added tested-by from Peter Geis, Jasper Korten and David Heidelberg
> who tested these patches on Ouya, TF300T and Nexus 7 devices.

I forgot to mention that both cpufreq and cpuidle patchsets were also
tested on AC100 by Nicolas Chauvet and I forgot to ask for the explicit
t-b. Nicolas, thank you very much for all the testing of the
grate-kernel! Please feel free to give yours t-b :)

> - Temporarily dropped the "cpuidle: tegra: Support CPU cluster power-down
> state on Tegra30" patch because MichaÅ MirosÅaw reported that it didn't
> work well on his TF300T. After some testing we found that changing
> a way in which firmware performs L2 cache maintenance helps, but later
> on we also found that the current v9 series works just fine without the
> extra firmware changes using recent linux-next and the reason why v8
> didn't work before is still unknown (need more testing). So I decided
> that it will be better to postpone the dropped patch until we know for
> sure that it works well for everyone in every possible configuration.

MichaÅ, please let me know if you'll spot any problems with the recent
version of the patches and please feel free to give yours t-b if it
works well.