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 <gneukum1@xxxxxxxxx> --- 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 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel