[U-Boot] ARM Pull Request

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Mon Mar 30 22:28:36 CEST 2009


On 21:49 Mon 30 Mar     , Wolfgang Denk wrote:
> Dear Dirk Behme,
> 
> In message <49D11E7D.5030701 at googlemail.com> you wrote:
> > 
> > >> http://lists.denx.de/pipermail/u-boot/2009-March/049762.html
> > >>
> > >> Please remove it from the pull request until everybody agreed
> > > this duplicated code 
> > 
> > It would be nice if you could give the exact file in which you found 
> > the other/original version of this code. As already mentioned I'd like 
> > to compare/review it.
> > 
> > > so I'll remove it anyway I do not want duplicated code
> > 
> > It would be nice if we could agree on a common way how to improve 
> > (remove?) this code. And not just "I'll remove it anyway". Sorry if I 
> > misunderstood this, but it could be understood as "I will remove it 
> > independent of what other people think and do".
> 
> I have to admit that I, too, hesitated what that statement was
> supposed to mean. Jean-Christophe, it would really be nice to raise
> your complaints on the mailing list, and without too long delays, so
> we can discuss things.
It's simple I really hate duplicated that's all
so when I find copy&paste I remote it
> 
> > > I'm not statisfy about this one I'm looking on an other implementation
> > > so this patch will wait a few more days
> > 
> > It would be nice if you could share your ideas/doubts/findings so that 
> > we can work together on improving this code.
> 
> Again, I agree 100%. Jean-Christophe, please share your concerns on
> the mailing list, and without so long delays after the postings. It
> is not your job as a custodian to fix everything yourself. It is
> sufficient (and actually usualy much better, think for example about
> the long-term educational aspect) just to raise your concerns and ask
> the original poster to change things (or maybe he has good reasons
> for doing it this way, which you didn't see). But please keep such
> discussion on the mailing list, and without big delays.  Thanks.
I need to time to study a way to clean it up. Without necessarely implement it
I'm just bored by reading duplicated code everytime as the timer
implementations.

So I need few days to study it

Best Regards,
J.


More information about the U-Boot mailing list