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

Subject: media: imx335: Parse fwnode properties
Author:  Kieran Bingham <kieran.bing...@ideasonboard.com>
Date:    Sun Apr 14 19:36:17 2024 +0530

Call the V4L2 fwnode device parser to handle controls that are
standardised by the framework.

Signed-off-by: Kieran Bingham <kieran.bing...@ideasonboard.com>
Signed-off-by: Umang Jain <umang.j...@ideasonboard.com>
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/i2c/imx335.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

---

diff --git a/drivers/media/i2c/imx335.c b/drivers/media/i2c/imx335.c
index fbf6bf662028..f6e42bef12a7 100644
--- a/drivers/media/i2c/imx335.c
+++ b/drivers/media/i2c/imx335.c
@@ -1227,10 +1227,16 @@ static int imx335_init_controls(struct imx335 *imx335)
 {
        struct v4l2_ctrl_handler *ctrl_hdlr = &imx335->ctrl_handler;
        const struct imx335_mode *mode = imx335->cur_mode;
+       struct v4l2_fwnode_device_properties props;
        u32 lpfr;
        int ret;
 
-       ret = v4l2_ctrl_handler_init(ctrl_hdlr, 7);
+       ret = v4l2_fwnode_device_parse(imx335->dev, &props);
+       if (ret)
+               return ret;
+
+       /* v4l2_fwnode_device_properties can add two more controls */
+       ret = v4l2_ctrl_handler_init(ctrl_hdlr, 9);
        if (ret)
                return ret;
 
@@ -1295,6 +1301,8 @@ static int imx335_init_controls(struct imx335 *imx335)
        if (imx335->hblank_ctrl)
                imx335->hblank_ctrl->flags |= V4L2_CTRL_FLAG_READ_ONLY;
 
+       v4l2_ctrl_new_fwnode_properties(ctrl_hdlr, &imx335_ctrl_ops, &props);
+
        if (ctrl_hdlr->error) {
                dev_err(imx335->dev, "control init failed: %d\n",
                        ctrl_hdlr->error);

Reply via email to