Re: [PATCH] drm/stm: dsi: compute the transition time from LP to HS and back

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 7/13/21 6:47 PM, Philippe CORNU wrote:
Hi Antonio,

On 7/13/21 4:49 PM, Antonio Borneo wrote:
The driver uses a conservative set of hardcoded values for the
maximum time delay of the transitions between LP and HS, either
for data and clock lanes.

By using the info in STM32MP157 datasheet, valid also for other ST
devices, compute the actual delay from the lane's bps.

Signed-off-by: Antonio Borneo <antonio.borneo@xxxxxxxxxxx>
---
To: Yannick Fertre <yannick.fertre@xxxxxxxxxxx>
To: Philippe Cornu <philippe.cornu@xxxxxxxxxxx>
To: Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxx>
To: David Airlie <airlied@xxxxxxxx>
To: Daniel Vetter <daniel@xxxxxxxx>
To: Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx>
To: Alexandre Torgue <alexandre.torgue@xxxxxxxxxxx>
To: Raphael Gallais-Pou <raphael.gallais-pou@xxxxxxxxxxx>
To: dri-devel@xxxxxxxxxxxxxxxxxxxxx
To: linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx
To: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
Cc: linux-kernel@xxxxxxxxxxxxxxx

  drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 17 +++++++++++++----
  1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c b/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c
index 8399d337589d..32cb41b2202f 100644
--- a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c
+++ b/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c
@@ -309,14 +309,23 @@ dw_mipi_dsi_get_lane_mbps(void *priv_data, const struct drm_display_mode *mode,
      return 0;
  }
+#define DSI_PHY_DELAY(fp, vp, mbps) DIV_ROUND_UP((fp) * (mbps) + 1000 * (vp), 8000)
+
  static int
  dw_mipi_dsi_phy_get_timing(void *priv_data, unsigned int lane_mbps,
                 struct dw_mipi_dsi_dphy_timing *timing)
  {
-    timing->clk_hs2lp = 0x40;
-    timing->clk_lp2hs = 0x40;
-    timing->data_hs2lp = 0x40;
-    timing->data_lp2hs = 0x40;
+    /*
+     * From STM32MP157 datasheet, valid for STM32F469, STM32F7x9, STM32H747
+     * phy_clkhs2lp_time = (272+136*UI)/(8*UI)
+     * phy_clklp2hs_time = (512+40*UI)/(8*UI)
+     * phy_hs2lp_time = (192+64*UI)/(8*UI)
+     * phy_lp2hs_time = (256+32*UI)/(8*UI)
+     */
+    timing->clk_hs2lp = DSI_PHY_DELAY(272, 136, lane_mbps);
+    timing->clk_lp2hs = DSI_PHY_DELAY(512, 40, lane_mbps);
+    timing->data_hs2lp = DSI_PHY_DELAY(192, 64, lane_mbps);
+    timing->data_lp2hs = DSI_PHY_DELAY(256, 32, lane_mbps);

Many thanks for your patch.

Reviewed-by: Philippe Cornu <philippe.cornu@xxxxxxxxxxx>
Acked-by: Philippe Cornu <philippe.cornu@xxxxxxxxxxx>

I will apply it on drm-misc-next early next week,

Philippe :-)

      return 0;
  }

base-commit: 35d283658a6196b2057be562096610c6793e1219


Applied on drm-misc-next.
Many thanks for your patch,
Philippe :-)



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux