summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJean Delvare <khali@linux-fr.org>2009-09-25 14:15:59 +0200
committerSamuel Ortiz <sameo@linux.intel.com>2009-12-13 19:20:40 +0100
commit29f02646f0fa2b3c3a91f9145653e6a6ae8e7eb1 (patch)
tree80f9a44d3ed34217c7a82c9302ace4cb1873fa80
parent956266da703e3885924fe21bfc20f32d8f875474 (diff)
downloadop-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.c3
-rw-r--r--drivers/mfd/twl4030-core.c2
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);
}
OpenPOWER on IntegriCloud