[git:media_tree/master] media: atomisp: Simplify atomisp_s_input() input argument checking

[Mauro Carvalho Chehab]

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

Subject: media: atomisp: Simplify atomisp_s_input() input argument checking
Author:  Hans de Goede <hdego...@redhat.com>
Date:    Thu Apr 11 18:47:41 2024 +0100

Simplify the checking of the input argument to atomisp_s_input():
- Remove the check for input >= ATOM_ISP_MAX_INPUTS, input_cnt
  never is > ATOM_ISP_MAX_INPUTS so checking only that is sufficient
- Remove the unnecessary camera local variable
- Move the 2 invalid argument checks to above the atomisp_pipe_check()
  which checks if the pipe is busy or not

Reviewed-by: Andy Shevchenko <a...@kernel.org>
Signed-off-by: Hans de Goede <hdego...@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@kernel.org>

 drivers/staging/media/atomisp/pci/atomisp_ioctl.c | 18 ++++++------------
 1 file changed, 6 insertions(+), 12 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c 
b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
index b3ad53449cb8..1f8d44900a82 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
@@ -449,23 +449,17 @@ static int atomisp_s_input(struct file *file, void *fh, 
unsigned int input)
        struct video_device *vdev = video_devdata(file);
        struct atomisp_device *isp = video_get_drvdata(vdev);
        struct atomisp_video_pipe *pipe = atomisp_to_video_pipe(vdev);
-       struct v4l2_subdev *camera = NULL;
        int ret;
 
-       ret = atomisp_pipe_check(pipe, true);
-       if (ret)
-               return ret;
-
-       if (input >= ATOM_ISP_MAX_INPUTS || input >= isp->input_cnt) {
-               dev_dbg(isp->dev, "input_cnt: %d\n", isp->input_cnt);
+       if (input >= isp->input_cnt)
                return -EINVAL;
-       }
 
-       camera = isp->inputs[input].camera;
-       if (!camera) {
-               dev_err(isp->dev, "%s, no camera\n", __func__);
+       if (!isp->inputs[input].camera)
                return -EINVAL;
-       }
+
+       ret = atomisp_pipe_check(pipe, true);
+       if (ret)
+               return ret;
 
        return atomisp_select_input(isp, input);
 }