Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S948026AbcJaX6R (ORCPT ); Mon, 31 Oct 2016 19:58:17 -0400 Received: from us01smtprelay-2.synopsys.com ([198.182.60.111]:60214 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S947258AbcJaX6P (ORCPT ); Mon, 31 Oct 2016 19:58:15 -0400 Subject: Re: [PATCH v2] ARC: Enable PERF_EVENTS in nSIM driven platforms To: Alexey Brodkin , "linux-snps-arc@lists.infradead.org" References: <1477918751-28058-1-git-send-email-abrodkin@synopsys.com> CC: "linux-kernel@vger.kernel.org" From: Vineet Gupta Message-ID: Date: Mon, 31 Oct 2016 16:58:07 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <1477918751-28058-1-git-send-email-abrodkin@synopsys.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.10.161.40] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1763 Lines: 47 On 10/31/2016 06:00 AM, Alexey Brodkin wrote: > Now when we have properly working performance counters in nSIM > even with interrupt support (fix should be a part of upcoming > nSIM engineering build 2016.12-005) we may enable perf support > by default for all platforms that use nSIM for ARC cores simulation. > > Note 1: PCT node was missing for some reason in nsimosci.dts > while all other nSIM-related .dts files already had > PCT node for quite some time, so adding it now. > > Note 2: All defconfigs were regenerated with "make savedefconfig" > which led to some clean-ups in nsimosci_hs_smp_defconfig: > CONFIG_FRAMEBUFFER_CONSOLE=y was removed because it is > automatically selected now by DRM. > > Signed-off-by: Alexey Brodkin > --- > > changes v1 -> v2: > * Added missing PCT Device Tree node in nsimosci.dts > > arch/arc/boot/dts/nsimosci.dts | 4 ++++ > arch/arc/configs/nsim_700_defconfig | 1 + > arch/arc/configs/nsim_hs_defconfig | 1 + > arch/arc/configs/nsim_hs_smp_defconfig | 1 + > arch/arc/configs/nsimosci_defconfig | 1 + > arch/arc/configs/nsimosci_hs_defconfig | 1 + > arch/arc/configs/nsimosci_hs_smp_defconfig | 3 +-- > 7 files changed, 10 insertions(+), 2 deletions(-) > > diff --git a/arch/arc/boot/dts/nsimosci.dts b/arch/arc/boot/dts/nsimosci.dts > index bcf603142a33..7fe184435956 100644 > --- a/arch/arc/boot/dts/nsimosci.dts > +++ b/arch/arc/boot/dts/nsimosci.dts > @@ -83,5 +83,9 @@ > reg = <0xf0003000 0x44>; > interrupts = <7>; > }; > + > + arcpct0: pct { > + compatible = "snps,archs-pct"; > + }; > }; This is not the right binding for 700 pct - I've fixed it and pushed to for-next ! -Vineet