Hi André On Fri, Oct 27, 2023 at 11:23 PM André Apitzsch <git@xxxxxxxxxxx> wrote: > > Hi Ricardo, > > Am Freitag, dem 27.10.2023 um 14:25 +0200 schrieb Ricardo Ribalda > Delgado: > > Hi Andre > > On Mon, Oct 23, 2023 at 11:49 PM André Apitzsch <git@xxxxxxxxxxx> > > wrote: > > > > > > Code refinement, no functional changes. > > > > > > Signed-off-by: André Apitzsch <git@xxxxxxxxxxx> > > > > With Jacopos comments (don't use de_err_probe()) > > Reviewed-by: Ricardo Ribalda <ribalda@xxxxxxxxxxxx> > > > > > + ret = imx214_ctrls_init(imx214); > > > + if (ret < 0) > > > goto free_ctrl; > > > > It seems like we can mutex_destroy a non inited mutex. Could you send > > a follow-up patch to fix that? > > > Sorry, I don't get it. Could you explain what you mean. Thanks. > If the controls are initialized incorrectly we will jump to free_ctrl in line 1046, which calls mutex_destroy(&imx214->mutex); But that mutex initialized in line 1050. You did not introduce the bug, but since you have the hardware and are sending the other patches it would be great if you could add a new patch to fix it :) Thanks! > > Thanks! >
