[U-Boot] [RFC] [PATCH 2/2] crazy: Sort u_boot_cmd at runtime

Wolfgang Denk wd at denx.de
Sat Jul 28 21:53:00 CEST 2012


Dear Marek Vasut,

In message <201207282039.34518.marex at denx.de> you wrote:
> 
> > Seems incomplete in several aspects:
> 
> Below the section:
> 
> * NOTE * THIS PATCH IS CRAZY

Then what is actually the purpose of such a posting?  Just dumping
unsorted thoughts to community?

You are experienced enough to know what would be needed for a
semi-clean patch, even if it's "just for RFC"...

> There are a few notes. I'd actually like to know if this approach is correct at 
> all, it might break on some crazy configurations or such.

Define "correct".  It may be possible - but what would be the
advantage?  Which problem does it solve?  In which way is it better
than the current code?

> > 1) what about all the non-ARM architecures and the board specific
> >    linker scripts?
> 
> - This patch affects only arm926t, obviously to make it proper, every
>   linkerscript would have to be adjusted
> 
> Which sucks, since there're a lot of them. But it can probably be automated.

Actually I doubt it makes sense at all.

I envision a situation where some "pluggable" code (say, a standalone
application, or some form of loadable module whatever) can add new
commands - it would be nice if these would still appear in sorted
order, but this cannot be done at compile-time.

So please explain which actual problem you are rying to solve.

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
It [being a Vulcan] means to adopt a philosophy, a way of life  which
is logical and beneficial. We cannot disregard that philosophy merely
for personal gain, no matter how important that gain might be.
	-- Spock, "Journey to Babel", stardate 3842.4


More information about the U-Boot mailing list