[PATCH] drm/amdgpu: access register without KIQ

[Christian König]

Am 12.12.18 um 09:54 schrieb Emily Deng:
> There is no need to access register such as mmSMC_IND_INDEX_11
> and mmSMC_IND_DATA_11, PCIE_INDEX, PCIE_DATA through KIQ because
> they are VF-copy.
>
> Signed-off-by: Emily Deng <Emily.Deng at amd.com>

Acked-by: Christian König <christian.koenig at amd.com>

> ---
>   drivers/gpu/drm/amd/amdgpu/vi.c | 18 +++++++++---------
>   1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c
> index ff2906c..77e3674 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vi.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vi.c
> @@ -87,9 +87,9 @@ static u32 vi_pcie_rreg(struct amdgpu_device *adev, u32 reg)
>   	u32 r;
>   
>   	spin_lock_irqsave(&adev->pcie_idx_lock, flags);
> -	WREG32(mmPCIE_INDEX, reg);
> -	(void)RREG32(mmPCIE_INDEX);
> -	r = RREG32(mmPCIE_DATA);
> +	WREG32_NO_KIQ(mmPCIE_INDEX, reg);
> +	(void)RREG32_NO_KIQ(mmPCIE_INDEX);
> +	r = RREG32_NO_KIQ(mmPCIE_DATA);
>   	spin_unlock_irqrestore(&adev->pcie_idx_lock, flags);
>   	return r;
>   }
> @@ -99,10 +99,10 @@ static void vi_pcie_wreg(struct amdgpu_device *adev, u32 reg, u32 v)
>   	unsigned long flags;
>   
>   	spin_lock_irqsave(&adev->pcie_idx_lock, flags);
> -	WREG32(mmPCIE_INDEX, reg);
> -	(void)RREG32(mmPCIE_INDEX);
> -	WREG32(mmPCIE_DATA, v);
> -	(void)RREG32(mmPCIE_DATA);
> +	WREG32_NO_KIQ(mmPCIE_INDEX, reg);
> +	(void)RREG32_NO_KIQ(mmPCIE_INDEX);
> +	WREG32_NO_KIQ(mmPCIE_DATA, v);
> +	(void)RREG32_NO_KIQ(mmPCIE_DATA);
>   	spin_unlock_irqrestore(&adev->pcie_idx_lock, flags);
>   }
>   
> @@ -123,8 +123,8 @@ static void vi_smc_wreg(struct amdgpu_device *adev, u32 reg, u32 v)
>   	unsigned long flags;
>   
>   	spin_lock_irqsave(&adev->smc_idx_lock, flags);
> -	WREG32(mmSMC_IND_INDEX_11, (reg));
> -	WREG32(mmSMC_IND_DATA_11, (v));
> +	WREG32_NO_KIQ(mmSMC_IND_INDEX_11, (reg));
> +	WREG32_NO_KIQ(mmSMC_IND_DATA_11, (v));
>   	spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
>   }
>