Re: [PATCH] thermal: Remove core header inclusion from drivers
From: Ricardo Neri
Date: Fri Feb 10 2023 - 21:00:44 EST
On Mon, Feb 06, 2023 at 04:34:29PM +0100, Daniel Lezcano wrote:
> As the name states "thermal_core.h" is the header file for the core
> components of the thermal framework.
>
> Too many drivers are including it. Hopefully the recent cleanups
> helped to self encapsulate the code a bit more and prevented the
> drivers to need this header.
>
> Remove this inclusion in every place where it is possible.
>
> Some other drivers did a confusion with the core header and the one
> exported in linux/thermal.h. They include the former instead of the
> latter. The changes also fix this.
>
> The tegra/soctherm driver still remains as it uses an internal
> function which need to be replaced.
>
> The Intel HFI driver uses the netlink internal framework core and
> should be changed to prevent to deal with the internals.
I don't see any of the thermal netlink functionality exposed. Is
there any work in progress?
FWIW, Acked-by: Ricardo Neri <ricardo.neri-calderon@xxxxxxxxxxxxxxx>