[PATCH] i2c-mux-pca9541: fix setup_timer.cocci warnings

* [PATCH] i2c-mux-pca9541: fix setup_timer.cocci warnings
@ 2016-04-27 19:06 Julia Lawall
  2016-04-27 21:42   ` Peter Rosin
  2016-05-11 15:14 ` Wolfram Sang
  0 siblings, 2 replies; 4+ messages in thread
From: Julia Lawall @ 2016-04-27 19:06 UTC (permalink / raw)
  To: Corey Minyard, Guenter Roeck, Wolfram Sang
  Cc: kbuild-all, linux-i2c, linux-kernel

Use setup_timer function instead of initializing timer with the function
and data fields

Generated by: scripts/coccinelle/api/setup_timer.cocci

Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
---

tree:   git://git.code.sf.net/p/openipmi/linux-ipmi v4.5-ipmi
head:   671fec027e30fdac67b341f0271841f02db0a027
commit: 4a199ab1bf7b7a2b5f7e0b0eb4162b4e98b9cd7f [25/33] i2c-mux-pca9541: 
Add support for non-blocking handling

 i2c-mux-pca9541.c |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

--- a/drivers/i2c/muxes/i2c-mux-pca9541.c
+++ b/drivers/i2c/muxes/i2c-mux-pca9541.c
@@ -547,9 +547,7 @@ static int pca9541_probe(struct i2c_clie
 	data->op_e.smbus.data = &data->op_data;
 	data->op_e.smbus.size = I2C_SMBUS_BYTE_DATA;
 
-	init_timer(&data->timer);
-	data->timer.data = (unsigned long) data;
-	data->timer.function = pca9541_timeout;
+	setup_timer(&data->timer, pca9541_timeout, (unsigned long)data);
 
 	/*
 	 * I2C accesses are unprotected here.

^ permalink raw reply	[flat|nested] 4+ messages in thread