[PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support

[PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support

[Ulrich Hecht]

Enable r8a73a4 Multiplatform support for the generic r8a73a4
machine vector.

Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
---
 arch/arm/mach-shmobile/Kconfig | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index 1b4fafe..d107b93 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -51,6 +51,11 @@ config ARCH_R7S72100
 	bool "RZ/A1H (R7S72100)"
 	select SYS_SUPPORTS_SH_MTU2
 
+config ARCH_R8A73A4
+	bool "R-Mobile APE6 (R8A73A40)"
+	select ARCH_RMOBILE
+	select RENESAS_IRQC
+
 config ARCH_R8A7740
 	bool "R-Mobile A1 (R8A77400)"
 	select ARCH_RMOBILE
-- 
1.8.4.5

Re: [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support

[Magnus Damm]

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

Re: [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support

[Simon Horman]

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?

Re: [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support

[Magnus Damm]

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

Re: [PATCH 1/5] ARM: shmobile: r8a73a4: Multiplatform support

[Simon Horman]

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.