* [PATCH] Staging: wilc1000 :Insert blank line after declaration
@ 2015-07-31 5:25 Shraddha Barke
2015-07-31 20:52 ` Greg Kroah-Hartman
0 siblings, 1 reply; 3+ messages in thread
From: Shraddha Barke @ 2015-07-31 5:25 UTC (permalink / raw)
To: Dean Lee, Rachel Kim, Johnny Kim, Chris Park, Greg Kroah-Hartman,
linux-wireless, devel, linux-kernel
Cc: Shraddha Barke
This patch fixes the following checkpatch.pl warning:
WARNING: Missing a blank line after declarations
Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com>
---
drivers/staging/wilc1000/coreconfigurator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/wilc1000/coreconfigurator.c
index 1143282..0f31d63 100644
--- a/drivers/staging/wilc1000/coreconfigurator.c
+++ b/drivers/staging/wilc1000/coreconfigurator.c
@@ -1094,6 +1094,7 @@ void ProcessCharWid(char *pcPacket, s32 *ps32PktLen,
u8 *pu8val = (u8 *)ps8WidVal;
u8 u8val = 0;
s32 s32PktLen = *ps32PktLen;
+
if (pstrWID == NULL) {
PRINT_WRN(CORECONFIG_DBG, "Can't set CHAR val 0x%x ,NULL structure\n", u8val);
return;
--
2.1.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] Staging: wilc1000 :Insert blank line after declaration
2015-07-31 5:25 [PATCH] Staging: wilc1000 :Insert blank line after declaration Shraddha Barke
@ 2015-07-31 20:52 ` Greg Kroah-Hartman
0 siblings, 0 replies; 3+ messages in thread
From: Greg Kroah-Hartman @ 2015-07-31 20:52 UTC (permalink / raw)
To: Shraddha Barke
Cc: Dean Lee, Rachel Kim, Johnny Kim, Chris Park, linux-wireless,
devel, linux-kernel
On Fri, Jul 31, 2015 at 10:55:40AM +0530, Shraddha Barke wrote:
> This patch fixes the following checkpatch.pl warning:
>
> WARNING: Missing a blank line after declarations
> Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com>
Please add a blank line and resend.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] Staging: wilc1000 :Insert blank line after declaration
@ 2015-08-02 10:51 Shraddha Barke
0 siblings, 0 replies; 3+ messages in thread
From: Shraddha Barke @ 2015-08-02 10:51 UTC (permalink / raw)
To: Johnny Kim, Rachel Kim, Dean Lee, Chris Park, Greg Kroah-Hartman,
linux-wireless, devel, linux-kernel
Cc: Shraddha Barke
This patch fixes the following checkpatch.pl warning:
WARNING: Missing a blank line after declarations
Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com>
---
drivers/staging/wilc1000/coreconfigurator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/wilc1000/coreconfigurator.c
index 9cb46bb..a6d809f 100644
--- a/drivers/staging/wilc1000/coreconfigurator.c
+++ b/drivers/staging/wilc1000/coreconfigurator.c
@@ -1152,6 +1152,7 @@ void ProcessShortWid(char *pcPacket, s32 *ps32PktLen,
u16 *pu16val = (u16 *)ps8WidVal;
u16 u16val = 0;
s32 s32PktLen = *ps32PktLen;
+
if (pstrWID == NULL) {
PRINT_WRN(CORECONFIG_DBG, "Can't set SHORT val 0x%x ,NULL structure\n", u16val);
return;
--
2.1.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-08-02 10:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-31 5:25 [PATCH] Staging: wilc1000 :Insert blank line after declaration Shraddha Barke
2015-07-31 20:52 ` Greg Kroah-Hartman
2015-08-02 10:51 Shraddha Barke
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).