public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 2/3] am335x_evm: Enable CMD_EXT4 and CMD_FS_GENERIC, add bootpart to env
Date: Wed, 13 Mar 2013 11:12:08 -0400	[thread overview]
Message-ID: <51409748.8020601@ti.com> (raw)
In-Reply-To: <877glbs5m9.fsf@dell.be.48ers.dk>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 03/13/2013 11:04 AM, Peter Korsgaard wrote:
>>>>>> "Tom" == Tom Rini <trini@ti.com> writes:
> 
> Tom> From: Koen Kooi <koen@dominion.thruhere.net> Tom> The kernel
> is loaded from some form of ext[234] or FAT, depending on the Tom>
> distribution used.  We add a bootpart variable to the environment
> so Tom> that we can load from the correct mmc partition as well.
> 
> Tom> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net> Tom>
> Signed-off-by: Tom Rini <trini@ti.com> Tom> --- Tom>
> include/configs/am335x_evm.h |   10 ++++++---- Tom>  1 file
> changed, 6 insertions(+), 4 deletions(-)
> 
> Tom> diff --git a/include/configs/am335x_evm.h
> b/include/configs/am335x_evm.h Tom> index abf4e39..ec4ddbc 100644 
> Tom> --- a/include/configs/am335x_evm.h Tom> +++
> b/include/configs/am335x_evm.h Tom> @@ -61,6 +61,7 @@ Tom>
> "mmcdev=0\0" \ Tom>  	"mmcroot=/dev/mmcblk0p2 ro\0" \ Tom>
> "mmcrootfstype=ext4 rootwait\0" \ Tom> +	"bootpart=0:2\0" \
> 
> You could argue this should be ${mmcdev}:2 instead.

Should but can't, sadly.  I did that at first but we don't get
re-expansion and it becomes later on 'load mmc ${mmcdev}:2 ...'.

> Tom> @@ -147,6 +147,8 @@ Tom>  #define CONFIG_DOS_PARTITION Tom>
> #define CONFIG_CMD_FAT Tom>  #define CONFIG_CMD_EXT2 Tom> +#define
> CONFIG_CMD_EXT4 Tom> +#define CONFIG_CMD_FS_GENERIC
> 
> CMD_EXT2 can presumably be dropped now?

I'd rather not in case people have scripts using ext2load already.

- -- 
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJRQJdIAAoJENk4IS6UOR1WumgP+wZRgnKi0Zds9g7a9rODiu0U
2WyzUE912hc1ifq0o2IEHzksrVBvg6wSbQtHFFPJKbecJf11V/SZINWnQXVbd8xU
IRPmyd12CbdYK+XiJbIRVCfJueHYIK5kgZo96vm0w3Hh90yuUVfvJtrYu2nlqad5
QOXXwfWRuD/VYZSjIUVKxG4RJBUVv2RrIUCFOMdxoC8osu/Ea/bCm178+kdIGvSC
Ij3dRefn6IsnEoa+VswY4oA9IRP69nTbIGZBXUQk+haIEjMKcYV2RknkKN9YW0Cj
6viKXJJm8HenBdbZ0ZSv7HWKPqtN5VWvzqHbMjj09r1P1WQ1jT+CERdNdIN5DYI0
YvTgBnyuOmw4b0WI5GQVnk3gNhX5PEw/OQqsI2WTlV/5kmckhz8+xNby5t8khxET
lphYKI7bLedpOETrwXl6SdzkeHQ8NfT2vSKKy2GBKBA9JtY3mMpxx2A89tgItw2B
QSLYZn425mMLPQzH6eOt7kXphqyWodbiypn/WFDRD6lhtD3yMZwmpZ5wHIkptw9r
GA1LIp7R+MMDu4srJvsF8VH1bKuDFB8Nde44iCD9JwSMqtiycbf3wMXrsKbQxLld
KQXvMiFZMz4VESiIGNEFvKMTsmbDeKnKsFt37eoUWyAeHCkbP/Or5gpD7uhkyaEm
zByAl7YtpdYbfSlEco+o
=tasN
-----END PGP SIGNATURE-----

  reply	other threads:[~2013-03-13 15:12 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-13 14:20 [U-Boot] [PATCH v2 0/3] am335x_evm: Switch to DT boot, add another platform Tom Rini
2013-03-13 14:20 ` [U-Boot] [PATCH v2 1/3] am335x_evm: add support for BeagleBone Black DT name Tom Rini
2013-03-13 14:35   ` Nishanth Menon
2013-03-13 14:57     ` Koen Kooi
2013-03-13 15:02       ` Nishanth Menon
2013-03-13 15:05         ` Koen Kooi
2013-03-13 15:07           ` Nishanth Menon
2013-03-13 19:02             ` Koen Kooi
2013-03-13 19:26               ` Nishanth Menon
2013-03-13 20:19               ` Matt Porter
2013-03-14  6:53                 ` Koen Kooi
2013-03-14 15:07                   ` Matt Porter
2013-03-13 14:20 ` [U-Boot] [PATCH v2 2/3] am335x_evm: Enable CMD_EXT4 and CMD_FS_GENERIC, add bootpart to env Tom Rini
2013-03-13 15:04   ` Peter Korsgaard
2013-03-13 15:12     ` Tom Rini [this message]
2013-03-13 17:40       ` Peter Korsgaard
2013-03-13 14:20 ` [U-Boot] [PATCH v2 3/3] am335x_evm: Add more variables and switch to DT booting Tom Rini
2013-03-13 15:06   ` Peter Korsgaard

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=51409748.8020601@ti.com \
    --to=trini@ti.com \
    --cc=u-boot@lists.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