[AMD Official Use Only - General]

Reviewed-by: Xiaojian Du <xiaojian...@amd.com> 

Thanks,
Xiaojian

-----Original Message-----
From: Zhang, Yifan <yifan1.zh...@amd.com> 
Sent: Tuesday, September 6, 2022 1:45 PM
To: amd-gfx@lists.freedesktop.org
Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Huang, Tim 
<tim.hu...@amd.com>; Du, Xiaojian <xiaojian...@amd.com>; Zhang, Yifan 
<yifan1.zh...@amd.com>
Subject: [PATCH] drm/amdgpu: correct doorbell range/size value for 
CSDMA_DOORBELL_RANGE

current function mixes CSDMA_DOORBELL_RANGE and SDMA0_DOORBELL_RANGE range/size 
manipulation, while these 2 registers have difference size field mask. Remove 
range/size manipulation for SDMA0_DOORBELL_RANGE.

Signed-off-by: Yifan Zhang <yifan1.zh...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/nbio_v7_7.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/nbio_v7_7.c 
b/drivers/gpu/drm/amd/amdgpu/nbio_v7_7.c
index 1dc95ef21da6..f30bc826a878 100644
--- a/drivers/gpu/drm/amd/amdgpu/nbio_v7_7.c
+++ b/drivers/gpu/drm/amd/amdgpu/nbio_v7_7.c
@@ -68,12 +68,6 @@ static void nbio_v7_7_sdma_doorbell_range(struct 
amdgpu_device *adev, int instan
                doorbell_range = REG_SET_FIELD(doorbell_range,
                                               GDC0_BIF_CSDMA_DOORBELL_RANGE,
                                               SIZE, doorbell_size);
-               doorbell_range = REG_SET_FIELD(doorbell_range,
-                                              GDC0_BIF_SDMA0_DOORBELL_RANGE,
-                                              OFFSET, doorbell_index);
-               doorbell_range = REG_SET_FIELD(doorbell_range,
-                                              GDC0_BIF_SDMA0_DOORBELL_RANGE,
-                                              SIZE, doorbell_size);
        } else {
                doorbell_range = REG_SET_FIELD(doorbell_range,
                                               GDC0_BIF_SDMA0_DOORBELL_RANGE,
--
2.37.1

Reply via email to