From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753270AbdBAROO (ORCPT ); Wed, 1 Feb 2017 12:14:14 -0500 Received: from smtprelay.synopsys.com ([198.182.60.111]:44901 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752861AbdBAROM (ORCPT ); Wed, 1 Feb 2017 12:14:12 -0500 From: Alexey Brodkin To: Vineet Gupta CC: "linux-kernel@vger.kernel.org" , Ruud Derwig , "linux-snps-arc@lists.infradead.org" Subject: Re: [PATCH 1/3] arc: vdk: Disable halt on reset Thread-Topic: [PATCH 1/3] arc: vdk: Disable halt on reset Thread-Index: AQHSfKpDbM5U8Om5YECRbZ2HwwCL1KFUTSWAgAAF84A= Date: Wed, 1 Feb 2017 17:14:08 +0000 Message-ID: <1485969247.5515.27.camel@synopsys.com> References: <1485967373-15273-1-git-send-email-abrodkin@synopsys.com> <1485967373-15273-2-git-send-email-abrodkin@synopsys.com> <8b857bc4-71f0-9b3b-0cd5-19b49edb2910@synopsys.com> In-Reply-To: <8b857bc4-71f0-9b3b-0cd5-19b49edb2910@synopsys.com> Accept-Language: en-US, ru-RU Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.121.8.77] Content-Type: text/plain; charset="utf-8" Content-ID: <88A48C7DB529084E85EF5B1E70622531@internal.synopsys.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id v11HEWDv012462 Hi Vineet, On Wed, 2017-02-01 at 08:52 -0800, Vineet Gupta wrote: > On 02/01/2017 08:42 AM, Alexey Brodkin wrote: > > > > In recent VDKs ARC cores are configured as "run on reset" > > which made existing kernel configuration outdated to effect that > > slave cores never start execution of the code keeping only master > > online. > > > > With that fix we're again in sync with VDK platform. > > > > And while at it we regenerate defconfig via savedefconfig so default > > options are now excluded. > > > > Signed-off-by: Alexey Brodkin > > --- > >  arch/arc/configs/vdk_hs38_smp_defconfig | 4 +--- > >  1 file changed, 1 insertion(+), 3 deletions(-) > > > > diff --git a/arch/arc/configs/vdk_hs38_smp_defconfig b/arch/arc/configs/vdk_hs38_smp_defconfig > > index 573028f19de7..f6361de41eec 100644 > > --- a/arch/arc/configs/vdk_hs38_smp_defconfig > > +++ b/arch/arc/configs/vdk_hs38_smp_defconfig > > @@ -15,7 +15,7 @@ CONFIG_ARC_PLAT_AXS10X=y > >  CONFIG_AXS103=y > >  CONFIG_ISA_ARCV2=y > >  CONFIG_SMP=y > > -# CONFIG_ARC_TIMERS_64BIT is not set > > Are you sure abut this part. Ater the timers driver rework, this would enable GFRC > for SMP builds and AFAIKR there were some issues with time with GFRC + nSIM etc.. Not anymore :) Probably I missed something in discussions. As a matter of fact I did run-test resulting vmlinux and it worked very nice. More over ARC_TIMERS_64BIT is selected automatically by ISA_ARCV2, see http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/arch/arc/Kconfig#n119 That said even if "# CONFIG_ARC_TIMERS_64BIT is not set" is left in place the option will be effectively enabled, no? -Alexey From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexey.Brodkin@synopsys.com (Alexey Brodkin) Date: Wed, 1 Feb 2017 17:14:08 +0000 Subject: [PATCH 1/3] arc: vdk: Disable halt on reset In-Reply-To: <8b857bc4-71f0-9b3b-0cd5-19b49edb2910@synopsys.com> References: <1485967373-15273-1-git-send-email-abrodkin@synopsys.com> <1485967373-15273-2-git-send-email-abrodkin@synopsys.com> <8b857bc4-71f0-9b3b-0cd5-19b49edb2910@synopsys.com> List-ID: Message-ID: <1485969247.5515.27.camel@synopsys.com> To: linux-snps-arc@lists.infradead.org Hi Vineet, On Wed, 2017-02-01@08:52 -0800, Vineet Gupta wrote: > On 02/01/2017 08:42 AM, Alexey Brodkin wrote: > > > > In recent VDKs ARC cores are configured as "run on reset" > > which made existing kernel configuration outdated to effect that > > slave cores never start execution of the code keeping only master > > online. > > > > With that fix we're again in sync with VDK platform. > > > > And while at it we regenerate defconfig via savedefconfig so default > > options are now excluded. > > > > Signed-off-by: Alexey Brodkin > > --- > > ?arch/arc/configs/vdk_hs38_smp_defconfig | 4 +--- > > ?1 file changed, 1 insertion(+), 3 deletions(-) > > > > diff --git a/arch/arc/configs/vdk_hs38_smp_defconfig b/arch/arc/configs/vdk_hs38_smp_defconfig > > index 573028f19de7..f6361de41eec 100644 > > --- a/arch/arc/configs/vdk_hs38_smp_defconfig > > +++ b/arch/arc/configs/vdk_hs38_smp_defconfig > > @@ -15,7 +15,7 @@ CONFIG_ARC_PLAT_AXS10X=y > > ?CONFIG_AXS103=y > > ?CONFIG_ISA_ARCV2=y > > ?CONFIG_SMP=y > > -# CONFIG_ARC_TIMERS_64BIT is not set > > Are you sure abut this part. Ater the timers driver rework, this would enable GFRC > for SMP builds and AFAIKR there were some issues with time with GFRC + nSIM etc.. Not anymore :) Probably I missed something in discussions. As a matter of fact I did run-test resulting vmlinux and it worked very nice. More over?ARC_TIMERS_64BIT is selected automatically by?ISA_ARCV2, see http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/arch/arc/Kconfig#n119 That said even if "# CONFIG_ARC_TIMERS_64BIT is not set" is left in place the option will be effectively enabled, no? -Alexey