Re: [PATCH 08/43] drm/amd/display: FEC overhead should be checked once for mst slot nums

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 28. 03. 24, 20:50, Roman.Li@xxxxxxx wrote:
From: Hersen Wu <hersenxs.wu@xxxxxxx>

[Why] Mst slot nums equals to pbn / pbn_div.

Today, pbn_div refers to dm_mst_get_pbn_divider ->
dc_link_bandwidth_kbps. In dp_link_bandwidth_kbps,
which includes effect of FEC overhead already. As
result, we should not include effect of FEC overhead
again while calculating pbn by kpbs_to_peak_pbn
(stream_kbps).

[How] Include FEC overhead within dp_link_bandwidth_kbps.
Remove FEC overhead from kbps_to_peak_pbn.

Hi, this 6.10 commit (8b2cb32cf0c613) causes my secondary monitor not to come up at all.

Reverting this on top of 6.10 makes it work again.

Reported as (more details):
https://gitlab.freedesktop.org/drm/amd/-/issues/3495
https://bugzilla.suse.com/show_bug.cgi?id=1228093

Reviewed-by: Wayne Lin <wayne.lin@xxxxxxx>
Acked-by: Roman Li <roman.li@xxxxxxx>
Signed-off-by: Hersen Wu <hersenxs.wu@xxxxxxx>
Tested-by: Daniel Wheeler <daniel.wheeler@xxxxxxx>
---
  .../display/amdgpu_dm/amdgpu_dm_mst_types.c   | 37 +++++--------------
  .../display/amdgpu_dm/amdgpu_dm_mst_types.h   |  3 --
  2 files changed, 10 insertions(+), 30 deletions(-)

thanks,
--
js
suse labs




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux