Commit Message
Hans de Goede
Jan. 23, 2023, 12:51 p.m. UTC
Call atomisp_gmin_remove_subdev() on probe failure to properly free
the GPIOs and other resources acquired by the gmin_camera_platform_data()
call earlier.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/staging/media/atomisp/i2c/atomisp-ov2722.c | 1 +
1 file changed, 1 insertion(+)
Comments
On Mon, Jan 23, 2023 at 01:51:59PM +0100, Hans de Goede wrote: > Call atomisp_gmin_remove_subdev() on probe failure to properly free > the GPIOs and other resources acquired by the gmin_camera_platform_data() > call earlier. Fixes? Reviewed-by: Andy Shevchenko <andy@kernel.org> > Signed-off-by: Hans de Goede <hdegoede@redhat.com> > --- > drivers/staging/media/atomisp/i2c/atomisp-ov2722.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c b/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c > index d819ab5de28a..d874e12da8cc 100644 > --- a/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c > +++ b/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c > @@ -994,6 +994,7 @@ static int ov2722_probe(struct i2c_client *client) > v4l2_ctrl_handler_free(&dev->ctrl_handler); > > out_free: > + atomisp_gmin_remove_subdev(&dev->sd); > v4l2_device_unregister_subdev(&dev->sd); > kfree(dev); > return ret; > -- > 2.39.0 >
diff --git a/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c b/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c index d819ab5de28a..d874e12da8cc 100644 --- a/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c +++ b/drivers/staging/media/atomisp/i2c/atomisp-ov2722.c @@ -994,6 +994,7 @@ static int ov2722_probe(struct i2c_client *client) v4l2_ctrl_handler_free(&dev->ctrl_handler); out_free: + atomisp_gmin_remove_subdev(&dev->sd); v4l2_device_unregister_subdev(&dev->sd); kfree(dev); return ret;