* [PATCH] Staging: rtl8188eu: Fixed prohibited spaces after open '('
@ 2014-05-06 20:00 Rémi Pommarel
2014-05-06 21:10 ` Remi Pommarel
0 siblings, 1 reply; 5+ messages in thread
From: Rémi Pommarel @ 2014-05-06 20:00 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: devel, linux-kernel, Remi Pommarel
This patch fixes all prohibited spaces after open '(' checkpatch.pl errors
for rtl8188eu.
Signed-off-by: Remi Pommarel <repk@triplefau.lt>
---
drivers/staging/rtl8188eu/include/odm_debug.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8188eu/include/odm_debug.h b/drivers/staging/rtl8188eu/include/odm_debug.h
index e8c4cab..db7b44e 100644
--- a/drivers/staging/rtl8188eu/include/odm_debug.h
+++ b/drivers/staging/rtl8188eu/include/odm_debug.h
@@ -85,7 +85,7 @@
/*------------------------Export Marco Definition---------------------------*/
#define DbgPrint pr_info
#define RT_PRINTK(fmt, args...) \
- DbgPrint( "%s(): " fmt, __func__, ## args);
+ DbgPrint("%s(): " fmt, __func__, ## args);
#ifndef ASSERT
#define ASSERT(expr)
@@ -106,8 +106,8 @@
#define ODM_RT_ASSERT(pDM_Odm, expr, fmt) \
if (!(expr)) { \
- DbgPrint( "Assertion failed! %s at ......\n", #expr); \
- DbgPrint( " ......%s,%s,line=%d\n", __FILE__, \
+ DbgPrint("Assertion failed! %s at ......\n", #expr); \
+ DbgPrint(" ......%s,%s,line=%d\n", __FILE__, \
__func__, __LINE__); \
RT_PRINTK fmt; \
ASSERT(false); \
--
1.8.5.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] Staging: rtl8188eu: Fixed prohibited spaces after open '('
2014-05-06 20:00 [PATCH] Staging: rtl8188eu: Fixed prohibited spaces after open '(' Rémi Pommarel
@ 2014-05-06 21:10 ` Remi Pommarel
2014-05-06 21:27 ` Dan Carpenter
0 siblings, 1 reply; 5+ messages in thread
From: Remi Pommarel @ 2014-05-06 21:10 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: devel, linux-kernel
Oups sorry msmtp messed up the from mail address. Do you want me to
re-submit this or you can fix it while applying the patch ?
--
Remi Pommarel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Staging: rtl8188eu: Fixed prohibited spaces after open '('
2014-05-06 21:10 ` Remi Pommarel
@ 2014-05-06 21:27 ` Dan Carpenter
2014-05-06 21:40 ` Remi Pommarel
0 siblings, 1 reply; 5+ messages in thread
From: Dan Carpenter @ 2014-05-06 21:27 UTC (permalink / raw)
To: Remi Pommarel; +Cc: Greg Kroah-Hartman, devel, linux-kernel
On Tue, May 06, 2014 at 11:10:11PM +0200, Remi Pommarel wrote:
> Oups sorry msmtp messed up the from mail address. Do you want me to
> re-submit this or you can fix it while applying the patch ?
>
Best to fix it yourself and Greg can take which ever version is easiest.
https://www.google.com/search?q=how+to+send+a+v2+patch
regards,
dan carpenter
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Staging: rtl8188eu: Fixed prohibited spaces after open '('
2014-05-06 21:27 ` Dan Carpenter
@ 2014-05-06 21:40 ` Remi Pommarel
0 siblings, 0 replies; 5+ messages in thread
From: Remi Pommarel @ 2014-05-06 21:40 UTC (permalink / raw)
To: Dan Carpenter; +Cc: Greg Kroah-Hartman, devel, linux-kernel
On Wed, May 07, 2014 at 12:27:21AM +0300, Dan Carpenter wrote:
> Best to fix it yourself and Greg can take which ever version is easiest.
Ok thank you so I'll re-submitted it.
Regards
--
Remi Pommarel
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] Staging: rtl8188eu: Fixed prohibited spaces after open '('
@ 2014-05-06 21:40 Remi Pommarel
0 siblings, 0 replies; 5+ messages in thread
From: Remi Pommarel @ 2014-05-06 21:40 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: devel, linux-kernel, Remi Pommarel
This patch fixes all prohibited spaces after open '(' checkpatch.pl errors
for rtl8188eu.
Signed-off-by: Remi Pommarel <repk@triplefau.lt>
---
drivers/staging/rtl8188eu/include/odm_debug.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8188eu/include/odm_debug.h b/drivers/staging/rtl8188eu/include/odm_debug.h
index e8c4cab..db7b44e 100644
--- a/drivers/staging/rtl8188eu/include/odm_debug.h
+++ b/drivers/staging/rtl8188eu/include/odm_debug.h
@@ -85,7 +85,7 @@
/*------------------------Export Marco Definition---------------------------*/
#define DbgPrint pr_info
#define RT_PRINTK(fmt, args...) \
- DbgPrint( "%s(): " fmt, __func__, ## args);
+ DbgPrint("%s(): " fmt, __func__, ## args);
#ifndef ASSERT
#define ASSERT(expr)
@@ -106,8 +106,8 @@
#define ODM_RT_ASSERT(pDM_Odm, expr, fmt) \
if (!(expr)) { \
- DbgPrint( "Assertion failed! %s at ......\n", #expr); \
- DbgPrint( " ......%s,%s,line=%d\n", __FILE__, \
+ DbgPrint("Assertion failed! %s at ......\n", #expr); \
+ DbgPrint(" ......%s,%s,line=%d\n", __FILE__, \
__func__, __LINE__); \
RT_PRINTK fmt; \
ASSERT(false); \
--
1.8.5.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-05-06 21:40 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-06 20:00 [PATCH] Staging: rtl8188eu: Fixed prohibited spaces after open '(' Rémi Pommarel
2014-05-06 21:10 ` Remi Pommarel
2014-05-06 21:27 ` Dan Carpenter
2014-05-06 21:40 ` Remi Pommarel
-- strict thread matches above, loose matches on Subject: below --
2014-05-06 21:40 Remi Pommarel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox