devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: Pawel Moll <pawel.moll@arm.com>
Cc: Rob Herring <robherring2@gmail.com>,
	"grant.likely@linaro.org" <grant.likely@linaro.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Mark Rutland <Mark.Rutland@arm.com>,
	Ian Campbell <ian.campbell@citrix.com>,
	"rob.herring@calxeda.com" <rob.herring@calxeda.com>,
	Olof Johansson <olof@lixom.net>,
	"devicetree-discuss@lists.ozlabs.org"
	<devicetree-discuss@lists.ozlabs.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: [PATCH 3/3] MAINTAINERS: Refactor device tree maintainership
Date: Tue, 23 Jul 2013 10:59:54 -0700	[thread overview]
Message-ID: <51EEC49A.4080909@wwwdotorg.org> (raw)
In-Reply-To: <1374599667.25700.92.camel@hornet>

On 07/23/2013 10:14 AM, Pawel Moll wrote:
> On Mon, 2013-07-22 at 21:03 +0100, Rob Herring wrote:
>> On 07/22/2013 11:50 AM, Pawel Moll wrote:
>>> On Sat, 2013-07-20 at 04:19 +0100, Grant Likely wrote:
>>>> +F:	include/dt-bindings
>>>
>>> One thing we didn't finish talking about was the question if this
>>> directory is supposed to contain *.dtsi files as well? The obvious
>>> problem I have is a vexpress motherboard being (well, actually not bein
>>> right now) shared between arch/arm/boot/dts and arch/arm64/boot/dts.
>>
>> Please no. 
> 
> No as in: no don't put *.dtsi files into include/dt-bindings; or: no, do
> not duplicate the motherboard file?

Don't put *.dtsi into include/dt-bindings, I believe.

> It you meant the latter, this is exactly what I wanted to say: I don't
> want to do that, but there's no way of avoiding it right now.

I think the solution is to introduce some new shared/common location for
shared/common *.dtsi files, into the kernel tree, in the interim.

When *.dts move out of the kernel, this common location can simply be
consumed as part of the DT tree re-organization.

Or perhaps, we could move *.dts around in the kernel to match the
proposed DT tree structure before that point in time?

>> we will still need
>> to copy dt-bindings into the kernel. Also, I think we should move all
>> dts files out of arch subdirs and arrange by vendor or soc family. I'm
>> sure there are some cases that structure doesn't fit well, but there is
>> very little in a dts tied to a cpu architecture.
> 
> I couldn't agree more. So:
> 
> <root>/include/dt-bindings/vendor/*?

> <root>/dts/vendor/*?

I would tend to prefer that option, ...

> <root>/of/vendor/*?
> <root>/dt/vendor/*?

or perhaps that one.

> <root>/drivers/of/vendor/*?

  reply	other threads:[~2013-07-23 17:59 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1374290388-19308-1-git-send-email-grant.likely@linaro.org>
2013-07-20  3:19 ` [PATCH 2/3] MAINTAINERS: Change device tree mailing list Grant Likely
     [not found] ` <1374290388-19308-1-git-send-email-grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2013-07-20  3:19   ` [PATCH 3/3] MAINTAINERS: Refactor device tree maintainership Grant Likely
2013-07-20 16:20     ` Linus Walleij
     [not found]       ` <CACRpkdb2bPQOD2CBzbF0K33mZZwKwGSCO6B4djesVd5DR-Z1tA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-07-21 13:46         ` Jon Loeliger
     [not found]           ` <E1V0txk-0007v7-0J-CYoMK+44s/E@public.gmane.org>
2013-07-22  0:33             ` Grant Likely
2013-07-22  1:27           ` David Gibson
2013-07-22 17:38             ` Jon Loeliger
2013-07-21  0:17     ` Olof Johansson
2013-07-21  0:29       ` Joe Perches
2013-07-21  9:46         ` Grant Likely
2013-07-22  0:36           ` Grant Likely
2013-07-22 17:29           ` Olof Johansson
     [not found]     ` <1374290388-19308-3-git-send-email-grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2013-07-21 18:56       ` Rob Herring
2013-07-22  0:35         ` Grant Likely
2013-07-22 20:31           ` Rob Herring
2013-07-22 21:36             ` Jon Loeliger
2013-07-23  3:27               ` Rob Herring
2013-07-22 16:50     ` Pawel Moll
2013-07-22 20:03       ` Rob Herring
2013-07-23 17:14         ` Pawel Moll
2013-07-23 17:59           ` Stephen Warren [this message]
2013-07-23 18:09             ` Ian Campbell
2013-07-23 18:44               ` Rob Herring
2013-07-23 18:55                 ` Ian Campbell
2013-07-23 19:28                   ` Rob Herring
2013-07-23 19:53                   ` Stephen Warren
2013-08-07 10:53                 ` Ian Campbell
2013-07-23 19:26             ` Nicolas Pitre

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51EEC49A.4080909@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --cc=Mark.Rutland@arm.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=devicetree@vger.kernel.org \
    --cc=grant.likely@linaro.org \
    --cc=ian.campbell@citrix.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=olof@lixom.net \
    --cc=pawel.moll@arm.com \
    --cc=rob.herring@calxeda.com \
    --cc=robherring2@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).