On 23 January 2014 12:56, Stephen Warren <swar...@wwwdotorg.org> wrote:

> From: Stephen Warren <swar...@nvidia.com>
>
> Fix a few issues with the generic "save" shell command, and fs_write()
> function.
>
> 1) fstypes[].write wasn't filled in for some file-systems, and isn't
>    checked when used, which could cause crashes/... if executing save
>    on e.g. fat/ext filesystems.
>
> 2) fs_write() requires the length argument to be non-zero, since it needs
>    to know exactly how many bytes to write. Adjust the comments and code
>    according to this.
>
> 3) fs_write() wasn't prototyped in <fs.h> like other generic functions;
>    other code should be able to call this directly rather than invoking
>    the "save" shell command.
>
> Cc: Simon Glass <s...@chromium.org>
> Signed-off-by: Stephen Warren <swar...@nvidia.com>
>

Not sure about the change log for v2, but the patch looks good.

Acked-by: Simon Glass <s...@chromium.org>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to