diff options
author | Jean Delvare <jdelvare@suse.de> | 2014-04-14 06:55:38 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-05-27 16:40:51 -0400 |
commit | 8283b4919e00fb110ac338d6b9984ca92d5030fa (patch) | |
tree | 3b8f95f677972af401d715ef9e5e24bbead2b475 /drivers/vfio | |
parent | 1bb6c08abfb653ce6e65d8ab4ddef403227afedf (diff) |
driver core: dev_set_drvdata can no longer fail
So there is no point in checking its return value, which will soon
disappear.
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/vfio')
-rw-r--r-- | drivers/vfio/vfio.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/drivers/vfio/vfio.c b/drivers/vfio/vfio.c index 512f479d8a50..f018d8d0f975 100644 --- a/drivers/vfio/vfio.c +++ b/drivers/vfio/vfio.c | |||
@@ -349,7 +349,6 @@ struct vfio_device *vfio_group_create_device(struct vfio_group *group, | |||
349 | void *device_data) | 349 | void *device_data) |
350 | { | 350 | { |
351 | struct vfio_device *device; | 351 | struct vfio_device *device; |
352 | int ret; | ||
353 | 352 | ||
354 | device = kzalloc(sizeof(*device), GFP_KERNEL); | 353 | device = kzalloc(sizeof(*device), GFP_KERNEL); |
355 | if (!device) | 354 | if (!device) |
@@ -360,12 +359,7 @@ struct vfio_device *vfio_group_create_device(struct vfio_group *group, | |||
360 | device->group = group; | 359 | device->group = group; |
361 | device->ops = ops; | 360 | device->ops = ops; |
362 | device->device_data = device_data; | 361 | device->device_data = device_data; |
363 | 362 | dev_set_drvdata(dev, device); | |
364 | ret = dev_set_drvdata(dev, device); | ||
365 | if (ret) { | ||
366 | kfree(device); | ||
367 | return ERR_PTR(ret); | ||
368 | } | ||
369 | 363 | ||
370 | /* No need to get group_lock, caller has group reference */ | 364 | /* No need to get group_lock, caller has group reference */ |
371 | vfio_group_get(group); | 365 | vfio_group_get(group); |