| Message ID | 1400054637-21505-6-git-send-email-chander.kashyap@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Chander, On 14.05.2014 10:03, Chander Kashyap wrote: > Exynos5420 is big.Little Soc. It uses cpuidle-big-litle generic cpuidle driver. > Hence do not allow exynos cpuidle driver registration for Exynos5420. > > Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org> > Signed-off-by: Chander Kashyap <k.chander@samsung.com> > Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> > --- > arch/arm/mach-exynos/cpuidle.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/arm/mach-exynos/cpuidle.c b/arch/arm/mach-exynos/cpuidle.c > index 3dd385e..807a386 100644 > --- a/arch/arm/mach-exynos/cpuidle.c > +++ b/arch/arm/mach-exynos/cpuidle.c > @@ -218,6 +218,9 @@ static int exynos_cpuidle_probe(struct platform_device *pdev) > int cpu_id, ret; > struct cpuidle_device *device; > > + if (soc_is_exynos5420()) > + return -ENODEV; > + > if (soc_is_exynos5250()) > exynos5_core_down_clk(); > > Why not put this in exynos.c in exynos_dt_machine_init(), so exynos_cpuidle_init() is called only if not running on Exynos5420? Best regards, Tomasz
Hi Tomasz, On 16 May 2014 02:56, Tomasz Figa <tomasz.figa@gmail.com> wrote: > Hi Chander, > > On 14.05.2014 10:03, Chander Kashyap wrote: >> Exynos5420 is big.Little Soc. It uses cpuidle-big-litle generic cpuidle driver. >> Hence do not allow exynos cpuidle driver registration for Exynos5420. >> >> Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org> >> Signed-off-by: Chander Kashyap <k.chander@samsung.com> >> Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org> >> --- >> arch/arm/mach-exynos/cpuidle.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/arch/arm/mach-exynos/cpuidle.c b/arch/arm/mach-exynos/cpuidle.c >> index 3dd385e..807a386 100644 >> --- a/arch/arm/mach-exynos/cpuidle.c >> +++ b/arch/arm/mach-exynos/cpuidle.c >> @@ -218,6 +218,9 @@ static int exynos_cpuidle_probe(struct platform_device *pdev) >> int cpu_id, ret; >> struct cpuidle_device *device; >> >> + if (soc_is_exynos5420()) >> + return -ENODEV; >> + >> if (soc_is_exynos5250()) >> exynos5_core_down_clk(); >> >> > > Why not put this in exynos.c in exynos_dt_machine_init(), so > exynos_cpuidle_init() is called only if not running on Exynos5420? > That makes more sense. I will update this Thanks > Best regards, > Tomasz
Exynos5420 is a big-little Soc from Samsung. It has 4 A15 and 4 A7 cores.
This patchset adds cpuidle support for Exynos5420 SoC based on
generic big.little cpuidle driver.
Tested on SMDK5420.
This patch set depends on:
1. [PATCH 0/5] MCPM backend for Exynos5420
http://www.spinics.net/lists/arm-kernel/msg331100.html
Changelog is in respective patches.
Chander Kashyap (5):
driver: cpuidle-big-little: add of_device_id structure
arm: exynos: add generic function to calculate cpu number
cpuidle: config: Add ARCH_EXYNOS entry to select cpuidle-big-little
driver
driver: cpuidle: cpuidle-big-little: init driver for Exynos5420
exynos: cpuidle: do not allow cpuidle registration for Exynos5420
mcpm: exynos: populate suspend and powered_up callbacks
arch/arm/mach-exynos/exynos.c | 4 +++-
arch/arm/mach-exynos/mcpm-exynos.c | 36 ++++++++++++++++++++++++++++++++++
arch/arm/mach-exynos/regs-pmu.h | 9 +++++++++
drivers/cpuidle/Kconfig.arm | 2 +-
drivers/cpuidle/cpuidle-big_little.c | 12 +++++++++++-
5 files changed, 60 insertions(+), 3 deletions(-)
Hi Daniel/Kgene, On 16 May 2014 13:33, Chander Kashyap <chander.kashyap@linaro.org> wrote: > Exynos5420 is a big-little Soc from Samsung. It has 4 A15 and 4 A7 cores. > > This patchset adds cpuidle support for Exynos5420 SoC based on > generic big.little cpuidle driver. > > Tested on SMDK5420. > > This patch set depends on: > 1. [PATCH 0/5] MCPM backend for Exynos5420 > http://www.spinics.net/lists/arm-kernel/msg331100.html > Changelog is in respective patches. > Chander Kashyap (5): > driver: cpuidle-big-little: add of_device_id structure > arm: exynos: add generic function to calculate cpu number > cpuidle: config: Add ARCH_EXYNOS entry to select cpuidle-big-little > driver > driver: cpuidle: cpuidle-big-little: init driver for Exynos5420 > exynos: cpuidle: do not allow cpuidle registration for Exynos5420 > mcpm: exynos: populate suspend and powered_up callbacks > > arch/arm/mach-exynos/exynos.c | 4 +++- > arch/arm/mach-exynos/mcpm-exynos.c | 36 ++++++++++++++++++++++++++++++++++ > arch/arm/mach-exynos/regs-pmu.h | 9 +++++++++ > drivers/cpuidle/Kconfig.arm | 2 +- > drivers/cpuidle/cpuidle-big_little.c | 12 +++++++++++- > 5 files changed, 60 insertions(+), 3 deletions(-) > > -- > 1.7.9.5 > As dependency patches are merged. If their are no further comment, can these patches be taken?
On 19 May 2014 11:10, Chander Kashyap <chander.kashyap@linaro.org> wrote: > Hi Daniel/Kgene, > > On 16 May 2014 13:33, Chander Kashyap <chander.kashyap@linaro.org> wrote: >> Exynos5420 is a big-little Soc from Samsung. It has 4 A15 and 4 A7 cores. >> >> This patchset adds cpuidle support for Exynos5420 SoC based on >> generic big.little cpuidle driver. >> >> Tested on SMDK5420. >> >> This patch set depends on: >> 1. [PATCH 0/5] MCPM backend for Exynos5420 >> http://www.spinics.net/lists/arm-kernel/msg331100.html >> Changelog is in respective patches. >> Chander Kashyap (5): >> driver: cpuidle-big-little: add of_device_id structure >> arm: exynos: add generic function to calculate cpu number >> cpuidle: config: Add ARCH_EXYNOS entry to select cpuidle-big-little >> driver >> driver: cpuidle: cpuidle-big-little: init driver for Exynos5420 >> exynos: cpuidle: do not allow cpuidle registration for Exynos5420 >> mcpm: exynos: populate suspend and powered_up callbacks >> >> arch/arm/mach-exynos/exynos.c | 4 +++- >> arch/arm/mach-exynos/mcpm-exynos.c | 36 ++++++++++++++++++++++++++++++++++ >> arch/arm/mach-exynos/regs-pmu.h | 9 +++++++++ >> drivers/cpuidle/Kconfig.arm | 2 +- >> drivers/cpuidle/cpuidle-big_little.c | 12 +++++++++++- >> 5 files changed, 60 insertions(+), 3 deletions(-) >> >> -- >> 1.7.9.5 >> > > As dependency patches are merged. If their are no further comment, can > these patches be taken? > ping > -- > with warm regards, > Chander Kashyap
Hi Chander, On 16.05.2014 10:03, Chander Kashyap wrote: > Exynos5420 is a big-little Soc from Samsung. It has 4 A15 and 4 A7 cores. > > This patchset adds cpuidle support for Exynos5420 SoC based on > generic big.little cpuidle driver. > > Tested on SMDK5420. > > This patch set depends on: > 1. [PATCH 0/5] MCPM backend for Exynos5420 > http://www.spinics.net/lists/arm-kernel/msg331100.html > Changelog is in respective patches. > Chander Kashyap (5): > driver: cpuidle-big-little: add of_device_id structure > arm: exynos: add generic function to calculate cpu number > cpuidle: config: Add ARCH_EXYNOS entry to select cpuidle-big-little > driver > driver: cpuidle: cpuidle-big-little: init driver for Exynos5420 > exynos: cpuidle: do not allow cpuidle registration for Exynos5420 > mcpm: exynos: populate suspend and powered_up callbacks > > arch/arm/mach-exynos/exynos.c | 4 +++- > arch/arm/mach-exynos/mcpm-exynos.c | 36 ++++++++++++++++++++++++++++++++++ > arch/arm/mach-exynos/regs-pmu.h | 9 +++++++++ > drivers/cpuidle/Kconfig.arm | 2 +- > drivers/cpuidle/cpuidle-big_little.c | 12 +++++++++++- > 5 files changed, 60 insertions(+), 3 deletions(-) > For the whole series, Reviewed-by: Tomasz Figa <t.figa@samsung.com> Best regards, Tomasz
On 26 May 2014 15:59, Tomasz Figa <tomasz.figa@gmail.com> wrote: > Hi Chander, > > On 16.05.2014 10:03, Chander Kashyap wrote: >> Exynos5420 is a big-little Soc from Samsung. It has 4 A15 and 4 A7 cores. >> >> This patchset adds cpuidle support for Exynos5420 SoC based on >> generic big.little cpuidle driver. >> >> Tested on SMDK5420. >> >> This patch set depends on: >> 1. [PATCH 0/5] MCPM backend for Exynos5420 >> http://www.spinics.net/lists/arm-kernel/msg331100.html >> Changelog is in respective patches. >> Chander Kashyap (5): >> driver: cpuidle-big-little: add of_device_id structure >> arm: exynos: add generic function to calculate cpu number >> cpuidle: config: Add ARCH_EXYNOS entry to select cpuidle-big-little >> driver >> driver: cpuidle: cpuidle-big-little: init driver for Exynos5420 >> exynos: cpuidle: do not allow cpuidle registration for Exynos5420 >> mcpm: exynos: populate suspend and powered_up callbacks >> >> arch/arm/mach-exynos/exynos.c | 4 +++- >> arch/arm/mach-exynos/mcpm-exynos.c | 36 ++++++++++++++++++++++++++++++++++ >> arch/arm/mach-exynos/regs-pmu.h | 9 +++++++++ >> drivers/cpuidle/Kconfig.arm | 2 +- >> drivers/cpuidle/cpuidle-big_little.c | 12 +++++++++++- >> 5 files changed, 60 insertions(+), 3 deletions(-) >> > > For the whole series, > > Reviewed-by: Tomasz Figa <t.figa@samsung.com> Thanks Tomasz. Dear Kukjin, Can you take these patches. > > Best regards, > Tomasz
Chander Kashyap wrote: > > On 26 May 2014 15:59, Tomasz Figa <tomasz.figa@gmail.com> wrote: > > Hi Chander, > > > > On 16.05.2014 10:03, Chander Kashyap wrote: > >> Exynos5420 is a big-little Soc from Samsung. It has 4 A15 and 4 A7 > cores. > >> > >> This patchset adds cpuidle support for Exynos5420 SoC based on > >> generic big.little cpuidle driver. > >> > >> Tested on SMDK5420. > >> > >> This patch set depends on: > >> 1. [PATCH 0/5] MCPM backend for Exynos5420 > >> http://www.spinics.net/lists/arm-kernel/msg331100.html > >> Changelog is in respective patches. > >> Chander Kashyap (5): > >> driver: cpuidle-big-little: add of_device_id structure > >> arm: exynos: add generic function to calculate cpu number > >> cpuidle: config: Add ARCH_EXYNOS entry to select cpuidle-big-little > >> driver > >> driver: cpuidle: cpuidle-big-little: init driver for Exynos5420 > >> exynos: cpuidle: do not allow cpuidle registration for Exynos5420 > >> mcpm: exynos: populate suspend and powered_up callbacks > >> > >> arch/arm/mach-exynos/exynos.c | 4 +++- > >> arch/arm/mach-exynos/mcpm-exynos.c | 36 > ++++++++++++++++++++++++++++++++++ > >> arch/arm/mach-exynos/regs-pmu.h | 9 +++++++++ > >> drivers/cpuidle/Kconfig.arm | 2 +- > >> drivers/cpuidle/cpuidle-big_little.c | 12 +++++++++++- > >> 5 files changed, 60 insertions(+), 3 deletions(-) > >> > > > > For the whole series, > > > > Reviewed-by: Tomasz Figa <t.figa@samsung.com> > > Thanks Tomasz. > > Dear Kukjin, > Can you take these patches. > > When I looked at this series quickly, looks good to me but I need to get ack from cpuidle maintainer Rafael or Daniel. Thanks, Kukjin
On 05/28/2014 06:35 AM, Kukjin Kim wrote: > Chander Kashyap wrote: >> >> On 26 May 2014 15:59, Tomasz Figa <tomasz.figa@gmail.com> wrote: >>> Hi Chander, >>> >>> On 16.05.2014 10:03, Chander Kashyap wrote: >>>> Exynos5420 is a big-little Soc from Samsung. It has 4 A15 and 4 A7 >> cores. >>>> >>>> This patchset adds cpuidle support for Exynos5420 SoC based on >>>> generic big.little cpuidle driver. >>>> >>>> Tested on SMDK5420. >>>> >>>> This patch set depends on: >>>> 1. [PATCH 0/5] MCPM backend for Exynos5420 >>>> http://www.spinics.net/lists/arm-kernel/msg331100.html >>>> Changelog is in respective patches. >>>> Chander Kashyap (5): >>>> driver: cpuidle-big-little: add of_device_id structure >>>> arm: exynos: add generic function to calculate cpu number >>>> cpuidle: config: Add ARCH_EXYNOS entry to select cpuidle-big-little >>>> driver >>>> driver: cpuidle: cpuidle-big-little: init driver for Exynos5420 >>>> exynos: cpuidle: do not allow cpuidle registration for Exynos5420 >>>> mcpm: exynos: populate suspend and powered_up callbacks >>>> >>>> arch/arm/mach-exynos/exynos.c | 4 +++- >>>> arch/arm/mach-exynos/mcpm-exynos.c | 36 >> ++++++++++++++++++++++++++++++++++ >>>> arch/arm/mach-exynos/regs-pmu.h | 9 +++++++++ >>>> drivers/cpuidle/Kconfig.arm | 2 +- >>>> drivers/cpuidle/cpuidle-big_little.c | 12 +++++++++++- >>>> 5 files changed, 60 insertions(+), 3 deletions(-) >>>> >>> >>> For the whole series, >>> >>> Reviewed-by: Tomasz Figa <t.figa@samsung.com> >> >> Thanks Tomasz. >> >> Dear Kukjin, >> Can you take these patches. >>> > When I looked at this series quickly, looks good to me but I need to get ack from cpuidle maintainer Rafael or Daniel. Acked the different cpuidle bits. Thanks -- Daniel
On 28 May 2014 14:32, Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > On 05/28/2014 06:35 AM, Kukjin Kim wrote: >> >> Chander Kashyap wrote: >>> >>> >>> On 26 May 2014 15:59, Tomasz Figa <tomasz.figa@gmail.com> wrote: >>>> >>>> Hi Chander, >>>> >>>> On 16.05.2014 10:03, Chander Kashyap wrote: >>>>> >>>>> Exynos5420 is a big-little Soc from Samsung. It has 4 A15 and 4 A7 >>> >>> cores. >>>>> >>>>> >>>>> This patchset adds cpuidle support for Exynos5420 SoC based on >>>>> generic big.little cpuidle driver. >>>>> >>>>> Tested on SMDK5420. >>>>> >>>>> This patch set depends on: >>>>> 1. [PATCH 0/5] MCPM backend for Exynos5420 >>>>> http://www.spinics.net/lists/arm-kernel/msg331100.html >>>>> Changelog is in respective patches. >>>>> Chander Kashyap (5): >>>>> driver: cpuidle-big-little: add of_device_id structure >>>>> arm: exynos: add generic function to calculate cpu number >>>>> cpuidle: config: Add ARCH_EXYNOS entry to select cpuidle-big-little >>>>> driver >>>>> driver: cpuidle: cpuidle-big-little: init driver for Exynos5420 >>>>> exynos: cpuidle: do not allow cpuidle registration for Exynos5420 >>>>> mcpm: exynos: populate suspend and powered_up callbacks >>>>> >>>>> arch/arm/mach-exynos/exynos.c | 4 +++- >>>>> arch/arm/mach-exynos/mcpm-exynos.c | 36 >>> >>> ++++++++++++++++++++++++++++++++++ >>>>> >>>>> arch/arm/mach-exynos/regs-pmu.h | 9 +++++++++ >>>>> drivers/cpuidle/Kconfig.arm | 2 +- >>>>> drivers/cpuidle/cpuidle-big_little.c | 12 +++++++++++- >>>>> 5 files changed, 60 insertions(+), 3 deletions(-) >>>>> >>>> >>>> For the whole series, >>>> >>>> Reviewed-by: Tomasz Figa <t.figa@samsung.com> >>> >>> >>> Thanks Tomasz. >>> >>> Dear Kukjin, >>> Can you take these patches. >>>> >>>> >> When I looked at this series quickly, looks good to me but I need to get >> ack from cpuidle maintainer Rafael or Daniel. > > > Acked the different cpuidle bits. Hi Kukjin, Can you take it now? > > Thanks > -- Daniel > > > -- > <http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs > > Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | > <http://twitter.com/#!/linaroorg> Twitter | > <http://www.linaro.org/linaro-blog/> Blog >
On Thu, May 29, 2014 at 10:07 AM, Chander Kashyap <chander.kashyap@linaro.org> wrote: > On 28 May 2014 14:32, Daniel Lezcano <daniel.lezcano@linaro.org> wrote: >> On 05/28/2014 06:35 AM, Kukjin Kim wrote: >>> >>> Chander Kashyap wrote: >>>> >>>> >>>> On 26 May 2014 15:59, Tomasz Figa <tomasz.figa@gmail.com> wrote: >>>>> >>>>> Hi Chander, >>>>> >>>>> On 16.05.2014 10:03, Chander Kashyap wrote: >>>>>> >>>>>> Exynos5420 is a big-little Soc from Samsung. It has 4 A15 and 4 A7 >>>> >>>> cores. >>>>>> >>>>>> >>>>>> This patchset adds cpuidle support for Exynos5420 SoC based on >>>>>> generic big.little cpuidle driver. >>>>>> >>>>>> Tested on SMDK5420. >>>>>> >>>>>> This patch set depends on: >>>>>> 1. [PATCH 0/5] MCPM backend for Exynos5420 >>>>>> http://www.spinics.net/lists/arm-kernel/msg331100.html >>>>>> Changelog is in respective patches. >>>>>> Chander Kashyap (5): >>>>>> driver: cpuidle-big-little: add of_device_id structure >>>>>> arm: exynos: add generic function to calculate cpu number >>>>>> cpuidle: config: Add ARCH_EXYNOS entry to select cpuidle-big-little >>>>>> driver >>>>>> driver: cpuidle: cpuidle-big-little: init driver for Exynos5420 >>>>>> exynos: cpuidle: do not allow cpuidle registration for Exynos5420 >>>>>> mcpm: exynos: populate suspend and powered_up callbacks >>>>>> >>>>>> arch/arm/mach-exynos/exynos.c | 4 +++- >>>>>> arch/arm/mach-exynos/mcpm-exynos.c | 36 >>>> >>>> ++++++++++++++++++++++++++++++++++ >>>>>> >>>>>> arch/arm/mach-exynos/regs-pmu.h | 9 +++++++++ >>>>>> drivers/cpuidle/Kconfig.arm | 2 +- >>>>>> drivers/cpuidle/cpuidle-big_little.c | 12 +++++++++++- >>>>>> 5 files changed, 60 insertions(+), 3 deletions(-) >>>>>> >>>>> >>>>> For the whole series, >>>>> >>>>> Reviewed-by: Tomasz Figa <t.figa@samsung.com> >>>> >>>> >>>> Thanks Tomasz. >>>> >>>> Dear Kukjin, >>>> Can you take these patches. >>>>> >>>>> >>> When I looked at this series quickly, looks good to me but I need to get >>> ack from cpuidle maintainer Rafael or Daniel. >> >> >> Acked the different cpuidle bits. > > Hi Kukjin, > Can you take it now? ping Sorry for previous html messages. > > >> >> Thanks >> -- Daniel >> >> >> -- >> <http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs >> >> Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | >> <http://twitter.com/#!/linaroorg> Twitter | >> <http://www.linaro.org/linaro-blog/> Blog >> > > > > -- > with warm regards, > Chander Kashyap
diff --git a/arch/arm/mach-exynos/cpuidle.c b/arch/arm/mach-exynos/cpuidle.c index 3dd385e..807a386 100644 --- a/arch/arm/mach-exynos/cpuidle.c +++ b/arch/arm/mach-exynos/cpuidle.c @@ -218,6 +218,9 @@ static int exynos_cpuidle_probe(struct platform_device *pdev) int cpu_id, ret; struct cpuidle_device *device; + if (soc_is_exynos5420()) + return -ENODEV; + if (soc_is_exynos5250()) exynos5_core_down_clk();