I plan to debug MST with this patches next week but I guess at least the 4 first patches can be merged. The 5th too if someone else reviews it, the selection of the master transcoder was tested by me and Lucas and the problem in not in any of this patches. On Tue, 2019-10-29 at 18:24 -0700, Lucas De Marchi wrote: > Additional code to support more than one display when using MST with > TGL. It's still WIP! From what I could check on my tests we are > correctly > tracking the master transcoder and setting it accordingly on > TRANS_DDI_FUNC_CTL and DP_TP_CTL. > > I tried also setting MST mode on the slave's DP_TP_CTL. I could get > the > second display to show up some times and we don't get stuck on > "timeout > waiting for ACT" error message. This is not according to the spec > though, and it still doesn't work most of the time. So... I didn't > add > it here. > > I guess now I need another pair of eyes to check what I'm doing wrong > since continuing to stare at the spec and code isn't helping. José, > Imre? > > José Roberto de Souza (2): > drm/i915: Add for_each_new_intel_connector_in_state() > drm/i915/tgl: Select master transcoder in DP MST > > Lucas De Marchi (3): > drm/i915: add wrappers to get intel connector state > drm/i915/tgl: do not enable transcoder clock twice on MST > drm/i915: avoid reading DP_TP_CTL twice > > drivers/gpu/drm/i915/display/intel_ddi.c | 54 +++--- > drivers/gpu/drm/i915/display/intel_display.c | 16 ++ > drivers/gpu/drm/i915/display/intel_display.h | 8 + > .../drm/i915/display/intel_display_types.h | 21 +++ > drivers/gpu/drm/i915/display/intel_dp_mst.c | 159 > +++++++++++++++++- > drivers/gpu/drm/i915/display/intel_dp_mst.h | 2 + > 6 files changed, 236 insertions(+), 24 deletions(-) > _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
