All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marian Balakowicz <m8@semihalf.com>
To: Grant Likely <grant.likely@secretlab.ca>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: [PATCH v3 04/13] [POWERPC] Add generic support for simple MPC5200 based boards
Date: Fri, 09 Nov 2007 15:43:38 +0100	[thread overview]
Message-ID: <4734721A.6030603@semihalf.com> (raw)
In-Reply-To: <fa686aa40711061525y79f9f99eneda32b09a77ddcd5@mail.gmail.com>


Hi Grant,

Grant Likely wrote:
> On 11/6/07, Wolfgang Denk <wd@denx.de> wrote:
>>
>> 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"

I can imagine that we may get into various trouble (or at least the
situation is less flexible) if we are unable to update .dts file along
with the kernel image on a deployed board. If so, then in fact there
is little sens in using "mpc5200-simple-platform" compatible.

But how serious is that, does such situation frequently happen in
field? If we are able to update kernel image than what prevents .dts
file update?

Cheers,
m.

  reply	other threads:[~2007-11-09 14:43 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
2007-11-09 14:43         ` Marian Balakowicz [this message]
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=4734721A.6030603@semihalf.com \
    --to=m8@semihalf.com \
    --cc=grant.likely@secretlab.ca \
    --cc=linuxppc-dev@ozlabs.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 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.