[U-Boot] [PATCH 1/2] OMAP3: Remove unused board-types

Wolfgang Denk wd at denx.de
Sun Apr 26 20:58:59 CEST 2009


Dear Dirk Behme,

In message <49F3E966.701 at googlemail.com> you wrote:
>
> >> I would not dare to use such a function in my code given the
> >> "test-only" comment.
> > sorry I've no time to clean every part of the arm as noone else are
> > interrested in old code
> > 
> > so yes it will be cleanup but later asI work on other part of the arm actually
> > which I will finish first
> 
> Uups :( And this is what I really have a problem with.

Problem solved.

I removed this comment. The patch has been checked into mainline,

> Then we find that the changes we are asked to do rely on code that is 
> marked with 'test only' and needs documentation.

Actualy I removed only the comment. I left the documentation for those
who know better than me what's going on. 

> And now? What are we supposed to do?
> 
> Change our patch based on 'test only' undocumented code?

The test-only has been removed, and the documentation will be added
ASAP. Please base your patch on this code as is now (even though the
documentation is still missing).

> Or will a trivial 'remove dead code only' patch delayed until e.g. the 
> Kconfig framework or e.g. the new clock framework or e.g. <add what 
> you want> will be ready? And when will this be?

No, it will not be delayed. I will take personal care that it goes
into this release. If needed I will apply it myself.

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
A committee is a group that keeps the minutes and loses hours.
                                                      -- Milton Berle


More information about the U-Boot mailing list