In message 463B3463.1030702@freescale.com you wrote:
If you apply my patch, then the comment becomes correct. My goal was to lock the variables 'len' and 'data' into one purpose. The reason the
I think you actually missed a few places where len was also used.
bug existed is because the other developer didn't realize this. He used 'len' thinking it was available. In a sense, I was trying to implement
This type of thinking was plain wrong. You must never use any variable without understanding what it's being used for.
Stefan said he had a testing repo of some kind. How about we just use that? If Stefan is willing, he can apply my emailed patches to his repo.
The u-boot-testing repo is actually *my* guardian repo where I put all stuff in that does not fit anywhere else.
As I explained before, you need a normal developer's repo, not a guardian one. And I repeat: I offer you such a repo, it's available right now, I just need the IP address from where you will connect.
Best regards,
Wolfgang Denk