[PATCH 7/7] drm/amd/amdgpu: Comment out currently unused SI DPM struct

[StDenis, Tom]

Hi Alex,


Would love to but unfortunately I don't have a tahiti_le (or any SI) part to test it on.


Cheers,

Tom


________________________________
From: Deucher, Alexander
Sent: Wednesday, September 7, 2016 10:45
To: 'Tom St Denis'; amd-gfx at lists.freedesktop.org
Cc: StDenis, Tom
Subject: RE: [PATCH 7/7] drm/amd/amdgpu: Comment out currently unused SI DPM struct

> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Tom St Denis
> Sent: Tuesday, September 06, 2016 12:19 PM
> To: amd-gfx at lists.freedesktop.org
> Cc: StDenis, Tom
> Subject: [PATCH 7/7] drm/amd/amdgpu: Comment out currently unused SI
> DPM struct
>
> The tahiti_le structure is not currently used.  Comment it out
> to avoid warnings.
>
> Signed-off-by: Tom St Denis <tom.stdenis at amd.com>

Might be worth assigning this on the Tahiti LE boards to see if it helps with the Tahiti LE bug.

Alex

> ---
>  drivers/gpu/drm/amd/amdgpu/si_dpm.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/si_dpm.c
> b/drivers/gpu/drm/amd/amdgpu/si_dpm.c
> index 7b9debefd64a..92a041755e8e 100644
> --- a/drivers/gpu/drm/amd/amdgpu/si_dpm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/si_dpm.c
> @@ -340,6 +340,7 @@ static const struct si_dte_data dte_data_tahiti =
>        false
>  };
>
> +#if 0
>  static const struct si_dte_data dte_data_tahiti_le =
>  {
>        { 0x1E8480, 0x7A1200, 0x2160EC0, 0x3938700, 0 },
> @@ -357,6 +358,7 @@ static const struct si_dte_data dte_data_tahiti_le =
>        85,
>        true
>  };
> +#endif
>
>  static const struct si_dte_data dte_data_tahiti_pro =
>  {
> --
> 2.9.3
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20160907/f430c6d1/attachment.html>