Re: [PATCH v7 0/6] cpufreq: use generic cpufreq drivers for exynos platforms

[Thomas Abraham <ta.omasab@gmail.com>]

Hi Mike,

On Tue, Jul 15, 2014 at 9:20 AM, Thomas Abraham <ta.omasab@gmail.com> wrote:
> Hi Tomasz,
>
> On Mon, Jul 14, 2014 at 7:08 PM, Thomas Abraham <thomas.ab@samsung.com> wrote:
>> Changes since v6:
>> - Fixes suggested by Amit Daniel <amit.daniel@samsung.com>.
>>
>> This patch series removes the use of Exynos4210 and Exynos5250 specific cpufreq
>> drivers and enables the use of cpufreq-cpu0 driver for these platforms. This
>> series also enabled cpufreq support for Exynos5420 using arm_big_little cpufreq
>> driver.
>>
>> Thomas Abraham (6):
>>   clk: samsung: add infrastructure to register cpu clocks
>>   clk: samsung: register exynos5420 apll/kpll configuration data
>>   clk: exynos: use cpu-clock provider type to represent arm clock
>>   ARM: dts: Exynos: add cpu nodes, opp and cpu clock configuration data
>>   ARM: Exynos: switch to using generic cpufreq driver for exynos4210/5250
>>   cpufreq: exynos: remove exynos4210/5250 specific cpufreq driver support
>
> In this series, support for Exynos5420 has been included. If there are
> any more changes required, could you please let me know.

This patch series migrates exynos cpufreq support to use generic
cpufreq drivers. For this, cpu clock blocks are encapsulated into a
cpu clock type and cpufreq driver operates the cpu clock type.

We had discussed in the [1] about using coordinated clocks. This
series is not using the concept of coordinated clocks but the code in
this series can be migrated to use coordinated clocks when it is
available.

Is it okay to consider this series for v3.17-rc1. I will update this
code to use coordinated clocks when it is available. This series helps
with cpufreq support for newer Exynos SoCs such as Exynos5420/3250.

[1] http://www.spinics.net/lists/cpufreq/msg10042.html

Thanks,
Thomas

>
> Thanks,
> Thomas.
>
>>
>>  arch/arm/boot/dts/exynos4210-origen.dts         |    6 +
>>  arch/arm/boot/dts/exynos4210-trats.dts          |    6 +
>>  arch/arm/boot/dts/exynos4210-universal_c210.dts |    6 +
>>  arch/arm/boot/dts/exynos4210.dtsi               |   27 ++
>>  arch/arm/boot/dts/exynos5250-arndale.dts        |    6 +
>>  arch/arm/boot/dts/exynos5250-cros-common.dtsi   |    6 +
>>  arch/arm/boot/dts/exynos5250-smdk5250.dts       |    6 +
>>  arch/arm/boot/dts/exynos5250.dtsi               |   23 +
>>  arch/arm/boot/dts/exynos5420-smdk5420.dts       |    6 +
>>  arch/arm/boot/dts/exynos5420.dtsi               |   32 ++
>>  arch/arm/mach-exynos/exynos.c                   |   15 +-
>>  drivers/clk/samsung/Makefile                    |    2 +-
>>  drivers/clk/samsung/clk-cpu.c                   |  576 +++++++++++++++++++++++
>>  drivers/clk/samsung/clk-exynos4.c               |   25 +-
>>  drivers/clk/samsung/clk-exynos5250.c            |   16 +-
>>  drivers/clk/samsung/clk-exynos5420.c            |   60 ++-
>>  drivers/clk/samsung/clk.h                       |    5 +
>>  drivers/cpufreq/Kconfig.arm                     |   22 -
>>  drivers/cpufreq/Makefile                        |    2 -
>>  drivers/cpufreq/exynos4210-cpufreq.c            |  184 --------
>>  drivers/cpufreq/exynos5250-cpufreq.c            |  210 ---------
>>  include/dt-bindings/clock/exynos5250.h          |    1 +
>>  include/dt-bindings/clock/exynos5420.h          |    2 +
>>  23 files changed, 802 insertions(+), 442 deletions(-)
>>  create mode 100644 drivers/clk/samsung/clk-cpu.c
>>  delete mode 100644 drivers/cpufreq/exynos4210-cpufreq.c
>>  delete mode 100644 drivers/cpufreq/exynos5250-cpufreq.c
>>
>> --
>> 1.7.9.5
>>