Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933295AbcKCRn7 (ORCPT ); Thu, 3 Nov 2016 13:43:59 -0400 Received: from us01smtprelay-2.synopsys.com ([198.182.47.9]:35492 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932866AbcKCRn5 (ORCPT ); Thu, 3 Nov 2016 13:43:57 -0400 Subject: Re: [PATCH 1/9] ARC: timer: gfrc, rtc: Read BCR to detect whether hardware exists ... To: Daniel Lezcano References: <35bde193-8492-83e0-fb03-8385d8afd007@synopsys.com> <1477954096-770-1-git-send-email-vgupta@synopsys.com> <1477954096-770-2-git-send-email-vgupta@synopsys.com> <20161103170056.GB1859@mai> CC: , , , Noam Camus , Newsgroups: gmane.linux.kernel,gmane.linux.kernel.arc From: Vineet Gupta Message-ID: Date: Thu, 3 Nov 2016 10:41:45 -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: <20161103170056.GB1859@mai> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.10.161.44] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1997 Lines: 61 On 11/03/2016 10:00 AM, Daniel Lezcano wrote: > On Mon, Oct 31, 2016 at 03:48:08PM -0700, Vineet Gupta wrote: >> ... don't rely on cpuinfo populated in arc boot code. This paves wat for >> moving this code in drivers/clocksource/ >> >> Signed-off-by: Vineet Gupta >> --- >> arch/arc/kernel/time.c | 10 ++++++---- >> 1 file changed, 6 insertions(+), 4 deletions(-) >> >> diff --git a/arch/arc/kernel/time.c b/arch/arc/kernel/time.c >> index f927b8dc6edd..a2db010cde18 100644 >> --- a/arch/arc/kernel/time.c >> +++ b/arch/arc/kernel/time.c >> @@ -118,10 +118,11 @@ static struct clocksource arc_counter_gfrc = { >> >> static int __init arc_cs_setup_gfrc(struct device_node *node) >> { >> - int exists = cpuinfo_arc700[0].extn.gfrc; >> + struct mcip_bcr mp; >> int ret; >> >> - if (WARN(!exists, "Global-64-bit-Ctr clocksource not detected")) >> + READ_BCR(ARC_REG_MCIP_BCR, mp); >> + if (WARN(!mp.gfrc, "Global-64-bit-Ctr clocksource not detected")) > > Take the opportunity to replace this WARN by a pr_err. OK. > >> return -ENXIO; >> >> ret = arc_get_timer_clk(node); >> @@ -174,10 +175,11 @@ static struct clocksource arc_counter_rtc = { >> >> static int __init arc_cs_setup_rtc(struct device_node *node) >> { >> - int exists = cpuinfo_arc700[smp_processor_id()].extn.rtc; >> + struct bcr_timer timer; >> int ret; >> >> - if (WARN(!exists, "Local-64-bit-Ctr clocksource not detected")) >> + READ_BCR(ARC_REG_TIMERS_BCR, timer); >> + if (WARN(!timer.rtc, "Local-64-bit-Ctr clocksource not detected")) >> return -ENXIO;o > > Ditto and ^^ > > So the READ_BCR() is only there to check the timer is physically present ? Yep, due to configurable nature of cores, we have Build Config Registers to detect at runtime what is present or not. This allows for boot printing at the minimum. This is defined in arcregs.h and in newly introduced soc/arc/aux.h > >> /* Local to CPU hence not usable in SMP */ >> -- >> 2.7.4 >> >