drm/amd/display: Handle HPD_IRQ for internal link

Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git

kernel os linux

[Why]
TCON data is corrupted after electro static discharge test.
Once the TCON data get corrupted, they will get themselves
reset and send HPD_IRQ to source side.

[How]
Support HPD_IRQ for internal link, and restore the PSR/Replay setup.

Reviewed-by: Robin Chen <robin.chen@amd.com>
Acked-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: Sung-huai Wang <danny.wang@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

authored by

Sung-huai Wang and committed by

Alex Deucher 2 years ago 239b31bd d0f2181d

+16 -9

1 changed file

expand all

drivers

gpu

drm

amd

display

link

protocols

link_dp_irq_handler.c

+16 -9

drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_irq_handler.c

··· 373 373 union device_service_irq device_service_clear = {0}; 374 374 enum dc_status result; 375 375 bool status = false; 376 + bool allow_active = false; 376 377 377 378 if (out_link_loss) 378 379 *out_link_loss = false; ··· 428 427 return false; 429 428 } 430 429 431 - if (handle_hpd_irq_psr_sink(link)) 432 - /* PSR-related error was detected and handled */ 433 - return true; 434 - 435 - handle_hpd_irq_replay_sink(link); 436 - 437 430 /* If PSR-related error handled, Main link may be off, 438 431 * so do not handle as a normal sink status change interrupt. 439 432 */ ··· 449 454 * If we got sink count changed it means 450 455 * Downstream port status changed, 451 456 * then DM should call DC to do the detection. 452 - * NOTE: Do not handle link loss on eDP since it is internal link*/ 453 - if ((link->connector_signal != SIGNAL_TYPE_EDP) && 454 - dp_parse_link_loss_status( 457 + */ 458 + if (dp_parse_link_loss_status( 455 459 link, 456 460 &hpd_irq_dpcd_data)) { 457 461 /* Connectivity log: link loss */ ··· 458 464 hpd_irq_dpcd_data.raw, 459 465 sizeof(hpd_irq_dpcd_data), 460 466 "Status: "); 467 + 468 + if (link->psr_settings.psr_feature_enabled) 469 + edp_set_psr_allow_active(link, &allow_active, true, false, NULL); 470 + else if (link->replay_settings.replay_allow_active) 471 + edp_set_replay_allow_active(link, &allow_active, true, false, NULL); 461 472 462 473 if (defer_handling && has_left_work) 463 474 *has_left_work = true; ··· 474 475 *out_link_loss = true; 475 476 476 477 dp_trace_link_loss_increment(link); 478 + } 479 + 480 + if (*out_link_loss == false) { 481 + if (handle_hpd_irq_psr_sink(link)) 482 + /* PSR-related error was detected and handled */ 483 + return true; 484 + 485 + handle_hpd_irq_replay_sink(link); 477 486 } 478 487 479 488 if (link->type == dc_connection_sst_branch &&

Configure Feed

Configure Feed