openwrt/target/linux/bcm27xx/patches-6.6/950-1050-DRM-rp1-rp1-dsi-Fix-escape-clock-divider-and-timeout.patch
Álvaro Fernández Rojas 8c405cdccc bcm27xx: add 6.6 kernel patches
The patches were generated from the RPi repo with the following command:
git format-patch v6.6.34..rpi-6.1.y

Some patches needed rebasing and, as usual, the applied and reverted, wireless
drivers, Github workflows, READMEs and defconfigs patches were removed.

Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
2024-06-18 18:52:49 +02:00

108 lines
4.1 KiB
Diff

From 85cc26d5496f073fc7e5dc33f8c9fd5c7aea93c6 Mon Sep 17 00:00:00 2001
From: Nick Hollinghurst <nick.hollinghurst@raspberrypi.com>
Date: Mon, 22 Apr 2024 13:06:21 +0100
Subject: [PATCH 1050/1085] DRM: rp1: rp1-dsi: Fix escape clock divider and
timeouts.
Escape clock divider was fixed at 5, which is correct at 800Mbps/lane
but increasingly out of spec for higher rates. Compute it correctly.
High speed timeout was fixed at 5*512 == 2560 byte-clocks per lane.
Compute it conservatively to be 8/7 times the line period (assuming
there will be a transition to LP some time during each scanline?)
keeping the old value as a lower bound. Increase LPRX TO to 1024,
and BTA TO to 0xb00 (same value as in bridge/synopsys/dw-mipi-dsi).
(No change to LP_CMD_TIM. To do: compute this correctly.)
Signed-off-by: Nick Hollinghurst <nick.hollinghurst@raspberrypi.com>
---
drivers/gpu/drm/rp1/rp1-dsi/rp1_dsi_dsi.c | 39 ++++++++++++++---------
1 file changed, 24 insertions(+), 15 deletions(-)
--- a/drivers/gpu/drm/rp1/rp1-dsi/rp1_dsi_dsi.c
+++ b/drivers/gpu/drm/rp1/rp1-dsi/rp1_dsi_dsi.c
@@ -1336,10 +1336,17 @@ static u32 get_colorcode(enum mipi_dsi_p
return 0x005;
}
+/* Maximum frequency for LP escape clock (20MHz), and some magic numbers */
+#define RP1DSI_ESC_CLK_KHZ 20000
+#define RP1DSI_TO_CLK_DIV 5
+#define RP1DSI_HSTX_TO_MIN 0x200
+#define RP1DSI_LPRX_TO_VAL 0x400
+#define RP1DSI_BTA_TO_VAL 0xd00
+
void rp1dsi_dsi_setup(struct rp1_dsi *dsi, struct drm_display_mode const *mode)
{
u32 timeout, mask, vid_mode_cfg;
- u32 freq_khz;
+ int lane_kbps;
unsigned int bpp = mipi_dsi_pixel_format_to_bpp(dsi->display_format);
DSI_WRITE(DSI_PHY_IF_CFG, dsi->lanes - 1);
@@ -1349,28 +1356,33 @@ void rp1dsi_dsi_setup(struct rp1_dsi *ds
/* a conservative guess (LP escape is slow!) */
DSI_WRITE(DSI_DPI_LP_CMD_TIM, 0x00100000);
- /* Drop to LP where possible */
+ /* Drop to LP where possible; use LP Escape for all commands */
vid_mode_cfg = 0xbf00;
if (!(dsi->display_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE))
vid_mode_cfg |= 0x01;
if (dsi->display_flags & MIPI_DSI_MODE_VIDEO_BURST)
vid_mode_cfg |= 0x02;
DSI_WRITE(DSI_VID_MODE_CFG, vid_mode_cfg);
-
- /* Use LP Escape Data signalling for all commands */
DSI_WRITE(DSI_CMD_MODE_CFG, 0x10F7F00);
+
/* Select Command Mode */
DSI_WRITE(DSI_MODE_CFG, 1);
- /* XXX magic number */
- DSI_WRITE(DSI_TO_CNT_CFG, 0x02000200);
- /* XXX magic number */
- DSI_WRITE(DSI_BTA_TO_CNT, 0x800);
+ /* Set timeouts and clock dividers */
+ DSI_WRITE(DSI_TO_CNT_CFG,
+ (max((bpp * mode->htotal) / (7 * RP1DSI_TO_CLK_DIV * dsi->lanes),
+ RP1DSI_HSTX_TO_MIN) << 16) |
+ RP1DSI_LPRX_TO_VAL);
+ DSI_WRITE(DSI_BTA_TO_CNT, RP1DSI_BTA_TO_VAL);
+ lane_kbps = (bpp * mode->clock) / dsi->lanes;
+ DSI_WRITE(DSI_CLKMGR_CFG,
+ (RP1DSI_TO_CLK_DIV << 8) |
+ max(2, lane_kbps / (8 * RP1DSI_ESC_CLK_KHZ) + 1));
+
+ /* Configure video timings */
DSI_WRITE(DSI_VID_PKT_SIZE, mode->hdisplay);
DSI_WRITE(DSI_VID_NUM_CHUNKS, 0);
DSI_WRITE(DSI_VID_NULL_SIZE, 0);
-
- /* Note, unlike Argon firmware, here we DON'T consider sync to be concurrent with porch */
DSI_WRITE(DSI_VID_HSA_TIME,
(bpp * (mode->hsync_end - mode->hsync_start)) / (8 * dsi->lanes));
DSI_WRITE(DSI_VID_HBP_TIME,
@@ -1381,9 +1393,8 @@ void rp1dsi_dsi_setup(struct rp1_dsi *ds
DSI_WRITE(DSI_VID_VFP_LINES, (mode->vsync_start - mode->vdisplay));
DSI_WRITE(DSI_VID_VACTIVE_LINES, mode->vdisplay);
- freq_khz = (bpp * mode->clock) / dsi->lanes;
-
- dphy_init_khz(dsi, rp1dsi_refclk_freq(dsi) / 1000, freq_khz);
+ /* Init PHY */
+ dphy_init_khz(dsi, rp1dsi_refclk_freq(dsi) / 1000, lane_kbps);
DSI_WRITE(DSI_PHY_TMR_LPCLK_CFG,
(hsfreq_table[dsi->hsfreq_index].clk_lp2hs << DSI_PHY_TMR_LP2HS_LSB) |
@@ -1392,8 +1403,6 @@ void rp1dsi_dsi_setup(struct rp1_dsi *ds
(hsfreq_table[dsi->hsfreq_index].data_lp2hs << DSI_PHY_TMR_LP2HS_LSB) |
(hsfreq_table[dsi->hsfreq_index].data_hs2lp << DSI_PHY_TMR_HS2LP_LSB));
- DSI_WRITE(DSI_CLKMGR_CFG, 0x00000505);
-
/* Wait for PLL lock */
for (timeout = (1 << 14); timeout != 0; --timeout) {
usleep_range(10, 50);