From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933028AbcLMMUe (ORCPT ); Tue, 13 Dec 2016 07:20:34 -0500 Received: from mout.kundenserver.de ([212.227.126.135]:63012 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932549AbcLMMUb (ORCPT ); Tue, 13 Dec 2016 07:20:31 -0500 From: Arnd Bergmann To: linux-arm-kernel@lists.infradead.org Cc: Andrew Jeffery , Lee Jones , Mark Rutland , Corey Minyard , devicetree@vger.kernel.org, Linus Walleij , linux-kernel@vger.kernel.org, =?ISO-8859-1?Q?C=E9dric?= Le Goater , Joel Stanley Subject: Re: [PATCH v3 6/6] mfd: dt: Move syscon bindings to syscon subdirectory Date: Tue, 13 Dec 2016 13:17:16 +0100 Message-ID: <5094120.Y7DXfRGTEm@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-34-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <1481630734.3112.40.camel@aj.id.au> References: <20161206025321.1792-1-andrew@aj.id.au> <20161213110710.GV3625@dell.home> <1481630734.3112.40.camel@aj.id.au> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:VaY5cjvlvQ8Md5NIT2RMtRTbg2BroX6wvj9qPmpD3Sak+jY0wtK FuFCjRRkCb5FXgk4qlrfm911G7M4vleT9P2it9I7e4WG/4WOG3sDC76eHHwFV9Bf3nY5PqG 3I4o38vpMJW7JYfyLPTHSSvldD/FJPfMvFIGCaAj3cV1PmW/keCFy/a8jXS7pF1CTxhsaQx j5tZelYfmbmfyOgGselsA== X-UI-Out-Filterresults: notjunk:1;V01:K0:5Zu8quGSt00=:7gTzxOkxRJJYUN9Pv021x2 2vs+elfbt7Wa9ax9mbrkVua+IyUxCceAHOx74D9pAR1ALGfDSffEVJ4T4Mk6z0nmjQz9JfcAU IYrewLVzHkl0hRVzTuwpfftpTQaYDAAEp057tZTh8jIrQoiRAHEbQUi1srutoEtVbFQBuNmJN nXvDK4T1rKfG6+SJUEVFC00whRfKdYRv5n253xVWoX1yzDKThC/fGZHMh6gTEU5LzsgP6Jnea PhOsHLwZf7jzrK8nrif5eoyAh4QrYujf0UZ5praT8go2NLXLJ2AneU82+H1V3ar9mCOiZdZMe rs4KSdOkfVpQYbDvd/VNTIEFyKe/5er3mkJSHpFaJ+H89YVjngqSa22VB9kOimhJ/uQ1t89dp TH7fT4pyGLs1IXlsg0prYI/XyFWYuCC/WzGbhNCemg/dE9DttLQYW3jcuIH8G6iB2jiZgVKkR yX2M2GRqhHJwVWLgcq9uaypLO7biUtHSiyxlA+FS1nc2QaSHUjF8kGO/yXzVsNvvvT/f+93RQ ois/UaZ6UeW+l5gLxXtMC2Ruwfua6UOxVC9+vOwF/LiiJ/PgcCfcOzomoa6MjD1bJ+ILPRnJa 85IWnacAIL63LstZynrWfZo+Mrp5YNzc9XKc+/BniT14dUexrsvg/QrdNkrp9He+b+DUwcswx dOCxP4WF53pv+eV9VVD9LdC9deqPA3wmQhXsJy8RjaKvvpVv6Cs1oAu00RNb0aFg0kirvGG1U UIcJ48oHmIbKecFV Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday, December 13, 2016 10:35:34 PM CET Andrew Jeffery wrote: > On Tue, 2016-12-13 at 11:07 +0000, Lee Jones wrote: > > On Tue, 13 Dec 2016, Andrew Jeffery wrote: > > > On Mon, 2016-12-12 at 09:39 -0600, Rob Herring wrote: > > > > On Tue, Dec 06, 2016 at 01:53:21PM +1100, Andrew Jeffery wrote: > > > > > > Lee's next email in the chain poked Arnd for an opinion, but Arnd > > > didn't reply. > > > > > > I don't mind. I moved these bindings separately so we could just drop > > > the patch if there was push-back. If we drop the whole idea I'll need > > > to apply a small fix to patch 5/6 to avoid creating the syscon > > > subdirectory. > > > > The sub-directory is a good idea for drivers who are *solely* syscon > > based. > > > > Yes, I wasn't saying otherwise, just commenting on my motivation and > approach. > > As far as I can tell all of the bindings I move here describe solely > syscon-based devices. > But do we know which ones they are? In principle, any syscon device node can have a specialized driver exporting an interface, the bindings always allow it to be done one way or the other, and we may change the driver or run a different OS that has decided differently. Arnd