]> git.hungrycats.org Git - linux/commitdiff
media: cadence: csi2rx: configure DPHY before starting source stream
authorPratyush Yadav <p.yadav@ti.com>
Fri, 23 Feb 2024 08:23:01 +0000 (13:53 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 30 May 2024 07:49:41 +0000 (09:49 +0200)
[ Upstream commit fd64dda48f7e3f67ada1e1fe47e784ab350da72e ]

When the source device is operating above 1.5 Gbps per lane, it needs to
send the Skew Calibration Sequence before sending any HS data. If the
DPHY is initialized after the source stream is started, then it might
miss the sequence and not be able to receive data properly. Move the
start of source subdev to the end of the sequence to make sure
everything is ready to receive data before the source starts streaming.

Signed-off-by: Pratyush Yadav <p.yadav@ti.com>
Fixes: 3295cf1241d3 ("media: cadence: Add support for external dphy")
Tested-by: Julien Massot <julien.massot@collabora.com>
Tested-by: Changhuang Liang <Changhuang.liang@starfivetech.com>
Reviewed-by: Julien Massot <julien.massot@collabora.com>
Reviewed-by: Changhuang Liang <Changhuang.liang@starfivetech.com>
Signed-off-by: Jai Luthra <j-luthra@ti.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/media/platform/cadence/cdns-csi2rx.c

index 0ea5fa956fe9a3161688d96c71815e108de9d999..5be75bd4add39172944c97d269244eb758528a41 100644 (file)
@@ -235,10 +235,6 @@ static int csi2rx_start(struct csi2rx_priv *csi2rx)
 
        writel(reg, csi2rx->base + CSI2RX_STATIC_CFG_REG);
 
-       ret = v4l2_subdev_call(csi2rx->source_subdev, video, s_stream, true);
-       if (ret)
-               goto err_disable_pclk;
-
        /* Enable DPHY clk and data lanes. */
        if (csi2rx->dphy) {
                reg = CSI2RX_DPHY_CL_EN | CSI2RX_DPHY_CL_RST;
@@ -248,6 +244,13 @@ static int csi2rx_start(struct csi2rx_priv *csi2rx)
                }
 
                writel(reg, csi2rx->base + CSI2RX_DPHY_LANE_CTRL_REG);
+
+               ret = csi2rx_configure_ext_dphy(csi2rx);
+               if (ret) {
+                       dev_err(csi2rx->dev,
+                               "Failed to configure external DPHY: %d\n", ret);
+                       goto err_disable_pclk;
+               }
        }
 
        /*
@@ -287,14 +290,9 @@ static int csi2rx_start(struct csi2rx_priv *csi2rx)
 
        reset_control_deassert(csi2rx->sys_rst);
 
-       if (csi2rx->dphy) {
-               ret = csi2rx_configure_ext_dphy(csi2rx);
-               if (ret) {
-                       dev_err(csi2rx->dev,
-                               "Failed to configure external DPHY: %d\n", ret);
-                       goto err_disable_sysclk;
-               }
-       }
+       ret = v4l2_subdev_call(csi2rx->source_subdev, video, s_stream, true);
+       if (ret)
+               goto err_disable_sysclk;
 
        clk_disable_unprepare(csi2rx->p_clk);
 
@@ -308,6 +306,10 @@ err_disable_pixclk:
                clk_disable_unprepare(csi2rx->pixel_clk[i - 1]);
        }
 
+       if (csi2rx->dphy) {
+               writel(0, csi2rx->base + CSI2RX_DPHY_LANE_CTRL_REG);
+               phy_power_off(csi2rx->dphy);
+       }
 err_disable_pclk:
        clk_disable_unprepare(csi2rx->p_clk);