[PATCH 1/2] drm/amd/display: Fix warning storm on Raven2

Huang, Ray Ray.Huang at amd.com
Wed Oct 10 03:19:49 UTC 2018


> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Roman.Li at amd.com
> Sent: Wednesday, October 10, 2018 1:50 AM
> To: amd-gfx at lists.freedesktop.org
> Cc: Li, Sun peng (Leo) <Sunpeng.Li at amd.com>; Wu, Hersen
> <hersenxs.wu at amd.com>; Wentland, Harry <Harry.Wentland at amd.com>;
> Li, Roman <Roman.Li at amd.com>
> Subject: [PATCH 1/2] drm/amd/display: Fix warning storm on Raven2
> 
> From: Roman Li <Roman.Li at amd.com>
> 
> [Why]
> Wrong index for pstate debug test register
> 
> [How]
> Add correct index value for dcn1_01 in hubbub1_construct()
> 
> Signed-off-by: Hersen Wu <hersenxs.wu at amd.com>
> Signed-off-by: Roman Li <Roman.Li at amd.com>

Thanks, Roman.

Series are also
Reviewed-by: Huang Rui <ray.huang at amd.com>

I will apply them directly to drm-next and dkms-4.18 for testing.

Thanks,
Ray	

> ---
>  .../gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c    | 43
> +++++++++++++++++++++-
>  1 file changed, 41 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c
> b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c
> index 69345ce6..4254e7e 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c
> @@ -133,7 +133,43 @@ bool hubbub1_verify_allow_pstate_change_high(
>  		forced_pstate_allow = false;
>  	}
> 
> -	/* RV1:
> +	/* RV2:
> +	 * dchubbubdebugind, at: 0xB
> +	 * description
> +	 * 0:     Pipe0 Plane0 Allow Pstate Change
> +	 * 1:     Pipe0 Plane1 Allow Pstate Change
> +	 * 2:     Pipe0 Cursor0 Allow Pstate Change
> +	 * 3:     Pipe0 Cursor1 Allow Pstate Change
> +	 * 4:     Pipe1 Plane0 Allow Pstate Change
> +	 * 5:     Pipe1 Plane1 Allow Pstate Change
> +	 * 6:     Pipe1 Cursor0 Allow Pstate Change
> +	 * 7:     Pipe1 Cursor1 Allow Pstate Change
> +	 * 8:     Pipe2 Plane0 Allow Pstate Change
> +	 * 9:     Pipe2 Plane1 Allow Pstate Change
> +	 * 10:    Pipe2 Cursor0 Allow Pstate Change
> +	 * 11:    Pipe2 Cursor1 Allow Pstate Change
> +	 * 12:    Pipe3 Plane0 Allow Pstate Change
> +	 * 13:    Pipe3 Plane1 Allow Pstate Change
> +	 * 14:    Pipe3 Cursor0 Allow Pstate Change
> +	 * 15:    Pipe3 Cursor1 Allow Pstate Change
> +	 * 16:    Pipe4 Plane0 Allow Pstate Change
> +	 * 17:    Pipe4 Plane1 Allow Pstate Change
> +	 * 18:    Pipe4 Cursor0 Allow Pstate Change
> +	 * 19:    Pipe4 Cursor1 Allow Pstate Change
> +	 * 20:    Pipe5 Plane0 Allow Pstate Change
> +	 * 21:    Pipe5 Plane1 Allow Pstate Change
> +	 * 22:    Pipe5 Cursor0 Allow Pstate Change
> +	 * 23:    Pipe5 Cursor1 Allow Pstate Change
> +	 * 24:    Pipe6 Plane0 Allow Pstate Change
> +	 * 25:    Pipe6 Plane1 Allow Pstate Change
> +	 * 26:    Pipe6 Cursor0 Allow Pstate Change
> +	 * 27:    Pipe6 Cursor1 Allow Pstate Change
> +	 * 28:    WB0 Allow Pstate Change
> +	 * 29:    WB1 Allow Pstate Change
> +	 * 30:    Arbiter's allow_pstate_change
> +	 * 31:    SOC pstate change request"
> +	 *
> +	 * RV1:
>  	 * dchubbubdebugind, at: 0x7
>  	 * description "3-0:   Pipe0 cursor0 QOS
>  	 * 7-4:   Pipe1 cursor0 QOS
> @@ -157,7 +193,6 @@ bool hubbub1_verify_allow_pstate_change_high(
>  	 * 31:    SOC pstate change request
>  	 */
> 
> -
>  	REG_WRITE(DCHUBBUB_TEST_DEBUG_INDEX, hubbub-
> >debug_test_index_pstate);
> 
>  	for (i = 0; i < pstate_wait_timeout_us; i++) { @@ -819,5 +854,9 @@
> void hubbub1_construct(struct hubbub *hubbub,
>  	hubbub->masks = hubbub_mask;
> 
>  	hubbub->debug_test_index_pstate = 0x7;
> +#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
> +	if (ctx->dce_version == DCN_VERSION_1_01)
> +		hubbub->debug_test_index_pstate = 0xB; #endif
>  }
> 
> --
> 2.7.4
> 
> _______________________________________________
> 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