Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753369AbbERKjB (ORCPT ); Mon, 18 May 2015 06:39:01 -0400 Received: from foss.arm.com ([217.140.101.70]:59265 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752763AbbERKix (ORCPT ); Mon, 18 May 2015 06:38:53 -0400 Message-ID: <5559C139.9030103@arm.com> Date: Mon, 18 May 2015 11:38:49 +0100 From: Sudeep Holla User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Catalin Marinas CC: Sudeep Holla , Russell King , Rob Herring , Arnd Bergmann , Daniel Lezcano , "linux-kernel@vger.kernel.org" , Olof Johansson , Thomas Gleixner , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH 1/2] ARM: simplify timer initialisation and remove arm_timer.h inclusion References: <1430390662-18246-1-git-send-email-sudeep.holla@arm.com> <1430390662-18246-2-git-send-email-sudeep.holla@arm.com> <554239D5.4080301@arm.com> <555634F6.9060008@arm.com> <20150518103302.GB21251@e104818-lin.cambridge.arm.com> In-Reply-To: <20150518103302.GB21251@e104818-lin.cambridge.arm.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2041 Lines: 49 On 18/05/15 11:33, Catalin Marinas wrote: > On Fri, May 15, 2015 at 07:03:34PM +0100, Sudeep Holla wrote: >> On 30/04/15 15:19, Sudeep Holla wrote: >>> On 30/04/15 15:09, Rob Herring wrote: >>>> On Thu, Apr 30, 2015 at 5:44 AM, Sudeep Holla wrote: >>>>> The header asm/hardware/arm_timer.h is included in various machine >>>>> specific files to access TIMER_CTRL and initialise to a known state. >>>>> However that's not required as the clock{source,event} driver timer-sp >>>>> initialises all the timer being used. >>>> >>>> I believe the idea is not to initialize the timers being used, but the >>>> ones not being used and perhaps left running by the bootloader. Cases >>>> where the interrupt is shared could cause a problem. >> >> Russell, can you confirm if that's the case ? > > Unless you want to test all these platforms, I would suggest we assume > this is the case. The comments even state "Initialise to a known state > (all timers off)". > OK makes sense. >>> Ah OK, makes sense. I will wait for Russell to confirm. The main idea >>> was to keep the header file having offsets local to driver/clocksource >>> and avoid sharing it in include/linux but looks like that's not possible. >> >> Since we need this driver on ARM64, we might have to end up sharing the >> header file with offsets if required for ARM platforms(though it would >> be good to avoid it if there's any better alternative solution than that) > > Can you not just move the definitions to > include/clocksource/timer-sp804.h and add some SP804_ prefix to avoid > name collisions? > Yes I can do that and that's the alternate plan, wanted to avoid having SP804 timer internal register offsets in that header file hoping to contain those in the driver files if possible. Regards, Sudeep -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/