Re: [PATCH] devreq: move governor.h to a public header location

[Jon Hunter]

On 03/09/2025 14:43, Dmitry Baryshkov wrote:
> Some device drivers (and out-of-tree modules) might want to define
> device-specific device governors. Rather than restricting all of them to
> be a part of drivers/devfreq/ (which is not possible for out-of-tree
> drivers anyway) move governor.h to include/linux/devfreq-governor.h and
> update all drivers to use it.
>
> The devfreq_cpu_data is only used internally, by the passive governor,
> so it is moved to the driver source rather than being a part of the
> public interface.
>
> Reported-by: Robie Basak <robibasa@xxxxxxxxxxxxxxxx>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> ---
>   drivers/devfreq/devfreq.c                          |  2 +-
>   drivers/devfreq/governor_passive.c                 | 27 +++++++++++++++++-
>   drivers/devfreq/governor_performance.c             |  2 +-
>   drivers/devfreq/governor_powersave.c               |  2 +-
>   drivers/devfreq/governor_simpleondemand.c          |  2 +-
>   drivers/devfreq/governor_userspace.c               |  2 +-
>   drivers/devfreq/hisi_uncore_freq.c                 |  3 +-
>   drivers/devfreq/tegra30-devfreq.c                  |  3 +-
>   .../governor.h => include/linux/devfreq-governor.h | 33 +++-------------------
>   9 files changed, 37 insertions(+), 39 deletions(-)

For the Tegra part ...

Acked-by: Jon Hunter <jonathanh@xxxxxxxxxx>

Jon

--
nvpublic