diff options
| author | Alex Deucher <alexander.deucher@amd.com> | 2012-08-06 17:06:03 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2012-08-13 10:50:53 -0400 |
| commit | ecd67955fd4c8e66e4df312098989d5fa7da624c (patch) | |
| tree | 2520777f3f6a0d84b15c4d186555c80525636b23 | |
| parent | 81ee8fb6b52ec69eeed37fe7943446af1dccecc5 (diff) | |
drm/radeon: fix ordering in pll picking on dce4+
No functional change, but re-order the cases so they
evaluate properly due to the way the DCE macros work.
Noticed by kallisti5 on IRC.
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
| -rw-r--r-- | drivers/gpu/drm/radeon/atombios_crtc.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/gpu/drm/radeon/atombios_crtc.c b/drivers/gpu/drm/radeon/atombios_crtc.c index dc279ebf7afb..c6fcb5b86a45 100644 --- a/drivers/gpu/drm/radeon/atombios_crtc.c +++ b/drivers/gpu/drm/radeon/atombios_crtc.c | |||
| @@ -1531,12 +1531,12 @@ static int radeon_atom_pick_pll(struct drm_crtc *crtc) | |||
| 1531 | * crtc virtual pixel clock. | 1531 | * crtc virtual pixel clock. |
| 1532 | */ | 1532 | */ |
| 1533 | if (ENCODER_MODE_IS_DP(atombios_get_encoder_mode(test_encoder))) { | 1533 | if (ENCODER_MODE_IS_DP(atombios_get_encoder_mode(test_encoder))) { |
| 1534 | if (ASIC_IS_DCE5(rdev)) | 1534 | if (rdev->clock.dp_extclk) |
| 1535 | return ATOM_DCPLL; | 1535 | return ATOM_PPLL_INVALID; |
| 1536 | else if (ASIC_IS_DCE6(rdev)) | 1536 | else if (ASIC_IS_DCE6(rdev)) |
| 1537 | return ATOM_PPLL0; | 1537 | return ATOM_PPLL0; |
| 1538 | else if (rdev->clock.dp_extclk) | 1538 | else if (ASIC_IS_DCE5(rdev)) |
| 1539 | return ATOM_PPLL_INVALID; | 1539 | return ATOM_DCPLL; |
| 1540 | } | 1540 | } |
| 1541 | } | 1541 | } |
| 1542 | } | 1542 | } |
