mbox

[GIT,PULL,3/5] Samsung exynos cpuidle update for v3.17

Message ID 53C9C159.4070305@samsung.com
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git

Message

Kukjin Kim July 19, 2014, 12:52 a.m. UTC
The following changes since commit 1795cd9b3a91d4b5473c97f491d63892442212ab:

   Linux 3.16-rc5 (2014-07-13 14:04:33 -0700)

are available in the git repository at:

   git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git 
tags/exynos-cpuidle

for you to fetch changes up to fc2cac41ebbfb16da8b036cba6ec6714ab780a6d:

   ARM: EXYNOS: populate suspend and powered_up callbacks for mcpm 
(2014-07-19 03:36:00 +0900)

----------------------------------------------------------------
exynos cpuidle update for v3.17

- add callbacks exynos_suspend() and exynos_powered_up()
   for support cpuidle through mcpm
- skip exynos_cpuidle for exynos5420 because is uses
   cpuidle-big-liggle generic cpuidle driver
- add generic functions to calculate cpu number is used
   for pmu and this is required for exynos5420 multi-cluster
- add of_device_id structure for big.LITTLE cpuidle and
   add "samsung,exynos5420" compatible string for exynos5420

----------------------------------------------------------------
Chander Kashyap (6):
       cpuidle: big.LITTLE: add of_device_id structure
       ARM: EXYNOS: add generic function to calculate cpu number
       cpuidle: big.LITTLE: Add ARCH_EXYNOS entry in config
       cpuidle: big.LITTLE: init driver for exynos5420
       ARM: EXYNOS: do not allow cpuidle registration for exynos5420
       ARM: EXYNOS: populate suspend and powered_up callbacks for mcpm

  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(-)

Comments

Olof Johansson July 19, 2014, 10:03 p.m. UTC | #1
On Sat, Jul 19, 2014 at 09:52:41AM +0900, Kukjin Kim wrote:
> The following changes since commit 1795cd9b3a91d4b5473c97f491d63892442212ab:
> 
>   Linux 3.16-rc5 (2014-07-13 14:04:33 -0700)
> 
> are available in the git repository at:
> 

> git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
> tags/exynos-cpuidle
> 
> for you to fetch changes up to fc2cac41ebbfb16da8b036cba6ec6714ab780a6d:
> 
>   ARM: EXYNOS: populate suspend and powered_up callbacks for mcpm
> (2014-07-19 03:36:00 +0900)

Merged, thanks.


-Olof
Kevin Hilman Aug. 8, 2014, 10:19 p.m. UTC | #2
Hello,

On Fri, Jul 18, 2014 at 5:52 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> The following changes since commit 1795cd9b3a91d4b5473c97f491d63892442212ab:
>
>   Linux 3.16-rc5 (2014-07-13 14:04:33 -0700)
>
> are available in the git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
> tags/exynos-cpuidle
>
> for you to fetch changes up to fc2cac41ebbfb16da8b036cba6ec6714ab780a6d:
>
>   ARM: EXYNOS: populate suspend and powered_up callbacks for mcpm
> (2014-07-19 03:36:00 +0900)
>
> ----------------------------------------------------------------
> exynos cpuidle update for v3.17
>
> - add callbacks exynos_suspend() and exynos_powered_up()
>   for support cpuidle through mcpm
> - skip exynos_cpuidle for exynos5420 because is uses
>   cpuidle-big-liggle generic cpuidle driver
> - add generic functions to calculate cpu number is used
>   for pmu and this is required for exynos5420 multi-cluster
> - add of_device_id structure for big.LITTLE cpuidle and
>   add "samsung,exynos5420" compatible string for exynos5420

I'm curious what platforms this is expected to work on, and where it's
been tested.

I tried it on exynos5800-peach-pi (chromebook2) and it hangs up (no
kernel messages) shortly after the driver loads, and never finishes to
boot.

I needed an extra compatible entry for the 5800 in the driver for the
driver to load, which suggests it hasn't been tested on 5800, but at
least in theory, this should be compatible with the 542x, right?

For testing, I tried today's linux-next and arm-soc/for-next.

Kevin
Kukjin Kim Aug. 9, 2014, 12:35 a.m. UTC | #3
Kevin Hilman wrote:
> 
> Hello,
> 
Hi,

> On Fri, Jul 18, 2014 at 5:52 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> > The following changes since commit 1795cd9b3a91d4b5473c97f491d63892442212ab:
> >
> >   Linux 3.16-rc5 (2014-07-13 14:04:33 -0700)
> >
> > are available in the git repository at:
> >
> >   git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
> > tags/exynos-cpuidle
> >
> > for you to fetch changes up to fc2cac41ebbfb16da8b036cba6ec6714ab780a6d:
> >
> >   ARM: EXYNOS: populate suspend and powered_up callbacks for mcpm
> > (2014-07-19 03:36:00 +0900)
> >
> > ----------------------------------------------------------------
> > exynos cpuidle update for v3.17
> >
> > - add callbacks exynos_suspend() and exynos_powered_up()
> >   for support cpuidle through mcpm
> > - skip exynos_cpuidle for exynos5420 because is uses
> >   cpuidle-big-liggle generic cpuidle driver
> > - add generic functions to calculate cpu number is used
> >   for pmu and this is required for exynos5420 multi-cluster
> > - add of_device_id structure for big.LITTLE cpuidle and
> >   add "samsung,exynos5420" compatible string for exynos5420
> 
> I'm curious what platforms this is expected to work on, and where it's
> been tested.
> 
exynos5420 and its reference board, smdk5420.

> I tried it on exynos5800-peach-pi (chromebook2) and it hangs up (no
> kernel messages) shortly after the driver loads, and never finishes to
> boot.
> 
Oh, sorry about that, if so, it should be fixed before -rc1. Just note that I
thought since exynos5800 is very similar with exynos5420 so I didn't ask
Chander to test the series on exynos5800 based board...Just asked to test with
Nico's mcpm series. BTW, unfortunately I have no exynos5800-peach-pi...

> I needed an extra compatible entry for the 5800 in the driver for the
> driver to load, which suggests it hasn't been tested on 5800, but at
> least in theory, this should be compatible with the 542x, right?
> 
Yes I think so. As you can see in exynos5800.dtsi, it is including exynos5420.

> For testing, I tried today's linux-next and arm-soc/for-next.
> 
Hmm...why is the problem happened at last? Not at that time? :(

I need to figure it out...

Thanks,
Kukjin
Chander M. Kashyap Aug. 11, 2014, 6:36 a.m. UTC | #4
Hi Kukjin,

On Sat, Aug 9, 2014 at 6:05 AM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> Kevin Hilman wrote:
>>
>> Hello,
>>
> Hi,
>
>> On Fri, Jul 18, 2014 at 5:52 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
>> > The following changes since commit 1795cd9b3a91d4b5473c97f491d63892442212ab:
>> >
>> >   Linux 3.16-rc5 (2014-07-13 14:04:33 -0700)
>> >
>> > are available in the git repository at:
>> >
>> >   git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
>> > tags/exynos-cpuidle
>> >
>> > for you to fetch changes up to fc2cac41ebbfb16da8b036cba6ec6714ab780a6d:
>> >
>> >   ARM: EXYNOS: populate suspend and powered_up callbacks for mcpm
>> > (2014-07-19 03:36:00 +0900)
>> >
>> > ----------------------------------------------------------------
>> > exynos cpuidle update for v3.17
>> >
>> > - add callbacks exynos_suspend() and exynos_powered_up()
>> >   for support cpuidle through mcpm
>> > - skip exynos_cpuidle for exynos5420 because is uses
>> >   cpuidle-big-liggle generic cpuidle driver
>> > - add generic functions to calculate cpu number is used
>> >   for pmu and this is required for exynos5420 multi-cluster
>> > - add of_device_id structure for big.LITTLE cpuidle and
>> >   add "samsung,exynos5420" compatible string for exynos5420
>>
>> I'm curious what platforms this is expected to work on, and where it's
>> been tested.
>>
> exynos5420 and its reference board, smdk5420.
>
>> I tried it on exynos5800-peach-pi (chromebook2) and it hangs up (no
>> kernel messages) shortly after the driver loads, and never finishes to
>> boot.
>>
> Oh, sorry about that, if so, it should be fixed before -rc1. Just note that I
> thought since exynos5800 is very similar with exynos5420 so I didn't ask
> Chander to test the series on exynos5800 based board...Just asked to test with
> Nico's mcpm series. BTW, unfortunately I have no exynos5800-peach-pi...
>
>> I needed an extra compatible entry for the 5800 in the driver for the
>> driver to load, which suggests it hasn't been tested on 5800, but at
>> least in theory, this should be compatible with the 542x, right?
>>
> Yes I think so. As you can see in exynos5800.dtsi, it is including exynos5420.
>
>> For testing, I tried today's linux-next and arm-soc/for-next.
>>
> Hmm...why is the problem happened at last? Not at that time? :(

Patch has been sent for the same:
http://lists.infradead.org/pipermail/linux-arm-kernel/2014-July/270472.html

This patch has been dropped as Tomasz has sent the fix in his patch.

Below is the link for the patch:

https://lkml.org/lkml/2014/8/1/103

which is yet to be merged.


>
> I need to figure it out...
>
> Thanks,
> Kukjin
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel