On Thu, Nov 25, 2021 at 2:18 PM Maulik Shah <quic_mkshah@xxxxxxxxxxx> wrote: > > Hi Rafael, > > On 11/25/2021 5:57 PM, Rafael J. Wysocki wrote: > > On Thu, Nov 25, 2021 at 9:55 AM Maulik Shah <quic_mkshah@xxxxxxxxxxx> wrote: > >> This series makes changes to allow cpuidle governors menu, ladder and teo > >> to compiled as modules when building with allmodconfig. > > What's the purpose of this? > > There are two purposes of this series. > > 1. The series enables cpuidle governors to be allowed to compiled as > modules. > This is something similar to what cpufreq/devfreq governors do > today as they can be be compiled as modules or built-in depending on the > build config. Which is not the case for all of them, though, and I don't see why this would imply that making cpuidle governors modular would be useful. > 2. The series will also enable custom cpuidle governor to be able to > register with cpuidle framework by using cpuidle_register_governor() API. > This will be already achieved by (1) since it will export the > required APIs for menu/ladder/teo governors to be compiled as module. No custom cpuidle governors in the mainline, please. If you have one you want to be included, please submit it. So from the mainline perspective this series doesn't serve any useful purpose at all. Sorry about that.
