All of lore.kernel.org
 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: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-20  3:19 [PATCH 1/3] MAINTAINERS: Remove Grant Likely Grant Likely
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  3:19     ` 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
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  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-21  9:46           ` Grant Likely
2013-07-22  0:36           ` Grant Likely
2013-07-22  0:43             ` Joe Perches
2013-07-22  0:48               ` 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-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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.