Skip to content
/ linux Public

Commit be04fe4

Browse files
Hans de GoedeSasha Levin
authored andcommitted
media: i2c: ov01a10: Add missing v4l2_subdev_cleanup() calls
[ Upstream commit 0dfec6e ] Add missing v4l2_subdev_cleanup() calls to cleanup after v4l2_subdev_init_finalize(). Fixes: 0827b58 ("media: i2c: add ov01a10 image sensor driver") Cc: stable@vger.kernel.org Signed-off-by: Hans de Goede <hansg@kernel.org> Tested-by: Mehdi Djait <mehdi.djait@linux.intel.com> # Dell XPS 9315 Reviewed-by: Mehdi Djait <mehdi.djait@linux.intel.com> Reviewed-by: Bingbu Cao <bingbu.cao@intel.com> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
1 parent 2195f2b commit be04fe4

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

drivers/media/i2c/ov01a10.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -864,6 +864,7 @@ static void ov01a10_remove(struct i2c_client *client)
864864
struct v4l2_subdev *sd = i2c_get_clientdata(client);
865865

866866
v4l2_async_unregister_subdev(sd);
867+
v4l2_subdev_cleanup(sd);
867868
media_entity_cleanup(&sd->entity);
868869
v4l2_ctrl_handler_free(sd->ctrl_handler);
869870

@@ -934,6 +935,7 @@ static int ov01a10_probe(struct i2c_client *client)
934935
err_pm_disable:
935936
pm_runtime_disable(dev);
936937
pm_runtime_set_suspended(&client->dev);
938+
v4l2_subdev_cleanup(&ov01a10->sd);
937939

938940
err_media_entity_cleanup:
939941
media_entity_cleanup(&ov01a10->sd.entity);

0 commit comments

Comments
 (0)