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

Subject: media: atomisp: Remove isp->need_gfx_throttle field
Author:  Hans de Goede <[email protected]>
Date:    Sun May 7 16:23:48 2023 +0100

Remove the isp->need_gfx_throttle field it is only ever set and
never read.

Also the code setting it is broken, comparing run_mode->val to
ATOMISP_SUBDEV_PAD_SOURCE_VIDEO which are not of the same type /
not part of the same enum.

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_cmd.c      | 10 ----------
 drivers/staging/media/atomisp/pci/atomisp_fops.c     |  1 -
 drivers/staging/media/atomisp/pci/atomisp_internal.h |  2 --
 3 files changed, 13 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_cmd.c 
b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
index bb49d6f2e67f..ed1a534b7c05 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_cmd.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
@@ -4693,16 +4693,6 @@ int atomisp_set_fmt(struct video_device *vdev, struct 
v4l2_format *f)
        f->fmt.pix = pipe->pix;
        f->fmt.pix.priv = PAGE_ALIGN(pipe->pix.width *
                                     pipe->pix.height * 2);
-
-       /*
-        * If in video 480P case, no GFX throttle
-        */
-       if (asd->run_mode->val == ATOMISP_SUBDEV_PAD_SOURCE_VIDEO &&
-           f->fmt.pix.width == 720 && f->fmt.pix.height == 480)
-               isp->need_gfx_throttle = false;
-       else
-               isp->need_gfx_throttle = true;
-
        /* Report the needed sizes */
        f->fmt.pix.sizeimage = pipe->pix.sizeimage;
        f->fmt.pix.bytesperline = pipe->pix.bytesperline;
diff --git a/drivers/staging/media/atomisp/pci/atomisp_fops.c 
b/drivers/staging/media/atomisp/pci/atomisp_fops.c
index fa362c8a37e8..a95c41d084c2 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_fops.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_fops.c
@@ -539,7 +539,6 @@ static void atomisp_dev_init_struct(struct atomisp_device 
*isp)
 {
        unsigned int i;
 
-       isp->need_gfx_throttle = true;
        isp->isp_fatal_error = false;
        isp->mipi_frame_size = 0;
 
diff --git a/drivers/staging/media/atomisp/pci/atomisp_internal.h 
b/drivers/staging/media/atomisp/pci/atomisp_internal.h
index 1fac99f4e2b0..e531f0c71a15 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_internal.h
+++ b/drivers/staging/media/atomisp/pci/atomisp_internal.h
@@ -211,8 +211,6 @@ struct atomisp_device {
 
        spinlock_t lock; /* Protects asd.streaming */
 
-       bool need_gfx_throttle;
-
        unsigned int mipi_frame_size;
        const struct atomisp_dfs_config *dfs;
        unsigned int hpll_freq;

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

Reply via email to