From: Segher Boessenkool <segher@kernel.crashing.org>
To: Josh Boyer <jwboyer@linux.vnet.ibm.com>
Cc: linuxppc-dev@ozlabs.org, Stefan Roese <sr@denx.de>,
david@gibson.dropbear.id.au
Subject: Re: [patch 3/4] 4xx: Convert Walnut flash mappings to new binding
Date: Sat, 15 Sep 2007 21:20:10 +0200 [thread overview]
Message-ID: <addcdbe0ab5b83aa58fc4e7df37b0925@kernel.crashing.org> (raw)
In-Reply-To: <1189877748.17593.9.camel@localhost.localdomain>
>> Maybe we can have U-Boot add the partition information if it's
>> missing in the device tree, and extend the mtdparts command in U-Boot
>> to add / adjust settings so they match what is defined in U-Boot.
>
> That would be great for newer U-Boots. For existing older ones, it
> doesn't really solve the problem. But then again, there's no way to
> possibly define all the partitioning schemes people may have adopted to
> their needs on their boards. And I believe that is why David has
> RedBoot and command line partitioning override what is in the DTS
> today.
Yeah, partitioning information really doesn't belong in the device
tree -- with the possible exception of the partitions that the
firmware (uboot in this case) needs to know about anyway.
You *can* put all partitioning info in the device tree, but that
doesn't mean you *should* :-)
Segher
next prev parent reply other threads:[~2007-09-15 19:20 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-14 18:54 [patch 0/4] Small 4xx updates for 2.6.24 Josh Boyer
2007-09-14 18:54 ` [patch 1/4] cuimage for Bamboo board Josh Boyer
2007-09-15 2:19 ` David Gibson
2007-09-15 14:26 ` Josh Boyer
2007-09-14 18:54 ` [patch 2/4] Make partitions optional in physmap_of Josh Boyer
2007-09-15 2:21 ` David Gibson
2007-09-14 18:54 ` [patch 3/4] 4xx: Convert Walnut flash mappings to new binding Josh Boyer
2007-09-15 2:22 ` David Gibson
2007-09-15 14:25 ` Josh Boyer
2007-09-15 3:23 ` Stefan Roese
2007-09-15 14:26 ` Josh Boyer
2007-09-15 15:09 ` Wolfgang Denk
2007-09-15 17:35 ` Josh Boyer
2007-09-15 19:20 ` Segher Boessenkool [this message]
2007-09-17 2:03 ` David Gibson
2007-09-17 2:02 ` David Gibson
2007-09-14 18:54 ` [patch 4/4] 4xx: Convert Seqouia " Josh Boyer
2007-09-15 2:23 ` David Gibson
2007-09-15 3:32 ` Stefan Roese
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=addcdbe0ab5b83aa58fc4e7df37b0925@kernel.crashing.org \
--to=segher@kernel.crashing.org \
--cc=david@gibson.dropbear.id.au \
--cc=jwboyer@linux.vnet.ibm.com \
--cc=linuxppc-dev@ozlabs.org \
--cc=sr@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 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.