[U-Boot] [PATCH RESEND] Introduced btrfs file-system with btrload command

Adnan Ali adnan.ali at codethink.co.uk
Thu Mar 28 18:45:33 CET 2013


On 28/03/13 17:04, Tom Rini wrote:
> On Thu, Mar 21, 2013 at 02:04:45PM +0000, Adnan Ali wrote:
>
>> Introduces btrfs file-system to read file from
>> volume/sub-volumes with btrload command. This
>> implementation has read-only support.
>> This btrfs implementation is based on syslinux btrfs
>> code, commit 269ebc845ebc8b46ef4b0be7fa0005c7fdb95b8d.
>>
>> v9:     patch problem reworked.
>> v8:     patch problem reworked.
>> v5:     merged with master.
>> v4:     btrls command added.
>>
>> Signed-off-by: Adnan Ali <adnan.ali at codethink.co.uk>
> First, this should be v10 not "RESEND".  And that means the next version
> should be v11.
      ok
>
> One of the rules of adding code to U-Boot is we don't add "dead" code
> that's not built somewhere.  So you should be adding CONFIG_CMD_BTR
> somewhere like to say include/configs/sandbox.h (which has ext4/fat
> support, but can't yet use them).
      You mean adding to one of the configs, I used include/configs/mx53loco
   the reason i didn't added config file to patch as some one told not to
add config file as part of port. Is that what you mean.
>
> Another rule is no adding code with compiler warning.  This code has
> _a_lot_ of warnings.  This is due in part to how you've ported the
> syslinux generic inode/fs_info structure over to U-Boot (as we need to
> spend some time here cleaning up our code, but I'm not asking you to do
> that).  I started digging into fixing these warnings but then found your
     Thanks if you are doing it.
> changes to btrfs_read_super_block() where you removed the syslinux check
> over all possible superblock areas.  Why are we deviating here?  Are
> there other deviations from syslinux in the driver itself?  Thanks.
>
      The first super is the real one the rest of them are just mirrors
of the first .  If you want it to be added i can do it.





More information about the U-Boot mailing list