Re: [U-Boot] [PATCH v2 2/3] am335x_evm: Enable CMD_EXT4 and CMD_FS_GENERIC, add bootpart to env

...

...

...

...

"Tom" == Tom Rini trini@ti.com writes:

Hi,

...

You could argue this should be ${mmcdev}:2 instead.

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

Ahh, ok.

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?

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

Ok, could be good to mention that in the commit message.