On Thu, Nov 25, 2021 at 2:18 PM Maulik Shah [off-list ref] wrote:

Hi Rafael,

On 11/25/2021 5:57 PM, Rafael J. Wysocki wrote:

quoted

On Thu, Nov 25, 2021 at 9:55 AM Maulik Shah [off-list ref] wrote:

quoted

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.