linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: "Grant Likely" <grant.likely@secretlab.ca>
To: "Wolfgang Denk" <wd@denx.de>
Cc: linuxppc-dev@ozlabs.org, Marian Balakowicz <m8@semihalf.com>
Subject: Re: [PATCH v3 04/13] [POWERPC] Add generic support for simple MPC5200 based boards
Date: Tue, 6 Nov 2007 16:25:21 -0700	[thread overview]
Message-ID: <fa686aa40711061525y79f9f99eneda32b09a77ddcd5@mail.gmail.com> (raw)
In-Reply-To: <20071106222224.D67BA2479C@gemini.denx.de>

On 11/6/07, Wolfgang Denk <wd@denx.de> wrote:
> Dear Grant,
>
> in message <fa686aa40711061304k4779d01cu7fd1b17d1d34e5a2@mail.gmail.com> you wrote:
> >
> > In other words; make the assumption that it is easier to change the
> > kernel than it is to change the device tree.
>
> Are you serious about this?
>
> Reading this from someone with your experience with device trees if
> feeding my worst fears...

I think I better clarify.

Once a device tree is written and shipped on a deployed board, it may
never change again.  Or, the kernel version may be updated more
frequently than the device tree.

Say, for example, that in kernel 2.6.25 tqm5200 and cm5200 are both
handled by the same platform code.  And lets say that in 2.6.26 we
decide that they really need to have separate platform code (perhaps
due to a firmware bug that needs to be worked around on one board).
In this case, "mpc5200-simple-platform" has suddenly become useless.
Or, does mpc5200-simple-platform now describe the cm5200 or the
tqm5200?  (an assumption which cannot be made due to deployed boards
of both types claiming "mpc5200-simple-platform").

Trying to claim "compatible" at the board level is far more difficult
than claiming it at the device level.

Segher suggested on IRC: "for boards it is pretty much useless most of
the time, i think -- use "model" instead"

Cheers,
g.



>
> Best regards,
>
> Wolfgang Denk
>
> --
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de
> "Text processing has made it possible to right-justify any idea, even
> one which cannot be justified on any other grounds."
>                                                  -- J. Finnegan, USC.
>


-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
grant.likely@secretlab.ca
(403) 399-0195

  reply	other threads:[~2007-11-06 23:25 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-06 20:04 [PATCH v3 00/13] [POWERPC] Add TQM5200/CM5200/Motion-PRO board support Marian Balakowicz
2007-11-06 20:04 ` [PATCH v3 01/13] [POWERPC] Add 'model: ...' line to common show_cpuinfo() Marian Balakowicz
2007-11-06 20:05 ` [PATCH v3 02/13] [POWERPC] Add 'fsl, lpb' bus type for MPC5200 LocalPlus Bus Marian Balakowicz
2007-11-06 20:05 ` [PATCH v3 03/13] [POWERPC] Add common mpc52xx_setup_pci() routine Marian Balakowicz
2007-11-06 20:05 ` [PATCH v3 04/13] [POWERPC] Add generic support for simple MPC5200 based boards Marian Balakowicz
2007-11-06 21:04   ` Grant Likely
2007-11-06 22:22     ` Wolfgang Denk
2007-11-06 23:25       ` Grant Likely [this message]
2007-11-09 14:43         ` Marian Balakowicz
2007-11-09 14:52           ` Grant Likely
2007-11-07  2:34   ` Stephen Rothwell
2007-11-09 14:11     ` Marian Balakowicz
2007-11-06 20:05 ` [PATCH v3 05/13] [POWERPC] Export mpc52xx_map_node() routine symbol Marian Balakowicz
2007-11-06 20:05 ` [PATCH v3 06/13] [POWERPC] Use EXPORT_SYMBOL_GPL for 52xx common routines symbol export Marian Balakowicz
2007-11-06 20:05 ` [PATCH v3 07/13] [POWERPC] TQM5200 DTS Marian Balakowicz
2007-11-06 22:36   ` David Gibson
2007-11-09 14:15     ` Marian Balakowicz
2007-11-06 20:05 ` [PATCH v3 08/13] [POWERPC] TQM5200 defconfig Marian Balakowicz
2007-11-06 20:06 ` [PATCH v3 09/13] [POWERPC] CM5200 DTS Marian Balakowicz
2007-11-06 20:06 ` [PATCH v3 10/13] [POWERPC] CM5200 defconfig Marian Balakowicz
2007-11-06 20:06 ` [PATCH v3 11/13] [POWERPC] Motion-PRO: Add LED support Marian Balakowicz
2007-11-06 20:06 ` [PATCH v3 12/13] [POWERPC] Promess Motion-PRO DTS Marian Balakowicz
2007-11-06 22:42   ` David Gibson
2007-11-09 14:22     ` Marian Balakowicz
2007-11-09 17:49     ` Grant Likely
2007-11-06 20:06 ` [PATCH v3 13/13] [POWERPC] Promess Motion-PRO defconfig Marian Balakowicz

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=fa686aa40711061525y79f9f99eneda32b09a77ddcd5@mail.gmail.com \
    --to=grant.likely@secretlab.ca \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=m8@semihalf.com \
    --cc=wd@denx.de \
    /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).