On Monday 09 March 2009, Jean-Christophe PLAGNIOL-VILLARD wrote:
It's already done by Jean-Christophe PLAGNIOL-VILLARD. Isn't it?
Is it? I have not seen such a patch. Do you have a link? (I added Jean-Christophe to CC).
http://git.denx.de/?p=u-boot/u-boot-arm.git;a=commit;h=9163868e5575d841 089a 84d97f287b2ca15486a1
Hmm. Now I'm a little confused. It seems that you added this patch with some changes to the "for-next" branch of your arm git repository. Did you post the changed version of this patch on the list? I couldn't find it.
Don't get me wrong. I appreciate that you did the required change to the patch. But please post the path on the list after you changed it.
yes but was too big and automaticly-reject
Ahh, I see. :-(
And also please incorporate the change to the patch description as I suggested.
which?
In my first review.
Additionally I think that this patch doesn't really belong into your arm repository.
I've put here due to the ML limit that's all
I'm hoping that Wolfgang will sent this patch to the list a a little while.
Best regards, Stefan
===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office@denx.de =====================================================================