[U-Boot-Users] Can U-boot Autodetect arch/ppcversusarch/powerpc from info in the uImage?

[Wolfgang Denk]

In message <1197205927.937.51.camel at gentoo-jocke.transmode.se> you wrote:
> 
> Made a new patch with Jerrys comments addressed. Also renamed
> DEFAULT_OF_TREE to CFG_OF_TREE. OK?

I still object against this modification.

> +#ifdef CFG_OF_TREE
> +	char	*of_flat_tree = CFG_OF_TREE;
> +#else
>  	char	*of_flat_tree = NULL;
> +#endif
>  	ulong	of_data = 0;
>  #endif

I hereby NAK this patch for 3 reasons:

1) The patch does not solve a problem. Instead of harwiring the
   address, you can just pass it as argument to the bootm command
   which seems more straightforward to me

2) The patch causes confusion. Documented behaviour is that "bootm"
   with one or two arguments (kernel address, or kernel plus ramdisk
   addresses) will boot a non-OF enabled kernel. With this patch,
   "bootm" will behave different on systems where the CFG_OF_TREE has
   been silected - which is usually not known to and cannot be
   checked by the end user, thus causing confusion.

3) With the patch applied and CFG_OF_TREE defined, there is no way to
   boot a non-OF kernel, thus breaking backward compatibility.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
If you're not part of the solution, you're part of the problem.