From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id ABD421EB5FD; Sun, 23 Nov 2025 15:03:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763910216; cv=none; b=Kl6vOh0/nJwUt+dlRvZPdJozvvV/S8VOp1S4neX0ELFgh3EwwywKGPHpDbdzDolb0r7HnbO4k7Rk4B2btS8+WOy9ZmlVi5uUXr4YGoMIAskYB35UfE7kxlzuOn2VPdsxOvYRUibgdab0BR2OPvyLKk7IlMIVAqHwxI9LWKm//6s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763910216; c=relaxed/simple; bh=rrfJ05CXKUZoV4GUX2/6fkvMYwZCqajzZcU6N60hraw=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ncQuJm0a1J5s9TtAx/RIGQQM1Rtp68wQBEiNRExW1l6/8xyxhjBgvYseXfdpjG8Q+iJ8oBHKKFmjXDK0o6wzxdQmmeXE5KhOHXPTg8zd77mrUkNrxfMpfCWyUeQ/LAoH8o3O+ou6REaOfnCPJ5nv6vQzkrcJYHhLGAUsxfDZzSI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=pTtRWJ3a; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="pTtRWJ3a" Received: by smtp.kernel.org (Postfix) with ESMTPSA id DBD22C113D0; Sun, 23 Nov 2025 15:03:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1763910216; bh=rrfJ05CXKUZoV4GUX2/6fkvMYwZCqajzZcU6N60hraw=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=pTtRWJ3a4whfVc7vO9ZUd45+XV0m0wGyBi6hMCB9JnvKwVocL3g7byC92oMEF6S+P iiZRTbe+6eT2L/BIK7lnMLQKGi3e9dTkX4cbP1IOzYSvhXQKss5nmSIVNu3QMPy/Qk FrNw6sFlxC87Zub2IT4IBMc5W8Mn9/kUMJoNZ8Le7pgaL9kJmTmq3e9rYcUxTbBCdk ffsOeqGUiBo3bLHRyNeT4rAOVYVfhXsZFgaAAhaQE1zcUe9lu2ANfnPZFttODEdZ/S N8fyGgoAzFBgAdaFX1uo8tDHZuVTkUZfJvXGmGUwUZfJiY3J3YzUCrGVw01jFy601U 0dy0Th9vUCf+A== Message-ID: <310a04c8-c268-4ff8-b185-94fe456b85f9@kernel.org> Date: Mon, 24 Nov 2025 00:03:32 +0900 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RESEND v2] devfreq: move governor.h to a public header location To: Dmitry Baryshkov , MyungJoo Ham , Kyungmin Park , Chanwoo Choi , Dmitry Osipenko , Thierry Reding , Jonathan Hunter Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-tegra@vger.kernel.org, linux-arm-msm@vger.kernel.org, Robie Basak References: <20251030-governor-public-v2-1-432a11a9975a@oss.qualcomm.com> From: Chanwoo Choi Content-Language: en-US In-Reply-To: <20251030-governor-public-v2-1-432a11a9975a@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Hi I applied with following consistent patch title format of devfreq patches. - PM / devfreq: Move governor.h to a public header location Thanks, Chanwoo Choi 25. 10. 31. 03:26에 Dmitry Baryshkov 이(가) 쓴 글: > 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 > Acked-by: Jon Hunter > Signed-off-by: Dmitry Baryshkov > --- > Changes in v2: > - Fixed typo in commit subject (Mikko Perttunen) > - Link to v1: https://lore.kernel.org/r/20250903-governor-public-v1-1-111abd89a89a@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(-) > > diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c > index 2e8d01d47f6996a634a8ad5ddf20c5a68d1a299d..00979f2e0e276a05ee073dcf5cd8e930bdd539fb 100644 > --- a/drivers/devfreq/devfreq.c > +++ b/drivers/devfreq/devfreq.c > @@ -20,6 +20,7 @@ > #include > #include > #include > +#include > #include > #include > #include > @@ -28,7 +29,6 @@ > #include > #include > #include > -#include "governor.h" > > #define CREATE_TRACE_POINTS > #include > diff --git a/drivers/devfreq/governor_passive.c b/drivers/devfreq/governor_passive.c > index 953cf9a1e9f7f93804cc889db38883bf97ae005d..8cd6f9a59f6422ccd138ff4b264dc8a547ad574f 100644 > --- a/drivers/devfreq/governor_passive.c > +++ b/drivers/devfreq/governor_passive.c > @@ -14,8 +14,33 @@ > #include > #include > #include > +#include > #include > -#include "governor.h" > + > +/** > + * struct devfreq_cpu_data - Hold the per-cpu data > + * @node: list node > + * @dev: reference to cpu device. > + * @first_cpu: the cpumask of the first cpu of a policy. > + * @opp_table: reference to cpu opp table. > + * @cur_freq: the current frequency of the cpu. > + * @min_freq: the min frequency of the cpu. > + * @max_freq: the max frequency of the cpu. > + * > + * This structure stores the required cpu_data of a cpu. > + * This is auto-populated by the governor. > + */ > +struct devfreq_cpu_data { > + struct list_head node; > + > + struct device *dev; > + unsigned int first_cpu; > + > + struct opp_table *opp_table; > + unsigned int cur_freq; > + unsigned int min_freq; > + unsigned int max_freq; > +}; > > static struct devfreq_cpu_data * > get_parent_cpu_data(struct devfreq_passive_data *p_data, > diff --git a/drivers/devfreq/governor_performance.c b/drivers/devfreq/governor_performance.c > index 2e4e981446fa8ea39f65b09dddff198c0b8e3338..fdb22bf512cf134d75f1eaf3edb80e562dd28bec 100644 > --- a/drivers/devfreq/governor_performance.c > +++ b/drivers/devfreq/governor_performance.c > @@ -7,8 +7,8 @@ > */ > > #include > +#include > #include > -#include "governor.h" > > static int devfreq_performance_func(struct devfreq *df, > unsigned long *freq) > diff --git a/drivers/devfreq/governor_powersave.c b/drivers/devfreq/governor_powersave.c > index f059e881480465b051f27d740348adaf779aebf0..ee2d6ec8a512248f070b2c5bee8146320b7be312 100644 > --- a/drivers/devfreq/governor_powersave.c > +++ b/drivers/devfreq/governor_powersave.c > @@ -7,8 +7,8 @@ > */ > > #include > +#include > #include > -#include "governor.h" > > static int devfreq_powersave_func(struct devfreq *df, > unsigned long *freq) > diff --git a/drivers/devfreq/governor_simpleondemand.c b/drivers/devfreq/governor_simpleondemand.c > index c234357363675508c12732a08c1cd26c349039d1..9c69b96df5f97306e9be46aa6bb1d9d2f8e58490 100644 > --- a/drivers/devfreq/governor_simpleondemand.c > +++ b/drivers/devfreq/governor_simpleondemand.c > @@ -9,8 +9,8 @@ > #include > #include > #include > +#include > #include > -#include "governor.h" > > /* Default constants for DevFreq-Simple-Ondemand (DFSO) */ > #define DFSO_UPTHRESHOLD (90) > diff --git a/drivers/devfreq/governor_userspace.c b/drivers/devfreq/governor_userspace.c > index 175de0c0b50e087861313060eab70a35b757fd20..395174f93960d0762456238654f4d356e21cf57c 100644 > --- a/drivers/devfreq/governor_userspace.c > +++ b/drivers/devfreq/governor_userspace.c > @@ -9,11 +9,11 @@ > #include > #include > #include > +#include > #include > #include > #include > #include > -#include "governor.h" > > struct userspace_data { > unsigned long user_frequency; > diff --git a/drivers/devfreq/hisi_uncore_freq.c b/drivers/devfreq/hisi_uncore_freq.c > index 96d1815059e32c4e70a1d3c257655cc6b162f745..b8e4621c57ebc76513e4eba978aa54f2b884e210 100644 > --- a/drivers/devfreq/hisi_uncore_freq.c > +++ b/drivers/devfreq/hisi_uncore_freq.c > @@ -9,6 +9,7 @@ > #include > #include > #include > +#include > #include > #include > #include > @@ -26,8 +27,6 @@ > #include > #include > > -#include "governor.h" > - > struct hisi_uncore_pcc_data { > u16 status; > u16 resv; > diff --git a/drivers/devfreq/tegra30-devfreq.c b/drivers/devfreq/tegra30-devfreq.c > index 4a4f0106ab9ddcfb106a1860370cbf8a3579322a..77cbb204087c970c1fec0c1597b1e76c1a11b390 100644 > --- a/drivers/devfreq/tegra30-devfreq.c > +++ b/drivers/devfreq/tegra30-devfreq.c > @@ -9,6 +9,7 @@ > #include > #include > #include > +#include > #include > #include > #include > @@ -21,8 +22,6 @@ > > #include > > -#include "governor.h" > - > #define ACTMON_GLB_STATUS 0x0 > #define ACTMON_GLB_PERIOD_CTRL 0x4 > > diff --git a/drivers/devfreq/governor.h b/include/linux/devfreq-governor.h > similarity index 80% > rename from drivers/devfreq/governor.h > rename to include/linux/devfreq-governor.h > index 0adfebc0467a3db39278814fa66d2b1f25d61f7a..dfdd0160a29f35f5608575b07b450cf5157420ff 100644 > --- a/drivers/devfreq/governor.h > +++ b/include/linux/devfreq-governor.h > @@ -5,11 +5,11 @@ > * Copyright (C) 2011 Samsung Electronics > * MyungJoo Ham > * > - * This header is for devfreq governors in drivers/devfreq/ > + * This header is for devfreq governors > */ > > -#ifndef _GOVERNOR_H > -#define _GOVERNOR_H > +#ifndef __LINUX_DEVFREQ_DEVFREQ_H__ > +#define __LINUX_DEVFREQ_DEVFREQ_H__ > > #include > > @@ -47,31 +47,6 @@ > #define DEVFREQ_GOV_ATTR_POLLING_INTERVAL BIT(0) > #define DEVFREQ_GOV_ATTR_TIMER BIT(1) > > -/** > - * struct devfreq_cpu_data - Hold the per-cpu data > - * @node: list node > - * @dev: reference to cpu device. > - * @first_cpu: the cpumask of the first cpu of a policy. > - * @opp_table: reference to cpu opp table. > - * @cur_freq: the current frequency of the cpu. > - * @min_freq: the min frequency of the cpu. > - * @max_freq: the max frequency of the cpu. > - * > - * This structure stores the required cpu_data of a cpu. > - * This is auto-populated by the governor. > - */ > -struct devfreq_cpu_data { > - struct list_head node; > - > - struct device *dev; > - unsigned int first_cpu; > - > - struct opp_table *opp_table; > - unsigned int cur_freq; > - unsigned int min_freq; > - unsigned int max_freq; > -}; > - > /** > * struct devfreq_governor - Devfreq policy governor > * @node: list node - contains registered devfreq governors > @@ -124,4 +99,4 @@ static inline int devfreq_update_stats(struct devfreq *df) > > return df->profile->get_dev_status(df->dev.parent, &df->last_status); > } > -#endif /* _GOVERNOR_H */ > +#endif /* __LINUX_DEVFREQ_DEVFREQ_H__ */ > > --- > base-commit: 8cd53fb40a304576fa86ba985f3045d5c55b0ae3 > change-id: 20250903-governor-public-d9cd4198f858 > > Best regards, -- Best Regards, Samsung Electronics Chanwoo Choi