On Thursday 19 July 2012 19:17:31 Benoît Thébaudeau wrote:
> On Friday 20 July 2012 00:44:52 Mike Frysinger wrote:
> > On Thursday 19 July 2012 18:03:06 Benoît Thébaudeau wrote:
> > > --- u-boot-66714b1.orig/fs/fat/fat.c
> > > +++ u-boot-66714b1/fs/fat/fat.c
> > > 
> > > +__u8 get_contents_vfatname_block[MAX_CLUSTSIZE]
> > > + __aligned(ARCH_DMA_MINALIGN);
> > 
> > is there any reason for this to be exported ?  could you mark it
> > static ?
> 
> I'll do that. There are probably other variables in fat.c missing static,
> so I'll do a dedicated patch before this series. That will shift the patch
> numbers. How should I repost the new version of the series? Should I keep
> the message ID of each patch even if the numbering changes, or should I
> post a v2 completely separately from v1?

it's easier on everyone to do this.  you might want to look at tools/patman/ 
as a way for managing the review process.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to