| Message ID | 1420511727-8242-2-git-send-email-wens@csie.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi, On Tue, Jan 06, 2015 at 10:35:11AM +0800, Chen-Yu Tsai wrote: > On sun[45678]i, we have one cluster of identical cores sharing a > clock, which is ideal for using cpufreq-dt. Register a platform > device for cpufreq-dt. > > Signed-off-by: Chen-Yu Tsai <wens@csie.org> While I don't have anything against this patch, I seem to recall that it was not necessary anymore, or that there was some work in the direction of removing that kind of code, have you looked into that? Thanks, Maxime
Hi, On Tue, Jan 6, 2015 at 11:55 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > Hi, > > On Tue, Jan 06, 2015 at 10:35:11AM +0800, Chen-Yu Tsai wrote: >> On sun[45678]i, we have one cluster of identical cores sharing a >> clock, which is ideal for using cpufreq-dt. Register a platform >> device for cpufreq-dt. >> >> Signed-off-by: Chen-Yu Tsai <wens@csie.org> > > While I don't have anything against this patch, I seem to recall that > it was not necessary anymore, or that there was some work in the > direction of removing that kind of code, have you looked into that? I believe you are referring to the work done by Viresh Kumar (CCed), specifically the 2 series "[RFC V1 0/8] CPUFreq: create platform-dev for DT based cpufreq drivers" and "[RFC] OPP: Redefine bindings to overcome shortcomings". So far the new code has not appeared in linux-next. If it does, I will happily use it. ChenYu
On 7 January 2015 at 14:37, Chen-Yu Tsai <wens@csie.org> wrote: > I believe you are referring to the work done by Viresh Kumar (CCed), > specifically the 2 series "[RFC V1 0/8] CPUFreq: create platform-dev > for DT based cpufreq drivers" and "[RFC] OPP: Redefine bindings to > overcome shortcomings". So far the new code has not appeared in > linux-next. If it does, I will happily use it. Yes. You still have to create these platform devices for cpufreq-dt.
On Tue, Jan 06, 2015 at 10:35:11AM +0800, Chen-Yu Tsai wrote: > On sun[45678]i, we have one cluster of identical cores sharing a > clock, which is ideal for using cpufreq-dt. Register a platform > device for cpufreq-dt. > > Signed-off-by: Chen-Yu Tsai <wens@csie.org> Applied, thanks! Maxime
diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c index d4bb2395d39c..1bc811a74a9f 100644 --- a/arch/arm/mach-sunxi/sunxi.c +++ b/arch/arm/mach-sunxi/sunxi.c @@ -13,9 +13,15 @@ #include <linux/clk-provider.h> #include <linux/clocksource.h> #include <linux/init.h> +#include <linux/platform_device.h> #include <asm/mach/arch.h> +static void __init sunxi_dt_cpufreq_init(void) +{ + platform_device_register_simple("cpufreq-dt", -1, NULL, 0); +} + static const char * const sunxi_board_dt_compat[] = { "allwinner,sun4i-a10", "allwinner,sun5i-a10s", @@ -25,6 +31,7 @@ static const char * const sunxi_board_dt_compat[] = { DT_MACHINE_START(SUNXI_DT, "Allwinner A1X (Device Tree)") .dt_compat = sunxi_board_dt_compat, + .init_late = sunxi_dt_cpufreq_init, MACHINE_END static const char * const sun6i_board_dt_compat[] = { @@ -45,6 +52,7 @@ static void __init sun6i_timer_init(void) DT_MACHINE_START(SUN6I_DT, "Allwinner sun6i (A31) Family") .init_time = sun6i_timer_init, .dt_compat = sun6i_board_dt_compat, + .init_late = sunxi_dt_cpufreq_init, MACHINE_END static const char * const sun7i_board_dt_compat[] = { @@ -54,6 +62,7 @@ static const char * const sun7i_board_dt_compat[] = { DT_MACHINE_START(SUN7I_DT, "Allwinner sun7i (A20) Family") .dt_compat = sun7i_board_dt_compat, + .init_late = sunxi_dt_cpufreq_init, MACHINE_END static const char * const sun8i_board_dt_compat[] = { @@ -63,6 +72,7 @@ static const char * const sun8i_board_dt_compat[] = { DT_MACHINE_START(SUN8I_DT, "Allwinner sun8i (A23) Family") .dt_compat = sun8i_board_dt_compat, + .init_late = sunxi_dt_cpufreq_init, MACHINE_END static const char * const sun9i_board_dt_compat[] = {
On sun[45678]i, we have one cluster of identical cores sharing a clock, which is ideal for using cpufreq-dt. Register a platform device for cpufreq-dt. Signed-off-by: Chen-Yu Tsai <wens@csie.org> --- arch/arm/mach-sunxi/sunxi.c | 10 ++++++++++ 1 file changed, 10 insertions(+)