[U-Boot] [PATCH v11 2/2] Enable btrfs support in mx53loco config

[Robert Nelson]

On Tue, Apr 2, 2013 at 10:38 AM, Adnan Ali <adnan.ali at codethink.co.uk> wrote:
> On 02/04/13 16:19, Robert Nelson wrote:
>>
>> On Tue, Apr 2, 2013 at 9:17 AM, Adnan Ali <adnan.ali at codethink.co.uk>
>> wrote:
>>>
>>> Enable btrfs support in mx53loco config
>>>
>>> Signed-off-by: Adnan Ali <adnan.ali at codethink.co.uk>
>>> ---
>>>   include/configs/mx53loco.h |    4 +++-
>>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h
>>> index a4b610f..62e9a76 100644
>>> --- a/include/configs/mx53loco.h
>>> +++ b/include/configs/mx53loco.h
>>> @@ -56,6 +56,8 @@
>>>   #define CONFIG_GENERIC_MMC
>>>   #define CONFIG_CMD_FAT
>>>   #define CONFIG_CMD_EXT2
>>> +#define CONFIG_CMD_BTR
>>> +#define CONFIG_CMD_FS_GENERIC
>>>   #define CONFIG_DOS_PARTITION
>>>
>>>   /* Eth Configs */
>>> @@ -128,7 +130,7 @@
>>>          "mmcroot=/dev/mmcblk0p3 rw rootwait\0" \
>>>          "mmcargs=setenv bootargs console=ttymxc0,${baudrate}
>>> root=${mmcroot}\0" \
>>>          "loadbootscript=" \
>>> -               "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr}
>>> ${script};\0" \
>>> +               "btrload mmc ${mmcdev}:${mmcpart} ${loadaddr}
>>> ${script};\0" \
>>
>> Instead of changing this to btrload for everyone, wouldn't it make
>> more sense to use the generic "load" command? As your already setting
>> "CONFIG_CMD_FS_GENERIC"
>
>    Well idea of adding that was to enable btrfs and to show its associated
> commands.
> Yes you can use generic 'load' command. Defaults was using fatload so i
> change
> it to btrload.

That's perfectly fine for showing the btrfs command's as an RFC patch,
but if this was heading for mainline as-is, it would be nice to use
the "load" command instead of moving from one partition format that's
been default for a couple years to a new format with less users. (not
that I don't like the btrfs format. ;) as i've been running it on a
few omap boards for a couple years now..)

Regards,

-- 
Robert Nelson
http://www.rcn-ee.com/