diff options
author | Axel Lin <axel.lin@gmail.com> | 2010-08-09 02:48:16 -0400 |
---|---|---|
committer | Liam Girdwood <lrg@slimlogic.co.uk> | 2010-08-11 06:38:02 -0400 |
commit | 8f1f151ed8ae4063837221cc9f5386a7af59f4e0 (patch) | |
tree | cde2de0cc29c272dfcf489f168dc6c698db006f7 /drivers/mfd/max8998.c | |
parent | 1bddc2f5c1727577183761581d5d4d1b00a8bd63 (diff) |
mfd: max8998 - fix incorrect kfree(i2c) in i2c_driver probe callback handler
The i2c_client received in probe() should not be kfree()'d.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Signed-off-by: Liam Girdwood <lrg@slimlogic.co.uk>
Diffstat (limited to 'drivers/mfd/max8998.c')
-rw-r--r-- | drivers/mfd/max8998.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/mfd/max8998.c b/drivers/mfd/max8998.c index 0d68de21ea9e..73e6f5c4efc9 100644 --- a/drivers/mfd/max8998.c +++ b/drivers/mfd/max8998.c | |||
@@ -89,10 +89,8 @@ static int max8998_i2c_probe(struct i2c_client *i2c, | |||
89 | int ret = 0; | 89 | int ret = 0; |
90 | 90 | ||
91 | max8998 = kzalloc(sizeof(struct max8998_dev), GFP_KERNEL); | 91 | max8998 = kzalloc(sizeof(struct max8998_dev), GFP_KERNEL); |
92 | if (max8998 == NULL) { | 92 | if (max8998 == NULL) |
93 | kfree(i2c); | ||
94 | return -ENOMEM; | 93 | return -ENOMEM; |
95 | } | ||
96 | 94 | ||
97 | i2c_set_clientdata(i2c, max8998); | 95 | i2c_set_clientdata(i2c, max8998); |
98 | max8998->dev = &i2c->dev; | 96 | max8998->dev = &i2c->dev; |