[PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

2017-07-20 Thread Gaurav Pathak
staging: rts5208: Fix coding style issue caught by checkpatch.pl related to function name in debug print Signed-off-by: Gaurav Pathak --- drivers/staging/rts5208/ms.c| 5 +++-- drivers/staging/rts5208/rtsx.c | 4 ++-- drivers/staging/rts5208/rtsx_chip.c

Re: [PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

2017-07-18 Thread Greg KH
On Mon, Jul 17, 2017 at 08:19:45PM +0530, Gaurav Pathak wrote: > staging: rts5208: Fix coding style issue caught by checkpatch.pl related to > function name in debug print and shorten few lines over 80 char No, only do one-type-of-thing per patch. It would make sense to maybe fix an 80 char line

[PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

2017-07-17 Thread Gaurav Pathak
staging: rts5208: Fix coding style issue caught by checkpatch.pl related to function name in debug print and shorten few lines over 80 char Signed-off-by: Gaurav Pathak --- drivers/staging/rts5208/ms.c| 5 +++-- drivers/staging/rts5208/rtsx.c | 4 ++--

Re: [PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

2017-07-17 Thread Greg KH
On Fri, Jul 14, 2017 at 07:38:43PM +0530, Gaurav Pathak wrote: > staging: rts5208: Refactored code to avoid few defects. That's the subject line, why repeat it here? Please put a "real" description of the patch here. > > Signed-off-by: Gaurav Pathak > --- > Hi,

[PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

2017-07-14 Thread Gaurav Pathak
staging: rts5208: Refactored code to avoid few defects. Signed-off-by: Gaurav Pathak --- Hi, Gaurav hope this text appaers in the email, and i put it at the right place in patch. drivers/staging/rts5208/ms.c| 5 +++-- drivers/staging/rts5208/rtsx.c | 4