From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Date: Thu, 11 Sep 2014 01:05:05 +0000 Subject: Re: [PATCH] ARM: shmobile: Enable r8a7794 SoC in shmobile_defconfig Message-Id: <20140911010505.GE10233@verge.net.au> List-Id: References: <1410309887-15492-1-git-send-email-horms+renesas@verge.net.au> <5410F40D.7090704@renesas.com> In-Reply-To: <5410F40D.7090704@renesas.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On Thu, Sep 11, 2014 at 09:59:57AM +0900, Khiem Nguyen wrote: > Hi Simon, > cc Ulrich, > > Thanks for your patch. > > I think it's a bit early to enable r8a7794 > in shmobile_defconfig. > > The accepted part of r8a7794 initial code lacked of > arch. timer initialization code for Cortex-A7 in r8a7794 SoC. > [PATCH 1/3] ARM: shmobile: rcar-gen2: Add CA7 arch_timer initialization for r8a7794 > http://www.spinics.net/lists/linux-sh/msg34978.html > > I guess (at least) that code should be merged (in current form or another) > before r8a7794 can be enabled in shmobile_defconfig. > > On 9/10/2014 9:44 AM, Simon Horman wrote: > > Support for the r8a7794 was recently added. > > Enable it in the shmobile_defconfig to increase build coverage. > > IMO, the support is only enabled after the code under that config > run correctly. > Please correct me if I'm wrong. The purpose of enabling r8a7794 at this point is to increase build coverage. As the SoC support is new it doesn't create any regressions even though the support may not be in a working form yet. > > Signed-off-by: Simon Horman > > --- > > > > Based on renesas-devel-20140910v2-v3.17-rc4 > > arch/arm/configs/shmobile_defconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/arch/arm/configs/shmobile_defconfig b/arch/arm/configs/shmobile_defconfig > > index 522275a..d7346ad 100644 > > --- a/arch/arm/configs/shmobile_defconfig > > +++ b/arch/arm/configs/shmobile_defconfig > > @@ -16,6 +16,7 @@ CONFIG_ARCH_R8A7740=y > > CONFIG_ARCH_R8A7779=y > > CONFIG_ARCH_R8A7790=y > > CONFIG_ARCH_R8A7791=y > > +CONFIG_ARCH_R8A7794=y > > CONFIG_MACH_KOELSCH=y > > CONFIG_MACH_LAGER=y > > CONFIG_MACH_MARZEN=y > > > > -- > Best regards, > KHIEM Nguyen >