From mboxrd@z Thu Jan 1 00:00:00 1970 From: thierry.reding@gmail.com (Thierry Reding) Date: Wed, 4 Dec 2013 09:49:56 +0100 Subject: [PATCH 01/31] ARM: tegra: add missing clock documentation to DT bindings In-Reply-To: <529E24A3.3080804@wwwdotorg.org> References: <1384548866-13141-1-git-send-email-swarren@wwwdotorg.org> <1384548866-13141-2-git-send-email-swarren@wwwdotorg.org> <20131129114900.GN22771@ulmo.nvidia.com> <529E24A3.3080804@wwwdotorg.org> Message-ID: <20131204084956.GG19943@ulmo.nvidia.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Dec 03, 2013 at 11:36:19AM -0700, Stephen Warren wrote: > On 11/29/2013 04:49 AM, Thierry Reding wrote: > > On Fri, Nov 15, 2013 at 01:53:56PM -0700, Stephen Warren wrote: > ... > >> + - clock-names : Must include the following entries: > > > > One other thing I noticed here is that you use a space between the > > property name and the :. None of the other properties have that, so > > it looks somewhat out of place. The same is true for other > > bindings, but there seem to be inconsistencies in some places > > anyway, so perhaps we don't care? Well, I do care, don't know about > > you. =) > > I've fixed those up locally. I assume you don't want a repost for such > a trivial change? I'll double-check the "resets" patch for the same issue. No need for a repost. I'll trust you on that one. But I'll urge you to consider what I said about the "disp" clock entry for the display controllers before merging this. Thierry -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: not available URL: