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-----
next prev parent 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