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

Wolfgang Denk
2012-07-28T19:53:00Z

Dear Marek Vasut,

In message <201207282039.34518.marex@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