From: Sascha Hauer <s.hauer@pengutronix.de>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 8/9] Add configurability via devicetree
Date: Tue, 16 Jul 2013 16:27:22 +0200 [thread overview]
Message-ID: <20130716142722.GA10380@pengutronix.de> (raw)
In-Reply-To: <20130715101119.GH19197@ns203013.ovh.net>
On Mon, Jul 15, 2013 at 12:11:19PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 22:17 Wed 10 Jul , Sascha Hauer wrote:
> > On Wed, Jul 10, 2013 at 05:50:28PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > > > > > It's the same binding, yes.
> > > > >
> > > > > I do not like to duplicate the information can de avoid this
> > > >
> > > > Which information? The Documentation?
> > >
> > > no the fact we need to describe the nor-partitions twice, for barebox and for
> > > linux in the DTB
> >
> > No, barebox should copy (or move) the partition nodes to where Linux
> > finds them before starting the kernel. This is not yet implemented
> > though.
> >
> > Another option of course is to do the obvious and put the partitions
> > under the mtd device in the first place. I decided against this since
> > I have the strong feeling that the hardware description should be
> > separated from clearly board (or project or usecase) specific partition
> > descriptions.
>
> here we need to be more generic as it's booloader specific not really barebox
As mentioned I moved from /chosen/barebox to /chosen. I'd like to keep
the "barebox," prefixes in the compatible strings as long as barebox is
the only bootloader doing such stuff. It's more convenient to drop the
prefix later than it is to realize that we introduced a too generic
binding.
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2013-07-16 14:27 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-10 10:51 [PATCH] Allow configuration from the devicetree Sascha Hauer
2013-07-10 10:52 ` [PATCH 1/9] devfs: let devfs_add_partition return the new partition Sascha Hauer
2013-07-10 10:52 ` [PATCH 2/9] of: export of_default_bus_match_table Sascha Hauer
2013-07-10 10:52 ` [PATCH 3/9] of: partitions: factor out function to parse a single partition Sascha Hauer
2013-07-10 10:52 ` [PATCH 4/9] cdev: introduce partition names Sascha Hauer
2013-07-10 10:52 ` [PATCH 5/9] cdev: allow to open a struct cdev Sascha Hauer
2013-07-10 10:52 ` [PATCH 6/9] cdev: add device_find_partition Sascha Hauer
2013-07-10 10:52 ` [PATCH 7/9] mci: set partnames of eMMC boot partitions Sascha Hauer
2013-07-10 10:52 ` [PATCH 8/9] Add configurability via devicetree Sascha Hauer
2013-07-10 13:17 ` Jean-Christophe PLAGNIOL-VILLARD
2013-07-10 13:34 ` Sascha Hauer
2013-07-10 14:33 ` Jean-Christophe PLAGNIOL-VILLARD
2013-07-10 14:42 ` Sascha Hauer
2013-07-10 15:50 ` Jean-Christophe PLAGNIOL-VILLARD
2013-07-10 20:17 ` Sascha Hauer
2013-07-15 10:11 ` Jean-Christophe PLAGNIOL-VILLARD
2013-07-16 14:27 ` Sascha Hauer [this message]
2013-07-10 10:52 ` [PATCH 9/9] ARM: i.MX Datamodul edmqx6: configure environment from devicetree Sascha Hauer
2013-07-11 7:37 ` [PATCH] Allow configuration from the devicetree Sascha Hauer
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=20130716142722.GA10380@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=barebox@lists.infradead.org \
--cc=plagnioj@jcrosoft.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox