diff options
author | Lad, Prabhakar <prabhakar.lad@ti.com> | 2013-01-03 08:09:19 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2013-02-05 11:05:06 -0500 |
commit | d41d81983a9eb44626ab3ddd1f0184e4815f5bcc (patch) | |
tree | b7b918eb9c8ac0a3882066be5143c6e93ee31268 | |
parent | b4c13d3df1dfabac3d7737924a0dec665a971484 (diff) |
[media] ths7303: use devm_kzalloc() instead of kzalloc()
I2C drivers can use devm_kzalloc() too in their .probe() methods. Doing so
simplifies their clean up paths.
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/i2c/ths7303.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/i2c/ths7303.c b/drivers/media/i2c/ths7303.c index c31cc04fffd2..e747524ba6ed 100644 --- a/drivers/media/i2c/ths7303.c +++ b/drivers/media/i2c/ths7303.c | |||
@@ -175,7 +175,7 @@ static int ths7303_probe(struct i2c_client *client, | |||
175 | v4l_info(client, "chip found @ 0x%x (%s)\n", | 175 | v4l_info(client, "chip found @ 0x%x (%s)\n", |
176 | client->addr << 1, client->adapter->name); | 176 | client->addr << 1, client->adapter->name); |
177 | 177 | ||
178 | sd = kzalloc(sizeof(struct v4l2_subdev), GFP_KERNEL); | 178 | sd = devm_kzalloc(&client->dev, sizeof(struct v4l2_subdev), GFP_KERNEL); |
179 | if (sd == NULL) | 179 | if (sd == NULL) |
180 | return -ENOMEM; | 180 | return -ENOMEM; |
181 | 181 | ||
@@ -189,7 +189,6 @@ static int ths7303_remove(struct i2c_client *client) | |||
189 | struct v4l2_subdev *sd = i2c_get_clientdata(client); | 189 | struct v4l2_subdev *sd = i2c_get_clientdata(client); |
190 | 190 | ||
191 | v4l2_device_unregister_subdev(sd); | 191 | v4l2_device_unregister_subdev(sd); |
192 | kfree(sd); | ||
193 | 192 | ||
194 | return 0; | 193 | return 0; |
195 | } | 194 | } |