Re: [PATCH 1/7] thermal/drivers/core: Remove the module Kconfig's option

From: Paul Burton
Date: Wed Apr 03 2019 - 18:55:54 EST


Hi Daniel,

On Tue, Apr 02, 2019 at 06:12:44PM +0200, Daniel Lezcano wrote:
> The module support for the thermal subsystem makes little sense:
> - some subsystems relying on it are not modules, thus forcing the
> framework to be compiled in
> - it is compiled in for almost every configs, the remaining ones
> are a few platforms where I don't see why we can not switch the thermal
> to 'y'. The drivers can stay in tristate.
> - platforms need the thermal to be ready as soon as possible at boot time
> in order to mitigate

Nit: mitigate what? High temperatures? It feels like this sentence was
cut short.

> Usually the subsystems framework are compiled-in and the plugs are as module.
>
> Remove the module option. The removal of the module related dead code will
> come after this patch gets in or is acked.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
> Acked-by: Guenter Roeck <groeck@xxxxxxxxxxxx>
> For mini2440:
> Acked-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>

Acked-by: Paul Burton <paul.burton@xxxxxxxx> # MIPS part

Thanks,
Paul