[U-Boot] [PATCH v4] sf: Macronix additional chips supported

Prafulla Wadaskar prafulla at marvell.com
Mon Jul 6 11:37:19 CEST 2009


 

> -----Original Message-----
> From: Mike Frysinger [mailto:vapier at gentoo.org] 
> Sent: Monday, July 06, 2009 1:58 PM
> To: u-boot at lists.denx.de
> Cc: Prafulla Wadaskar; Manas Saksena; Ronen Shitrit; Nicolas 
> Pitre; Ashish Karkare; Prabhanjan Sarnaik; Lennert Buijtenhek
> Subject: Re: [U-Boot] [PATCH v4] sf: Macronix additional 
> chips supported
> 
> On Monday 06 July 2009 08:21:16 Prafulla Wadaskar wrote:
> > new chips supported:-
> > MX25L1605D, MX25L3205D, MX25L6405D, MX25L12855E out of which 
> > MX25L6405D and MX25L12855E tested on Kirkwood platforms
> >
> > Modified the Macronix flash support to use 2 bytes of device id 
> > instead of
> > 1 This was required to support MX25L12855E
> >
> > Done some indentation fixes
> >
> > Signed-off-by: Piyush Shah <spiyush at marvell.com>
> > Signed-off-by: Prafulla Wadaskar <prafulla at marvell.com>
> > ---
> > Change log:-
> > v2: leading spaces removed for clean patch apply build error fixed 
> > tested for new added chip MX25L6405D
> >
> > v3: updated contributors info
> > ids combined to u16
> > ids list of defines removed, used values in the struct
> >
> > v4: indentation fixes,
> > line lengths brought below 78 chars
> > alligned access fixed for idcode
> 
> blah, this just got much worse.  code style cleanups should 
> be kept separate.  
> i.e. the v3 patch without whitespace damage should be one 
> commit and then if you still want to do style changes, that 
> needs to be on top of that.
Okay..
I will repost v4 for this patch without whitespcae change.
And will post separate patch for indentation.

Regards..
Prafulla . .
 
> -mike
> 


More information about the U-Boot mailing list