* Re: [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support
2014-12-02 8:37 [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support Ulrich Hecht
@ 2014-12-11 12:31 ` Magnus Damm
2014-12-12 0:13 ` Simon Horman
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Magnus Damm @ 2014-12-11 12:31 UTC (permalink / raw)
To: linux-sh
Hi Ulrich,
On Tue, Dec 2, 2014 at 5:37 PM, Ulrich Hecht
<ulrich.hecht+renesas@gmail.com> wrote:
> Enable r8a73a4 Multiplatform support for the generic r8a73a4
> machine vector.
>
> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
This for your efforts. This particular patch is fine with me.
Acked-by: Magnus Damm <damm+renesas@opensource.se>
Regarding the other patches, as discussed over chat please consider
optimising the patch order to allow merge of multiplatform merge
independently of board removal and renaming of files. Basically I
think you can add multiplatform support using the non-reference DT
file directly and not making use of any C board code.
Cheers,
/ magnus
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support
2014-12-02 8:37 [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support Ulrich Hecht
2014-12-11 12:31 ` Magnus Damm
@ 2014-12-12 0:13 ` Simon Horman
2014-12-12 0:51 ` Magnus Damm
2014-12-12 7:07 ` Simon Horman
3 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2014-12-12 0:13 UTC (permalink / raw)
To: linux-sh
Hi Magnus, Hi Ulrich,
On Thu, Dec 11, 2014 at 09:31:13PM +0900, Magnus Damm wrote:
> Hi Ulrich,
>
> On Tue, Dec 2, 2014 at 5:37 PM, Ulrich Hecht
> <ulrich.hecht+renesas@gmail.com> wrote:
> > Enable r8a73a4 Multiplatform support for the generic r8a73a4
> > machine vector.
> >
> > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
>
> This for your efforts. This particular patch is fine with me.
>
> Acked-by: Magnus Damm <damm+renesas@opensource.se>
>
> Regarding the other patches, as discussed over chat please consider
> optimising the patch order to allow merge of multiplatform merge
> independently of board removal and renaming of files. Basically I
> think you can add multiplatform support using the non-reference DT
> file directly and not making use of any C board code.
Would you like me to queue up this patch now?
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support
2014-12-02 8:37 [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support Ulrich Hecht
2014-12-11 12:31 ` Magnus Damm
2014-12-12 0:13 ` Simon Horman
@ 2014-12-12 0:51 ` Magnus Damm
2014-12-12 7:07 ` Simon Horman
3 siblings, 0 replies; 5+ messages in thread
From: Magnus Damm @ 2014-12-12 0:51 UTC (permalink / raw)
To: linux-sh
Hi Simon,
On Fri, Dec 12, 2014 at 9:13 AM, Simon Horman <horms@verge.net.au> wrote:
> Hi Magnus, Hi Ulrich,
>
> On Thu, Dec 11, 2014 at 09:31:13PM +0900, Magnus Damm wrote:
>> Hi Ulrich,
>>
>> On Tue, Dec 2, 2014 at 5:37 PM, Ulrich Hecht
>> <ulrich.hecht+renesas@gmail.com> wrote:
>> > Enable r8a73a4 Multiplatform support for the generic r8a73a4
>> > machine vector.
>> >
>> > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
>>
>> This for your efforts. This particular patch is fine with me.
>>
>> Acked-by: Magnus Damm <damm+renesas@opensource.se>
>>
>> Regarding the other patches, as discussed over chat please consider
>> optimising the patch order to allow merge of multiplatform merge
>> independently of board removal and renaming of files. Basically I
>> think you can add multiplatform support using the non-reference DT
>> file directly and not making use of any C board code.
>
> Would you like me to queue up this patch now?
Thanks for the offer, but I think we should wait for Ulrich to rework
this series first. Once a couple of patches are OK as-is then it would
be nice to merge them before the CCF bits are in place. But I guess it
much depends on how Ulrich will split the code.
Cheers,
/ magnus
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support
2014-12-02 8:37 [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support Ulrich Hecht
` (2 preceding siblings ...)
2014-12-12 0:51 ` Magnus Damm
@ 2014-12-12 7:07 ` Simon Horman
3 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2014-12-12 7:07 UTC (permalink / raw)
To: linux-sh
On Fri, Dec 12, 2014 at 09:51:43AM +0900, Magnus Damm wrote:
> Hi Simon,
>
> On Fri, Dec 12, 2014 at 9:13 AM, Simon Horman <horms@verge.net.au> wrote:
> > Hi Magnus, Hi Ulrich,
> >
> > On Thu, Dec 11, 2014 at 09:31:13PM +0900, Magnus Damm wrote:
> >> Hi Ulrich,
> >>
> >> On Tue, Dec 2, 2014 at 5:37 PM, Ulrich Hecht
> >> <ulrich.hecht+renesas@gmail.com> wrote:
> >> > Enable r8a73a4 Multiplatform support for the generic r8a73a4
> >> > machine vector.
> >> >
> >> > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
> >>
> >> This for your efforts. This particular patch is fine with me.
> >>
> >> Acked-by: Magnus Damm <damm+renesas@opensource.se>
> >>
> >> Regarding the other patches, as discussed over chat please consider
> >> optimising the patch order to allow merge of multiplatform merge
> >> independently of board removal and renaming of files. Basically I
> >> think you can add multiplatform support using the non-reference DT
> >> file directly and not making use of any C board code.
> >
> > Would you like me to queue up this patch now?
>
> Thanks for the offer, but I think we should wait for Ulrich to rework
> this series first. Once a couple of patches are OK as-is then it would
> be nice to merge them before the CCF bits are in place. But I guess it
> much depends on how Ulrich will split the code.
Thanks, I will immediately do nothing.
^ permalink raw reply [flat|nested] 5+ messages in thread