[PATCH v3 0/4] Initial r8a7794/Alt support

[PATCH v3 0/4] Initial r8a7794/Alt support

[Ulrich Hecht]

Hi!

This is a rebased version of v2 requested by Magnus with the parts that are
supposed to go through Simon's tree.  It also splits board and SoC DT into
two patches.

CU
Uli


Changes since v2:
- dropped bindings stuff
- split SoC and board DT into two patches

Changes since v1:
- removed loadaddr
- added init_late method to DT_MACHINE_START
- removed board file
- renamed serial10 to serial0 because we're not popular enough to be
  different
- reordered patches according to dependencies


Ulrich Hecht (4):
  ARM: shmobile: support Cortex-A7 in shmobile_init_delay()
  ARM: shmobile: Initial r8a7794 SoC support
  ARM: shmobile: Initial r8a7794 SoC device tree
  ARM: shmobile: Initial Alt board device tree

 arch/arm/boot/dts/Makefile                |   3 +-
 arch/arm/boot/dts/r8a7794-alt.dts         |  47 +++
 arch/arm/boot/dts/r8a7794.dtsi            | 531 ++++++++++++++++++++++++++++++
 arch/arm/mach-shmobile/Kconfig            |   4 +
 arch/arm/mach-shmobile/Makefile           |   1 +
 arch/arm/mach-shmobile/setup-r8a7794.c    |  35 ++
 arch/arm/mach-shmobile/timer.c            |   9 +-
 include/dt-bindings/clock/r8a7794-clock.h |  82 +++++
 8 files changed, 707 insertions(+), 5 deletions(-)
 create mode 100644 arch/arm/boot/dts/r8a7794-alt.dts
 create mode 100644 arch/arm/boot/dts/r8a7794.dtsi
 create mode 100644 arch/arm/mach-shmobile/setup-r8a7794.c
 create mode 100644 include/dt-bindings/clock/r8a7794-clock.h

-- 
1.8.4.5

Re: [PATCH v3 0/4] Initial r8a7794/Alt support

[Magnus Damm]

On Thu, Sep 4, 2014 at 11:02 PM, Ulrich Hecht
<ulrich.hecht+renesas@gmail.com> wrote:
> Hi!
>
> This is a rebased version of v2 requested by Magnus with the parts that are
> supposed to go through Simon's tree.  It also splits board and SoC DT into
> two patches.
>
> CU
> Uli
>
>
> Changes since v2:
> - dropped bindings stuff
> - split SoC and board DT into two patches

Hi Ulrich,

This series looks good to me.

Acked-by: Magnus Damm <damm+renesas@opensource.se>

Like Khiem-san pointed out, perhaps the DTS needs some rework? From my
side it would be fine to handle that as incremental changes over time.

Simon, would it be possible for you to pick up these patches? Do you
think the DTS patch shall be reworked once more, or is incremental
fixing OK?

Cheers,

/ magnus

Re: [PATCH v3 0/4] Initial r8a7794/Alt support

[Simon Horman]

On Fri, Sep 05, 2014 at 12:34:17PM +0900, Magnus Damm wrote:
> On Thu, Sep 4, 2014 at 11:02 PM, Ulrich Hecht
> <ulrich.hecht+renesas@gmail.com> wrote:
> > Hi!
> >
> > This is a rebased version of v2 requested by Magnus with the parts that are
> > supposed to go through Simon's tree.  It also splits board and SoC DT into
> > two patches.
> >
> > CU
> > Uli
> >
> >
> > Changes since v2:
> > - dropped bindings stuff
> > - split SoC and board DT into two patches
> 
> Hi Ulrich,
> 
> This series looks good to me.
> 
> Acked-by: Magnus Damm <damm+renesas@opensource.se>
> 
> Like Khiem-san pointed out, perhaps the DTS needs some rework? From my
> side it would be fine to handle that as incremental changes over time.
> 
> Simon, would it be possible for you to pick up these patches? Do you
> think the DTS patch shall be reworked once more, or is incremental
> fixing OK?

I would prefer if Ulrich re-spun the patches.

I have had push-back from Olof in the past about including
changes and fixes to said changes in the same pull-request.

Re: [PATCH v3 0/4] Initial r8a7794/Alt support

[Magnus Damm]

Hi Simon, Ulrich,

On Fri, Sep 5, 2014 at 1:18 PM, Simon Horman <horms@verge.net.au> wrote:
> On Fri, Sep 05, 2014 at 12:34:17PM +0900, Magnus Damm wrote:
>> On Thu, Sep 4, 2014 at 11:02 PM, Ulrich Hecht
>> <ulrich.hecht+renesas@gmail.com> wrote:
>> > Hi!
>> >
>> > This is a rebased version of v2 requested by Magnus with the parts that are
>> > supposed to go through Simon's tree.  It also splits board and SoC DT into
>> > two patches.
>> >
>> > CU
>> > Uli
>> >
>> >
>> > Changes since v2:
>> > - dropped bindings stuff
>> > - split SoC and board DT into two patches
>>
>> Hi Ulrich,
>>
>> This series looks good to me.
>>
>> Acked-by: Magnus Damm <damm+renesas@opensource.se>
>>
>> Like Khiem-san pointed out, perhaps the DTS needs some rework? From my
>> side it would be fine to handle that as incremental changes over time.
>>
>> Simon, would it be possible for you to pick up these patches? Do you
>> think the DTS patch shall be reworked once more, or is incremental
>> fixing OK?
>
> I would prefer if Ulrich re-spun the patches.
>
> I have had push-back from Olof in the past about including
> changes and fixes to said changes in the same pull-request.

Of course, resubmitting is fine. If you however think it is OK to
merge some individual patches as-is - like for instance 1-2 - then
that would of course be greatly appreciated.

Can please you tell Ulrich which patches you want him to resend?

Cheers,

/ magnus

Re: [PATCH v3 0/4] Initial r8a7794/Alt support

[Simon Horman]

On Fri, Sep 05, 2014 at 03:32:59PM +0900, Magnus Damm wrote:
> Hi Simon, Ulrich,
> 
> On Fri, Sep 5, 2014 at 1:18 PM, Simon Horman <horms@verge.net.au> wrote:
> > On Fri, Sep 05, 2014 at 12:34:17PM +0900, Magnus Damm wrote:
> >> On Thu, Sep 4, 2014 at 11:02 PM, Ulrich Hecht
> >> <ulrich.hecht+renesas@gmail.com> wrote:
> >> > Hi!
> >> >
> >> > This is a rebased version of v2 requested by Magnus with the parts that are
> >> > supposed to go through Simon's tree.  It also splits board and SoC DT into
> >> > two patches.
> >> >
> >> > CU
> >> > Uli
> >> >
> >> >
> >> > Changes since v2:
> >> > - dropped bindings stuff
> >> > - split SoC and board DT into two patches
> >>
> >> Hi Ulrich,
> >>
> >> This series looks good to me.
> >>
> >> Acked-by: Magnus Damm <damm+renesas@opensource.se>
> >>
> >> Like Khiem-san pointed out, perhaps the DTS needs some rework? From my
> >> side it would be fine to handle that as incremental changes over time.
> >>
> >> Simon, would it be possible for you to pick up these patches? Do you
> >> think the DTS patch shall be reworked once more, or is incremental
> >> fixing OK?
> >
> > I would prefer if Ulrich re-spun the patches.
> >
> > I have had push-back from Olof in the past about including
> > changes and fixes to said changes in the same pull-request.
> 
> Of course, resubmitting is fine. If you however think it is OK to
> merge some individual patches as-is - like for instance 1-2 - then
> that would of course be greatly appreciated.

In general I prefer to pick up entire series once they are entirely ready.
But as you ask so nicely I have queued up patches 1 and 2.

> Can please you tell Ulrich which patches you want him to resend?

Please repost patches 3 and 4.