mbox

[GIT,PULL] ARM: mvebu: SoC changes for v3.17 (round 3)

Message ID 20140722205706.GA23220@titan.lakedaemon.net
State New
Headers show

Pull-request

git://git.infradead.org/linux-mvebu.git tags/mvebu-soc-3.17-3

Message

Jason Cooper July 22, 2014, 8:57 p.m. UTC
All,

Here's the cpufreq changes I mentioned over the weekend.  It's been in
-next for a while.  Once I get the new version of cpuidle from ThomasP,
I'll push that as quickly as possible.

As usual (do I need to type this each time any more? :) ), this is an
incremental pull request from tags/mvebu-soc-3.17-2 up to
tags/mvebu-soc-3.17-3 on the mvebu/soc branch.

Please pull.

thx,

Jason.


The following changes since commit ba364fc752daeded072a5ef31e43b84cb1f9e5fd:

  ARM: Kirkwood: Remove mach-kirkwood (2014-07-13 22:13:39 +0000)

are available in the git repository at:

  git://git.infradead.org/linux-mvebu.git tags/mvebu-soc-3.17-3

for you to fetch changes up to ba3ec5780bba27819bbc4f669e6c77418a00f14b:

  Merge branch 'mvebu/soc-cpufreq' into mvebu/soc (2014-07-22 20:46:48 +0000)

----------------------------------------------------------------

mvebu SoC changes for v3.17 (round 3)

 - mvebu:
    - Add cpufreq support for Armada XP

----------------------------------------------------------------
Jason Cooper (1):
      Merge branch 'mvebu/soc-cpufreq' into mvebu/soc

Thomas Petazzoni (3):
      ARM: mvebu: ensure CPU clocks are enabled
      ARM: mvebu: extend PMSU code to support dynamic frequency scaling
      clk: mvebu: extend clk-cpu for dynamic frequency scaling

 .../devicetree/bindings/clock/mvebu-cpu-clock.txt  |   5 +-
 arch/arm/mach-mvebu/platsmp.c                      |   1 +
 arch/arm/mach-mvebu/pmsu.c                         | 162 +++++++++++++++++++++
 drivers/clk/mvebu/clk-cpu.c                        |  80 +++++++++-
 include/linux/mvebu-pmsu.h                         |  20 +++
 5 files changed, 261 insertions(+), 7 deletions(-)
 create mode 100644 include/linux/mvebu-pmsu.h

Comments

Arnd Bergmann July 23, 2014, 8:31 p.m. UTC | #1
On Tuesday 22 July 2014, Jason Cooper wrote:
> All,
> 
> Here's the cpufreq changes I mentioned over the weekend.  It's been in
> -next for a while.  Once I get the new version of cpuidle from ThomasP,
> I'll push that as quickly as possible.

Just for my information, is this coordinated with the cpufreq maintainers?
I remember being asked to ensure all cpufreq/cpuidle stuff has an Ack
from them, even stuff that doesn't touch drivers/cpufreq at all.

The contents look good to me. I can merge them if there are no objections
from Viresh, feel free to send the next batch already without waiting
for me to pull.

> As usual (do I need to type this each time any more? :) ), this is an
> incremental pull request from tags/mvebu-soc-3.17-2 up to
> tags/mvebu-soc-3.17-3 on the mvebu/soc branch.

It certainly helps me to have this information, but you can write it
more briefly, e.g. "based on tags/mvebu-soc-3.17-2".

	Arnd
 
> The following changes since commit ba364fc752daeded072a5ef31e43b84cb1f9e5fd:
> 
>   ARM: Kirkwood: Remove mach-kirkwood (2014-07-13 22:13:39 +0000)
> 
> are available in the git repository at:
> 
>   git://git.infradead.org/linux-mvebu.git tags/mvebu-soc-3.17-3
> 
> for you to fetch changes up to ba3ec5780bba27819bbc4f669e6c77418a00f14b:
> 
>   Merge branch 'mvebu/soc-cpufreq' into mvebu/soc (2014-07-22 20:46:48 +0000)
> 
> ----------------------------------------------------------------
> 
> mvebu SoC changes for v3.17 (round 3)
> 
>  - mvebu:
>     - Add cpufreq support for Armada XP
> 
> ----------------------------------------------------------------
> Jason Cooper (1):
>       Merge branch 'mvebu/soc-cpufreq' into mvebu/soc
> 
> Thomas Petazzoni (3):
>       ARM: mvebu: ensure CPU clocks are enabled
>       ARM: mvebu: extend PMSU code to support dynamic frequency scaling
>       clk: mvebu: extend clk-cpu for dynamic frequency scaling
> 
>  .../devicetree/bindings/clock/mvebu-cpu-clock.txt  |   5 +-
>  arch/arm/mach-mvebu/platsmp.c                      |   1 +
>  arch/arm/mach-mvebu/pmsu.c                         | 162 +++++++++++++++++++++
>  drivers/clk/mvebu/clk-cpu.c                        |  80 +++++++++-
>  include/linux/mvebu-pmsu.h                         |  20 +++
>  5 files changed, 261 insertions(+), 7 deletions(-)
>  create mode 100644 include/linux/mvebu-pmsu.h
>
Jason Cooper July 23, 2014, 11:41 p.m. UTC | #2
On Wed, Jul 23, 2014 at 10:31:41PM +0200, Arnd Bergmann wrote:
> On Tuesday 22 July 2014, Jason Cooper wrote:
> > All,
> > 
> > Here's the cpufreq changes I mentioned over the weekend.  It's been in
> > -next for a while.  Once I get the new version of cpuidle from ThomasP,
> > I'll push that as quickly as possible.
> 
> Just for my information, is this coordinated with the cpufreq maintainers?
> I remember being asked to ensure all cpufreq/cpuidle stuff has an Ack
> from them, even stuff that doesn't touch drivers/cpufreq at all.

Good to know.

> The contents look good to me. I can merge them if there are no objections
> from Viresh, feel free to send the next batch already without waiting
> for me to pull.

We've been coordinating with Viresh.  He has an unsettled issue on his
side he hopes to have resolved soon.  Worst case scenario, he doesn't
get the change in for v3.17, we'll have to do a minor DT fix on top of
-rc1.

> > As usual (do I need to type this each time any more? :) ), this is an
> > incremental pull request from tags/mvebu-soc-3.17-2 up to
> > tags/mvebu-soc-3.17-3 on the mvebu/soc branch.
> 
> It certainly helps me to have this information, but you can write it
> more briefly, e.g. "based on tags/mvebu-soc-3.17-2".

Ahhhh....  Much more succinct.  Thank you.  On a side note, I wonder
what it would take to get 'git request-pull' smart enough to add that
information when handed a tag?

> > The following changes since commit ba364fc752daeded072a5ef31e43b84cb1f9e5fd:
> > 
> >   ARM: Kirkwood: Remove mach-kirkwood (2014-07-13 22:13:39 +0000)

eg:

  ARM: Kirkwood: Remove mach-kirkwood (tags/mvebu-soc-3.17-2)

> > 
> > are available in the git repository at:
> > 
> >   git://git.infradead.org/linux-mvebu.git tags/mvebu-soc-3.17-3
> > 
> > for you to fetch changes up to ba3ec5780bba27819bbc4f669e6c77418a00f14b:
> > 
> >   Merge branch 'mvebu/soc-cpufreq' into mvebu/soc (2014-07-22 20:46:48 +0000)
> > 
> > ----------------------------------------------------------------

Is the date of the base commit useful in any way in the pull request?

thx,

Jason.
Olof Johansson July 23, 2014, 11:45 p.m. UTC | #3
On Wed, Jul 23, 2014 at 4:41 PM, Jason Cooper <jason@lakedaemon.net> wrote:
> On Wed, Jul 23, 2014 at 10:31:41PM +0200, Arnd Bergmann wrote:
>> On Tuesday 22 July 2014, Jason Cooper wrote:
>> > All,
>> >
>> > Here's the cpufreq changes I mentioned over the weekend.  It's been in
>> > -next for a while.  Once I get the new version of cpuidle from ThomasP,
>> > I'll push that as quickly as possible.
>>
>> Just for my information, is this coordinated with the cpufreq maintainers?
>> I remember being asked to ensure all cpufreq/cpuidle stuff has an Ack
>> from them, even stuff that doesn't touch drivers/cpufreq at all.
>
> Good to know.
>
>> The contents look good to me. I can merge them if there are no objections
>> from Viresh, feel free to send the next batch already without waiting
>> for me to pull.
>
> We've been coordinating with Viresh.  He has an unsettled issue on his
> side he hopes to have resolved soon.  Worst case scenario, he doesn't
> get the change in for v3.17, we'll have to do a minor DT fix on top of
> -rc1.
>
>> > As usual (do I need to type this each time any more? :) ), this is an
>> > incremental pull request from tags/mvebu-soc-3.17-2 up to
>> > tags/mvebu-soc-3.17-3 on the mvebu/soc branch.
>>
>> It certainly helps me to have this information, but you can write it
>> more briefly, e.g. "based on tags/mvebu-soc-3.17-2".
>
> Ahhhh....  Much more succinct.  Thank you.  On a side note, I wonder
> what it would take to get 'git request-pull' smart enough to add that
> information when handed a tag?
>
>> > The following changes since commit ba364fc752daeded072a5ef31e43b84cb1f9e5fd:
>> >
>> >   ARM: Kirkwood: Remove mach-kirkwood (2014-07-13 22:13:39 +0000)
>
> eg:
>
>   ARM: Kirkwood: Remove mach-kirkwood (tags/mvebu-soc-3.17-2)

That only maps to the tag in your tree, which might or might not map
directly to something that we know or care about. We do tend to keep
naming similar, but not always identical. For example, the branch for
that tag would be mvebu/soc2 in our tree.

>> >
>> > are available in the git repository at:
>> >
>> >   git://git.infradead.org/linux-mvebu.git tags/mvebu-soc-3.17-3
>> >
>> > for you to fetch changes up to ba3ec5780bba27819bbc4f669e6c77418a00f14b:
>> >
>> >   Merge branch 'mvebu/soc-cpufreq' into mvebu/soc (2014-07-22 20:46:48 +0000)
>> >
>> > ----------------------------------------------------------------
>
> Is the date of the base commit useful in any way in the pull request?

I find the SHA to be useful, I will sometimes run "git branch
--contains ba3ec57<...>" to see where the base is merged already. The
date, for me, less so.


-Olof
Jason Cooper July 24, 2014, 12:08 a.m. UTC | #4
On Wed, Jul 23, 2014 at 04:45:15PM -0700, Olof Johansson wrote:
> On Wed, Jul 23, 2014 at 4:41 PM, Jason Cooper <jason@lakedaemon.net> wrote:
> > On Wed, Jul 23, 2014 at 10:31:41PM +0200, Arnd Bergmann wrote:
> >> On Tuesday 22 July 2014, Jason Cooper wrote:
...
> >> > As usual (do I need to type this each time any more? :) ), this is an
> >> > incremental pull request from tags/mvebu-soc-3.17-2 up to
> >> > tags/mvebu-soc-3.17-3 on the mvebu/soc branch.
> >>
> >> It certainly helps me to have this information, but you can write it
> >> more briefly, e.g. "based on tags/mvebu-soc-3.17-2".
> >
> > Ahhhh....  Much more succinct.  Thank you.  On a side note, I wonder
> > what it would take to get 'git request-pull' smart enough to add that
> > information when handed a tag?
> >
> >> > The following changes since commit ba364fc752daeded072a5ef31e43b84cb1f9e5fd:
> >> >
> >> >   ARM: Kirkwood: Remove mach-kirkwood (2014-07-13 22:13:39 +0000)
> >
> > eg:
> >
> >   ARM: Kirkwood: Remove mach-kirkwood (tags/mvebu-soc-3.17-2)
> 
> That only maps to the tag in your tree, which might or might not map
> directly to something that we know or care about. We do tend to keep
> naming similar, but not always identical. For example, the branch for
> that tag would be mvebu/soc2 in our tree.

Right, the goal was to automate 'based on tags/mvebu-soc-3.17-2' which
lets you guys know it's an incremental pull request.

> > Is the date of the base commit useful in any way in the pull request?
> 
> I find the SHA to be useful, I will sometimes run "git branch
> --contains ba3ec57<...>" to see where the base is merged already. The
> date, for me, less so.

Ok, good to know.

thx,

Jason.