media: staging: imx: controls are from another device, mark this
authorHans Verkuil <hverkuil-cisco@xs4all.nl>
Wed, 17 Jan 2024 11:46:45 +0000 (12:46 +0100)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Fri, 16 Feb 2024 10:46:32 +0000 (11:46 +0100)
The last argument of v4l2_ctrl_add_handler() indicates whether the controls
you add are from a control handler owned by another driver (true) or from
the same driver (false). In this case the last argument was incorrectly set
to false. The control handlers are from different devices.

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/staging/media/imx/imx-media-fim.c

index e28a33d9dec75572fc46fe3129b749f0e74cfce4..ccbc0371fba2e4a42cd76ac805e3468bf726c0fa 100644 (file)
@@ -401,7 +401,7 @@ int imx_media_fim_add_controls(struct imx_media_fim *fim)
 {
        /* add the FIM controls to the calling subdev ctrl handler */
        return v4l2_ctrl_add_handler(fim->sd->ctrl_handler,
-                                    &fim->ctrl_handler, NULL, false);
+                                    &fim->ctrl_handler, NULL, true);
 }
 
 /* Called by the subdev in its subdev registered callback */