Re: [PATCH 3/3] cpufreq: exynos: Add exynos5420 cpufreq driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 6 December 2013 15:18, Arun Kumar K <arun.kk@xxxxxxxxxxx> wrote:
> From: "Arjun.K.V" <arjun.kv@xxxxxxxxxxx>
>
> The patch adds cpufreq driver for exynos5420.
>
> Signed-off-by: Arjun.K.V <arjun.kv@xxxxxxxxxxx>
> Signed-off-by: Andrew Bresticker <abrestic@xxxxxxxxxxxx>
> Signed-off-by: Arun Kumar K <arun.kk@xxxxxxxxxxx>
> ---
>  drivers/cpufreq/Kconfig.arm          |   11 ++
>  drivers/cpufreq/Makefile             |    1 +
>  drivers/cpufreq/exynos-cpufreq.c     |    2 +
>  drivers/cpufreq/exynos-cpufreq.h     |    8 +
>  drivers/cpufreq/exynos5420-cpufreq.c |  346 ++++++++++++++++++++++++++++++++++
>  5 files changed, 368 insertions(+)
>  create mode 100644 drivers/cpufreq/exynos5420-cpufreq.c

Hmm.. That's the second big LITTLE SoC after TC2..
And we already have a generic enough driver for all big LITTLE platforms:
arm_big_little.c

Please use that with a glue clk and regulator driver if required. We don't
want to get any new driver for big LITTLE unless there is a real limitation.

Also, the generic big LITTLE driver supports IKS (In Kernel Switcher)
which may or maynot be used by Exynos.

@Lukasz: We can't use cpu0 driver here as we have multiple clusters here
A7 and A15, and so there are two clock domains instead of one. That's why
we have written big LITTLE driver.

@Arun: And you logs must be a bit enhanced as well.. I knew 5420 is big
LITTLE but others may not.. And hence please provide a summary of your
platform whenever you add new drivers..

--
viresh
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux