From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benoit Cousson Subject: Re: [PATCH 2/2] ARM: dts: OMAP2+: Add PMU nodes Date: Mon, 17 Dec 2012 17:04:14 +0100 Message-ID: <50CF427E.3000804@ti.com> References: <1355519930-3851-1-git-send-email-jon-hunter@ti.com> <1355519930-3851-3-git-send-email-jon-hunter@ti.com> <50CED4D3.9080106@ti.com> <50CF411F.4060105@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <50CF411F.4060105@ti.com> Sender: linux-omap-owner@vger.kernel.org To: Jon Hunter Cc: Tony Lindgren , linux-omap , linux-arm , device-tree List-Id: devicetree@vger.kernel.org Hi Jon, On 12/17/2012 04:58 PM, Jon Hunter wrote: > > On 12/17/2012 02:16 AM, Benoit Cousson wrote: >> Hi Jon, >> >> On 12/14/2012 10:18 PM, Jon Hunter wrote: >>> Add PMU nodes for OMAP2, OMAP3 and OMAP4460 devices. >>> >>> Please note that the node for OMAP4460 has been placed in a separate >>> header file for OMAP4460, because the node is not compatible with >>> OMAP4430. >> >> But where is the omap4430 node then? > > I have left it out deliberately because OMAP4430 is not yet supported by > PMU as it is dependent on having a driver for the cross-trigger interface. > > If you prefer to stick the node in the omap4.dtsi for now then that is > ok, but I wanted to make it clear that this is for omap4460. No, that's fine, I was just wondering. You should just add that comment in the cover letter to make it explicit. Thanks, Benoit