media: atomisp: asc.acc.pipeline is always NULL
authorHans de Goede <hdegoede@redhat.com>
Wed, 15 Jun 2022 20:50:18 +0000 (21:50 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 8 Jul 2022 15:22:50 +0000 (16:22 +0100)
With the removal of the ACC ioctls and atomisp_acc.c asc.acc.pipeline
never gets set, so it is always NULL.

Remove asc.acc.pipeline and drop checks for it being NULL / !NULL.

Link: https://lore.kernel.org/linux-media/20220615205037.16549-22-hdegoede@redhat.com
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/staging/media/atomisp/pci/atomisp_cmd.c
drivers/staging/media/atomisp/pci/atomisp_compat_css20.c
drivers/staging/media/atomisp/pci/atomisp_ioctl.c
drivers/staging/media/atomisp/pci/atomisp_subdev.h

index 18f0633..d8b3896 100644 (file)
@@ -1888,9 +1888,6 @@ out:
                    && isp->sw_contex.file_input)
                        v4l2_subdev_call(isp->inputs[asd->input_curr].camera,
                                         video, s_stream, 1);
-               /* FIXME! FIX ACC implementation */
-               if (asd->acc.pipeline && css_pipe_done[asd->index])
-                       atomisp_css_acc_done(asd);
        }
        dev_dbg(isp->dev, "<%s\n", __func__);
 
index 51b31e6..646404a 100644 (file)
@@ -3857,8 +3857,7 @@ static struct atomisp_sub_device *__get_atomisp_subdev(
 
        for (i = 0; i < isp->num_of_streams; i++) {
                asd = &isp->asd[i];
-               if (asd->streaming == ATOMISP_DEVICE_STREAMING_DISABLED &&
-                   !asd->acc.pipeline)
+               if (asd->streaming == ATOMISP_DEVICE_STREAMING_DISABLED)
                        continue;
                for (j = 0; j < ATOMISP_INPUT_STREAM_NUM; j++) {
                        stream_env = &asd->stream_env[j];
index 69deeb5..725b3b1 100644 (file)
@@ -626,12 +626,6 @@ unsigned int atomisp_streaming_count(struct atomisp_device *isp)
 
 unsigned int atomisp_is_acc_enabled(struct atomisp_device *isp)
 {
-       unsigned int i;
-
-       for (i = 0; i < isp->num_of_streams; i++)
-               if (isp->asd[i].acc.pipeline)
-                       return 1;
-
        return 0;
 }
 
index f1e87c3..f611fd4 100644 (file)
@@ -323,7 +323,6 @@ struct atomisp_sub_device {
        struct v4l2_ctrl *disable_dz;
 
        struct {
-               struct ia_css_pipe *pipeline;
                struct completion acc_done;
        } acc;