[PATCH] Drivers: Staging: CSR: made corrections on the indentation errors

Greg KH gregkh at linuxfoundation.org
Sun Oct 14 11:26:52 UTC 2012


On Sun, Oct 14, 2012 at 08:20:11PM +0900, Sangho Yi wrote:
> Originally there were a lot mis-indentation, so I made changes to use
> only the tabs (no spaces) on indenting code.
> - Still, the typedef struct ... tells warning message on checkpatch.pl.

Yes, we shouldn't be using a typedef, but why did you change the typedef
itself:

> -typedef struct
> -{
> -    u32 sec;
> -    u16 msec;
> +typedef struct _CsrTimeUtc {
> +	u32 sec;
> +	u16 msec;
>  } CsrTimeUtc;

Why add the _CsrTimeUtc here?  That's not really needed, right?

thanks,

greg k-h



More information about the devel mailing list