Re: [PATCH] Drivers: Staging: CSR: Fixed coding style warnings

From: Greg KH
Date: Sun Oct 14 2012 - 09:11:42 EST


On Sun, Oct 14, 2012 at 09:48:29PM +0900, Sangho Yi wrote:
> On drivers/staging/csr/csr_time.h, there were a few warnings on
> indentation, braces, and typedef struct usage.
> All those were changed to the proper way.

No, as I explained in the talk, only do one thing at a time here.

So, one patch for the indentation, one patch for the braces, and then,
one patch for the typedef.

And, with the typedef change, did you just break the build? If not,
then the whole structure can be removed, right?

So, care to redo this?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/