From: Arnd Bergmann <arnd@arndb.de> To: Jonas Jensen <jonas.jensen@gmail.com> Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, arm@kernel.org, linux@arm.linux.org.uk, olof@lixom.net Subject: Re: [PATCH v4 1/2] ARM: mach-moxart: add MOXA ART SoC platform files Date: Fri, 13 Dec 2013 17:17:25 +0100 [thread overview] Message-ID: <201312131717.25435.arnd@arndb.de> (raw) In-Reply-To: <1386945188-8316-2-git-send-email-jonas.jensen@gmail.com> On Friday 13 December 2013, Jonas Jensen wrote: > The MOXA ART SoC is based on Faraday's FA526. This is a ARMv4 32-bit > 192 MHz CPU with MMU and 16KB/8KB D/I-cache. > > Add platform support for this SoC. > > Also add UC-7112-LX as a machine. > > Signed-off-by: Jonas Jensen <jonas.jensen@gmail.com> Let's make sure we get this merged this time. I'm definitely fine with this version going in. My questions are just for clarification on my part for potential future follow-ups. Acked-by: Arnd Bergmann <arnd@arndb.de> > diff --git a/arch/arm/configs/moxart_defconfig b/arch/arm/configs/moxart_defconfig > new file mode 100644 > index 0000000..51f0f03 > --- /dev/null > +++ b/arch/arm/configs/moxart_defconfig > @@ -0,0 +1,148 @@ > +# CONFIG_LOCALVERSION_AUTO is not set > +# CONFIG_SWAP is not set > +CONFIG_SYSVIPC=y > +CONFIG_NO_HZ=y > +CONFIG_IKCONFIG=y IIRC the separate defconfig is needed because we have never gotten a mixed fa526+arm9 kernel to boot, right? In theory it is supposed to work and there is most likely just a small bug somewhere. Once that is working, we can think about adding a generic multi_v4_v5_defconfig that handles all multipltform capable platforms along the lines of multi_v7_defconfig. > +static void __init moxart_dt_init(void) > +{ > + moxart_setup_restart(); > + of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL); > +} I've been out of the loop a bit here. I initially suggested having the restart handler be part of the watchdog driver, and I thought you had done that in an earlier version. What has changed your mind? Arnd
WARNING: multiple messages have this Message-ID (diff)
From: arnd@arndb.de (Arnd Bergmann) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v4 1/2] ARM: mach-moxart: add MOXA ART SoC platform files Date: Fri, 13 Dec 2013 17:17:25 +0100 [thread overview] Message-ID: <201312131717.25435.arnd@arndb.de> (raw) In-Reply-To: <1386945188-8316-2-git-send-email-jonas.jensen@gmail.com> On Friday 13 December 2013, Jonas Jensen wrote: > The MOXA ART SoC is based on Faraday's FA526. This is a ARMv4 32-bit > 192 MHz CPU with MMU and 16KB/8KB D/I-cache. > > Add platform support for this SoC. > > Also add UC-7112-LX as a machine. > > Signed-off-by: Jonas Jensen <jonas.jensen@gmail.com> Let's make sure we get this merged this time. I'm definitely fine with this version going in. My questions are just for clarification on my part for potential future follow-ups. Acked-by: Arnd Bergmann <arnd@arndb.de> > diff --git a/arch/arm/configs/moxart_defconfig b/arch/arm/configs/moxart_defconfig > new file mode 100644 > index 0000000..51f0f03 > --- /dev/null > +++ b/arch/arm/configs/moxart_defconfig > @@ -0,0 +1,148 @@ > +# CONFIG_LOCALVERSION_AUTO is not set > +# CONFIG_SWAP is not set > +CONFIG_SYSVIPC=y > +CONFIG_NO_HZ=y > +CONFIG_IKCONFIG=y IIRC the separate defconfig is needed because we have never gotten a mixed fa526+arm9 kernel to boot, right? In theory it is supposed to work and there is most likely just a small bug somewhere. Once that is working, we can think about adding a generic multi_v4_v5_defconfig that handles all multipltform capable platforms along the lines of multi_v7_defconfig. > +static void __init moxart_dt_init(void) > +{ > + moxart_setup_restart(); > + of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL); > +} I've been out of the loop a bit here. I initially suggested having the restart handler be part of the watchdog driver, and I thought you had done that in an earlier version. What has changed your mind? Arnd
next prev parent reply other threads:[~2013-12-13 16:17 UTC|newest] Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-12-13 14:33 [PATCH v4 0/2] ARM: mach-moxart: add MOXA ART SoC support Jonas Jensen 2013-12-13 14:33 ` Jonas Jensen 2013-12-13 14:33 ` [PATCH v4 1/2] ARM: mach-moxart: add MOXA ART SoC platform files Jonas Jensen 2013-12-13 14:33 ` Jonas Jensen [not found] ` <1386945188-8316-2-git-send-email-jonas.jensen-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2013-12-13 14:38 ` Fwd: " Jonas Jensen 2013-12-13 16:17 ` Arnd Bergmann [this message] 2013-12-13 16:17 ` Arnd Bergmann 2013-12-13 17:23 ` Jonas Jensen 2013-12-13 17:23 ` Jonas Jensen 2013-12-13 19:07 ` Guenter Roeck 2013-12-13 19:07 ` Guenter Roeck 2013-12-14 4:01 ` Arnd Bergmann 2013-12-14 4:01 ` Arnd Bergmann 2013-12-14 8:32 ` Jonas Jensen 2013-12-14 8:32 ` Jonas Jensen 2013-12-14 15:50 ` Guenter Roeck 2013-12-14 15:50 ` Guenter Roeck 2013-12-14 16:26 ` Jonas Jensen 2013-12-14 16:26 ` Jonas Jensen 2013-12-14 18:50 ` Arnd Bergmann 2013-12-14 18:50 ` Arnd Bergmann 2013-12-14 20:14 ` Guenter Roeck 2013-12-14 20:14 ` Guenter Roeck 2013-12-15 0:21 ` Arnd Bergmann 2013-12-15 0:21 ` Arnd Bergmann 2013-12-22 19:48 ` Olof Johansson 2013-12-22 19:48 ` Olof Johansson 2013-12-13 14:33 ` [PATCH v4 2/2] ARM: mach-moxart: add MOXA ART SoC device tree files Jonas Jensen 2013-12-13 14:33 ` Jonas Jensen [not found] ` <1386945188-8316-3-git-send-email-jonas.jensen-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2013-12-13 14:39 ` Fwd: " Jonas Jensen 2013-12-13 16:17 ` Arnd Bergmann 2013-12-13 16:17 ` Arnd Bergmann 2013-12-15 4:27 ` Peter Crosthwaite 2013-12-15 4:27 ` Peter Crosthwaite 2013-12-16 20:05 ` Jonas Jensen 2013-12-16 20:05 ` Jonas Jensen 2013-12-16 23:53 ` Peter Crosthwaite 2013-12-16 23:53 ` Peter Crosthwaite 2013-12-16 23:53 ` Peter Crosthwaite 2013-12-17 0:09 ` Sören Brinkmann 2013-12-17 0:09 ` Sören Brinkmann 2013-12-17 0:09 ` Sören Brinkmann 2013-12-22 19:49 ` Olof Johansson 2013-12-22 19:49 ` Olof Johansson
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=201312131717.25435.arnd@arndb.de \ --to=arnd@arndb.de \ --cc=arm@kernel.org \ --cc=devicetree-discuss@lists.ozlabs.org \ --cc=jonas.jensen@gmail.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=olof@lixom.net \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.