* [PATCH] PM / devfreq: Kconfig: Drop explicit selection of PM_OPP @ 2019-11-11 18:16 ` Leonard Crestez 2019-11-12 0:40 ` Chanwoo Choi 0 siblings, 1 reply; 2+ messages in thread From: Leonard Crestez @ 2019-11-11 18:16 UTC (permalink / raw) To: MyungJoo Ham, Kyungmin Park, Chanwoo Choi Cc: Krzysztof Kozlowski, Rafael J. Wysocki, Viresh Kumar, Enric Balletbo i Serra, Dmitry Osipenko, linux-pm CONFIG_PM_OPP is already selected by CONFIG_PM_DEVFREQ since commit b9c69e043266 ("PM / devfreq: Add dependency on PM_OPP") This means that individual drivers shouldn't "select PM_OPP" explicitly. Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com> --- Spawned by review comments on a new driver: https://patchwork.kernel.org/patch/11235695/#22992159 This patch shouldn't have any dependencies. drivers/devfreq/Kconfig | 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig index defe1d438710..066e6c4efaa2 100644 --- a/drivers/devfreq/Kconfig +++ b/drivers/devfreq/Kconfig @@ -81,11 +81,10 @@ config ARM_EXYNOS_BUS_DEVFREQ depends on ARCH_EXYNOS || COMPILE_TEST select DEVFREQ_GOV_SIMPLE_ONDEMAND select DEVFREQ_GOV_PASSIVE select DEVFREQ_EVENT_EXYNOS_PPMU select PM_DEVFREQ_EVENT - select PM_OPP help This adds the common DEVFREQ driver for Exynos Memory bus. Exynos Memory bus has one more group of memory bus (e.g, MIF and INT block). Each memory bus group could contain many memoby bus block. It reads PPMU counters of memory controllers by using DEVFREQ-event device @@ -96,22 +95,20 @@ config ARM_TEGRA_DEVFREQ tristate "NVIDIA Tegra30/114/124/210 DEVFREQ Driver" depends on ARCH_TEGRA_3x_SOC || ARCH_TEGRA_114_SOC || \ ARCH_TEGRA_132_SOC || ARCH_TEGRA_124_SOC || \ ARCH_TEGRA_210_SOC || \ COMPILE_TEST - select PM_OPP help This adds the DEVFREQ driver for the Tegra family of SoCs. It reads ACTMON counters of memory controllers and adjusts the operating frequencies and voltages with OPP support. config ARM_TEGRA20_DEVFREQ tristate "NVIDIA Tegra20 DEVFREQ Driver" depends on (TEGRA_MC && TEGRA20_EMC) || COMPILE_TEST depends on COMMON_CLK select DEVFREQ_GOV_SIMPLE_ONDEMAND - select PM_OPP help This adds the DEVFREQ driver for the Tegra20 family of SoCs. It reads Memory Controller counters and adjusts the operating frequencies and voltages with OPP support. @@ -119,11 +116,10 @@ config ARM_RK3399_DMC_DEVFREQ tristate "ARM RK3399 DMC DEVFREQ Driver" depends on ARCH_ROCKCHIP select DEVFREQ_EVENT_ROCKCHIP_DFI select DEVFREQ_GOV_SIMPLE_ONDEMAND select PM_DEVFREQ_EVENT - select PM_OPP help This adds the DEVFREQ driver for the RK3399 DMC(Dynamic Memory Controller). It sets the frequency for the memory controller and reads the usage counts from hardware. -- 2.17.1 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] PM / devfreq: Kconfig: Drop explicit selection of PM_OPP 2019-11-11 18:16 ` [PATCH] PM / devfreq: Kconfig: Drop explicit selection of PM_OPP Leonard Crestez @ 2019-11-12 0:40 ` Chanwoo Choi 0 siblings, 0 replies; 2+ messages in thread From: Chanwoo Choi @ 2019-11-12 0:40 UTC (permalink / raw) To: Leonard Crestez, MyungJoo Ham, Kyungmin Park Cc: Krzysztof Kozlowski, Rafael J. Wysocki, Viresh Kumar, Enric Balletbo i Serra, Dmitry Osipenko, linux-pm On 11/12/19 3:16 AM, Leonard Crestez wrote: > CONFIG_PM_OPP is already selected by CONFIG_PM_DEVFREQ since > commit b9c69e043266 ("PM / devfreq: Add dependency on PM_OPP") > This means that individual drivers shouldn't "select PM_OPP" explicitly. > > Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com> > > --- > Spawned by review comments on a new driver: > https://patchwork.kernel.org/patch/11235695/#22992159 > > This patch shouldn't have any dependencies. > > drivers/devfreq/Kconfig | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig > index defe1d438710..066e6c4efaa2 100644 > --- a/drivers/devfreq/Kconfig > +++ b/drivers/devfreq/Kconfig > @@ -81,11 +81,10 @@ config ARM_EXYNOS_BUS_DEVFREQ > depends on ARCH_EXYNOS || COMPILE_TEST > select DEVFREQ_GOV_SIMPLE_ONDEMAND > select DEVFREQ_GOV_PASSIVE > select DEVFREQ_EVENT_EXYNOS_PPMU > select PM_DEVFREQ_EVENT > - select PM_OPP > help > This adds the common DEVFREQ driver for Exynos Memory bus. Exynos > Memory bus has one more group of memory bus (e.g, MIF and INT block). > Each memory bus group could contain many memoby bus block. It reads > PPMU counters of memory controllers by using DEVFREQ-event device > @@ -96,22 +95,20 @@ config ARM_TEGRA_DEVFREQ > tristate "NVIDIA Tegra30/114/124/210 DEVFREQ Driver" > depends on ARCH_TEGRA_3x_SOC || ARCH_TEGRA_114_SOC || \ > ARCH_TEGRA_132_SOC || ARCH_TEGRA_124_SOC || \ > ARCH_TEGRA_210_SOC || \ > COMPILE_TEST > - select PM_OPP > help > This adds the DEVFREQ driver for the Tegra family of SoCs. > It reads ACTMON counters of memory controllers and adjusts the > operating frequencies and voltages with OPP support. > > config ARM_TEGRA20_DEVFREQ > tristate "NVIDIA Tegra20 DEVFREQ Driver" > depends on (TEGRA_MC && TEGRA20_EMC) || COMPILE_TEST > depends on COMMON_CLK > select DEVFREQ_GOV_SIMPLE_ONDEMAND > - select PM_OPP > help > This adds the DEVFREQ driver for the Tegra20 family of SoCs. > It reads Memory Controller counters and adjusts the operating > frequencies and voltages with OPP support. > > @@ -119,11 +116,10 @@ config ARM_RK3399_DMC_DEVFREQ > tristate "ARM RK3399 DMC DEVFREQ Driver" > depends on ARCH_ROCKCHIP > select DEVFREQ_EVENT_ROCKCHIP_DFI > select DEVFREQ_GOV_SIMPLE_ONDEMAND > select PM_DEVFREQ_EVENT > - select PM_OPP > help > This adds the DEVFREQ driver for the RK3399 DMC(Dynamic Memory Controller). > It sets the frequency for the memory controller and reads the usage counts > from hardware. > > Applied it. Thanks. -- Best Regards, Chanwoo Choi Samsung Electronics ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-11-12 0:34 UTC | newest] Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <CGME20191111181617epcas4p37a34e64e080e11e7f14754dd8dce7d27@epcas4p3.samsung.com> 2019-11-11 18:16 ` [PATCH] PM / devfreq: Kconfig: Drop explicit selection of PM_OPP Leonard Crestez 2019-11-12 0:40 ` Chanwoo Choi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).