public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: tessolveupstream@gmail.com
To: Luca Ceresoli <luca.ceresoli@bootlin.com>,
	andrzej.hajda@intel.com, neil.armstrong@linaro.org,
	rfoss@kernel.org
Cc: Laurent.pinchart@ideasonboard.com, jonas@kwiboo.se,
	jernej.skrabec@gmail.com, maarten.lankhorst@linux.intel.com,
	mripard@kernel.org, tzimmermann@suse.de, airlied@gmail.com,
	simona@ffwll.ch, marex@denx.de, valentin@compulab.co.il,
	philippe.schenker@toradex.com, alexander.stein@ew.tq-group.com,
	dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 1/1] drm: bridge: ti-sn65dsi83: Fix DSI mode flags for stable LVDS output
Date: Thu, 23 Apr 2026 14:46:26 +0530	[thread overview]
Message-ID: <60a24977-b181-40e4-bcf6-38b65af293e2@gmail.com> (raw)
In-Reply-To: <7056b23b-ed81-4d79-b782-5cfcb0102ef7@gmail.com>



On 21-04-2026 11:31, tessolveupstream@gmail.com wrote:
> 
> 
> On 16-04-2026 14:40, tessolveupstream@gmail.com wrote:
>>
>>
>> On 15-04-2026 12:43, Luca Ceresoli wrote:
>>> Hello Sudarshan,
>>>
>>> On Sun Apr 12, 2026 at 7:38 AM CEST, Sudarshan Shetty wrote:
>>>> The current DSI mode configuration enables VIDEO_BURST and disables
>>>> horizontal front porch (HFP) and back porch (HBP) transmission using
>>>> MIPI_DSI_MODE_VIDEO_NO_HFP and MIPI_DSI_MODE_VIDEO_NO_HBP.
>>>>
>>>> However, the SN65DSI83/84 bridge relies on receiving full horizontal
>>>> timing information over DSI in order to correctly reconstruct the LVDS
>>>> output timings. When HFP and HBP are not transmitted, the bridge cannot
>>>> recreate the required timing parameters, resulting in unstable or
>>>> missing display output on some panels.
>>>>
>>>> Additionally, while burst mode is supported by the hardware, its use
>>>> depends on continuous clock behavior from the DSI host. In practice,
>>>> burst mode may introduce instability depending on the host controller
>>>> implementation, as the DSI link may transition to low-power state
>>>> between bursts.
>>>>
>>>> In testing, removing burst mode and ensuring full horizontal timing
>>>> transmission results in stable LVDS output across affected panels.
>>>>
>>>> Update the DSI mode flags to:
>>>>   - Drop MIPI_DSI_MODE_VIDEO_BURST
>>>>   - Drop MIPI_DSI_MODE_VIDEO_NO_HFP
>>>>   - Drop MIPI_DSI_MODE_VIDEO_NO_HBP
>>>>
>>>> This aligns with common system configurations where non-burst mode is
>>>> preferred and full timing information is transmitted over DSI.
>>>>
>>>> Signed-off-by: Sudarshan Shetty <tessolveupstream@gmail.com>
>>>
>>> Thanks for having gone into the details and provide a good explanation for
>>> the removal of these three flags.
>>>
>>> It would be still great to have authoritative info from TI, possibly as an
>>> update to the documentation.
>>>
>>> I managed to test on a board with i.MX93, a TI SN65DSI84 and a 1920x1080p60
>>> panel, and:
>>>
>>>  - no image shown with the current mainline driver
>>>  - it works by removing only MIPI_DSI_MODE_VIDEO_BURST (my current setup)
>>>  - it works with this patch applied
>>>
>>> Tested-by: Luca Ceresoli <luca.ceresoli@bootlin.com> # imx93 1920x1080p60
>>>
>>
>> Thanks for testing the patch and for the Tested-by tag. 
>> I’ll reach out to TI for clarification on the DSI mode flag behavior and
>> any documentation updates, and will share any feedback I receive, though I
>> cannot guarantee a response.
> 
> I had reached out to TI for clarification and any related documentation 
> updates, but I have not received any response so far.Given this, it is 
> uncertain whether we will be able to obtain further details or official 
> confirmation from TI in the near term.
> 
> I would appreciate your guidance on how you would prefer us to proceed 
> from here.

I followed up with TI, and they pointed us to the relevant sections in the 
SN65DSI83/84/86 datasheets covering DSI video transmission specifications.

https://www.ti.com/lit/ds/symlink/sn65dsi84.pdf?ts=1776924088430&ref_url=https%253A%252F%252Fwww.ti.com%252Fproduct%252FSN65DSI84

As per datasheet Section 7.4.7, the device operates in DSI video mode with 
fixed horizontal timing, where HBP, and HFP are explicitly defined and expected 
to be present as part of the video line structure. The timing description in 
this section assumes standard non‑burst video transmission with all horizontal 
blanking intervals accounted for.
 
The datasheet does not document support for DSI video burst mode, nor does it 
mention support for omitting horizontal front or back porch intervals. In 
particular, there is no indication that the device can operate with collapsed 
or suppressed horizontal blanking, as enabled by MIPI_DSI_MODE_VIDEO_BURST, 
MIPI_DSI_MODE_VIDEO_NO_HFP, or MIPI_DSI_MODE_VIDEO_NO_HBP.

Enabling these modes results in a mismatch between the host‑generated DSI timing 
and the device’s expected fixed timing model described in the datasheet, which 
leads to incorrect display behavior. This is consistent with our observations: 
the display fails when these modes are enabled, and operates correctly when using 
standard non‑burst video mode with explicit HBP/HFP timing.
 
Therefore, removing these flags aligns the driver configuration with the 
datasheet‑defined timing requirements rather than acting as a workaround.
 
They also shared the following E2E FAQ for reference:
https://e2e.ti.com/support/interface-group/interface/f/interface-forum/1382976/faq-sn65dsi8x-programming-tools?tisearch=e2e-sitesearch&keymatch=sn65dsi86

However, they did not provide any additional clarification beyond what is already 
documented and indicated that there are currently no plans to further support or 
update the Linux driver.
 
Given this, it seems unlikely that we will receive more detailed or authoritative 
guidance from TI on the specific mode flag usage.
 
Please let me know how you would prefer us to proceed from here.
>>> Luca
>>>
>>> --
>>> Luca Ceresoli, Bootlin
>>> Embedded Linux and Kernel engineering
>>> https://bootlin.com
>>
> 


  reply	other threads:[~2026-04-23  9:16 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-12  5:38 [PATCH v3 0/1] drm: bridge: ti-sn65dsi83: Fix DSI mode flags for stable LVDS output Sudarshan Shetty
2026-04-12  5:38 ` [PATCH v3 1/1] " Sudarshan Shetty
2026-04-15  7:13   ` Luca Ceresoli
2026-04-15  8:07     ` Luca Ceresoli
2026-04-16  9:10     ` tessolveupstream
2026-04-21  6:01       ` tessolveupstream
2026-04-23  9:16         ` tessolveupstream [this message]
2026-04-24  8:25           ` Luca Ceresoli
2026-04-15 13:51   ` Alexander Stein

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=60a24977-b181-40e4-bcf6-38b65af293e2@gmail.com \
    --to=tessolveupstream@gmail.com \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=airlied@gmail.com \
    --cc=alexander.stein@ew.tq-group.com \
    --cc=andrzej.hajda@intel.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jernej.skrabec@gmail.com \
    --cc=jonas@kwiboo.se \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luca.ceresoli@bootlin.com \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=marex@denx.de \
    --cc=mripard@kernel.org \
    --cc=neil.armstrong@linaro.org \
    --cc=philippe.schenker@toradex.com \
    --cc=rfoss@kernel.org \
    --cc=simona@ffwll.ch \
    --cc=tzimmermann@suse.de \
    --cc=valentin@compulab.co.il \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox