Signed-off-by: Axel Lin <axel....@gmail.com>
---
 drivers/gpio/gpio-mc9s08dz60.c |   21 +++------------------
 1 file changed, 3 insertions(+), 18 deletions(-)

diff --git a/drivers/gpio/gpio-mc9s08dz60.c b/drivers/gpio/gpio-mc9s08dz60.c
index 2738cc4..0ab7000 100644
--- a/drivers/gpio/gpio-mc9s08dz60.c
+++ b/drivers/gpio/gpio-mc9s08dz60.c
@@ -91,10 +91,9 @@ static int mc9s08dz60_direction_output(struct gpio_chip *gc,
 static int mc9s08dz60_probe(struct i2c_client *client,
                            const struct i2c_device_id *id)
 {
-       int ret = 0;
        struct mc9s08dz60 *mc9s;
 
-       mc9s = kzalloc(sizeof(*mc9s), GFP_KERNEL);
+       mc9s = devm_kzalloc(&client->dev, sizeof(*mc9s), GFP_KERNEL);
        if (!mc9s)
                return -ENOMEM;
 
@@ -110,30 +109,16 @@ static int mc9s08dz60_probe(struct i2c_client *client,
        mc9s->client = client;
        i2c_set_clientdata(client, mc9s);
 
-       ret = gpiochip_add(&mc9s->chip);
-       if (ret)
-               goto error;
-
-       return 0;
-
- error:
-       kfree(mc9s);
-       return ret;
+       return gpiochip_add(&mc9s->chip);
 }
 
 static int mc9s08dz60_remove(struct i2c_client *client)
 {
        struct mc9s08dz60 *mc9s;
-       int ret;
 
        mc9s = i2c_get_clientdata(client);
 
-       ret = gpiochip_remove(&mc9s->chip);
-       if (!ret)
-               kfree(mc9s);
-
-       return ret;
-
+       return gpiochip_remove(&mc9s->chip);
 }
 
 static const struct i2c_device_id mc9s08dz60_id[] = {
-- 
1.7.9.5



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to