Hi Tomi, Thank you for the patch. On Mon, May 24, 2021 at 02:08:33PM +0300, Tomi Valkeinen wrote: > cal_camerarx_create() doesn't handle error returned from > cal_camerarx_sd_init_cfg(). Fix this. > > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxxxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/media/platform/ti-vpe/cal-camerarx.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/platform/ti-vpe/cal-camerarx.c b/drivers/media/platform/ti-vpe/cal-camerarx.c > index 124a4e2bdefe..e2e384a887ac 100644 > --- a/drivers/media/platform/ti-vpe/cal-camerarx.c > +++ b/drivers/media/platform/ti-vpe/cal-camerarx.c > @@ -845,7 +845,9 @@ struct cal_camerarx *cal_camerarx_create(struct cal_dev *cal, > if (ret) > goto error; > > - cal_camerarx_sd_init_cfg(sd, NULL); > + ret = cal_camerarx_sd_init_cfg(sd, NULL); > + if (ret) > + goto error; > > ret = v4l2_device_register_subdev(&cal->v4l2_dev, sd); > if (ret) -- Regards, Laurent Pinchart
