Re: [U-Boot] [PATCH v2 1/3] am335x-evm: enable ext4
On Mon, Feb 18, 2013 at 07:26:42AM +0100, Peter Korsgaard wrote: Koen == Koen Kooi k...@dominion.thruhere.net writes: Koen #define CONFIG_CMD_EXT2 Koen +#define CONFIG_CMD_EXT4 Shouldn't the bootcmd then also be changed to use ext4load instead? Why keep CMD_EXT2 enabled? Koen that will be a followup patch Ok. It imho makes sense to do both changes at once as they are strongly related and in the same file. And we ought to add CONFIG_CMD_FS_GENERIC and just 'load' so that this can more easily work with folks still using FAT for boot. -- Tom ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/3] am335x-evm: enable ext4
Koen == Koen Kooi k...@dominion.thruhere.net writes: Koen The kernel is loaded from an ext4 partition, not ext2 on beaglebone boards. Koen Signed-off-by: Koen Kooi k...@dominion.thruhere.net Koen --- Koen include/configs/am335x_evm.h | 1 + Koen 1 file changed, 1 insertion(+) Koen diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h Koen index 72459d8..2190a7d 100644 Koen --- a/include/configs/am335x_evm.h Koen +++ b/include/configs/am335x_evm.h Koen @@ -144,6 +144,7 @@ Koen #define CONFIG_DOS_PARTITION Koen #define CONFIG_CMD_FAT Koen #define CONFIG_CMD_EXT2 Koen +#define CONFIG_CMD_EXT4 Shouldn't the bootcmd then also be changed to use ext4load instead? Why keep CMD_EXT2 enabled? -- Bye, Peter Korsgaard ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/3] am335x-evm: enable ext4
Op 17 feb. 2013 om 14:03 heeft Peter Korsgaard jac...@sunsite.dk het volgende geschreven: Koen == Koen Kooi k...@dominion.thruhere.net writes: Koen The kernel is loaded from an ext4 partition, not ext2 on beaglebone boards. Koen Signed-off-by: Koen Kooi k...@dominion.thruhere.net Koen --- Koen include/configs/am335x_evm.h | 1 + Koen 1 file changed, 1 insertion(+) Koen diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h Koen index 72459d8..2190a7d 100644 Koen --- a/include/configs/am335x_evm.h Koen +++ b/include/configs/am335x_evm.h Koen @@ -144,6 +144,7 @@ Koen #define CONFIG_DOS_PARTITION Koen #define CONFIG_CMD_FAT Koen #define CONFIG_CMD_EXT2 Koen +#define CONFIG_CMD_EXT4 Shouldn't the bootcmd then also be changed to use ext4load instead? Why keep CMD_EXT2 enabled? that will be a followup patch -- Bye, Peter Korsgaard ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/3] am335x-evm: enable ext4
Koen == Koen Kooi k...@dominion.thruhere.net writes: Koen #define CONFIG_CMD_EXT2 Koen +#define CONFIG_CMD_EXT4 Shouldn't the bootcmd then also be changed to use ext4load instead? Why keep CMD_EXT2 enabled? Koen that will be a followup patch Ok. It imho makes sense to do both changes at once as they are strongly related and in the same file. -- Bye, Peter Korsgaard ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH v2 1/3] am335x-evm: enable ext4
The kernel is loaded from an ext4 partition, not ext2 on beaglebone boards. Signed-off-by: Koen Kooi k...@dominion.thruhere.net --- include/configs/am335x_evm.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h index 72459d8..2190a7d 100644 --- a/include/configs/am335x_evm.h +++ b/include/configs/am335x_evm.h @@ -144,6 +144,7 @@ #define CONFIG_DOS_PARTITION #define CONFIG_CMD_FAT #define CONFIG_CMD_EXT2 +#define CONFIG_CMD_EXT4 #define CONFIG_SPI #define CONFIG_OMAP3_SPI -- 1.8.1.2 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
