From: Rickard Strandqvist <rickard.strandqvist@xxxxxxxxxxxxxx> Variable ar assigned a value that is never used. I have also removed all the code that thereby serves no purpose. This was found using a static code analysis program called cppcheck Signed-off-by: Rickard Strandqvist <rickard_strandqvist@xxxxxxxxxxxxxxxxxx> --- drivers/video/fbdev/mb862xx/mb862xx-i2c.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/drivers/video/fbdev/mb862xx/mb862xx-i2c.c b/drivers/video/fbdev/mb862xx/mb862xx-i2c.c index c87e17a..6b9769f 100644 --- a/drivers/video/fbdev/mb862xx/mb862xx-i2c.c +++ b/drivers/video/fbdev/mb862xx/mb862xx-i2c.c @@ -20,7 +20,6 @@ static int mb862xx_i2c_wait_event(struct i2c_adapter *adap) { - struct mb862xxfb_par *par = adap->algo_data; u32 reg; do { @@ -48,8 +47,6 @@ static int mb862xx_i2c_do_address(struct i2c_adapter *adap, int addr) static int mb862xx_i2c_write_byte(struct i2c_adapter *adap, u8 byte) { - struct mb862xxfb_par *par = adap->algo_data; - outreg(i2c, GC_I2C_DAR, byte); outreg(i2c, GC_I2C_BCR, I2C_START); if (!mb862xx_i2c_wait_event(adap)) @@ -59,8 +56,6 @@ static int mb862xx_i2c_write_byte(struct i2c_adapter *adap, u8 byte) static int mb862xx_i2c_read_byte(struct i2c_adapter *adap, u8 *byte, int last) { - struct mb862xxfb_par *par = adap->algo_data; - outreg(i2c, GC_I2C_BCR, I2C_START | (last ? 0 : I2C_ACK)); if (!mb862xx_i2c_wait_event(adap)) return 0; -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html