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

Subject: media: ov5647: Implement enum_frame_size()
Author:  Jacopo Mondi <[email protected]>
Date:    Thu Nov 19 17:32:29 2020 +0100

Implement the .enum_frame_size subdev pad operation.

As the driver only supports one format and one resolution at the moment
the implementation is trivial.

Signed-off-by: Jacopo Mondi <[email protected]>
Signed-off-by: Sakari Ailus <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/i2c/ov5647.c | 25 ++++++++++++++++++++++---
 1 file changed, 22 insertions(+), 3 deletions(-)

---

diff --git a/drivers/media/i2c/ov5647.c b/drivers/media/i2c/ov5647.c
index ad3397881c9a..2e7a6cb39689 100644
--- a/drivers/media/i2c/ov5647.c
+++ b/drivers/media/i2c/ov5647.c
@@ -484,6 +484,24 @@ static int ov5647_enum_mbus_code(struct v4l2_subdev *sd,
        return 0;
 }
 
+static int ov5647_enum_frame_size(struct v4l2_subdev *sd,
+                                 struct v4l2_subdev_pad_config *cfg,
+                                 struct v4l2_subdev_frame_size_enum *fse)
+{
+       if (fse->index)
+               return -EINVAL;
+
+       if (fse->code != MEDIA_BUS_FMT_SBGGR8_1X8)
+               return -EINVAL;
+
+       fse->min_width = 640;
+       fse->max_width = 640;
+       fse->min_height = 480;
+       fse->max_height = 480;
+
+       return 0;
+}
+
 static int ov5647_set_get_fmt(struct v4l2_subdev *sd,
                              struct v4l2_subdev_pad_config *cfg,
                              struct v4l2_subdev_format *format)
@@ -502,9 +520,10 @@ static int ov5647_set_get_fmt(struct v4l2_subdev *sd,
 }
 
 static const struct v4l2_subdev_pad_ops ov5647_subdev_pad_ops = {
-       .enum_mbus_code = ov5647_enum_mbus_code,
-       .set_fmt =        ov5647_set_get_fmt,
-       .get_fmt =        ov5647_set_get_fmt,
+       .enum_mbus_code         = ov5647_enum_mbus_code,
+       .enum_frame_size        = ov5647_enum_frame_size,
+       .set_fmt                = ov5647_set_get_fmt,
+       .get_fmt                = ov5647_set_get_fmt,
 };
 
 static const struct v4l2_subdev_ops ov5647_subdev_ops = {

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

Reply via email to