Message ID | 20200212235134.12638-1-digetx@gmail.com |
---|---|
Headers | show |
Series | Consolidate and improve NVIDIA Tegra CPUIDLE driver(s) | expand |
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.
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 ... > cpuidle: Refactor and move out NVIDIA Tegra20 driver into > drivers/cpuidle > cpuidle: tegra: Squash Tegra30 driver into the common driver > cpuidle: tegra: Squash Tegra114 driver into the common driver > cpuidle: tegra: Disable CC6 state if LP2 unavailable Hello Rafael and Daniel, Could you please let us know whether you're fine with the above patches by giving an ACK to them? My understanding is that Thierry can't take the cpuidle patches through the Tegra tree without yours ACK. Thanks in advance!
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. > > - 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. > > - Rebased this series on top of recent linux-next, in a result dropped > the "cpuidle: Avoid NULL dereference in cpuidle_driver_state_disabled()" > patch because it's not needed anymore. I just noticed that this patchset was versioned wrongly, it should've been v10. Sorry for the confusion, next version will be v11.