diff mbox

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

Message ID 1417509439-31739-2-git-send-email-ulrich.hecht+renesas@gmail.com (mailing list archive)
State Changes Requested
Delegated to: Simon Horman
Headers show

Commit Message

Ulrich Hecht Dec. 2, 2014, 8:37 a.m. UTC
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(+)

Comments

Magnus Damm Dec. 11, 2014, 12:31 p.m. UTC | #1
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
--
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
Simon Horman Dec. 12, 2014, 12:13 a.m. UTC | #2
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?
--
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
Magnus Damm Dec. 12, 2014, 12:51 a.m. UTC | #3
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
--
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
Simon Horman Dec. 12, 2014, 7:07 a.m. UTC | #4
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.
--
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
diff mbox

Patch

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