Hi Rafael, On 22/09/2020 19:27, Rafael J. Wysocki wrote: > Hi Lina, > > On Tue, Sep 22, 2020 at 6:12 PM Lina Iyer <ilina@xxxxxxxxxxxxxx> wrote: >> >> Hi Rafael, >> >> On Tue, Sep 22 2020 at 10:00 -0600, Rafael J. Wysocki wrote: >>> Sorry for the delay. >>> >>> On Wed, Sep 9, 2020 at 12:15 AM Lina Iyer <ilina@xxxxxxxxxxxxxx> wrote: >>>> >>>> Commit 83788c0caed3 ("cpuidle: remove unused exports") removed >>>> capability of registering cpuidle governors, which was unused at that >>>> time. By exporting the symbol, let's allow platform specific modules to >>>> register cpuidle governors and use cpuidle_governor_latency_req() to get >>>> the QoS for the CPU. >>> >>> Which platform-specific modules may want to do that and why? >>> >> We are planning a custom cpuidle governor for QCOM SoCs. With Android, >> the idea is to make them loadable modules so they can be in a separate >> partition. > > Well, the $subject patch is not applicable without a mainline user > requiring this, so it needs to be posted along with that user. Putting apart the custom cpuidle governor mentioned above, would it make sense to convert the governors into modules ? It is pointless to have all of them compiled in, especially with distros doing make allmodconfig, no? -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog