[U-Boot] [PATCH 7/7] add gc-sections to TPL boot

Albert ARIBAUD albert.aribaud at free.fr
Fri Jan 28 19:21:16 CET 2011


Le 28/01/2011 19:08, Haiying Wang a écrit :

>> I see patch 3/8, 4/8, 5/8, and 7/7.  Where are the rest?
> Sorry, patch 7/7 is a wrong number here. I kept the patch # as 3/8/,
> 4/8, 5/8, 8/8 to be consistent with the order in the patchset(v2) I
> submitted in last December. I thought it would be clearer to compare
> them with v2 version and review.  Patch 1/8,2/8 have been applied by
> Kumar, patch 6/8, 7/8 remain the same as v2 version. This patch, is a
> new patch because that TPL still needs --gc-sections in linker option to
> do partial link.
>
> If it is preferable to have new whole set of patch, I can reorder them
> from 3/8-8/8 plus this one to submit.

I would suggest to simply number patches from 1 to N for each version 
even if that means the same patch gets numbered differently across 
versions, because readers of a given version may not have read the 
previous one. A patchset should be self-sufficient and self-consistent IMO.

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list