[PATCH 0/7] Fixes for lt9211c

Philipp Zabel posted 7 patches 1 month, 3 weeks ago
drivers/gpu/drm/bridge/lontium-lt9211.c | 189 ++++++++----------------
1 file changed, 61 insertions(+), 128 deletions(-)
[PATCH 0/7] Fixes for lt9211c
Posted by Philipp Zabel 1 month, 3 weeks ago
With these fixes

Tested-by: Philipp Zabel <p.zabel@pengutronix.de>

on RK3576 with a 1080p60 panel @ 148.5 MHz.

Feel free to squash any of these into v7.

regards
Philipp

Oleksij Rempel (1):
  drm/bridge: lt9211: Enable DSI burst mode for LT9211C

Philipp Zabel (6):
  drm/bridge: lt9211: drop delayed work
  drm/bridge: lt9211: fix regmap_write error handling
  drm/bridge: lt9211: use regmap_write_bits
  drm/bridge: lt9211: reuse lt9211_configure_timing in
    lt9211c_configure_timing
  drm/bridge: lt9211: combine equal branches
  drm/bridge: lt9211: use devm_drm_bridge_add and drop lt9211_remove

 drivers/gpu/drm/bridge/lontium-lt9211.c | 189 ++++++++----------------
 1 file changed, 61 insertions(+), 128 deletions(-)

-- 
2.47.3
Re: [PATCH 0/7] Fixes for lt9211c
Posted by Vishnu Saini 1 month, 2 weeks ago
On Thu, Aug 06, 2026 at 03:17:08PM +0200, Philipp Zabel wrote:
> With these fixes
> 
> Tested-by: Philipp Zabel <p.zabel@pengutronix.de>
> 
> on RK3576 with a 1080p60 panel @ 148.5 MHz.
> 
> Feel free to squash any of these into v7.
Thank you for the fixes will use most of them in next revison.
> regards
> Philipp
> 
> Oleksij Rempel (1):
>   drm/bridge: lt9211: Enable DSI burst mode for LT9211C
This fix will be included in the next revision. 
> Philipp Zabel (6):
>   drm/bridge: lt9211: drop delayed work
This patch is not working autodetect failed to read format/width/height.
1. Tried ading more delay upto 5s before, autodetect did not help.
2. Tried adding a dummy drm_hook hw_timing_enabled, which is called after enabling the timing engine
with this synchronous approach is working.
In the next revision i will be keeping delayed work apprach with some improvements, by reducing the delay to minimum and wait in autect_rx for correct nonzero format/width/height. Let me know if this is ok, or if any alternative approach i should follow?

>   drm/bridge: lt9211: fix regmap_write error handling
This fix will be included in the next revision.
>   drm/bridge: lt9211: use regmap_write_bits
This fix will be included in the next revision.
>   drm/bridge: lt9211: reuse lt9211_configure_timing in
>     lt9211c_configure_timing
This fix will be included in the next revision.
>   drm/bridge: lt9211: combine equal branches
This fix will be included in the next revision.
>   drm/bridge: lt9211: use devm_drm_bridge_add and drop lt9211_remove
This fix will be included in the next revision.
>  drivers/gpu/drm/bridge/lontium-lt9211.c | 189 ++++++++----------------
>  1 file changed, 61 insertions(+), 128 deletions(-)
> 
> -- 
> 2.47.3
>
Re: [PATCH 0/7] Fixes for lt9211c
Posted by Vishnu Saini 1 month, 3 weeks ago
On Thu, Aug 06, 2026 at 03:17:08PM +0200, Philipp Zabel wrote:
> With these fixes
> 
> Tested-by: Philipp Zabel <p.zabel@pengutronix.de>
> 
> on RK3576 with a 1080p60 panel @ 148.5 MHz.
> 
> Feel free to squash any of these into v7.
Thanks Philipp for the review and fixes on V6, will be addressing them by EOW.

> regards
> Philipp
> 
> Oleksij Rempel (1):
>   drm/bridge: lt9211: Enable DSI burst mode for LT9211C
> 
> Philipp Zabel (6):
>   drm/bridge: lt9211: drop delayed work
currently droping delayed work is not working as expected.
lt9211_autodetect_rx failed to read the format, Tried increasing the delay upto 3s
before reading the format buf, did not help:
lt9211 2-002d: Unsupported DSI format 0x0

I am further looking into this, let me know if you have any suggestions.

>   drm/bridge: lt9211: fix regmap_write error handling
>   drm/bridge: lt9211: use regmap_write_bits
>   drm/bridge: lt9211: reuse lt9211_configure_timing in
>     lt9211c_configure_timing
>   drm/bridge: lt9211: combine equal branches
>   drm/bridge: lt9211: use devm_drm_bridge_add and drop lt9211_remove
> 
>  drivers/gpu/drm/bridge/lontium-lt9211.c | 189 ++++++++----------------
>  1 file changed, 61 insertions(+), 128 deletions(-)
> 
> -- 
> 2.47.3
>