Skip to contentSkip to conversation

Staging: i4l: Fix coding style issues.

PATCH Superseded Sandhya Bankar 2 participants 6 messages Mar 20, 2016 Staging

lore
Share & tools
lore
#0 lore raw

v2 cover Staging: i4l: Fix coding style issues.

Fix coding style issues.
Changes in v2:
* Correcting the heading.
* Resending v2 as previously it is not showing in mailing list.

Sandhya Bankar (4):
Staging: i4l: pcbit: capi: Add parentheses to complex macro.
Staging: i4l: pcbit: drv: Use !x instead of x == NULL
Staging: i4l: pcbit: edss1: Do not initialise statics to 0.
Staging: i4l: pcbit: layer2: Add parentheses to complex macro.

 drivers/staging/i4l/pcbit/capi.h   | 2 +-
 drivers/staging/i4l/pcbit/drv.c    | 4 ++--
 drivers/staging/i4l/pcbit/edss1.c  | 2 +-
 drivers/staging/i4l/pcbit/layer2.h | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)
Signature · 1 line
1.8.3.4
#1 lore raw

PATCH v2 1/4 Staging: i4l: pcbit: layer2: Add parentheses to complex macro.

Add parentheses to complex macro.

---
 drivers/staging/i4l/pcbit/layer2.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/i4l/pcbit/layer2.h +1 -1
index be1327b..6b9063e 100644
@@ -109,7 +109,7 @@
 #define SCHED_READ    0x01
 #define SCHED_WRITE   0x02
 
-#define SET_RUN_TIMEOUT 2 * HZ /* 2 seconds */
+#define SET_RUN_TIMEOUT (2 * HZ) /* 2 seconds */
 
 struct frame_buf {
 	ulong msg;
Signature · 1 line
1.8.3.4
#2 lore raw
In reply to Sandhya Bankar

Actually, when you resend something, you should put RESEND after PATCH in the subject line.

julia

Quoted 27 lines · Sandhya Bankar

Fix coding style issues.
Changes in v2:
* Correcting the heading.
* Resending v2 as previously it is not showing in mailing list.

Sandhya Bankar (4):
Staging: i4l: pcbit: capi: Add parentheses to complex macro.
Staging: i4l: pcbit: drv: Use !x instead of x == NULL
Staging: i4l: pcbit: edss1: Do not initialise statics to 0.
Staging: i4l: pcbit: layer2: Add parentheses to complex macro.

 drivers/staging/i4l/pcbit/capi.h   | 2 +-
 drivers/staging/i4l/pcbit/drv.c    | 4 ++--
 drivers/staging/i4l/pcbit/edss1.c  | 2 +-
 drivers/staging/i4l/pcbit/layer2.h | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

--
1.8.3.4

--
You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected].
To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/cover.1458471223.git.bankarsandhya512%40gmail.com.
For more options, visit https://groups.google.com/d/optout.

#3 lore raw

PATCH v2 2/4 Staging: i4l: pcbit: edss1: Use !x instead of x == NULL.

In reply to Sandhya Bankar

Use !x instead of x == NULL.

---
 drivers/staging/i4l/pcbit/edss1.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/i4l/pcbit/edss1.c +1 -1
index b2262ba..e72c164 100644
@@ -254,7 +254,7 @@ static void pcbit_fsm_timer(unsigned long data)
 
 	dev = chan2dev(chan);
 
-	if (dev == NULL) {
+	if (!dev) {
 		printk(KERN_WARNING "pcbit: timer for unknown device\n");
 		return;
 	}
Signature · 1 line
1.8.3.4
#4 lore raw

PATCH v2 3/4 Staging: i4l: pcbit: drv: Do not initialise statics to 0.

In reply to Sandhya Bankar

Do not initialise statics to 0.

---
 drivers/staging/i4l/pcbit/drv.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/staging/i4l/pcbit/drv.c +2 -2
index 2087a52..c1b6f88 100644
@@ -699,8 +699,8 @@ void pcbit_l3_receive(struct pcbit_dev *dev, ulong msg,
  */
 
 static char statbuf[STATBUF_LEN];
-static int stat_st = 0;
-static int stat_end = 0;
+static int stat_st;
+static int stat_end;
 
 static int pcbit_stat(u_char __user *buf, int len, int driver, int channel)
 {
Signature · 1 line
1.8.3.4
#5 lore raw

PATCH v2 4/4 Staging: i4l: pcbit: capi: Add parentheses to complex macro.

In reply to Sandhya Bankar

Add parentheses to complex macro.

---
 drivers/staging/i4l/pcbit/capi.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/i4l/pcbit/capi.h +1 -1
index 635f634..6f6f4dd 100644
@@ -17,7 +17,7 @@
 #define REQ_DISPLAY       0x04
 #define REQ_USER_TO_USER  0x08
 
-#define AppInfoMask  REQ_CAUSE | REQ_DISPLAY | REQ_USER_TO_USER
+#define AppInfoMask  (REQ_CAUSE | REQ_DISPLAY | REQ_USER_TO_USER)
 
 /* Connection Setup */
 extern int capi_conn_req(const char *calledPN, struct sk_buff **buf,
Signature · 1 line
1.8.3.4