Skip to contentSkip to conversation

Drivers: Staging: CSR: Fixed coding style warnings

PATCH In review Sangho Yi 3 participants 3 messages Oct 14, 2012 – Oct 18, 2012 Staging

lore
Share & tools
lore

Sangho Yi

author
#0 lore raw

PATCH 1/1 Drivers: Staging: CSR: Fixed coding style warnings

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.

---
 drivers/staging/csr/csr_time.h |   72 ++++++++++++++++++++--------------------
 1 file changed, 36 insertions(+), 36 deletions(-)
drivers/staging/csr/csr_time.h +36 -36
index 2a45f3e..66f1c08 100644
@@ -2,11 +2,11 @@
 #define CSR_TIME_H__
 /*****************************************************************************
 
-            (c) Cambridge Silicon Radio Limited 2010
-            All rights reserved and confidential information of CSR
+(c) Cambridge Silicon Radio Limited 2010
+All rights reserved and confidential information of CSR
 
-            Refer to LICENSE.txt included with this source for details
-            on the license terms.
+Refer to LICENSE.txt included with this source for details
+on the license terms.
 
 *****************************************************************************/
 
@@ -18,13 +18,13 @@ extern "C" {
 
 /*******************************************************************************
 
-    NAME
-        CsrTime
+NAME
+	CsrTime
 
-    DESCRIPTION
-        Type to hold a value describing the current system time, which is a
-        measure of time elapsed since some arbitrarily defined fixed time
-        reference, usually associated with system startup.
+DESCRIPTION
+	Type to hold a value describing the current system time, which is a
+	measure of time elapsed since some arbitrarily defined fixed time
+	reference, usually associated with system startup.
 
 *******************************************************************************/
 typedef u32 CsrTime;
@@ -32,44 +32,44 @@ typedef u32 CsrTime;
 
 /*******************************************************************************
 
-    NAME
-        CsrTimeUtc
+NAME
+	CsrTimeUtc
 
-    DESCRIPTION
-        Type to hold a value describing a UTC wallclock time expressed in
-        seconds and milliseconds elapsed since midnight January 1st 1970.
+DESCRIPTION
+	Type to hold a value describing a UTC wallclock time expressed in
+	seconds and milliseconds elapsed since midnight January 1st 1970.
 
 *******************************************************************************/
-typedef struct
-{
-    u32 sec;
-    u16 msec;
-} CsrTimeUtc;
+struct CstTimeUtc {
+	u32 sec;
+	u16 msec;
+};
 
 
 /*******************************************************************************
 
-    NAME
-        CsrTimeGet
+NAME
+	CsrTimeGet
 
-    DESCRIPTION
-        Returns the current system time in a low and a high part. The low part
-        is expressed in microseconds. The high part is incremented when the low
-        part wraps to provide an extended range.
+DESCRIPTION
+	Returns the current system time in a low and a high part. The low part
+	is expressed in microseconds. The high part is incremented when the low
+	part wraps to provide an extended range.
 
-        The caller may provide a NULL pointer as the high parameter. In this case
-        the function just returns the low part and ignores the high parameter.
+	The caller may provide a NULL pointer as the high parameter.
+	In this case the function just returns the low part and ignores the
+	high parameter.
 
-        Although the time is expressed in microseconds the actual resolution is
-        platform dependent and can be less. It is recommended that the
-        resolution is at least 10 milliseconds.
+	Although the time is expressed in microseconds the actual resolution is
+	platform dependent and can be less. It is recommended that the
+	resolution is at least 10 milliseconds.
 
-    PARAMETERS
-        high - Pointer to variable that will receive the high part of the
-               current system time. Passing NULL is valid.
+PARAMETERS
+	high - Pointer to variable that will receive the high part of the
+	       current system time. Passing NULL is valid.
 
-    RETURNS
-        Low part of current system time in microseconds.
+RETURNS
+	Low part of current system time in microseconds.
 
 *******************************************************************************/
 CsrTime CsrTimeGet(CsrTime *high);
Signature · 1 line
1.7.9.5
maintainer
#1 lore raw
Quoted 3 lines · Sangho Yi

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