Title: [8850] trunk/drivers/staging/iio/adc/ad799x_core.c: [#5861] Enable driver support for ad799x ad converter in new IIO
Revision
8850
Author
hennerich
Date
2010-05-26 04:21:59 -0400 (Wed, 26 May 2010)

Log Message

[#5861] Enable driver support for ad799x ad converter in new IIO
Fix some issues reported by Mike Frysinger:
Remove void casts
Use __devexit
Remove redundant function
Declare some functions static

Modified Paths

Diff

Modified: trunk/drivers/staging/iio/adc/ad799x_core.c (8849 => 8850)


--- trunk/drivers/staging/iio/adc/ad799x_core.c	2010-05-25 20:32:37 UTC (rev 8849)
+++ trunk/drivers/staging/iio/adc/ad799x_core.c	2010-05-26 08:21:59 UTC (rev 8850)
@@ -136,20 +136,19 @@
 static IIO_DEVICE_ATTR(in_precision, S_IRUGO, ad799x_show_precision,
 		       NULL, 0);
 
-int ad7991_5_9_set_scan_mode(struct ad799x_state *st, unsigned mask)
+static int ad7991_5_9_set_scan_mode(struct ad799x_state *st, unsigned mask)
 {
 	return i2c_smbus_write_byte(st->client,
 		st->config | (mask << AD799X_CHANNEL_SHIFT));
 }
 
-int ad7992_3_4_set_scan_mode(struct ad799x_state *st, unsigned mask)
+static int ad7992_3_4_set_scan_mode(struct ad799x_state *st, unsigned mask)
 {
-
 	return ad799x_i2c_write8(st, AD7998_CONF_REG,
 		st->config | (mask << AD799X_CHANNEL_SHIFT));
 }
 
-int ad7997_8_set_scan_mode(struct ad799x_state *st, unsigned mask)
+static int ad7997_8_set_scan_mode(struct ad799x_state *st, unsigned mask)
 {
 	return ad799x_i2c_write16(st, AD7998_CONF_REG,
 		st->config | (mask << AD799X_CHANNEL_SHIFT));
@@ -324,7 +323,6 @@
 
 	ret = ad799x_i2c_write8(st, AD7998_CYCLE_TMR_REG, t);
 
-
 error_ret_mutex:
 	mutex_unlock(&dev_info->mlock);
 
@@ -671,7 +669,7 @@
 		.int_vref_mv = 4096,
 		.dev_attrs = &ad7991_5_9_3_4_dev_attr_group,
 		.scan_attrs = &ad7991_5_9_3_4_scan_el_group,
-		.ad799x_set_scan_mode = (void *) ad7991_5_9_set_scan_mode,
+		.ad799x_set_scan_mode = ad7991_5_9_set_scan_mode,
 	},
 	[ad7995] = {
 		.num_inputs = 4,
@@ -679,7 +677,7 @@
 		.int_vref_mv = 1024,
 		.dev_attrs = &ad7991_5_9_3_4_dev_attr_group,
 		.scan_attrs = &ad7991_5_9_3_4_scan_el_group,
-		.ad799x_set_scan_mode = (void *) ad7991_5_9_set_scan_mode,
+		.ad799x_set_scan_mode = ad7991_5_9_set_scan_mode,
 	},
 	[ad7999] = {
 		.num_inputs = 4,
@@ -687,7 +685,7 @@
 		.int_vref_mv = 1024,
 		.dev_attrs = &ad7991_5_9_3_4_dev_attr_group,
 		.scan_attrs = &ad7991_5_9_3_4_scan_el_group,
-		.ad799x_set_scan_mode = (void *) ad7991_5_9_set_scan_mode,
+		.ad799x_set_scan_mode = ad7991_5_9_set_scan_mode,
 	},
 	[ad7992] = {
 		.num_inputs = 2,
@@ -698,7 +696,7 @@
 		.dev_attrs = &ad7992_dev_attr_group,
 		.scan_attrs = &ad7992_scan_el_group,
 		.event_attrs = &ad7992_event_attrs_group,
-		.ad799x_set_scan_mode = (void *) ad7992_3_4_set_scan_mode,
+		.ad799x_set_scan_mode = ad7992_3_4_set_scan_mode,
 	},
 	[ad7993] = {
 		.num_inputs = 4,
@@ -709,7 +707,7 @@
 		.dev_attrs = &ad7991_5_9_3_4_dev_attr_group,
 		.scan_attrs = &ad7991_5_9_3_4_scan_el_group,
 		.event_attrs = &ad7993_4_7_8_event_attrs_group,
-		.ad799x_set_scan_mode = (void *) ad7992_3_4_set_scan_mode,
+		.ad799x_set_scan_mode = ad7992_3_4_set_scan_mode,
 	},
 	[ad7994] = {
 		.num_inputs = 4,
@@ -720,7 +718,7 @@
 		.dev_attrs = &ad7991_5_9_3_4_dev_attr_group,
 		.scan_attrs = &ad7991_5_9_3_4_scan_el_group,
 		.event_attrs = &ad7993_4_7_8_event_attrs_group,
-		.ad799x_set_scan_mode = (void *) ad7992_3_4_set_scan_mode,
+		.ad799x_set_scan_mode = ad7992_3_4_set_scan_mode,
 	},
 	[ad7997] = {
 		.num_inputs = 8,
@@ -731,7 +729,7 @@
 		.dev_attrs = &ad7997_8_dev_attr_group,
 		.scan_attrs = &ad7997_8_scan_el_group,
 		.event_attrs = &ad7993_4_7_8_event_attrs_group,
-		.ad799x_set_scan_mode = (void *) ad7997_8_set_scan_mode,
+		.ad799x_set_scan_mode = ad7997_8_set_scan_mode,
 	},
 	[ad7998] = {
 		.num_inputs = 8,
@@ -742,15 +740,10 @@
 		.dev_attrs = &ad7997_8_dev_attr_group,
 		.scan_attrs = &ad7997_8_scan_el_group,
 		.event_attrs = &ad7993_4_7_8_event_attrs_group,
-		.ad799x_set_scan_mode = (void *) ad7997_8_set_scan_mode,
+		.ad799x_set_scan_mode = ad7997_8_set_scan_mode,
 	},
 };
 
-static int ad799x_initial_setup(struct ad799x_state *st)
-{
-	return ad799x_set_scan_mode(st, 0);
-}
-
 static int __devinit ad799x_probe(struct i2c_client *client,
 				   const struct i2c_device_id *id)
 {
@@ -802,7 +795,7 @@
 	st->indio_dev->modes = INDIO_DIRECT_MODE;
 	st->indio_dev->num_interrupt_lines = 1;
 
-	ret = ad799x_initial_setup(st);
+	ret = ad799x_set_scan_mode(st, 0);
 	if (ret)
 		goto error_free_device;
 
@@ -853,12 +846,11 @@
 	if (!IS_ERR(st->reg))
 		regulator_put(st->reg);
 	kfree(st);
-
 error_ret:
 	return ret;
 }
 
-static int ad799x_remove(struct i2c_client *client)
+static __devexit int ad799x_remove(struct i2c_client *client)
 {
 	struct ad799x_state *st = i2c_get_clientdata(client);
 	struct iio_dev *indio_dev = st->indio_dev;
@@ -897,7 +889,7 @@
 		.name = "ad799x",
 	},
 	.probe = ad799x_probe,
-	.remove = ad799x_remove,
+	.remove = __devexit_p(ad799x_remove),
 	.id_table = ad799x_id,
 };
 
_______________________________________________
Linux-kernel-commits mailing list
[email protected]
https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits

Reply via email to