Hi Adnan,

On Fri, Mar 8, 2013 at 3:11 AM, Adnan Ali adnan.ali@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@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