* [Patch] Driver: Staging: checkpatch cleanup for drivers/staging/winbond/wb35tx_s.h
@ 2012-09-25 20:26 Harsh Kumar
2012-09-25 20:51 ` Dan Carpenter
2012-09-26 3:51 ` Harsh Kumar
0 siblings, 2 replies; 3+ messages in thread
From: Harsh Kumar @ 2012-09-25 20:26 UTC (permalink / raw)
To: kernel-janitors
checkpatch cleanup for drivers/staging/winbond/wb35tx_s.h
1) Changed c99 comments to c89 comments
2) Removed undesired spaces which were causing checkpatch errors.
Signed-off-by: Harsh Kumar <harsh1kumar@gmail.com>
diff -uprN a/drivers/staging/winbond/wb35tx_s.h b/drivers/staging/winbond/wb35tx_s.h
--- a/drivers/staging/winbond/wb35tx_s.h
+++ b/drivers/staging/winbond/wb35tx_s.h
@@ -3,45 +3,45 @@
#include "mds_s.h"
-//==================
-// IS89C35 Tx related definition
-//==================
-#define TX_INTERFACE 0 // Interface 1
-#define TX_PIPE 3 // endpoint 4
-#define TX_INTERRUPT 1 // endpoint 2
-#define MAX_INTERRUPT_LENGTH 64 // It must be 64 for EP2 hardware
+/**********************************
+* IS89C35 Tx related definition
+***********************************/
+#define TX_INTERFACE 0 /* Interface 1 */
+#define TX_PIPE 3 /* Endpoint 4 */
+#define TX_INTERRUPT 1 /* Endpoint 2 */
+#define MAX_INTERRUPT_LENGTH 64 /* It must be 64 for EP2 hardware */
-//==================
-// Internal variable for module
-//==================
+/**********************************
+* Internal variable for module
+***********************************/
struct wb35_tx {
- // For Tx buffer
- u8 TxBuffer[ MAX_USB_TX_BUFFER_NUMBER ][ MAX_USB_TX_BUFFER ];
+ /* For Tx buffer */
+ u8 TxBuffer[MAX_USB_TX_BUFFER_NUMBER][MAX_USB_TX_BUFFER];
- // For Interrupt pipe
+ /* For Interrupt pipe */
u8 EP2_buf[MAX_INTERRUPT_LENGTH];
- atomic_t TxResultCount;// For thread control of EP2 931130.4.m
- atomic_t TxFireCounter;// For thread control of EP4 931130.4.n
- u32 ByteTransfer;
-
- u32 TxSendIndex;// The next index of Mds array to be sent
- u32 EP2vm_state; // for EP2vm state
- u32 EP4vm_state; // for EP4vm state
- u32 tx_halt; // Stopping VM
+ atomic_t TxResultCount;/* For thread control of EP2 931130.4.m */
+ atomic_t TxFireCounter;/* For thread control of EP4 931130.4.n */
+ u32 ByteTransfer;
+
+ u32 TxSendIndex;/* The next index of Mds array to be sent */
+ u32 EP2vm_state; /* for EP2vm state */
+ u32 EP4vm_state; /* for EP4vm state */
+ u32 tx_halt; /* Stopping VM */
- struct urb * Tx4Urb;
- struct urb * Tx2Urb;
+ struct urb *Tx4Urb;
+ struct urb *Tx2Urb;
int EP2VM_status;
int EP4VM_status;
- u32 TxFillCount; // 20060928
- u32 TxTimer; // 20060928 Add if sending packet not great than 13
+ u32 TxFillCount; /* 20060928 */
+ u32 TxTimer; /* 20060928 Add if sending packet not great than 13 */
};
#endif
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Patch] Driver: Staging: checkpatch cleanup for drivers/staging/winbond/wb35tx_s.h
2012-09-25 20:26 [Patch] Driver: Staging: checkpatch cleanup for drivers/staging/winbond/wb35tx_s.h Harsh Kumar
@ 2012-09-25 20:51 ` Dan Carpenter
2012-09-26 3:51 ` Harsh Kumar
1 sibling, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2012-09-25 20:51 UTC (permalink / raw)
To: kernel-janitors
On Wed, Sep 26, 2012 at 01:44:48AM +0530, Harsh Kumar wrote:
> checkpatch cleanup for drivers/staging/winbond/wb35tx_s.h
>
> 1) Changed c99 comments to c89 comments
> 2) Removed undesired spaces which were causing checkpatch errors.
>
Can you break this up into two patches.
This doesn't apply for me on linux-next (please redo it so it
applies).
> Signed-off-by: Harsh Kumar <harsh1kumar@gmail.com>
>
> diff -uprN a/drivers/staging/winbond/wb35tx_s.h b/drivers/staging/winbond/wb35tx_s.h
> --- a/drivers/staging/winbond/wb35tx_s.h
> +++ b/drivers/staging/winbond/wb35tx_s.h
> @@ -3,45 +3,45 @@
>
> #include "mds_s.h"
>
> -//==================
> -// IS89C35 Tx related definition
> -//==================
> -#define TX_INTERFACE 0 // Interface 1
> -#define TX_PIPE 3 // endpoint 4
> -#define TX_INTERRUPT 1 // endpoint 2
> -#define MAX_INTERRUPT_LENGTH 64 // It must be 64 for EP2 hardware
> +/**********************************
> +* IS89C35 Tx related definition
> +***********************************/
Just make these a normal comments.
/* IS89C35 Tx related definitions */
Or for multi-line:
/*
* IS89C35 Tx related blah blah blha blah blah blah blah blha blah
* definitions
*/
There is a section on comments in CodingStyle.
> +#define TX_INTERFACE 0 /* Interface 1 */
> +#define TX_PIPE 3 /* Endpoint 4 */
> +#define TX_INTERRUPT 1 /* Endpoint 2 */
> +#define MAX_INTERRUPT_LENGTH 64 /* It must be 64 for EP2 hardware */
>
>
>
Only one blank line at a time. (Although that was there in the
original so that's fine).
> -//==================
> -// Internal variable for module
> -//==================
> +/**********************************
> +* Internal variable for module
> +***********************************/
>
>
> struct wb35_tx {
> - // For Tx buffer
> - u8 TxBuffer[ MAX_USB_TX_BUFFER_NUMBER ][ MAX_USB_TX_BUFFER ];
> + /* For Tx buffer */
> + u8 TxBuffer[MAX_USB_TX_BUFFER_NUMBER][MAX_USB_TX_BUFFER];
>
> - // For Interrupt pipe
> + /* For Interrupt pipe */
> u8 EP2_buf[MAX_INTERRUPT_LENGTH];
>
> - atomic_t TxResultCount;// For thread control of EP2 931130.4.m
> - atomic_t TxFireCounter;// For thread control of EP4 931130.4.n
> - u32 ByteTransfer;
> -
> - u32 TxSendIndex;// The next index of Mds array to be sent
> - u32 EP2vm_state; // for EP2vm state
> - u32 EP4vm_state; // for EP4vm state
> - u32 tx_halt; // Stopping VM
> + atomic_t TxResultCount;/* For thread control of EP2 931130.4.m */
^
Put a space there, between the semi-colon and the comment.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Patch] Driver: Staging: checkpatch cleanup for drivers/staging/winbond/wb35tx_s.h
2012-09-25 20:26 [Patch] Driver: Staging: checkpatch cleanup for drivers/staging/winbond/wb35tx_s.h Harsh Kumar
2012-09-25 20:51 ` Dan Carpenter
@ 2012-09-26 3:51 ` Harsh Kumar
1 sibling, 0 replies; 3+ messages in thread
From: Harsh Kumar @ 2012-09-26 3:51 UTC (permalink / raw)
To: kernel-janitors
Thanks Dan for the reply.
I am a kernel newbie so this will help me learn. I will make the
required changes and try to avoid them next time.
On Wed, Sep 26, 2012 at 2:21 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> On Wed, Sep 26, 2012 at 01:44:48AM +0530, Harsh Kumar wrote:
>> checkpatch cleanup for drivers/staging/winbond/wb35tx_s.h
>>
>> 1) Changed c99 comments to c89 comments
>> 2) Removed undesired spaces which were causing checkpatch errors.
>>
>
> Can you break this up into two patches.
>
> This doesn't apply for me on linux-next (please redo it so it
> applies).
>
>> Signed-off-by: Harsh Kumar <harsh1kumar@gmail.com>
>>
>> diff -uprN a/drivers/staging/winbond/wb35tx_s.h b/drivers/staging/winbond/wb35tx_s.h
>> --- a/drivers/staging/winbond/wb35tx_s.h
>> +++ b/drivers/staging/winbond/wb35tx_s.h
>> @@ -3,45 +3,45 @@
>>
>> #include "mds_s.h"
>>
>> -//==================
>> -// IS89C35 Tx related definition
>> -//==================
>> -#define TX_INTERFACE 0 // Interface 1
>> -#define TX_PIPE 3 // endpoint 4
>> -#define TX_INTERRUPT 1 // endpoint 2
>> -#define MAX_INTERRUPT_LENGTH 64 // It must be 64 for EP2 hardware
>> +/**********************************
>> +* IS89C35 Tx related definition
>> +***********************************/
>
> Just make these a normal comments.
>
> /* IS89C35 Tx related definitions */
>
> Or for multi-line:
>
> /*
> * IS89C35 Tx related blah blah blha blah blah blah blah blha blah
> * definitions
> */
>
> There is a section on comments in CodingStyle.
>
>> +#define TX_INTERFACE 0 /* Interface 1 */
>> +#define TX_PIPE 3 /* Endpoint 4 */
>> +#define TX_INTERRUPT 1 /* Endpoint 2 */
>> +#define MAX_INTERRUPT_LENGTH 64 /* It must be 64 for EP2 hardware */
>>
>>
>>
>
> Only one blank line at a time. (Although that was there in the
> original so that's fine).
>
>> -//==================
>> -// Internal variable for module
>> -//==================
>> +/**********************************
>> +* Internal variable for module
>> +***********************************/
>>
>>
>> struct wb35_tx {
>> - // For Tx buffer
>> - u8 TxBuffer[ MAX_USB_TX_BUFFER_NUMBER ][ MAX_USB_TX_BUFFER ];
>> + /* For Tx buffer */
>> + u8 TxBuffer[MAX_USB_TX_BUFFER_NUMBER][MAX_USB_TX_BUFFER];
>>
>> - // For Interrupt pipe
>> + /* For Interrupt pipe */
>> u8 EP2_buf[MAX_INTERRUPT_LENGTH];
>>
>> - atomic_t TxResultCount;// For thread control of EP2 931130.4.m
>> - atomic_t TxFireCounter;// For thread control of EP4 931130.4.n
>> - u32 ByteTransfer;
>> -
>> - u32 TxSendIndex;// The next index of Mds array to be sent
>> - u32 EP2vm_state; // for EP2vm state
>> - u32 EP4vm_state; // for EP4vm state
>> - u32 tx_halt; // Stopping VM
>> + atomic_t TxResultCount;/* For thread control of EP2 931130.4.m */
> ^
> Put a space there, between the semi-colon and the comment.
>
> regards,
> dan carpenter
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-09-26 3:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-25 20:26 [Patch] Driver: Staging: checkpatch cleanup for drivers/staging/winbond/wb35tx_s.h Harsh Kumar
2012-09-25 20:51 ` Dan Carpenter
2012-09-26 3:51 ` Harsh Kumar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox