Skip to content

Commit 219be9d

Browse files
Clark Zhengalexdeucher
authored andcommitted
drm/amd/display: Refine disable VGA
bad case won't follow normal sense, it will not enable vga1 as usual, but vga2,3,4 is on. Signed-off-by: Clark Zheng <[email protected]> Reviewed-by: Tony Cheng <[email protected]> Acked-by: Alex Deucher <[email protected]> Signed-off-by: Alex Deucher <[email protected]>
1 parent d1bb88e commit 219be9d

File tree

2 files changed

+22
-6
lines changed

2 files changed

+22
-6
lines changed

drivers/gpu/drm/amd/display/dc/dce/dce_hwseq.h

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -496,6 +496,9 @@ struct dce_hwseq_registers {
496496
HWS_SF(, DOMAIN7_PG_STATUS, DOMAIN7_PGFSM_PWR_STATUS, mask_sh), \
497497
HWS_SF(, DC_IP_REQUEST_CNTL, IP_REQUEST_EN, mask_sh), \
498498
HWS_SF(, D1VGA_CONTROL, D1VGA_MODE_ENABLE, mask_sh),\
499+
HWS_SF(, D2VGA_CONTROL, D2VGA_MODE_ENABLE, mask_sh),\
500+
HWS_SF(, D3VGA_CONTROL, D3VGA_MODE_ENABLE, mask_sh),\
501+
HWS_SF(, D4VGA_CONTROL, D4VGA_MODE_ENABLE, mask_sh),\
499502
HWS_SF(, VGA_TEST_CONTROL, VGA_TEST_ENABLE, mask_sh),\
500503
HWS_SF(, VGA_TEST_CONTROL, VGA_TEST_RENDER_START, mask_sh),\
501504
HWS_SF(, LVTMA_PWRSEQ_CNTL, LVTMA_BLON, mask_sh), \
@@ -591,7 +594,10 @@ struct dce_hwseq_registers {
591594
type DENTIST_DISPCLK_WDIVIDER; \
592595
type VGA_TEST_ENABLE; \
593596
type VGA_TEST_RENDER_START; \
594-
type D1VGA_MODE_ENABLE;
597+
type D1VGA_MODE_ENABLE; \
598+
type D2VGA_MODE_ENABLE; \
599+
type D3VGA_MODE_ENABLE; \
600+
type D4VGA_MODE_ENABLE;
595601

596602
struct dce_hwseq_shift {
597603
HWSEQ_REG_FIELD_LIST(uint8_t)

drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -238,14 +238,24 @@ static void enable_power_gating_plane(
238238
static void disable_vga(
239239
struct dce_hwseq *hws)
240240
{
241-
unsigned int in_vga_mode = 0;
242-
243-
REG_GET(D1VGA_CONTROL, D1VGA_MODE_ENABLE, &in_vga_mode);
244-
245-
if (in_vga_mode == 0)
241+
unsigned int in_vga1_mode = 0;
242+
unsigned int in_vga2_mode = 0;
243+
unsigned int in_vga3_mode = 0;
244+
unsigned int in_vga4_mode = 0;
245+
246+
REG_GET(D1VGA_CONTROL, D1VGA_MODE_ENABLE, &in_vga1_mode);
247+
REG_GET(D2VGA_CONTROL, D2VGA_MODE_ENABLE, &in_vga2_mode);
248+
REG_GET(D3VGA_CONTROL, D3VGA_MODE_ENABLE, &in_vga3_mode);
249+
REG_GET(D4VGA_CONTROL, D4VGA_MODE_ENABLE, &in_vga4_mode);
250+
251+
if (in_vga1_mode == 0 && in_vga2_mode == 0 &&
252+
in_vga3_mode == 0 && in_vga4_mode == 0)
246253
return;
247254

248255
REG_WRITE(D1VGA_CONTROL, 0);
256+
REG_WRITE(D2VGA_CONTROL, 0);
257+
REG_WRITE(D3VGA_CONTROL, 0);
258+
REG_WRITE(D4VGA_CONTROL, 0);
249259

250260
/* HW Engineer's Notes:
251261
* During switch from vga->extended, if we set the VGA_TEST_ENABLE and

0 commit comments

Comments
 (0)