Dear Dirk Behme,
In message 49F3E966.701@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