Re: ov2680: Fail to probe on imx7s-warp

[Fabio Estevam <festevam@xxxxxxxxx>]

On Thu, Mar 28, 2024 at 12:15 AM Fabio Estevam <festevam@xxxxxxxxx> wrote:

> I checked that bus_cfg.nr_of_link_frequencies == 0 which causes the
> dev_err_probe().
>
> Is there anything that needs to be adapted in
> arch/arm/boot/dts/nxp/imx/imx7s-warp.dts ?

I had to make the following changes to make ov2680 probe:

diff --git a/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
b/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
index ba7231b364bb..7bab113ca6da 100644
--- a/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx7s-warp.dts
@@ -210,6 +210,7 @@ ov2680_to_mipi: endpoint {
                                remote-endpoint = <&mipi_from_sensor>;
                                clock-lanes = <0>;
                                data-lanes = <1>;
+                               link-frequencies = /bits/ 64 <330000000>;
                        };
                };
        };
diff --git a/drivers/media/i2c/ov2680.c b/drivers/media/i2c/ov2680.c
index 39d321e2b7f9..cfa77a62b526 100644
--- a/drivers/media/i2c/ov2680.c
+++ b/drivers/media/i2c/ov2680.c
@@ -1125,7 +1125,7 @@ static int ov2680_parse_dt(struct ov2680_dev *sensor)

        for (i = 0; i < bus_cfg.nr_of_link_frequencies; i++)
                if (bus_cfg.link_frequencies[i] == sensor->link_freq[0])
-                       break;
+                       return 0;

        if (bus_cfg.nr_of_link_frequencies == 0 ||
            bus_cfg.nr_of_link_frequencies == i) {