From mboxrd@z Thu Jan 1 00:00:00 1970 From: shawn.guo@linaro.org (Shawn Guo) Date: Fri, 10 Jan 2014 10:41:29 +0800 Subject: DT include files (was: [GIT PULL 2/2] ARM: imx: device tree changes for 3.14) In-Reply-To: <20140104011056.GA3282@S2101-09.ap.freescale.net> References: <20131231054427.GA22383@S2101-09.ap.freescale.net> <20140102202108.GF19720@quad.lixom.net> <20140103023211.GA25079@S2101-09.ap.freescale.net> <20140103030455.GB25079@S2101-09.ap.freescale.net> <20140104011056.GA3282@S2101-09.ap.freescale.net> Message-ID: <20140110024124.GA6844@S2101-09.ap.freescale.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Sat, Jan 04, 2014 at 09:10:58AM +0800, Shawn Guo wrote: > On Fri, Jan 03, 2014 at 11:29:35AM -0800, Olof Johansson wrote: > > On Thu, Jan 2, 2014 at 7:04 PM, Shawn Guo wrote: > > > On Thu, Jan 02, 2014 at 06:41:30PM -0800, Olof Johansson wrote: > > >> Ok, then it's probably just the location of the header files that > > >> should be adjusted. Other subsystems have placed them under > > >> include/dt-bindings/, so that's likely a better place for > > >> these as well, don't you think? > > > > > > I had a little discussion with DT people when the headers were firstly > > > created. These pinctrl headers are a little different from the headers > > > in include/dt-bindings/. The latter are used by both kernel > > > and device tree sources, while the pinctrl headers are used by device > > > tree sources only, so I chose to put them just in the same folder as > > > dts files. And DT people are fine with my take. > > > > Since you don't provide references to this I had to go searching for > > it. All I find is some discussion from 8 months ago, and quite a bit > > of that seems to have been about changing bindings, and some about the > > preprocessor behavior. Also, the patches seem to have been too big to > > make it out on the lists. > > > > I'd like a fresh look from DT people on this just to make sure no > > opinions have changed -- lots of things have changed in the last 8 > > months w.r.t. DT. > > Indeed, it's been quite a long time. Let me restate my point. The > include/dt-bindings is introduced as a folder to hold headers that are > referenced by both kernel and DTS. That's why we create the folder in > the kernel include folder and have arch/arm/boot/dts/include/dt-bindings > being a symbol link to it. All the headers in there need to be > duplicated between kernel and DTS tree, when we move DTS files into > a separated repository. Putting DTS local headers into the folder is > absolutely unnecessary, and will only confuse people and bother > ourselves when moving DTS files out of kernel tree. Just a gentle ping to ensure we do not get the pull request lost. Or do you have any further comment? Shawn