aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorJean Delvare <khali@linux-fr.org>2005-04-19 00:16:58 -0400
committerGreg KH <greg@press.kroah.org>2005-04-19 00:16:58 -0400
commit86b5ac878d4a63c772d03c5017b72cc799a8f2f2 (patch)
treeef4138c02aa447610146df63749e07c1859c3509 /drivers
parent05985715756b8667cb35a3e05e1668bb8afe3298 (diff)
[PATCH] I2C: via686a cleanups
Here comes a small cleanup patch for the via686a driver. I noticed the following two non-fatal problems: 1* The device parent is explicitely set, but it's not needed because the i2c core will do as the client is registered. 2* snprintf is used where strlcpy would suffice. Fixing them brings the via686a driver in line with what other similar drivers do. Signed-off-by: Jean Delvare <khali@linux-fr.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/i2c/chips/via686a.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/i2c/chips/via686a.c b/drivers/i2c/chips/via686a.c
index 9b948f4531f5..ecd1e1de3f05 100644
--- a/drivers/i2c/chips/via686a.c
+++ b/drivers/i2c/chips/via686a.c
@@ -651,10 +651,9 @@ static int via686a_detect(struct i2c_adapter *adapter, int address, int kind)
651 new_client->adapter = adapter; 651 new_client->adapter = adapter;
652 new_client->driver = &via686a_driver; 652 new_client->driver = &via686a_driver;
653 new_client->flags = 0; 653 new_client->flags = 0;
654 new_client->dev.parent = &adapter->dev;
655 654
656 /* Fill in the remaining client fields and put into the global list */ 655 /* Fill in the remaining client fields and put into the global list */
657 snprintf(new_client->name, I2C_NAME_SIZE, client_name); 656 strlcpy(new_client->name, client_name, I2C_NAME_SIZE);
658 657
659 data->valid = 0; 658 data->valid = 0;
660 init_MUTEX(&data->update_lock); 659 init_MUTEX(&data->update_lock);