On Tue, Nov 6, 2018 at 8:37 PM Evan Quan <evan.quan@xxxxxxx> wrote:
>
> For Vega20, LBPW feature is disabled at default.
>
> Change-Id: I184520cbb03ab8cba9321cd94d1deb0ce38b7e17
> Signed-off-by: Evan Quan <evan.quan@xxxxxxx>
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> index dcf1f79ab347..5bb0cca5b71f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> @@ -2503,6 +2503,20 @@ static int gfx_v9_0_rlc_resume(struct amdgpu_device *adev)
> return r;
> }
>
> + if (amdgpu_lbpw == -1) {
> + switch (adev->asic_type) {
> + case CHIP_RAVEN:
> + amdgpu_lbpw = 1;
> + break;
> + case CHIP_VEGA20:
> + amdgpu_lbpw = 0;
> + break;
> + default:
> + amdgpu_lbpw = 0;
> + break;
> + }
> + }
I'd suggest using a local variable rather than changing the global one
since there may be multiple GPUs in the system.
Alex
> +
> if (adev->asic_type == CHIP_RAVEN ||
> adev->asic_type == CHIP_VEGA20) {
> if (amdgpu_lbpw != 0)
> --
> 2.19.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
