[PATCH char-misc-next 2/2] misc: mic: Fix reported static checker warning

From: Sudeep Dutt
Date: Wed Jun 10 2015 - 00:01:04 EST


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

Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Reviewed-by: Ashutosh Dixit <ashutosh.dixit@xxxxxxxxx>
Signed-off-by: Sudeep Dutt <sudeep.dutt@xxxxxxxxx>
---
drivers/misc/mic/scif/scif_main.c | 4 ----
1 file changed, 4 deletions(-)

diff --git a/drivers/misc/mic/scif/scif_main.c b/drivers/misc/mic/scif/scif_main.c
index f7187dc..6ce851f 100644
--- a/drivers/misc/mic/scif/scif_main.c
+++ b/drivers/misc/mic/scif/scif_main.c
@@ -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)
--
1.8.2.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/