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

[Jon Hunter <jonathanh@nvidia.com>]

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@qti.qualcomm.com>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> ---
>   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@nvidia.com>

Jon

-- 
nvpublic