All of lore.kernel.org
 help / color / mirror / Atom feed
* drm/msm/dsi Lane-swapping for 2-lane panels non-functional for msm8x74?
@ 2015-08-21  0:54 Johansson, Werner
  2015-08-21 20:14 ` Hai Li
  0 siblings, 1 reply; 4+ messages in thread
From: Johansson, Werner @ 2015-08-21  0:54 UTC (permalink / raw)
  To: dri-devel@lists.freedesktop.org

Hi,

I'm having issues with non-functional DSI output driving a 2-lane panel connected to DSI0 on MSM8x74 MDP5 v1.2 hardware. The code in drivers/gpu/drm/msm/dsi/dsi_host.c around line 703 enables lane 1 and 2 instead of lane 0 and 1 for performance reasons (and then enables lane swapping in the DSI PHY (LANE_SWAP_1230)). I cannot get this to work on the MSM8x74, no data observed on physical lane 0 (can't access lane1 easily unfortunately). Disabling this swap and enabling lane0 and 1 instead results in a fully functional panel.

Would it make sense to make this performance optimization a compile-time option that defaults to off (for certain problematic hardware or in general)?

/wj
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* drm/msm/dsi Lane-swapping for 2-lane panels non-functional for msm8x74?
@ 2015-08-21 18:33 Johansson, Werner
  0 siblings, 0 replies; 4+ messages in thread
From: Johansson, Werner @ 2015-08-21 18:33 UTC (permalink / raw)
  To: dri-devel@lists.freedesktop.org

Hi,

I'm having issues with non-functional DSI output driving a 2-lane panel connected to DSI0 on MSM8x74 MDP5 v1.2 hardware. The code in drivers/gpu/drm/msm/dsi/dsi_host.c around line 703 enables lane 1 and 2 instead of lane 0 and 1 for performance reasons (and then enables lane swapping in the DSI PHY (LANE_SWAP_1230)). I cannot get this to work on the MSM8x74, no data observed on physical lane 0 (can't access lane1 easily unfortunately). Disabling this swap and enabling lane0 and 1 instead results in a fully functional panel.

Would it make sense to make this performance optimization a compile-time option that defaults to off (for certain problematic hardware or in general)?

/wj

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: drm/msm/dsi Lane-swapping for 2-lane panels non-functional for msm8x74?
  2015-08-21  0:54 drm/msm/dsi Lane-swapping for 2-lane panels non-functional for msm8x74? Johansson, Werner
@ 2015-08-21 20:14 ` Hai Li
  2015-08-21 20:23   ` Johansson, Werner
  0 siblings, 1 reply; 4+ messages in thread
From: Hai Li @ 2015-08-21 20:14 UTC (permalink / raw)
  To: 'Johansson, Werner', dri-devel

Hi Werner,

I will prepare a change to make the lane swap configurable.

Thanks,
Hai

-----Original Message-----
From: dri-devel [mailto:dri-devel-bounces@lists.freedesktop.org] On Behalf Of Johansson, Werner
Sent: Thursday, August 20, 2015 8:54 PM
To: dri-devel@lists.freedesktop.org
Subject: drm/msm/dsi Lane-swapping for 2-lane panels non-functional for msm8x74?

Hi,

I'm having issues with non-functional DSI output driving a 2-lane panel connected to DSI0 on MSM8x74 MDP5 v1.2 hardware. The code in drivers/gpu/drm/msm/dsi/dsi_host.c around line 703 enables lane 1 and 2 instead of lane 0 and 1 for performance reasons (and then enables lane swapping in the DSI PHY (LANE_SWAP_1230)). I cannot get this to work on the MSM8x74, no data observed on physical lane 0 (can't access lane1 easily unfortunately). Disabling this swap and enabling lane0 and 1 instead results in a fully functional panel.

Would it make sense to make this performance optimization a compile-time option that defaults to off (for certain problematic hardware or in general)?

/wj
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: drm/msm/dsi Lane-swapping for 2-lane panels non-functional for msm8x74?
  2015-08-21 20:14 ` Hai Li
@ 2015-08-21 20:23   ` Johansson, Werner
  0 siblings, 0 replies; 4+ messages in thread
From: Johansson, Werner @ 2015-08-21 20:23 UTC (permalink / raw)
  To: Hai Li, dri-devel@lists.freedesktop.org

> From: Hai Li [mailto:hali@codeaurora.org]
> Sent: den 21 augusti 2015 13:14
> Hi Werner,
> 
> I will prepare a change to make the lane swap configurable.
> 
> Thanks,
> Hai

That sounds really great Hai!

Any idea as to why this lane swap would misbehave? In theory the swap should be completely transparent to the rest of the code (and panel). This behavior is seen on a Dragonboard 800 (the only device with a display with low enough resolution to fit in two lanes).

Thanks!
/wj


_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-08-21 20:23 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-21  0:54 drm/msm/dsi Lane-swapping for 2-lane panels non-functional for msm8x74? Johansson, Werner
2015-08-21 20:14 ` Hai Li
2015-08-21 20:23   ` Johansson, Werner
  -- strict thread matches above, loose matches on Subject: below --
2015-08-21 18:33 Johansson, Werner

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.