Skip to contentSkip to conversation

misc: mic: bug fixes

PATCH New Sudeep Dutt 1 participant 3 messages Jun 10, 2015 Char and misc drivers

lore
Share & tools
lore
#0 lore raw

cover misc: mic: bug fixes

This patch series fixes reported issues including a randconfig build error and a warning from a static checker.

Sudeep Dutt (2):
misc: mic: Fix randconfig build error by including errno.h
misc: mic: Fix reported static checker warning

 drivers/misc/mic/scif/scif_main.c | 4 ----
 drivers/misc/mic/scif/scif_rb.c   | 1 +
 2 files changed, 1 insertion(+), 4 deletions(-)
Signature · 1 line
1.8.2.1
#1 lore raw

PATCH 1/2 misc: mic: Fix randconfig build error by including errno.h

This issue was reported @ https://lkml.org/lkml/2015/6/9/731

---
 drivers/misc/mic/scif/scif_rb.c | 1 +
 1 file changed, 1 insertion(+)
drivers/misc/mic/scif/scif_rb.c +1 -0
index 47de222..637cc46 100644
@@ -18,6 +18,7 @@
 #include <linux/circ_buf.h>
 #include <linux/types.h>
 #include <linux/io.h>
+#include <linux/errno.h>
 
 #include "scif_rb.h"
 
Signature · 1 line
1.8.2.1
#2 lore raw

PATCH 2/2 misc: mic: Fix reported static checker warning

In reply to Sudeep Dutt

Delete unnecessary prints resulting in an "spdev could be null" warning from a static checker in scif_peer_remove(..).

---
 drivers/misc/mic/scif/scif_main.c | 4 ----
 1 file changed, 4 deletions(-)
drivers/misc/mic/scif/scif_main.c +0 -4
index f7187dc..6ce851f 100644
@@ -91,8 +91,6 @@ static int scif_peer_probe(struct scif_peer_dev *spdev)
 	rcu_assign_pointer(scifdev->spdev, spdev);
 
 	/* In the future SCIF kernel client devices will be added here */
-	dev_info(&spdev->dev, "Peer added dnode %d\n",
-		 spdev->dnode);
 	return 0;
 }
 
@@ -109,8 +107,6 @@ static void scif_peer_remove(struct scif_peer_dev *spdev)
 	mutex_lock(&scif_info.conflock);
 	scif_info.total--;
 	mutex_unlock(&scif_info.conflock);
-	dev_info(&spdev->dev, "Peer removed dnode %d\n",
-		 spdev->dnode);
 }
 
 static void scif_qp_setup_handler(struct work_struct *work)
Signature · 1 line
1.8.2.1