[PATCH 02/14] Staging: bcm: Properly format and remove comments as needed in nvm.h

Kevin McKinney klmckinney1 at gmail.com
Fri Nov 30 13:14:48 UTC 2012


On Fri, Nov 30, 2012 at 10:49:20AM +0300, Dan Carpenter wrote:
> On Thu, Nov 29, 2012 at 11:22:42PM -0500, Kevin McKinney wrote:
> > @@ -31,48 +30,28 @@ typedef struct _FLASH_SECTOR_INFO
> >  typedef struct _FLASH_CS_INFO
> >  {
> 
> >  	B_UINT32 MagicNumber;
> > -	// let the magic number be 0xBECE-F1A5 - F1A5 for "flas-h"
> 
> That's sort of useful to know how the magic number is formed.
> 
> > -	// MSC Timeout after reset to switch from MSC to NW Mode
> >  	B_UINT32 MassStorageTimeout;
> 
> This one as well is useful I think.
> 
> > -	// NO Data Activity timeout to switch from MSC to NW Mode
> >  	B_UINT32 CDLessInactivityTimeout;
> 
> This one as well, perhaps.
> 
Yeah, I thought about leaving these comments.  Okay, I will submit a version 2 patch set fixing this and the unsigned char
to u8. Thanks.

Thanks,
Kevin




More information about the devel mailing list