2019-05-26 01:20:54

by Geordan Neukum

[permalink] [raw]
Subject: [PATCH 1/8] staging: kpc2000: kpc_i2c: Remove unused rw_sem

In pi2c_probe, a rw_sem is initialized and stashed off in the
i2c_device private runtime state struct. This rw_sem is never used
after initialization. Remove the rw_sem and cleanup unneeded header
inclusion.

Signed-off-by: Geordan Neukum <[email protected]>
---
drivers/staging/kpc2000/kpc2000_i2c.c | 3 ---
1 file changed, 3 deletions(-)

diff --git a/drivers/staging/kpc2000/kpc2000_i2c.c b/drivers/staging/kpc2000/kpc2000_i2c.c
index fb9a8386bcce..2c272ad8eca6 100644
--- a/drivers/staging/kpc2000/kpc2000_i2c.c
+++ b/drivers/staging/kpc2000/kpc2000_i2c.c
@@ -25,7 +25,6 @@
#include <linux/slab.h>
#include <linux/platform_device.h>
#include <linux/fs.h>
-#include <linux/rwsem.h>
#include <linux/delay.h>
#include <linux/i2c.h>
#include "kpc.h"
@@ -38,7 +37,6 @@ struct i2c_device {
unsigned long smba;
struct i2c_adapter adapter;
struct platform_device *pldev;
- struct rw_semaphore rw_sem;
unsigned int features;
};

@@ -606,7 +604,6 @@ static int pi2c_probe(struct platform_device *pldev)
priv->features |= FEATURE_BLOCK_BUFFER;

//init_MUTEX(&lddata->sem);
- init_rwsem(&priv->rw_sem);

/* set up the sysfs linkage to our parent device */
priv->adapter.dev.parent = &pldev->dev;
--
2.21.0