Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
        struct wm8940_priv *wm8940;
        int ret;
 
-       wm8940 = kzalloc(sizeof(struct wm8940_priv), GFP_KERNEL);
+       wm8940 = devm_kzalloc(&i2c->dev, sizeof(struct wm8940_priv),
+                             GFP_KERNEL);
        if (wm8940 == NULL)
                return -ENOMEM;
 
 
        ret = snd_soc_register_codec(&i2c->dev,
                        &soc_codec_dev_wm8940, &wm8940_dai, 1);
-       if (ret < 0)
-               kfree(wm8940);
+
        return ret;
 }
 
 static __devexit int wm8940_i2c_remove(struct i2c_client *client)
 {
        snd_soc_unregister_codec(&client->dev);
-       kfree(i2c_get_clientdata(client));
+
        return 0;
 }