This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: atomisp: Remove unused fields from struct atomisp_input_subdev
Author:  Hans de Goede <[email protected]>
Date:    Sun May 14 19:59:13 2023 +0100

Remove unused fields from struct atomisp_input_subdev:

1. frame_size is never used at all
2. sensor_index is always 0, just directly pass 0 in the single user.

Link: https://lore.kernel.org/r/[email protected]

Reviewed-by: Andy Shevchenko <[email protected]>
Signed-off-by: Hans de Goede <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/staging/media/atomisp/pci/atomisp_internal.h | 3 ---
 drivers/staging/media/atomisp/pci/atomisp_ioctl.c    | 2 +-
 drivers/staging/media/atomisp/pci/atomisp_v4l2.c     | 7 -------
 3 files changed, 1 insertion(+), 11 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_internal.h 
b/drivers/staging/media/atomisp/pci/atomisp_internal.h
index feaf4037a389..ee0dd5eb4711 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_internal.h
+++ b/drivers/staging/media/atomisp/pci/atomisp_internal.h
@@ -126,15 +126,12 @@ struct atomisp_input_subdev {
        enum atomisp_camera_port port;
        struct v4l2_subdev *camera;
        struct v4l2_subdev *motor;
-       struct v4l2_frmsizeenum frame_size;
 
        /*
         * To show this resource is used by
         * which stream, in ISP multiple stream mode
         */
        struct atomisp_sub_device *asd;
-
-       int sensor_index;
 };
 
 enum atomisp_dfs_mode {
diff --git a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c 
b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
index 900e4c79cd78..2cde1af77a2d 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
@@ -673,7 +673,7 @@ static int atomisp_s_input(struct file *file, void *fh, 
unsigned int input)
 
        /* select operating sensor */
        ret = v4l2_subdev_call(isp->inputs[input].camera, video, s_routing,
-                              0, isp->inputs[input].sensor_index, 0);
+                              0, 0, 0);
        if (ret && (ret != -ENOIOCTLCMD)) {
                dev_err(isp->dev, "Failed to select sensor\n");
                return ret;
diff --git a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c 
b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c
index b36dc98f897b..e78c738b3008 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c
@@ -866,13 +866,6 @@ static int atomisp_subdev_probe(struct atomisp_device *isp)
                        isp->inputs[isp->input_cnt].type = subdevs->type;
                        isp->inputs[isp->input_cnt].port = subdevs->port;
                        isp->inputs[isp->input_cnt].camera = subdevs->subdev;
-                       isp->inputs[isp->input_cnt].sensor_index = 0;
-                       /*
-                        * initialize the subdev frame size, then next we can
-                        * judge whether frame_size store effective value via
-                        * pixel_format.
-                        */
-                       isp->inputs[isp->input_cnt].frame_size.pixel_format = 0;
                        isp->input_cnt++;
                        break;
                case CAMERA_MOTOR:

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to