media: staging: media: imx: imx7-media-csi: Remove imx_media_of_add_csi
authorPaul Elder <paul.elder@ideasonboard.com>
Fri, 6 May 2022 11:36:58 +0000 (12:36 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 8 Jul 2022 17:02:33 +0000 (18:02 +0100)
imx_media_of_add_csi() from imx-media-of.c validates that the fwnode for
the CSI bridge is enabled, and adds the fwnode to the async notifier of
the imxmd. The former is redundant, as if the CSI bridge driver probes,
clearly it is enabled in the DT. The latter is not needed as the imxmd
notifier isn't used anymore. The call is thus not needed and can be
dropped. This removes the dependency of imx7-media-csi.c on
imx-media-of.c.

Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
Acked-by: Rui Miguel Silva <rmfrfs@gmail.com>
Tested-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/staging/media/imx/imx7-media-csi.c

index 4b1fae5..8af5b69 100644 (file)
@@ -1206,10 +1206,6 @@ static int imx7_csi_media_init(struct imx7_csi *csi)
 
        csi->imxmd = imxmd;
 
-       ret = imx_media_of_add_csi(imxmd, csi->dev->of_node);
-       if (ret < 0 && ret != -ENODEV && ret != -EEXIST)
-               goto error;
-
        v4l2_subdev_init(&csi->sd, &imx7_csi_subdev_ops);
        v4l2_set_subdevdata(&csi->sd, csi);
        csi->sd.internal_ops = &imx7_csi_internal_ops;