[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