[PATCH] I2C: via686a cleanups
authorJean Delvare <khali@linux-fr.org>
Tue, 19 Apr 2005 04:16:58 +0000 (21:16 -0700)
committerGreg KH <greg@press.kroah.org>
Tue, 19 Apr 2005 04:16:58 +0000 (21:16 -0700)
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>
drivers/i2c/chips/via686a.c

index 9b948f4531f5ade8899ca3efee959c7c3d33da04..ecd1e1de3f05abd1ec1eec87c248bf2b137067a4 100644 (file)
@@ -651,10 +651,9 @@ static int via686a_detect(struct i2c_adapter *adapter, int address, int kind)
        new_client->adapter = adapter;
        new_client->driver = &via686a_driver;
        new_client->flags = 0;
-       new_client->dev.parent = &adapter->dev;
 
        /* Fill in the remaining client fields and put into the global list */
-       snprintf(new_client->name, I2C_NAME_SIZE, client_name);
+       strlcpy(new_client->name, client_name, I2C_NAME_SIZE);
 
        data->valid = 0;
        init_MUTEX(&data->update_lock);