Re: [PATCH V2 5/9] cpufreq: rockchip: Use generic platdev driver

From: Heiko Stübner
Date: Fri Apr 22 2016 - 09:58:37 EST


Am Freitag, 22. April 2016, 16:58:43 schrieb Viresh Kumar:
> From: Finley Xiao <finley.xiao@xxxxxxxxxxxxxx>
>
> This patch add rockchip's compatible string to the compat list and
> remove similar code from platform code for supporting generic platdev
> driver.
>
> Signed-off-by: Finley Xiao <finley.xiao@xxxxxxxxxxxxxx>
> Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
> Acked-by: Arnd Bergmann <arnd@xxxxxxxx>
> Signed-off-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

very cool to have that cpufreq stuff move somewhere else

Reviewed-by: Heiko Stuebner <heiko@xxxxxxxxx>

> ---
> arch/arm/mach-rockchip/rockchip.c | 1 -
> drivers/cpufreq/cpufreq-dt-platdev.c | 11 +++++++++++
> 2 files changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-rockchip/rockchip.c
> b/arch/arm/mach-rockchip/rockchip.c index 3f07cc5dfe5f..beb71da5d9c8 100644
> --- a/arch/arm/mach-rockchip/rockchip.c
> +++ b/arch/arm/mach-rockchip/rockchip.c
> @@ -74,7 +74,6 @@ static void __init rockchip_dt_init(void)
> {
> rockchip_suspend_init();
> of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
> - platform_device_register_simple("cpufreq-dt", 0, NULL, 0);
> }
>
> static const char * const rockchip_board_dt_compat[] = {
> diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c
> b/drivers/cpufreq/cpufreq-dt-platdev.c index 4e525f6ec59f..75fa921d8540
> 100644
> --- a/drivers/cpufreq/cpufreq-dt-platdev.c
> +++ b/drivers/cpufreq/cpufreq-dt-platdev.c
> @@ -29,6 +29,17 @@ static const struct of_device_id machines[] __initconst =
> { { .compatible = "samsung,exynos5800", },
> #endif
>
> + { .compatible = "rockchip,rk2928", },
> + { .compatible = "rockchip,rk3036", },
> + { .compatible = "rockchip,rk3066a", },
> + { .compatible = "rockchip,rk3066b", },
> + { .compatible = "rockchip,rk3188", },
> + { .compatible = "rockchip,rk3228", },
> + { .compatible = "rockchip,rk3288", },
> + { .compatible = "rockchip,rk3366", },
> + { .compatible = "rockchip,rk3368", },
> + { .compatible = "rockchip,rk3399", },
> +
> { .compatible = "ti,omap2", },
> { .compatible = "ti,omap3", },
> { .compatible = "ti,omap4", },