This series makes some small improvements to the tc358768 driver, and then adds support for long commands, commands that have more than 8 bytes of payload.
This has been tested on Toradex Verdin AM62 board, with a ST7703 based DSI panel which requires initial configuration using commands that have 8+ bytes. I tested the following combinations, by observing the panel visually and the DSI lanes with an oscilloscope: - HS command transfer, continuous clock - HS command transfer, non-continuous clock - LP command transfer, continuous clock - LP command transfer, non-continuous clock All except LP + continuous clock work fine. While observing the lanes with normal oscilloscope is a very high level and vague view of what exactly is going on the lanes, it still looks good to me: initial commands are sent in LP, and then video data is being sent in HS, and clock lane is continuous. However, the panel stays black, so something is not quite right. As it doesn't make sense to send commands in LP during initial configuration, especially with longer commands, I will leave that particular combination unresolved. I'd be interested to hear if it works on some other panel. Signed-off-by: Tomi Valkeinen <[email protected]> --- Parth Pancholi (1): drm/bridge: tc358768: Set pre_enable_prev_first for reverse order Tomi Valkeinen (6): drm/bridge: tc358768: Fix typo in TC358768_DSI_CONTROL_DIS_MODE drm/bridge: tc358768: Separate indirect register writes drm/bridge: tc358768: Support non-continuous clock drm/bridge: tc358768: Add LP mode command support drm/bridge: tc358768: Separate video format config drm/bridge: tc358768: Add support for long command tx via video buffer drivers/gpu/drm/bridge/tc358768.c | 202 +++++++++++++++++++++++++++++--------- 1 file changed, 157 insertions(+), 45 deletions(-) --- base-commit: e5f0a698b34ed76002dc5cff3804a61c80233a7a change-id: 20251021-tc358768-93eb23689b7a Best regards, -- Tomi Valkeinen <[email protected]>
