| Message ID | 7182e308a552d37064f61e724c709667a6fa098d.1502861444.git.viresh.kumar@linaro.org |
|---|---|
| State | Accepted |
| Commit | d79d148b6451307fd46e30673ce9de28690c8ced |
| Headers | show |
| Series | None | expand |
On Wed, Aug 16, 2017 at 08:51:04AM +0200, Geert Uytterhoeven wrote: > Hi Viresh, > > On Wed, Aug 16, 2017 at 7:37 AM, Viresh Kumar <viresh.kumar@linaro.org> wrote: > > Drop few ARM (32 and 64 bit) platforms from the whitelist which always > > use "operating-points-v2" property from their DT. They should continue > > to work after this patch. > > > > Tested on Hikey platform (only the "hisilicon,hi6220" entry). > > > > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > > --- > > drivers/cpufreq/cpufreq-dt-platdev.c | 11 ----------- > > 1 file changed, 11 deletions(-) > > > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c > > index 061b468512a2..45f2ec3b7f7a 100644 > > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > > @@ -27,7 +27,6 @@ static const struct of_device_id whitelist[] __initconst = { > > { .compatible = "allwinner,sun6i-a31s", }, > > { .compatible = "allwinner,sun7i-a20", }, > > { .compatible = "allwinner,sun8i-a23", }, > > - { .compatible = "allwinner,sun8i-a33", }, > > { .compatible = "allwinner,sun8i-a83t", }, > > { .compatible = "allwinner,sun8i-h3", }, > > I think "renesas,r8a7795" can be removed again, too. > Simon? I assume you are referring to "[PATCH 1/2] cpufreq: dt-platdev: Automatically create cpufreq device with OPP v2" (which you kindly forwarded me). If so, yes, that seems likely. I'll do some testing to confirm this.
On Thu, Aug 17, 2017 at 09:53:01AM +0200, Simon Horman wrote: > On Wed, Aug 16, 2017 at 08:51:04AM +0200, Geert Uytterhoeven wrote: > > Hi Viresh, > > > > On Wed, Aug 16, 2017 at 7:37 AM, Viresh Kumar <viresh.kumar@linaro.org> wrote: > > > Drop few ARM (32 and 64 bit) platforms from the whitelist which always > > > use "operating-points-v2" property from their DT. They should continue > > > to work after this patch. > > > > > > Tested on Hikey platform (only the "hisilicon,hi6220" entry). > > > > > > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > > > --- > > > drivers/cpufreq/cpufreq-dt-platdev.c | 11 ----------- > > > 1 file changed, 11 deletions(-) > > > > > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c > > > index 061b468512a2..45f2ec3b7f7a 100644 > > > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > > > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > > > @@ -27,7 +27,6 @@ static const struct of_device_id whitelist[] __initconst = { > > > { .compatible = "allwinner,sun6i-a31s", }, > > > { .compatible = "allwinner,sun7i-a20", }, > > > { .compatible = "allwinner,sun8i-a23", }, > > > - { .compatible = "allwinner,sun8i-a33", }, > > > { .compatible = "allwinner,sun8i-a83t", }, > > > { .compatible = "allwinner,sun8i-h3", }, > > > > I think "renesas,r8a7795" can be removed again, too. > > Simon? > > I assume you are referring to "[PATCH 1/2] cpufreq: dt-platdev: > Automatically create cpufreq device with OPP v2" (which you kindly > forwarded me). If so, yes, that seems likely. I'll do some testing to > confirm this. I have now done some testing. With "[PATCH 1/2]" applied the following can be removed: "renesas,r8a7795" "renesas,r8a7796"
diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c index 061b468512a2..45f2ec3b7f7a 100644 --- a/drivers/cpufreq/cpufreq-dt-platdev.c +++ b/drivers/cpufreq/cpufreq-dt-platdev.c @@ -27,7 +27,6 @@ static const struct of_device_id whitelist[] __initconst = { { .compatible = "allwinner,sun6i-a31s", }, { .compatible = "allwinner,sun7i-a20", }, { .compatible = "allwinner,sun8i-a23", }, - { .compatible = "allwinner,sun8i-a33", }, { .compatible = "allwinner,sun8i-a83t", }, { .compatible = "allwinner,sun8i-h3", }, @@ -37,7 +36,6 @@ static const struct of_device_id whitelist[] __initconst = { { .compatible = "arm,integrator-cp", }, { .compatible = "hisilicon,hi3660", }, - { .compatible = "hisilicon,hi6220", }, { .compatible = "fsl,imx27", }, { .compatible = "fsl,imx51", }, @@ -51,11 +49,8 @@ static const struct of_device_id whitelist[] __initconst = { { .compatible = "samsung,exynos3250", }, { .compatible = "samsung,exynos4210", }, { .compatible = "samsung,exynos4212", }, - { .compatible = "samsung,exynos4412", }, { .compatible = "samsung,exynos5250", }, #ifndef CONFIG_BL_SWITCHER - { .compatible = "samsung,exynos5420", }, - { .compatible = "samsung,exynos5433", }, { .compatible = "samsung,exynos5800", }, #endif @@ -87,11 +82,7 @@ static const struct of_device_id whitelist[] __initconst = { { .compatible = "rockchip,rk3368", }, { .compatible = "rockchip,rk3399", }, - { .compatible = "socionext,uniphier-pro5", }, - { .compatible = "socionext,uniphier-pxs2", }, { .compatible = "socionext,uniphier-ld6b", }, - { .compatible = "socionext,uniphier-ld11", }, - { .compatible = "socionext,uniphier-ld20", }, { .compatible = "ti,omap2", }, { .compatible = "ti,omap3", }, @@ -101,8 +92,6 @@ static const struct of_device_id whitelist[] __initconst = { { .compatible = "xlnx,zynq-7000", }, { .compatible = "xlnx,zynqmp", }, - { .compatible = "zte,zx296718", }, - { } };
Drop few ARM (32 and 64 bit) platforms from the whitelist which always use "operating-points-v2" property from their DT. They should continue to work after this patch. Tested on Hikey platform (only the "hisilicon,hi6220" entry). Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> --- drivers/cpufreq/cpufreq-dt-platdev.c | 11 ----------- 1 file changed, 11 deletions(-) -- 2.7.4