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

Simon Glass sjg at chromium.org
Fri Mar 8 19:15:30 CET 2013


Hi Adnan,

On Fri, Mar 8, 2013 at 3:11 AM, Adnan Ali <adnan.ali at codethink.co.uk> wrote:

> Hi
>
> On 08/03/13 05:03, Simon Glass wrote:
>
>> +U-Boot et al
>>
>> Hi,
>>
>> On Thu, Mar 7, 2013 at 2:35 AM, Adnan Ali <adnan.ali at codethink.co.uk>
>> 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 269ebc845ebc8b46ef4b0be7fa0005**c7fdb95b8d.
>>>
>> Here are some comments, hope it helps.
>>
>     Most of your comments in btrfs.c is about ported code.
>    which we have already discussed to leave it as it for easy
>    of future re sync. I will also wait for others to respond,
>    so that i know what to change before sending other
>    revision of patch.  Please find my comments below
>

Yes, I'm sorry, now I remember that discussion. I think if you can fit it
in to work with the new fs.c changes then all will be well. Please just
take account of my comments that relate to fitting it into U-Boot, and
ignore the ones that concern code style/readability.

Regards,
Simon


More information about the U-Boot mailing list