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

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to