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

Subject: media: ov2680: Wait for endpoint fwnode before continuing with probe()
Author:  Hans de Goede <hdego...@redhat.com>
Date:    Thu Aug 3 11:33:31 2023 +0200

Defer probe() until the endpoint fwnode is available. This is necessary
on ACPI platforms where the bridge code creating the fwnodes may also e.g.
set the "clock-frequency" device property and add GPIO mappings.

Acked-by: Rui Miguel Silva <rmf...@gmail.com>
Reviewed-by: Daniel Scally <dan.sca...@ideasonboard.com>
Signed-off-by: Hans de Goede <hdego...@redhat.com>
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@kernel.org>

 drivers/media/i2c/ov2680.c | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

---

diff --git a/drivers/media/i2c/ov2680.c b/drivers/media/i2c/ov2680.c
index a8c257f3bcd6..d4664581b49b 100644
--- a/drivers/media/i2c/ov2680.c
+++ b/drivers/media/i2c/ov2680.c
@@ -717,10 +717,22 @@ static int ov2680_check_id(struct ov2680_dev *sensor)
 static int ov2680_parse_dt(struct ov2680_dev *sensor)
 {
        struct device *dev = sensor->dev;
+       struct fwnode_handle *ep_fwnode;
        struct gpio_desc *gpio;
        unsigned int rate = 0;
        int i, ret;
 
+       /*
+        * Sometimes the fwnode graph is initialized by the bridge driver.
+        * Bridge drivers doing this may also add GPIO mappings, wait for this.
+        */
+       ep_fwnode = fwnode_graph_get_next_endpoint(dev_fwnode(dev), NULL);
+       if (!ep_fwnode)
+               return dev_err_probe(dev, -EPROBE_DEFER,
+                                    "waiting for fwnode graph endpoint\n");
+
+       fwnode_handle_put(ep_fwnode);
+
        /*
         * The pin we want is named XSHUTDN in the datasheet. Linux sensor
         * drivers have standardized on using "powerdown" as con-id name
@@ -801,7 +813,7 @@ static int ov2680_probe(struct i2c_client *client)
 
        ret = ov2680_parse_dt(sensor);
        if (ret < 0)
-               return -EINVAL;
+               return ret;
 
        ret = ov2680_mode_init(sensor);
        if (ret < 0)

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

Reply via email to