[U-Boot] [PATCH] image: Use ENOENT instead of ENOMEDIUM for better compatibility

[U-Boot] [PATCH] image: Use ENOENT instead of ENOMEDIUM for better compatibility

[Simon Glass]

This error may not be defined on some platforms such as MacOS so host
compilation will fail. Use one of the more common errors instead.

Signed-off-by: Simon Glass <sjg@chromium.org>
---
 common/image-fit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/common/image-fit.c b/common/image-fit.c
index 7bf82d3..f40f160 100644
--- a/common/image-fit.c
+++ b/common/image-fit.c
@@ -1557,7 +1557,7 @@ int fit_image_load(bootm_headers_t *images, const char *prop_name, ulong addr,
 	if (fit_image_get_data(fit, noffset, &buf, &size)) {
 		printf("Could not find %s subimage data!\n", prop_name);
 		bootstage_error(bootstage_id + BOOTSTAGE_SUB_GET_DATA);
-		return -ENOMEDIUM;
+		return -ENOENT;
 	}
 	len = (ulong)size;
 
-- 
1.8.3

[U-Boot] image: Use ENOENT instead of ENOMEDIUM for better compatibility

[Jeroen Hofstee]

Hello Simon,

On 06/16/2013 04:46 PM, Simon Glass wrote:
> This error may not be defined on some platforms such as MacOS so host
> compilation will fail. Use one of the more common errors instead.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
>
> ---
> common/image-fit.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/common/image-fit.c b/common/image-fit.c
> index 7bf82d3..f40f160 100644
> --- a/common/image-fit.c
> +++ b/common/image-fit.c
> @@ -1557,7 +1557,7 @@ int fit_image_load(bootm_headers_t *images, const char *prop_name, ulong addr,
>   	if (fit_image_get_data(fit, noffset, &buf, &size)) {
>   		printf("Could not find %s subimage data!\n", prop_name);
>   		bootstage_error(bootstage_id + BOOTSTAGE_SUB_GET_DATA);
> -		return -ENOMEDIUM;
> +		return -ENOENT;
>   	}
>   	len = (ulong)size;
>   
This fixes building for FreeBSD as well, thanks.

Regards,
Jeroen

[U-Boot] [PATCH] image: Use ENOENT instead of ENOMEDIUM for better compatibility

[Andreas Bießmann]

On 16.06.13 16:46, Simon Glass wrote:
> This error may not be defined on some platforms such as MacOS so host
> compilation will fail. Use one of the more common errors instead.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>

Tested-by: Andreas Bie?mann <andreas.devel@googlemail.com>

on OS X 10.7.5

[U-Boot] [PATCH] image: Use ENOENT instead of ENOMEDIUM for better compatibility

[Lubomir Popov]

On 16.06.2013 17:46, Simon Glass wrote:
> This error may not be defined on some platforms such as MacOS so host
> compilation will fail. Use one of the more common errors instead.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>

Tested-by: Lubomir Popov <lpopov@mm-sol.com>

on MacOS X 10.8.3 with the following tools used for building U-Boot:

arm-none-eabi-gcc (GCC) 4.7.2
GNU ld (GNU Binutils) 2.23.1

[U-Boot] [PATCH] image: Use ENOENT instead of ENOMEDIUM for better compatibility

[Tom Rini]

On Sun, Jun 16, 2013 at 07:46:49AM -0700, Simon Glass wrote:

> This error may not be defined on some platforms such as MacOS so host
> compilation will fail. Use one of the more common errors instead.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130617/f7c6d7ce/attachment.pgp>