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

Subject: media: atmel-isi: Use accessors for pad config 'try_*' fields
Author:  Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Date:    Mon Oct 23 23:40:05 2023 +0200

The 'try_*' fields of the v4l2_subdev_pad_config structure are meant to
be accessed through helper functions. Replace direct access with usage
of the v4l2_subdev_get_pad_format(), v4l2_subdev_get_pad_crop() and
v4l2_subdev_get_pad_compose() helpers.

Signed-off-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@kernel.org>

 drivers/media/platform/atmel/atmel-isi.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

---

diff --git a/drivers/media/platform/atmel/atmel-isi.c 
b/drivers/media/platform/atmel/atmel-isi.c
index 4046212d48b4..bb83117bab2e 100644
--- a/drivers/media/platform/atmel/atmel-isi.c
+++ b/drivers/media/platform/atmel/atmel-isi.c
@@ -559,11 +559,13 @@ static const struct isi_format 
*find_format_by_fourcc(struct atmel_isi *isi,
 static void isi_try_fse(struct atmel_isi *isi, const struct isi_format 
*isi_fmt,
                        struct v4l2_subdev_state *sd_state)
 {
-       int ret;
+       struct v4l2_rect *try_crop =
+               v4l2_subdev_get_pad_crop(isi->entity.subdev, sd_state, 0);
        struct v4l2_subdev_frame_size_enum fse = {
                .code = isi_fmt->mbus_code,
                .which = V4L2_SUBDEV_FORMAT_TRY,
        };
+       int ret;
 
        ret = v4l2_subdev_call(isi->entity.subdev, pad, enum_frame_size,
                               sd_state, &fse);
@@ -572,11 +574,11 @@ static void isi_try_fse(struct atmel_isi *isi, const 
struct isi_format *isi_fmt,
         * just use the maximum ISI can receive.
         */
        if (ret) {
-               sd_state->pads->try_crop.width = MAX_SUPPORT_WIDTH;
-               sd_state->pads->try_crop.height = MAX_SUPPORT_HEIGHT;
+               try_crop->width = MAX_SUPPORT_WIDTH;
+               try_crop->height = MAX_SUPPORT_HEIGHT;
        } else {
-               sd_state->pads->try_crop.width = fse.max_width;
-               sd_state->pads->try_crop.height = fse.max_height;
+               try_crop->width = fse.max_width;
+               try_crop->height = fse.max_height;
        }
 }
 

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to