diff options
author | Wolfram Sang <wsa+renesas@sang-engineering.com> | 2023-09-22 10:04:19 +0200 |
---|---|---|
committer | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2023-10-07 10:55:45 +0200 |
commit | b622e46690eb34a738bb295308f83c253b78afe0 (patch) | |
tree | da1c4f3a20436194f1458aaac534ce061b5ca6f7 | |
parent | 36fe515c1a3cd5eac148e8a591a82108d92d5522 (diff) | |
download | lwn-b622e46690eb34a738bb295308f83c253b78afe0.tar.gz lwn-b622e46690eb34a738bb295308f83c253b78afe0.zip |
media: i2c: drop check because i2c_unregister_device() is NULL safe
No need to check the argument of i2c_unregister_device() because the
function itself does it.
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
-rw-r--r-- | drivers/media/i2c/rdacm20.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/i2c/rdacm20.c b/drivers/media/i2c/rdacm20.c index f4e2e2f3972a..b4647bda8c21 100644 --- a/drivers/media/i2c/rdacm20.c +++ b/drivers/media/i2c/rdacm20.c @@ -625,8 +625,7 @@ error_free_ctrls: v4l2_ctrl_handler_free(&dev->ctrls); error: media_entity_cleanup(&dev->sd.entity); - if (dev->sensor) - i2c_unregister_device(dev->sensor); + i2c_unregister_device(dev->sensor); dev_err(&client->dev, "probe failed\n"); |