[U-Boot-Users] [PATCH] cmdbootz

[Wolfgang Denk]

In message <20040515145022.GK27774 at pengutronix.de> you wrote:
> 
> I've rewritten the bootm command to recognize an ARM zImage the same way
> as it already does with i368 zImages, hope that's ok with you. 

Sorry, but it is not. I reject this modification.

As the comment in bootm says:

> -#ifdef __I386__	/* correct image format not implemented yet - fake it */
> -		if (fake_header(hdr, (void*)addr, -1) != NULL) {

The fact that one implementation, which is far from  being  complete,
fails  to  implement  something  correctly, does not justuify to make
this failure a standard.

Also, it makes no sense to me to implement this feature (should it be
useful and needed) just for ARM - if it gets accepted, then I want to
have a consistent interface for ALL architectures.

Anyway. "bootm" is a command to boot U-Boot images, which include the
header. If you want to boot something else, then  bootm  is  not  the
right command to use.



Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
Killing is wrong.
	-- Losira, "That Which Survives", stardate unknown