On 06/17/2016 05:01 PM, Fabio Estevam wrote: > Hi Tom, > > On Mon, May 23, 2016 at 7:14 PM, Tom Rini <tr...@konsulko.com> wrote: >> On Fri, May 20, 2016 at 01:45:24AM +0200, Marek Vasut wrote: >> >>> The spl_parse_image_header() can return 0 and it is not an error. >>> Only treat non-zero return value as an error. >>> >>> Signed-off-by: Marek Vasut <ma...@denx.de> >>> Cc: Fabio Estevam <fabio.este...@nxp.com> >>> Cc: Peng Fan <van.free...@gmail.com> >>> Cc: Stefano Babic <sba...@denx.de> >>> Cc: Tom Rini <tr...@konsulko.com> >>> Reviewed-by: Fabio Estevam <fabio.este...@nxp.com> >>> Reviewed-by: Peng Fan <van.free...@gmail.com> >>> Reviewed-by: Stefano Babic <sba...@denx.de> >>> Reviewed-by: Tom Rini <tr...@konsulko.com> >> >> Applied to u-boot/master, thanks! > > Looks like something went wrong while applying this patch as Marek's > authorship got lost:
Ah, you're right, nice catch :) Tom, I don't really care, bug is fixed, you can leave it as it is. > commit d550e82e4efebf5995809e38e8fb029b8fe6e861 > Author: Tom Rini <tr...@konsulko.com> > Date: Mon May 23 11:51:13 2016 -0400 > > SPL: fat: Fix spl_parse_image_header() return value handling > > The spl_parse_image_header() can return 0 and it is not an error. > Only treat non-zero return value as an error. > > Signed-off-by: Marek Vasut <ma...@denx.de> > Reviewed-by: Fabio Estevam <fabio.este...@nxp.com> > Reviewed-by: Peng Fan <van.free...@gmail.com> > Reviewed-by: Stefano Babic <sba...@denx.de> > Reviewed-by: Tom Rini <tr...@konsulko.com> > -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot