Skip to content

Commit 02d2723

Browse files
committed
drm/amdgpu/dp: add back special handling for NUTMEG
When I fixed the dp rate selection in: 3b73b168cffd9c392584d3f665021fa2190f8612 drm/amdgpu: fix dp link rate selection (v2) I accidently dropped the special handling for NUTMEG DP bridge chips. They require a fixed link rate. Reviewed-by: Christian König <christian.koenig@amd.com> Reviewed-by: Ken Wang <Qingqing.Wang@amd.com> Reviewed-by: Harry Wentland <harry.wentland@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Cc: stable@vger.kernel.org
1 parent c8213a6 commit 02d2723

File tree

1 file changed

+16
-4
lines changed

1 file changed

+16
-4
lines changed

drivers/gpu/drm/amd/amdgpu/atombios_dp.c

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -265,15 +265,27 @@ static int amdgpu_atombios_dp_get_dp_link_config(struct drm_connector *connector
265265
unsigned max_lane_num = drm_dp_max_lane_count(dpcd);
266266
unsigned lane_num, i, max_pix_clock;
267267

268-
for (lane_num = 1; lane_num <= max_lane_num; lane_num <<= 1) {
269-
for (i = 0; i < ARRAY_SIZE(link_rates) && link_rates[i] <= max_link_rate; i++) {
270-
max_pix_clock = (lane_num * link_rates[i] * 8) / bpp;
268+
if (amdgpu_connector_encoder_get_dp_bridge_encoder_id(connector) ==
269+
ENCODER_OBJECT_ID_NUTMEG) {
270+
for (lane_num = 1; lane_num <= max_lane_num; lane_num <<= 1) {
271+
max_pix_clock = (lane_num * 270000 * 8) / bpp;
271272
if (max_pix_clock >= pix_clock) {
272273
*dp_lanes = lane_num;
273-
*dp_rate = link_rates[i];
274+
*dp_rate = 270000;
274275
return 0;
275276
}
276277
}
278+
} else {
279+
for (lane_num = 1; lane_num <= max_lane_num; lane_num <<= 1) {
280+
for (i = 0; i < ARRAY_SIZE(link_rates) && link_rates[i] <= max_link_rate; i++) {
281+
max_pix_clock = (lane_num * link_rates[i] * 8) / bpp;
282+
if (max_pix_clock >= pix_clock) {
283+
*dp_lanes = lane_num;
284+
*dp_rate = link_rates[i];
285+
return 0;
286+
}
287+
}
288+
}
277289
}
278290

279291
return -EINVAL;

0 commit comments

Comments
 (0)