All of lore.kernel.org
 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 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.