[U-Boot] [PATCH v8 4/4] EXYNOS: SMDK5250: Add MMC SPL support

Mike Frysinger vapier at gentoo.org
Fri Feb 3 20:31:47 CET 2012


On Friday 03 February 2012 11:05:57 Chander Kashyap wrote:
>  board/samsung/smdk5250/tools/mkexynos_image.c |  117
>  create mode 100644 tools/mksmdk5250spl.c

something funky in your diffstat ... says you're creating tools/ but the 
diffstat shows board/.../tools/.  but the diff below looks correct.

> --- /dev/null
> +++ b/tools/mksmdk5250spl.c

you could still name it "mkexynos_image".  up to you.

> +	if (fstat(ifd, &stat)) {
> +		close(ifd);
> +		close(ofd);
> +	}

looks like you should issue a warning + exit here
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120203/6cc939af/attachment.pgp>


More information about the U-Boot mailing list