[PATCH] Revert "drm/amd/display: disable dcn20 abm feature for bring up"
Alex Deucher
alexdeucher at gmail.com
Thu May 28 14:08:22 UTC 2020
On Thu, May 28, 2020 at 9:47 AM Harry Wentland <harry.wentland at amd.com> wrote:
>
> This reverts commit 96cb7cf13d8530099c256c053648ad576588c387.
>
> This change was used for DCN2 bringup and is no longer desired.
> In fact it breaks backlight on DCN2 systems.
>
> Cc: Alexander Monakov <amonakov at ispras.ru>
> Cc: Hersen Wu <hersenxs.wu at amd.com>
> Cc: Anthony Koo <Anthony.Koo at amd.com>
> Cc: Michael Chiu <Michael.Chiu at amd.com>
> Signed-off-by: Harry Wentland <harry.wentland at amd.com>
Acked-by: Alex Deucher <alexander.deucher at amd.com>
> ---
> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 11 ++++-------
> 1 file changed, 4 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index ddc979e3eebe..acd4874e0743 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -1356,7 +1356,7 @@ static int dm_late_init(void *handle)
> unsigned int linear_lut[16];
> int i;
> struct dmcu *dmcu = NULL;
> - bool ret = false;
> + bool ret;
>
> if (!adev->dm.fw_dmcu)
> return detect_mst_link_for_all_connectors(adev->ddev);
> @@ -1377,13 +1377,10 @@ static int dm_late_init(void *handle)
> */
> params.min_abm_backlight = 0x28F;
>
> - /* todo will enable for navi10 */
> - if (adev->asic_type <= CHIP_RAVEN) {
> - ret = dmcu_load_iram(dmcu, params);
> + ret = dmcu_load_iram(dmcu, params);
>
> - if (!ret)
> - return -EINVAL;
> - }
> + if (!ret)
> + return -EINVAL;
>
> return detect_mst_link_for_all_connectors(adev->ddev);
> }
> --
> 2.26.2
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
More information about the amd-gfx
mailing list