media: i2c: rdacm21: Remove an incorrect fwnode_handle_put() call

tititiou36 · Hans Verkuil · commit 12d21fc2ba88 · 2023-09-14T23:31:55.000+02:00
The commit in Fixes has removed an fwnode_handle_put() call in the error handling path of the probe. Remove the same call from the remove function. Fixes: 1029939 ("media: v4l: async: Simplify async sub-device fwnode matching") Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
diff --git a/drivers/media/i2c/rdacm21.c b/drivers/media/i2c/rdacm21.c
@@ -608,7 +608,6 @@ static void rdacm21_remove(struct i2c_client *client)
 	v4l2_async_unregister_subdev(&dev->sd);
 	v4l2_ctrl_handler_free(&dev->ctrls);
 	i2c_unregister_device(dev->isp);
-	fwnode_handle_put(dev->sd.fwnode);
 }
 
 static const struct of_device_id rdacm21_of_ids[] = {

Original file line number	Diff line number	Diff line change
`@@ -608,7 +608,6 @@ static void rdacm21_remove(struct i2c_client *client)`
`608`	`608`	`v4l2_async_unregister_subdev(&dev->sd);`
`609`	`609`	`v4l2_ctrl_handler_free(&dev->ctrls);`
`610`	`610`	`i2c_unregister_device(dev->isp);`
`611`		`- fwnode_handle_put(dev->sd.fwnode);`
`612`	`611`	`}`
`613`	`612`
`614`	`613`	`static const struct of_device_id rdacm21_of_ids[] = {`