diff options
author | Jean Delvare <khali@linux-fr.org> | 2009-09-25 14:15:59 +0200 |
---|---|---|
committer | Samuel Ortiz <sameo@linux.intel.com> | 2009-12-13 19:20:40 +0100 |
commit | 29f02646f0fa2b3c3a91f9145653e6a6ae8e7eb1 (patch) | |
tree | 80f9a44d3ed34217c7a82c9302ace4cb1873fa80 | |
parent | 956266da703e3885924fe21bfc20f32d8f875474 (diff) | |
download | op-kernel-dev-29f02646f0fa2b3c3a91f9145653e6a6ae8e7eb1.zip op-kernel-dev-29f02646f0fa2b3c3a91f9145653e6a6ae8e7eb1.tar.gz |
mfd: Don't abuse i2c_client.name
The name field of struct i2c_client is for i2c-core's use, it should
never be changed by the drivers themselves.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
-rw-r--r-- | drivers/mfd/ab3100-core.c | 3 | ||||
-rw-r--r-- | drivers/mfd/twl4030-core.c | 2 |
2 files changed, 0 insertions, 5 deletions
diff --git a/drivers/mfd/ab3100-core.c b/drivers/mfd/ab3100-core.c index 6134810..fd42a80 100644 --- a/drivers/mfd/ab3100-core.c +++ b/drivers/mfd/ab3100-core.c @@ -900,9 +900,6 @@ static int __init ab3100_probe(struct i2c_client *client, goto exit_no_testreg_client; } - strlcpy(ab3100->testreg_client->name, id->name, - sizeof(ab3100->testreg_client->name)); - err = ab3100_setup(ab3100); if (err) goto exit_no_setup; diff --git a/drivers/mfd/twl4030-core.c b/drivers/mfd/twl4030-core.c index 40449cd..e3abbf6 100644 --- a/drivers/mfd/twl4030-core.c +++ b/drivers/mfd/twl4030-core.c @@ -814,8 +814,6 @@ twl4030_probe(struct i2c_client *client, const struct i2c_device_id *id) status = -ENOMEM; goto fail; } - strlcpy(twl->client->name, id->name, - sizeof(twl->client->name)); } mutex_init(&twl->xfer_lock); } |