devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>
To: Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Michal Marek <mmarek-AlSwsSmVLrQ@public.gmane.org>,
	Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
Subject: Re: dtc: import latest upstream dtc
Date: Thu, 11 Oct 2012 10:16:23 +1100	[thread overview]
Message-ID: <20121010231623.GG28467@truffula.fritz.box> (raw)
In-Reply-To: <5075954B.8030008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

On Wed, Oct 10, 2012 at 10:33:31AM -0500, Rob Herring wrote:
> On 10/10/2012 10:16 AM, Stephen Warren wrote:
> > On 10/10/2012 01:24 AM, David Gibson wrote:
> >> On Tue, Oct 09, 2012 at 10:43:50PM -0600, Warner Losh wrote:
> >>> On Oct 9, 2012, at 6:04 PM, Scott Wood wrote:
[snip]
> > That's probably a reasonable idea, although I imagined that people would
> > actually split out the portions of any header file they wanted to use
> > with dtc, so that any headers included by *.dts would only include
> > #defines. Those headers could be used by both dtc and other .h files (or
> > .c files).
> 
> Used by what other files? kernel files? We ultimately want to split out
> dts files from the kernel, so whatever we add needs to be self
> contained. I don't see this as a huge issue though because the whole
> point of the DT data is to move that information out of the kernel. If
> it is needed in both places, then something is wrong.

People get very hung up on this idea of having the DT move device
information out of the kernel, but that was never really the
motivation behind it.  Or at least, not the only or foremost
motivation.

The DT provides a consistent, flexible way of describing device
information.  That allows the core runtime the kernel to operate the
same way, regardless of how the DT information was obtained.  The DT
could come from firmware, but it could also come from an intermediate
bootloader or from early kernel code.  All are perfectly acceptable
options depending on the constraints of the platform.

The idea of firmware supplying the DT is much touted, but while it's a
theoretically nice idea, I think it's frequently a bad idea for
practical reasons.  Those being, in essence that a) firmware usually
sucks, b) it's usually harder (or at least no easier) to replace
firmware with a fixed version than the kernel/bootwrapper and c)
firmware usually *really* sucks.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

  parent reply	other threads:[~2012-10-10 23:16 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-28 21:25 [PATCH] dtc: import latest upstream dtc Stephen Warren
2012-09-29 21:06 ` Jon Loeliger
2012-10-01 16:09 ` Rob Herring
     [not found]   ` <5069C042.40209-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-10-01 16:13     ` Stephen Warren
     [not found]       ` <5069C11C.6040505-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-10-01 17:56         ` Rob Herring
2012-10-01 18:33           ` Stephen Warren
     [not found]             ` <5069E1F0.5070902-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-10-01 18:39               ` Jon Loeliger
     [not found]                 ` <E1TIktZ-0000U4-Qh-CYoMK+44s/E@public.gmane.org>
2012-10-09 21:16                   ` Stephen Warren
     [not found]                     ` <50749441.8030307-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-10-09 23:20                       ` Mitch Bradley
2012-10-10  0:04                         ` Scott Wood
2012-10-10  4:43                           ` Warner Losh
2012-10-10  7:24                             ` David Gibson
     [not found]                               ` <20121010072401.GA28467-W9XWwYn+TF0XU02nzanrWNbf9cGiqdzd@public.gmane.org>
2012-10-10 14:41                                 ` Warner Losh
2012-10-10 23:06                                   ` David Gibson
2012-10-10 15:16                               ` Stephen Warren
2012-10-10 15:33                                 ` Rob Herring
2012-10-10 16:19                                   ` Stephen Warren
2012-10-10 17:18                                     ` Rob Herring
2012-10-10 18:42                                       ` Stephen Warren
     [not found]                                   ` <5075954B.8030008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-10-10 23:16                                     ` David Gibson [this message]
2012-10-11  1:42                                       ` Mitch Bradley
     [not found]                                         ` <50762409.5060105-D5eQfiDGL7eakBO8gow8eQ@public.gmane.org>
2012-10-11  5:11                                           ` David Gibson
     [not found]                                 ` <50759152.9050407-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-10-10 23:09                                   ` David Gibson
2012-10-10 15:15                           ` Stephen Warren
2012-10-10 16:09                             ` Scott Wood
2012-10-10 16:22                               ` Stephen Warren
2012-10-10 23:18                                 ` David Gibson
2012-10-12 17:24                                   ` Stephen Warren
     [not found]                                     ` <5078525B.9030008-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-10-13  6:24                                       ` David Gibson
     [not found]                                         ` <20121013062453.GH4640-W9XWwYn+TF0XU02nzanrWNbf9cGiqdzd@public.gmane.org>
2012-10-13 13:42                                           ` Segher Boessenkool
     [not found]                                             ` <3C5DD611-6F36-4D13-9A88-377A8E30AAA5-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2012-10-14  0:16                                               ` David Gibson
2012-10-10 17:09                     ` Rob Herring
     [not found]                       ` <5075ABB8.103-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-10-10 18:23                         ` Mitch Bradley
     [not found]                           ` <5075BD21.2070106-D5eQfiDGL7eakBO8gow8eQ@public.gmane.org>
2012-10-10 18:45                             ` Stephen Warren
     [not found]                               ` <5075C254.4040304-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-10-10 18:56                                 ` Mitch Bradley
2012-10-11  0:14                                   ` David Gibson
2012-10-10 23:54                               ` David Gibson
2012-10-10 18:40                       ` Stephen Warren
     [not found]                         ` <5075C10C.1030205-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-10-10 18:52                           ` Mitch Bradley
2012-10-01 18:02     ` Jon Loeliger

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=20121010231623.GG28467@truffula.fritz.box \
    --to=david-xt8fgy+axnrb3ne2bgzf6laj5h9x9tb+@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mmarek-AlSwsSmVLrQ@public.gmane.org \
    --cc=robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
    --cc=swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    /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).