On 06/17/2016 05:01 PM, Fabio Estevam wrote:
Hi Tom,
On Mon, May 23, 2016 at 7:14 PM, Tom Rini trini@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 marex@denx.de Cc: Fabio Estevam fabio.estevam@nxp.com Cc: Peng Fan van.freenix@gmail.com Cc: Stefano Babic sbabic@denx.de Cc: Tom Rini trini@konsulko.com Reviewed-by: Fabio Estevam fabio.estevam@nxp.com Reviewed-by: Peng Fan van.freenix@gmail.com Reviewed-by: Stefano Babic sbabic@denx.de Reviewed-by: Tom Rini trini@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 trini@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 <marex@denx.de> Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com> Reviewed-by: Peng Fan <van.freenix@gmail.com> Reviewed-by: Stefano Babic <sbabic@denx.de> Reviewed-by: Tom Rini <trini@konsulko.com>