* Re: [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support
2014-12-16 16:25 [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support Ulrich Hecht
@ 2014-12-17 1:23 ` Magnus Damm
2014-12-17 4:57 ` Simon Horman
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Magnus Damm @ 2014-12-17 1:23 UTC (permalink / raw)
To: linux-sh
Hi Ulrich and Simon,
On Wed, Dec 17, 2014 at 1:25 AM, Ulrich Hecht
<ulrich.hecht+renesas@gmail.com> wrote:
> Hi!
>
> This series implements multiplatform support for the APE6EVM board, and
> eliminates both legacy platforms.
>
> This version incorporates the "ARM: shmobile: ape6evm-reference: add missing
> devices" series. As suggested by Magnus, it synchronizes the ape6evm and
> ape6evm-reference dts files first, eliminating the big renaming at the end.
Thanks for resending following this style. It is easy to understand -
at least for me!
> It is split in such a way that the first five patches can be merged without
> breaking either of the two legacy platforms. Simon, please pick those up.
> (Or the first three, if adding a multiplatform target that does not build is
> a problem.) Thank you.
The first patch seems to have some issue reported by Laurent. It needs
to be reworked.
Patch 2-5 should be possible to merge as-is unless I'm mistaken.
Simon, if you don't mind, can you please consider picking up 2-5 before 1, 6-7?
As for patch 6-7 and the sh73a4 legacy removal code by Laurent, I
think the main blocker there is the r8a73a4 CCF support patch. I don't
want to remove legacy code before reference is at least half-complete.
How can we move forward on the r8a73a4 CCF bits?
Cheers,
/ magnus
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support
2014-12-16 16:25 [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support Ulrich Hecht
2014-12-17 1:23 ` Magnus Damm
@ 2014-12-17 4:57 ` Simon Horman
2014-12-17 16:20 ` Ulrich Hecht
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2014-12-17 4:57 UTC (permalink / raw)
To: linux-sh
On Wed, Dec 17, 2014 at 10:23:39AM +0900, Magnus Damm wrote:
> Hi Ulrich and Simon,
>
> On Wed, Dec 17, 2014 at 1:25 AM, Ulrich Hecht
> <ulrich.hecht+renesas@gmail.com> wrote:
> > Hi!
> >
> > This series implements multiplatform support for the APE6EVM board, and
> > eliminates both legacy platforms.
> >
> > This version incorporates the "ARM: shmobile: ape6evm-reference: add missing
> > devices" series. As suggested by Magnus, it synchronizes the ape6evm and
> > ape6evm-reference dts files first, eliminating the big renaming at the end.
>
> Thanks for resending following this style. It is easy to understand -
> at least for me!
>
> > It is split in such a way that the first five patches can be merged without
> > breaking either of the two legacy platforms. Simon, please pick those up.
> > (Or the first three, if adding a multiplatform target that does not build is
> > a problem.) Thank you.
>
> The first patch seems to have some issue reported by Laurent. It needs
> to be reworked.
>
> Patch 2-5 should be possible to merge as-is unless I'm mistaken.
>
> Simon, if you don't mind, can you please consider picking up 2-5 before 1, 6-7?
Sure, that looks like it should work out.
I'll see about making it so and get back to you accordingly.
>
> As for patch 6-7 and the sh73a4 legacy removal code by Laurent, I
> think the main blocker there is the r8a73a4 CCF support patch. I don't
> want to remove legacy code before reference is at least half-complete.
>
> How can we move forward on the r8a73a4 CCF bits?
>
> Cheers,
>
> / magnus
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support
2014-12-16 16:25 [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support Ulrich Hecht
2014-12-17 1:23 ` Magnus Damm
2014-12-17 4:57 ` Simon Horman
@ 2014-12-17 16:20 ` Ulrich Hecht
2014-12-18 0:19 ` Simon Horman
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Ulrich Hecht @ 2014-12-17 16:20 UTC (permalink / raw)
To: linux-sh
On Wed, Dec 17, 2014 at 2:23 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> Hi Ulrich and Simon,
>
> On Wed, Dec 17, 2014 at 1:25 AM, Ulrich Hecht
> <ulrich.hecht+renesas@gmail.com> wrote:
>> It is split in such a way that the first five patches can be merged without
>> breaking either of the two legacy platforms. Simon, please pick those up.
>> (Or the first three, if adding a multiplatform target that does not build is
>> a problem.) Thank you.
>
> The first patch seems to have some issue reported by Laurent. It needs
> to be reworked.
>
> Patch 2-5 should be possible to merge as-is unless I'm mistaken.
>
> Simon, if you don't mind, can you please consider picking up 2-5 before 1, 6-7?
I respun the series following Laurent's suggestions, there should be
no need for special-casing anymore.
> As for patch 6-7 and the sh73a4 legacy removal code by Laurent, I
> think the main blocker there is the r8a73a4 CCF support patch. I don't
> want to remove legacy code before reference is at least half-complete.
>
> How can we move forward on the r8a73a4 CCF bits?
For starters I have sent a new version of the series with the last
remaining trivialities fixed.
CU
Uli
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support
2014-12-16 16:25 [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support Ulrich Hecht
` (2 preceding siblings ...)
2014-12-17 16:20 ` Ulrich Hecht
@ 2014-12-18 0:19 ` Simon Horman
2014-12-18 1:22 ` Magnus Damm
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2014-12-18 0:19 UTC (permalink / raw)
To: linux-sh
On Wed, Dec 17, 2014 at 05:20:15PM +0100, Ulrich Hecht wrote:
> On Wed, Dec 17, 2014 at 2:23 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> > Hi Ulrich and Simon,
> >
> > On Wed, Dec 17, 2014 at 1:25 AM, Ulrich Hecht
> > <ulrich.hecht+renesas@gmail.com> wrote:
> >> It is split in such a way that the first five patches can be merged without
> >> breaking either of the two legacy platforms. Simon, please pick those up.
> >> (Or the first three, if adding a multiplatform target that does not build is
> >> a problem.) Thank you.
> >
> > The first patch seems to have some issue reported by Laurent. It needs
> > to be reworked.
> >
> > Patch 2-5 should be possible to merge as-is unless I'm mistaken.
> >
> > Simon, if you don't mind, can you please consider picking up 2-5 before 1, 6-7?
>
> I respun the series following Laurent's suggestions, there should be
> no need for special-casing anymore.
>
> > As for patch 6-7 and the sh73a4 legacy removal code by Laurent, I
> > think the main blocker there is the r8a73a4 CCF support patch. I don't
> > want to remove legacy code before reference is at least half-complete.
> >
> > How can we move forward on the r8a73a4 CCF bits?
>
> For starters I have sent a new version of the series with the last
> remaining trivialities fixed.
Thanks, that seems clean to me. And I see that Laurent has Acked the
changes. I will queue those up.
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support
2014-12-16 16:25 [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support Ulrich Hecht
` (3 preceding siblings ...)
2014-12-18 0:19 ` Simon Horman
@ 2014-12-18 1:22 ` Magnus Damm
2014-12-18 1:25 ` Simon Horman
2014-12-18 1:33 ` Magnus Damm
6 siblings, 0 replies; 8+ messages in thread
From: Magnus Damm @ 2014-12-18 1:22 UTC (permalink / raw)
To: linux-sh
Hi Simon,
On Thu, Dec 18, 2014 at 9:19 AM, Simon Horman <horms@verge.net.au> wrote:
> On Wed, Dec 17, 2014 at 05:20:15PM +0100, Ulrich Hecht wrote:
>> On Wed, Dec 17, 2014 at 2:23 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
>> > Hi Ulrich and Simon,
>> >
>> > On Wed, Dec 17, 2014 at 1:25 AM, Ulrich Hecht
>> > <ulrich.hecht+renesas@gmail.com> wrote:
>> >> It is split in such a way that the first five patches can be merged without
>> >> breaking either of the two legacy platforms. Simon, please pick those up.
>> >> (Or the first three, if adding a multiplatform target that does not build is
>> >> a problem.) Thank you.
>> >
>> > The first patch seems to have some issue reported by Laurent. It needs
>> > to be reworked.
>> >
>> > Patch 2-5 should be possible to merge as-is unless I'm mistaken.
>> >
>> > Simon, if you don't mind, can you please consider picking up 2-5 before 1, 6-7?
>>
>> I respun the series following Laurent's suggestions, there should be
>> no need for special-casing anymore.
>>
>> > As for patch 6-7 and the sh73a4 legacy removal code by Laurent, I
>> > think the main blocker there is the r8a73a4 CCF support patch. I don't
>> > want to remove legacy code before reference is at least half-complete.
>> >
>> > How can we move forward on the r8a73a4 CCF bits?
>>
>> For starters I have sent a new version of the series with the last
>> remaining trivialities fixed.
>
> Thanks, that seems clean to me. And I see that Laurent has Acked the
> changes. I will queue those up.
Thanks. I assume you guys work out the CCF vs the shmobile ordering
and keep at least one DT-reference implementation working at any given
time.
Cheers,
/ magnus
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support
2014-12-16 16:25 [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support Ulrich Hecht
` (4 preceding siblings ...)
2014-12-18 1:22 ` Magnus Damm
@ 2014-12-18 1:25 ` Simon Horman
2014-12-18 1:33 ` Magnus Damm
6 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2014-12-18 1:25 UTC (permalink / raw)
To: linux-sh
On Thu, Dec 18, 2014 at 10:22:00AM +0900, Magnus Damm wrote:
> Hi Simon,
>
> On Thu, Dec 18, 2014 at 9:19 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Wed, Dec 17, 2014 at 05:20:15PM +0100, Ulrich Hecht wrote:
> >> On Wed, Dec 17, 2014 at 2:23 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> >> > Hi Ulrich and Simon,
> >> >
> >> > On Wed, Dec 17, 2014 at 1:25 AM, Ulrich Hecht
> >> > <ulrich.hecht+renesas@gmail.com> wrote:
> >> >> It is split in such a way that the first five patches can be merged without
> >> >> breaking either of the two legacy platforms. Simon, please pick those up.
> >> >> (Or the first three, if adding a multiplatform target that does not build is
> >> >> a problem.) Thank you.
> >> >
> >> > The first patch seems to have some issue reported by Laurent. It needs
> >> > to be reworked.
> >> >
> >> > Patch 2-5 should be possible to merge as-is unless I'm mistaken.
> >> >
> >> > Simon, if you don't mind, can you please consider picking up 2-5 before 1, 6-7?
> >>
> >> I respun the series following Laurent's suggestions, there should be
> >> no need for special-casing anymore.
> >>
> >> > As for patch 6-7 and the sh73a4 legacy removal code by Laurent, I
> >> > think the main blocker there is the r8a73a4 CCF support patch. I don't
> >> > want to remove legacy code before reference is at least half-complete.
> >> >
> >> > How can we move forward on the r8a73a4 CCF bits?
> >>
> >> For starters I have sent a new version of the series with the last
> >> remaining trivialities fixed.
> >
> > Thanks, that seems clean to me. And I see that Laurent has Acked the
> > changes. I will queue those up.
>
> Thanks. I assume you guys work out the CCF vs the shmobile ordering
> and keep at least one DT-reference implementation working at any given
> time.
Yes, that is the assumption that I am working on.
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support
2014-12-16 16:25 [PATCH v2 0/7] ARM: shmobile: ape6evm: multiplatform support Ulrich Hecht
` (5 preceding siblings ...)
2014-12-18 1:25 ` Simon Horman
@ 2014-12-18 1:33 ` Magnus Damm
6 siblings, 0 replies; 8+ messages in thread
From: Magnus Damm @ 2014-12-18 1:33 UTC (permalink / raw)
To: linux-sh
Hi Simon,
On Thu, Dec 18, 2014 at 10:25 AM, Simon Horman <horms@verge.net.au> wrote:
> On Thu, Dec 18, 2014 at 10:22:00AM +0900, Magnus Damm wrote:
>> Hi Simon,
>>
>> On Thu, Dec 18, 2014 at 9:19 AM, Simon Horman <horms@verge.net.au> wrote:
>> > On Wed, Dec 17, 2014 at 05:20:15PM +0100, Ulrich Hecht wrote:
>> >> On Wed, Dec 17, 2014 at 2:23 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
>> >> > Hi Ulrich and Simon,
>> >> >
>> >> > On Wed, Dec 17, 2014 at 1:25 AM, Ulrich Hecht
>> >> > <ulrich.hecht+renesas@gmail.com> wrote:
>> >> >> It is split in such a way that the first five patches can be merged without
>> >> >> breaking either of the two legacy platforms. Simon, please pick those up.
>> >> >> (Or the first three, if adding a multiplatform target that does not build is
>> >> >> a problem.) Thank you.
>> >> >
>> >> > The first patch seems to have some issue reported by Laurent. It needs
>> >> > to be reworked.
>> >> >
>> >> > Patch 2-5 should be possible to merge as-is unless I'm mistaken.
>> >> >
>> >> > Simon, if you don't mind, can you please consider picking up 2-5 before 1, 6-7?
>> >>
>> >> I respun the series following Laurent's suggestions, there should be
>> >> no need for special-casing anymore.
>> >>
>> >> > As for patch 6-7 and the sh73a4 legacy removal code by Laurent, I
>> >> > think the main blocker there is the r8a73a4 CCF support patch. I don't
>> >> > want to remove legacy code before reference is at least half-complete.
>> >> >
>> >> > How can we move forward on the r8a73a4 CCF bits?
>> >>
>> >> For starters I have sent a new version of the series with the last
>> >> remaining trivialities fixed.
>> >
>> > Thanks, that seems clean to me. And I see that Laurent has Acked the
>> > changes. I will queue those up.
>>
>> Thanks. I assume you guys work out the CCF vs the shmobile ordering
>> and keep at least one DT-reference implementation working at any given
>> time.
>
> Yes, that is the assumption that I am working on.
Great, thank you!
/ magnus
^ permalink raw reply [flat|nested] 8+ messages in thread