On Thu, Jan 9, 2014 at 6:41 PM, Shawn Guo <shawn.guo@xxxxxxxxxx> wrote: > 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 <shawn.guo@xxxxxxxxxx> 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/<subsystem>, 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/<subsystem>. 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? Still waiting on DT maintainers to chime in. -Olof -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
