Patchwork [GIT,PULL] Renesas ARM-based SoC: SoC for 3.7

login
register
mail settings
Submitter Simon Horman
Date Sept. 6, 2012, 2:02 a.m.
Message ID <1346896928-5750-1-git-send-email-horms@verge.net.au>
Download mbox
Permalink /patch/181991/
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git soc

Comments

Simon Horman - Sept. 6, 2012, 2:02 a.m.
Hi Olof, Hi Arnd,

please consider the following SoC enhancement from
Phil Edworthy for inclusion in 3.7.

----------------------------------------------------------------
The following changes since commit 7952717adb69efc1d2443a1858f96d23c2fb93e0:

  ARM: shmobile: kzm9g: enable restarting (2012-08-25 14:39:44 +0900)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git soc

for you to fetch changes up to 263510ec774b490dffbd0b8db7b723025cf560c9:

  r8a7779: add SDHI clock support (2012-09-06 10:49:18 +0900)

----------------------------------------------------------------
Phil Edworthy (1):
      r8a7779: add SDHI clock support

 arch/arm/mach-shmobile/clock-r8a7779.c |   11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)
Olof Johansson - Sept. 7, 2012, 9:49 p.m.
Hi,

On Thu, Sep 06, 2012 at 11:02:07AM +0900, Simon Horman wrote:
> Hi Olof, Hi Arnd,
> 
> please consider the following SoC enhancement from
> Phil Edworthy for inclusion in 3.7.
> 
> ----------------------------------------------------------------
> The following changes since commit 7952717adb69efc1d2443a1858f96d23c2fb93e0:
> 
>   ARM: shmobile: kzm9g: enable restarting (2012-08-25 14:39:44 +0900)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git soc

The above is based on the previous branch we pulled from you (kzm9g), and
contains a patch to support rebooting of a board. I would have expected that to
be the base for your board branch but not the SoC branch.

Did you mean to base them like this for some other reason that I didn't
realize, or would you mind switching the two branches so that the board
one includes the restart change but not the other way around?


-Olof
Simon Horman - Sept. 8, 2012, 12:10 a.m.
On Fri, Sep 07, 2012 at 02:49:51PM -0700, Olof Johansson wrote:
> Hi,
> 
> On Thu, Sep 06, 2012 at 11:02:07AM +0900, Simon Horman wrote:
> > Hi Olof, Hi Arnd,
> > 
> > please consider the following SoC enhancement from
> > Phil Edworthy for inclusion in 3.7.
> > 
> > ----------------------------------------------------------------
> > The following changes since commit 7952717adb69efc1d2443a1858f96d23c2fb93e0:
> > 
> >   ARM: shmobile: kzm9g: enable restarting (2012-08-25 14:39:44 +0900)
> > 
> > are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git soc
> 
> The above is based on the previous branch we pulled from you (kzm9g), and
> contains a patch to support rebooting of a board. I would have expected that to
> be the base for your board branch but not the SoC branch.
> 
> Did you mean to base them like this for some other reason that I didn't
> realize, or would you mind switching the two branches so that the board
> one includes the restart change but not the other way around?

Sorry about that, I'll switch things around.
Simon Horman - Sept. 11, 2012, 4:51 a.m.
On Sat, Sep 08, 2012 at 09:10:26AM +0900, Simon Horman wrote:
> On Fri, Sep 07, 2012 at 02:49:51PM -0700, Olof Johansson wrote:
> > Hi,
> > 
> > On Thu, Sep 06, 2012 at 11:02:07AM +0900, Simon Horman wrote:
> > > Hi Olof, Hi Arnd,
> > > 
> > > please consider the following SoC enhancement from
> > > Phil Edworthy for inclusion in 3.7.
> > > 
> > > ----------------------------------------------------------------
> > > The following changes since commit 7952717adb69efc1d2443a1858f96d23c2fb93e0:
> > > 
> > >   ARM: shmobile: kzm9g: enable restarting (2012-08-25 14:39:44 +0900)
> > > 
> > > are available in the git repository at:
> > > 
> > >   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git soc
> > 
> > The above is based on the previous branch we pulled from you (kzm9g), and
> > contains a patch to support rebooting of a board. I would have expected that to
> > be the base for your board branch but not the SoC branch.
> > 
> > Did you mean to base them like this for some other reason that I didn't
> > realize, or would you mind switching the two branches so that the board
> > one includes the restart change but not the other way around?
> 
> Sorry about that, I'll switch things around.

Hi Olof,

I was going to switch things around as you suggest, but I noticed
that the reason that I did things this way is that my kzm9g branch
was pulled into the next/soc branch of the arm-soc tree (commit
301fd5c13257862df66c81c48d963d474e63e0ef).

With that in mind, do you want to take this pull-request as is?
Olof Johansson - Sept. 12, 2012, 5:55 a.m.
Hi,

On Mon, Sep 10, 2012 at 9:51 PM, Simon Horman <horms@verge.net.au> wrote:
> On Sat, Sep 08, 2012 at 09:10:26AM +0900, Simon Horman wrote:
>> On Fri, Sep 07, 2012 at 02:49:51PM -0700, Olof Johansson wrote:
>> > Hi,
>> >
>> > On Thu, Sep 06, 2012 at 11:02:07AM +0900, Simon Horman wrote:
>> > > Hi Olof, Hi Arnd,
>> > >
>> > > please consider the following SoC enhancement from
>> > > Phil Edworthy for inclusion in 3.7.
>> > >
>> > > ----------------------------------------------------------------
>> > > The following changes since commit 7952717adb69efc1d2443a1858f96d23c2fb93e0:
>> > >
>> > >   ARM: shmobile: kzm9g: enable restarting (2012-08-25 14:39:44 +0900)
>> > >
>> > > are available in the git repository at:
>> > >
>> > >   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git soc
>> >
>> > The above is based on the previous branch we pulled from you (kzm9g), and
>> > contains a patch to support rebooting of a board. I would have expected that to
>> > be the base for your board branch but not the SoC branch.
>> >
>> > Did you mean to base them like this for some other reason that I didn't
>> > realize, or would you mind switching the two branches so that the board
>> > one includes the restart change but not the other way around?
>>
>> Sorry about that, I'll switch things around.
>
> Hi Olof,
>
> I was going to switch things around as you suggest, but I noticed
> that the reason that I did things this way is that my kzm9g branch
> was pulled into the next/soc branch of the arm-soc tree (commit
> 301fd5c13257862df66c81c48d963d474e63e0ef).

Ah, gotcha, and also noticing my own slight confusion here.

First, since there's no overlap with the old and the new branch,
there's no need to base them on each other (since they won't cause
merge conflicts, and order of merging them in doesn't have other
dependencies either).

But it also makes sense from your development point of view to stack
them, since that's how you'll end up building up the branches anyway.
So:

> With that in mind, do you want to take this pull-request as is?

Nah, no need, sorry for the confusion. I'll pull it in as it is into next/soc.


-Olof
Simon Horman - Sept. 12, 2012, 7:20 a.m.
On Tue, Sep 11, 2012 at 10:55:47PM -0700, Olof Johansson wrote:
> Hi,
> 
> On Mon, Sep 10, 2012 at 9:51 PM, Simon Horman <horms@verge.net.au> wrote:
> > On Sat, Sep 08, 2012 at 09:10:26AM +0900, Simon Horman wrote:
> >> On Fri, Sep 07, 2012 at 02:49:51PM -0700, Olof Johansson wrote:
> >> > Hi,
> >> >
> >> > On Thu, Sep 06, 2012 at 11:02:07AM +0900, Simon Horman wrote:
> >> > > Hi Olof, Hi Arnd,
> >> > >
> >> > > please consider the following SoC enhancement from
> >> > > Phil Edworthy for inclusion in 3.7.
> >> > >
> >> > > ----------------------------------------------------------------
> >> > > The following changes since commit 7952717adb69efc1d2443a1858f96d23c2fb93e0:
> >> > >
> >> > >   ARM: shmobile: kzm9g: enable restarting (2012-08-25 14:39:44 +0900)
> >> > >
> >> > > are available in the git repository at:
> >> > >
> >> > >   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git soc
> >> >
> >> > The above is based on the previous branch we pulled from you (kzm9g), and
> >> > contains a patch to support rebooting of a board. I would have expected that to
> >> > be the base for your board branch but not the SoC branch.
> >> >
> >> > Did you mean to base them like this for some other reason that I didn't
> >> > realize, or would you mind switching the two branches so that the board
> >> > one includes the restart change but not the other way around?
> >>
> >> Sorry about that, I'll switch things around.
> >
> > Hi Olof,
> >
> > I was going to switch things around as you suggest, but I noticed
> > that the reason that I did things this way is that my kzm9g branch
> > was pulled into the next/soc branch of the arm-soc tree (commit
> > 301fd5c13257862df66c81c48d963d474e63e0ef).
> 
> Ah, gotcha, and also noticing my own slight confusion here.
> 
> First, since there's no overlap with the old and the new branch,
> there's no need to base them on each other (since they won't cause
> merge conflicts, and order of merging them in doesn't have other
> dependencies either).
> 
> But it also makes sense from your development point of view to stack
> them, since that's how you'll end up building up the branches anyway.
> So:
> 
> > With that in mind, do you want to take this pull-request as is?
> 
> Nah, no need, sorry for the confusion. I'll pull it in as it is into next/soc.

Thanks!