Dear Dirk Behme, In message <49f3e966....@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: w...@denx.de A committee is a group that keeps the minutes and loses hours. -- Milton Berle _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot