Hi Adnan, On Fri, Mar 8, 2013 at 3:11 AM, Adnan Ali <adnan....@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....@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
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
