From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Thu, 6 Sep 2012 14:19:07 +0000 Subject: [PATCH 4/6 v2] ARM: integrator: initial device tree support In-Reply-To: References: <1346471760-23993-1-git-send-email-linus.walleij@linaro.org> <5048A31E.2090704@gmail.com> Message-ID: <201209061419.07325.arnd@arndb.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thursday 06 September 2012, Linus Walleij wrote: > On Thu, Sep 6, 2012 at 3:20 PM, Rob Herring wrote: > > [Me] > >> I can of course skip the aliases and go for the timer nodes > >> I want directly, but that isn't any helpful for people reading the > >> device tree, is it? > > > > Well, the dts doesn't really need to describe how Linux is using things. > > The arm,integrator-clocksource and arm,integrator-clockevent > are renamed arm,timer-primary and arm,timer-secondary in the > last patch set. > > Are you OK with this for now, going forward? I'd be happy to > take a round refactoring it for the next cycle as outlined > below. If the timers are really free for use by anything and you want to encode how Linux uses them, I would suggest naming the aliases "linux.clocksource" and "linux,clockevent" to make it clear that you are not describing the hardware here but rather picking a configuration. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH 4/6 v2] ARM: integrator: initial device tree support Date: Thu, 6 Sep 2012 14:19:07 +0000 Message-ID: <201209061419.07325.arnd@arndb.de> References: <1346471760-23993-1-git-send-email-linus.walleij@linaro.org> <5048A31E.2090704@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: Linus Walleij Cc: Russell King , Pawel Moll , devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, Will Deacon , arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On Thursday 06 September 2012, Linus Walleij wrote: > On Thu, Sep 6, 2012 at 3:20 PM, Rob Herring wrote: > > [Me] > >> I can of course skip the aliases and go for the timer nodes > >> I want directly, but that isn't any helpful for people reading the > >> device tree, is it? > > > > Well, the dts doesn't really need to describe how Linux is using things. > > The arm,integrator-clocksource and arm,integrator-clockevent > are renamed arm,timer-primary and arm,timer-secondary in the > last patch set. > > Are you OK with this for now, going forward? I'd be happy to > take a round refactoring it for the next cycle as outlined > below. If the timers are really free for use by anything and you want to encode how Linux uses them, I would suggest naming the aliases "linux.clocksource" and "linux,clockevent" to make it clear that you are not describing the hardware here but rather picking a configuration. Arnd