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

Adnan Ali
2013-04-02T17:10:23Z

On 02/04/13 17:41, Otavio Salvador wrote:
> On Tue, Apr 2, 2013 at 12:38 PM, Adnan Ali <adnan.ali@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@codethink.co.uk>
>>> wrote:
>>>> Enable btrfs support in mx53loco config
>>>>
>>>> Signed-off-by: Adnan Ali <adnan.ali@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.
> If you use the generic load I would be OK in changing it.
I will send this as separate patch. But I leave it until
btrfs patch get accepted. If you are happy with that. ;)

>
> --
> Otavio Salvador O.S. Systems
> E-mail: otavio at ossystems.com.br http://www.ossystems.com.br
> Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br