When doorbells are 8-byte on SOC15, doorbell_index in rings no longer
reflects its true usage. So we should indicate its dword attribute as
a generic way to accommodate different doorbell sizes.

Change-Id: I053c69498af5d68df1783a7eb03106dd68f5e8cc
Signed-off-by: Yong Zhao <yong.z...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h |  2 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c    |  2 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c    |  6 +++---
 drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c    | 16 ++++++++--------
 drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c    | 20 ++++++++++----------
 drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c   |  6 +++---
 drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c   | 16 ++++++++--------
 drivers/gpu/drm/amd/amdgpu/soc15.c       |  2 +-
 drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c    |  8 ++++----
 drivers/gpu/drm/amd/amdgpu/vce_v4_0.c    |  8 ++++----
 11 files changed, 44 insertions(+), 44 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
index 97a60da62004..6389ef068f4b 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
@@ -250,7 +250,7 @@ int amdgpu_gfx_kiq_init_ring(struct amdgpu_device *adev,
        ring->adev = NULL;
        ring->ring_obj = NULL;
        ring->use_doorbell = true;
-       ring->doorbell_index = adev->doorbell_index.kiq;
+       ring->doorbell_dw_idx = adev->doorbell_index.kiq;
 
        r = amdgpu_gfx_kiq_acquire(adev, ring);
        if (r)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h
index d7fae2676269..0c6fa7576a9f 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h
@@ -198,7 +198,7 @@ struct amdgpu_ring {
        uint64_t                mqd_gpu_addr;
        void                    *mqd_ptr;
        uint64_t                eop_gpu_addr;
-       u32                     doorbell_index;
+       u32                     doorbell_dw_idx;
        bool                    use_doorbell;
        bool                    use_pollmem;
        unsigned                wptr_offs;
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
index 305276c7e4bf..a6aff3f9ab9d 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
@@ -3109,7 +3109,7 @@ static int gfx_v6_0_sw_init(void *handle)
                ring = &adev->gfx.compute_ring[i];
                ring->ring_obj = NULL;
                ring->use_doorbell = false;
-               ring->doorbell_index = 0;
+               ring->doorbell_dw_idx = 0;
                ring->me = 1;
                ring->pipe = i;
                ring->queue = i;
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
index 7984292f9282..3f11f0ac43fc 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
@@ -2641,7 +2641,7 @@ static void gfx_v7_0_ring_set_wptr_compute(struct 
amdgpu_ring *ring)
 
        /* XXX check if swapping is necessary on BE */
        adev->wb.wb[ring->wptr_offs] = lower_32_bits(ring->wptr);
-       WDOORBELL32(ring->doorbell_index, lower_32_bits(ring->wptr));
+       WDOORBELL32(ring->doorbell_dw_idx, lower_32_bits(ring->wptr));
 }
 
 /**
@@ -2957,7 +2957,7 @@ static void gfx_v7_0_mqd_init(struct amdgpu_device *adev,
                mqd->cp_hqd_pq_doorbell_control &=
                        ~CP_HQD_PQ_DOORBELL_CONTROL__DOORBELL_OFFSET_MASK;
                mqd->cp_hqd_pq_doorbell_control |=
-                       (ring->doorbell_index <<
+                       (ring->doorbell_dw_idx <<
                         CP_HQD_PQ_DOORBELL_CONTROL__DOORBELL_OFFSET__SHIFT);
                mqd->cp_hqd_pq_doorbell_control |=
                        CP_HQD_PQ_DOORBELL_CONTROL__DOORBELL_EN_MASK;
@@ -4363,7 +4363,7 @@ static int gfx_v7_0_compute_ring_init(struct 
amdgpu_device *adev, int ring_id,
 
        ring->ring_obj = NULL;
        ring->use_doorbell = true;
-       ring->doorbell_index = adev->doorbell_index.mec_ring0 + ring_id;
+       ring->doorbell_dw_idx = adev->doorbell_index.mec_ring0 + ring_id;
        sprintf(ring->name, "comp_%d.%d.%d", ring->me, ring->pipe, ring->queue);
 
        irq_type = AMDGPU_CP_IRQ_COMPUTE_MEC1_PIPE0_EOP
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
index a26747681ed6..a2ae1446e7a9 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
@@ -1890,7 +1890,7 @@ static int gfx_v8_0_compute_ring_init(struct 
amdgpu_device *adev, int ring_id,
 
        ring->ring_obj = NULL;
        ring->use_doorbell = true;
-       ring->doorbell_index = adev->doorbell_index.mec_ring0 + ring_id;
+       ring->doorbell_dw_idx = adev->doorbell_index.mec_ring0 + ring_id;
        ring->eop_gpu_addr = adev->gfx.mec.hpd_eop_gpu_addr
                                + (ring_id * GFX8_MEC_HPD_SIZE);
        sprintf(ring->name, "comp_%d.%d.%d", ring->me, ring->pipe, ring->queue);
@@ -2001,7 +2001,7 @@ static int gfx_v8_0_sw_init(void *handle)
                /* no gfx doorbells on iceland */
                if (adev->asic_type != CHIP_TOPAZ) {
                        ring->use_doorbell = true;
-                       ring->doorbell_index = adev->doorbell_index.gfx_ring0;
+                       ring->doorbell_dw_idx = adev->doorbell_index.gfx_ring0;
                }
 
                r = amdgpu_ring_init(adev, ring, 1024, &adev->gfx.eop_irq,
@@ -4204,7 +4204,7 @@ static void gfx_v8_0_set_cpg_door_bell(struct 
amdgpu_device *adev, struct amdgpu
 
        if (ring->use_doorbell) {
                tmp = REG_SET_FIELD(tmp, CP_RB_DOORBELL_CONTROL,
-                               DOORBELL_OFFSET, ring->doorbell_index);
+                               DOORBELL_OFFSET, ring->doorbell_dw_idx);
                tmp = REG_SET_FIELD(tmp, CP_RB_DOORBELL_CONTROL,
                                                DOORBELL_HIT, 0);
                tmp = REG_SET_FIELD(tmp, CP_RB_DOORBELL_CONTROL,
@@ -4357,7 +4357,7 @@ static int gfx_v8_0_kiq_kcq_enable(struct amdgpu_device 
*adev)
                amdgpu_ring_write(kiq_ring,
                                  PACKET3_MAP_QUEUES_NUM_QUEUES(1));
                amdgpu_ring_write(kiq_ring,
-                                 
PACKET3_MAP_QUEUES_DOORBELL_OFFSET(ring->doorbell_index) |
+                                 
PACKET3_MAP_QUEUES_DOORBELL_OFFSET(ring->doorbell_dw_idx) |
                                  PACKET3_MAP_QUEUES_QUEUE(ring->queue) |
                                  PACKET3_MAP_QUEUES_PIPE(ring->pipe) |
                                  PACKET3_MAP_QUEUES_ME(ring->me == 1 ? 0 : 
1)); /* doorbell */
@@ -4475,7 +4475,7 @@ static int gfx_v8_0_mqd_init(struct amdgpu_ring *ring)
        if (ring->use_doorbell) {
                tmp = RREG32(mmCP_HQD_PQ_DOORBELL_CONTROL);
                tmp = REG_SET_FIELD(tmp, CP_HQD_PQ_DOORBELL_CONTROL,
-                               DOORBELL_OFFSET, ring->doorbell_index);
+                               DOORBELL_OFFSET, ring->doorbell_dw_idx);
 
                tmp = REG_SET_FIELD(tmp, CP_HQD_PQ_DOORBELL_CONTROL,
                                         DOORBELL_EN, 1);
@@ -4803,7 +4803,7 @@ static int gfx_v8_0_kcq_disable(struct amdgpu_device 
*adev)
                                                
PACKET3_UNMAP_QUEUES_QUEUE_SEL(0) |
                                                
PACKET3_UNMAP_QUEUES_ENGINE_SEL(0) |
                                                
PACKET3_UNMAP_QUEUES_NUM_QUEUES(1));
-               amdgpu_ring_write(kiq_ring, 
PACKET3_UNMAP_QUEUES_DOORBELL_OFFSET0(ring->doorbell_index));
+               amdgpu_ring_write(kiq_ring, 
PACKET3_UNMAP_QUEUES_DOORBELL_OFFSET0(ring->doorbell_dw_idx));
                amdgpu_ring_write(kiq_ring, 0);
                amdgpu_ring_write(kiq_ring, 0);
                amdgpu_ring_write(kiq_ring, 0);
@@ -5993,7 +5993,7 @@ static void gfx_v8_0_ring_set_wptr_gfx(struct amdgpu_ring 
*ring)
        if (ring->use_doorbell) {
                /* XXX check if swapping is necessary on BE */
                adev->wb.wb[ring->wptr_offs] = lower_32_bits(ring->wptr);
-               WDOORBELL32(ring->doorbell_index, lower_32_bits(ring->wptr));
+               WDOORBELL32(ring->doorbell_dw_idx, lower_32_bits(ring->wptr));
        } else {
                WREG32(mmCP_RB0_WPTR, lower_32_bits(ring->wptr));
                (void)RREG32(mmCP_RB0_WPTR);
@@ -6169,7 +6169,7 @@ static void gfx_v8_0_ring_set_wptr_compute(struct 
amdgpu_ring *ring)
 
        /* XXX check if swapping is necessary on BE */
        adev->wb.wb[ring->wptr_offs] = lower_32_bits(ring->wptr);
-       WDOORBELL32(ring->doorbell_index, lower_32_bits(ring->wptr));
+       WDOORBELL32(ring->doorbell_dw_idx, lower_32_bits(ring->wptr));
 }
 
 static void gfx_v8_0_ring_set_pipe_percent(struct amdgpu_ring *ring,
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
index 262ee3cf6f1c..139515feef64 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
@@ -1580,7 +1580,7 @@ static int gfx_v9_0_compute_ring_init(struct 
amdgpu_device *adev, int ring_id,
 
        ring->ring_obj = NULL;
        ring->use_doorbell = true;
-       ring->doorbell_index = (adev->doorbell_index.mec_ring0 + ring_id) << 1;
+       ring->doorbell_dw_idx = (adev->doorbell_index.mec_ring0 + ring_id) << 1;
        ring->eop_gpu_addr = adev->gfx.mec.hpd_eop_gpu_addr
                                + (ring_id * GFX9_MEC_HPD_SIZE);
        sprintf(ring->name, "comp_%d.%d.%d", ring->me, ring->pipe, ring->queue);
@@ -1669,7 +1669,7 @@ static int gfx_v9_0_sw_init(void *handle)
                else
                        sprintf(ring->name, "gfx_%d", i);
                ring->use_doorbell = true;
-               ring->doorbell_index = adev->doorbell_index.gfx_ring0 << 1;
+               ring->doorbell_dw_idx = adev->doorbell_index.gfx_ring0 << 1;
                r = amdgpu_ring_init(adev, ring, 1024,
                                     &adev->gfx.eop_irq, AMDGPU_CP_IRQ_GFX_EOP);
                if (r)
@@ -2619,7 +2619,7 @@ static int gfx_v9_0_cp_gfx_resume(struct amdgpu_device 
*adev)
        tmp = RREG32_SOC15(GC, 0, mmCP_RB_DOORBELL_CONTROL);
        if (ring->use_doorbell) {
                tmp = REG_SET_FIELD(tmp, CP_RB_DOORBELL_CONTROL,
-                                   DOORBELL_OFFSET, ring->doorbell_index);
+                                   DOORBELL_OFFSET, ring->doorbell_dw_idx);
                tmp = REG_SET_FIELD(tmp, CP_RB_DOORBELL_CONTROL,
                                    DOORBELL_EN, 1);
        } else {
@@ -2628,7 +2628,7 @@ static int gfx_v9_0_cp_gfx_resume(struct amdgpu_device 
*adev)
        WREG32_SOC15(GC, 0, mmCP_RB_DOORBELL_CONTROL, tmp);
 
        tmp = REG_SET_FIELD(0, CP_RB_DOORBELL_RANGE_LOWER,
-                       DOORBELL_RANGE_LOWER, ring->doorbell_index);
+                       DOORBELL_RANGE_LOWER, ring->doorbell_dw_idx);
        WREG32_SOC15(GC, 0, mmCP_RB_DOORBELL_RANGE_LOWER, tmp);
 
        WREG32_SOC15(GC, 0, mmCP_RB_DOORBELL_RANGE_UPPER,
@@ -2769,7 +2769,7 @@ static int gfx_v9_0_kiq_kcq_enable(struct amdgpu_device 
*adev)
                                  PACKET3_MAP_QUEUES_ALLOC_FORMAT(0) | /* alloc 
format: all_on_one_pipe */
                                  PACKET3_MAP_QUEUES_ENGINE_SEL(0) | /* 
engine_sel: compute */
                                  PACKET3_MAP_QUEUES_NUM_QUEUES(1)); /* 
num_queues: must be 1 */
-               amdgpu_ring_write(kiq_ring, 
PACKET3_MAP_QUEUES_DOORBELL_OFFSET(ring->doorbell_index));
+               amdgpu_ring_write(kiq_ring, 
PACKET3_MAP_QUEUES_DOORBELL_OFFSET(ring->doorbell_dw_idx));
                amdgpu_ring_write(kiq_ring, lower_32_bits(mqd_addr));
                amdgpu_ring_write(kiq_ring, upper_32_bits(mqd_addr));
                amdgpu_ring_write(kiq_ring, lower_32_bits(wptr_addr));
@@ -2821,7 +2821,7 @@ static int gfx_v9_0_mqd_init(struct amdgpu_ring *ring)
 
        if (ring->use_doorbell) {
                tmp = REG_SET_FIELD(tmp, CP_HQD_PQ_DOORBELL_CONTROL,
-                                   DOORBELL_OFFSET, ring->doorbell_index);
+                                   DOORBELL_OFFSET, ring->doorbell_dw_idx);
                tmp = REG_SET_FIELD(tmp, CP_HQD_PQ_DOORBELL_CONTROL,
                                    DOORBELL_EN, 1);
                tmp = REG_SET_FIELD(tmp, CP_HQD_PQ_DOORBELL_CONTROL,
@@ -2887,7 +2887,7 @@ static int gfx_v9_0_mqd_init(struct amdgpu_ring *ring)
        if (ring->use_doorbell) {
                tmp = RREG32_SOC15(GC, 0, mmCP_HQD_PQ_DOORBELL_CONTROL);
                tmp = REG_SET_FIELD(tmp, CP_HQD_PQ_DOORBELL_CONTROL,
-                               DOORBELL_OFFSET, ring->doorbell_index);
+                               DOORBELL_OFFSET, ring->doorbell_dw_idx);
 
                tmp = REG_SET_FIELD(tmp, CP_HQD_PQ_DOORBELL_CONTROL,
                                         DOORBELL_EN, 1);
@@ -3288,7 +3288,7 @@ static int gfx_v9_0_kcq_disable(struct amdgpu_device 
*adev)
                                                
PACKET3_UNMAP_QUEUES_QUEUE_SEL(0) |
                                                
PACKET3_UNMAP_QUEUES_ENGINE_SEL(0) |
                                                
PACKET3_UNMAP_QUEUES_NUM_QUEUES(1));
-               amdgpu_ring_write(kiq_ring, 
PACKET3_UNMAP_QUEUES_DOORBELL_OFFSET0(ring->doorbell_index));
+               amdgpu_ring_write(kiq_ring, 
PACKET3_UNMAP_QUEUES_DOORBELL_OFFSET0(ring->doorbell_dw_idx));
                amdgpu_ring_write(kiq_ring, 0);
                amdgpu_ring_write(kiq_ring, 0);
                amdgpu_ring_write(kiq_ring, 0);
@@ -3933,7 +3933,7 @@ static void gfx_v9_0_ring_set_wptr_gfx(struct amdgpu_ring 
*ring)
        if (ring->use_doorbell) {
                /* XXX check if swapping is necessary on BE */
                atomic64_set((atomic64_t*)&adev->wb.wb[ring->wptr_offs], 
ring->wptr);
-               WDOORBELL64(ring->doorbell_index, ring->wptr);
+               WDOORBELL64(ring->doorbell_dw_idx, ring->wptr);
        } else {
                WREG32_SOC15(GC, 0, mmCP_RB0_WPTR, lower_32_bits(ring->wptr));
                WREG32_SOC15(GC, 0, mmCP_RB0_WPTR_HI, 
upper_32_bits(ring->wptr));
@@ -4202,7 +4202,7 @@ static void gfx_v9_0_ring_set_wptr_compute(struct 
amdgpu_ring *ring)
        /* XXX check if swapping is necessary on BE */
        if (ring->use_doorbell) {
                atomic64_set((atomic64_t*)&adev->wb.wb[ring->wptr_offs], 
ring->wptr);
-               WDOORBELL64(ring->doorbell_index, ring->wptr);
+               WDOORBELL64(ring->doorbell_dw_idx, ring->wptr);
        } else{
                BUG(); /* only DOORBELL method supported on gfx9 now */
        }
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c 
b/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
index 0ce8331baeb2..954a68e3aa80 100644
--- a/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
@@ -387,7 +387,7 @@ static void sdma_v3_0_ring_set_wptr(struct amdgpu_ring 
*ring)
                u32 *wb = (u32 *)&adev->wb.wb[ring->wptr_offs];
                /* XXX check if swapping is necessary on BE */
                WRITE_ONCE(*wb, (lower_32_bits(ring->wptr) << 2));
-               WDOORBELL32(ring->doorbell_index, lower_32_bits(ring->wptr) << 
2);
+               WDOORBELL32(ring->doorbell_dw_idx, lower_32_bits(ring->wptr) << 
2);
        } else if (ring->use_pollmem) {
                u32 *wb = (u32 *)&adev->wb.wb[ring->wptr_offs];
 
@@ -702,7 +702,7 @@ static int sdma_v3_0_gfx_resume(struct amdgpu_device *adev)
 
                if (ring->use_doorbell) {
                        doorbell = REG_SET_FIELD(doorbell, SDMA0_GFX_DOORBELL,
-                                                OFFSET, ring->doorbell_index);
+                                                OFFSET, ring->doorbell_dw_idx);
                        doorbell = REG_SET_FIELD(doorbell, SDMA0_GFX_DOORBELL, 
ENABLE, 1);
                } else {
                        doorbell = REG_SET_FIELD(doorbell, SDMA0_GFX_DOORBELL, 
ENABLE, 0);
@@ -1145,7 +1145,7 @@ static int sdma_v3_0_sw_init(void *handle)
                ring->ring_obj = NULL;
                if (!amdgpu_sriov_vf(adev)) {
                        ring->use_doorbell = true;
-                       ring->doorbell_index = 
adev->doorbell_index.sdma_engine[i];
+                       ring->doorbell_dw_idx = 
adev->doorbell_index.sdma_engine[i];
                } else {
                        ring->use_pollmem = true;
                }
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c 
b/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
index e65e9bc52bd4..9136428aea13 100644
--- a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
@@ -408,8 +408,8 @@ static void sdma_v4_0_ring_set_wptr(struct amdgpu_ring 
*ring)
                /* XXX check if swapping is necessary on BE */
                WRITE_ONCE(*wb, (ring->wptr << 2));
                DRM_DEBUG("calling WDOORBELL64(0x%08x, 0x%016llx)\n",
-                               ring->doorbell_index, ring->wptr << 2);
-               WDOORBELL64(ring->doorbell_index, ring->wptr << 2);
+                               ring->doorbell_dw_idx, ring->wptr << 2);
+               WDOORBELL64(ring->doorbell_dw_idx, ring->wptr << 2);
        } else {
                DRM_DEBUG("Not using doorbell -- "
                                "mmSDMA%i_GFX_RB_WPTR == 0x%08x "
@@ -465,7 +465,7 @@ static void sdma_v4_0_page_ring_set_wptr(struct amdgpu_ring 
*ring)
 
                /* XXX check if swapping is necessary on BE */
                WRITE_ONCE(*wb, (ring->wptr << 2));
-               WDOORBELL64(ring->doorbell_index, ring->wptr << 2);
+               WDOORBELL64(ring->doorbell_dw_idx, ring->wptr << 2);
        } else {
                uint64_t wptr = ring->wptr << 2;
 
@@ -831,7 +831,7 @@ static void sdma_v4_0_gfx_resume(struct amdgpu_device 
*adev, unsigned int i)
                                 ring->use_doorbell);
        doorbell_offset = REG_SET_FIELD(doorbell_offset,
                                        SDMA0_GFX_DOORBELL_OFFSET,
-                                       OFFSET, ring->doorbell_index);
+                                       OFFSET, ring->doorbell_dw_idx);
        WREG32_SDMA(i, mmSDMA0_GFX_DOORBELL, doorbell);
        WREG32_SDMA(i, mmSDMA0_GFX_DOORBELL_OFFSET, doorbell_offset);
 
@@ -921,7 +921,7 @@ static void sdma_v4_0_page_resume(struct amdgpu_device 
*adev, unsigned int i)
                                 ring->use_doorbell);
        doorbell_offset = REG_SET_FIELD(doorbell_offset,
                                        SDMA0_PAGE_DOORBELL_OFFSET,
-                                       OFFSET, ring->doorbell_index);
+                                       OFFSET, ring->doorbell_dw_idx);
        WREG32_SDMA(i, mmSDMA0_PAGE_DOORBELL, doorbell);
        WREG32_SDMA(i, mmSDMA0_PAGE_DOORBELL_OFFSET, doorbell_offset);
 
@@ -1520,7 +1520,7 @@ static int sdma_v4_0_sw_init(void *handle)
                                ring->use_doorbell?"true":"false");
 
                /* doorbell size is 2 dwords, get DWORD offset */
-               ring->doorbell_index = adev->doorbell_index.sdma_engine[i]
+               ring->doorbell_dw_idx = adev->doorbell_index.sdma_engine[i]
                                * adev->doorbell_index.entry_dw_size;
 
                sprintf(ring->name, "sdma%d", i);
@@ -1540,8 +1540,8 @@ static int sdma_v4_0_sw_init(void *handle)
                        /* paging queue use same doorbell index/routing as gfx 
queue
                         * with 0x400 (4096 dwords) offset on second doorbell 
page
                         */
-                       ring->doorbell_index = 
adev->doorbell_index.sdma_engine[i] << 1;
-                       ring->doorbell_index += 0x400;
+                       ring->doorbell_dw_idx = 
adev->doorbell_index.sdma_engine[i] << 1;
+                       ring->doorbell_dw_idx += 0x400;
 
                        sprintf(ring->name, "page%d", i);
                        r = amdgpu_ring_init(adev, ring, 1024,
diff --git a/drivers/gpu/drm/amd/amdgpu/soc15.c 
b/drivers/gpu/drm/amd/amdgpu/soc15.c
index fbf4dde22f6c..db2fa4482e2c 100644
--- a/drivers/gpu/drm/amd/amdgpu/soc15.c
+++ b/drivers/gpu/drm/amd/amdgpu/soc15.c
@@ -979,7 +979,7 @@ static void soc15_doorbell_range_init(struct amdgpu_device 
*adev)
        for (i = 0; i < adev->sdma.num_instances; i++) {
                ring = &adev->sdma.instance[i].ring;
                adev->nbio_funcs->sdma_doorbell_range(adev, i,
-                       ring->use_doorbell, ring->doorbell_index,
+                       ring->use_doorbell, ring->doorbell_dw_idx,
                        adev->doorbell_index.dw_range_per_sdma_eng);
        }
 
diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c 
b/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c
index dc461df48da0..6b3cd7b39394 100644
--- a/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c
@@ -154,7 +154,7 @@ static void uvd_v7_0_enc_ring_set_wptr(struct amdgpu_ring 
*ring)
        if (ring->use_doorbell) {
                /* XXX check if swapping is necessary on BE */
                adev->wb.wb[ring->wptr_offs] = lower_32_bits(ring->wptr);
-               WDOORBELL32(ring->doorbell_index, lower_32_bits(ring->wptr));
+               WDOORBELL32(ring->doorbell_dw_idx, lower_32_bits(ring->wptr));
                return;
        }
 
@@ -451,9 +451,9 @@ static int uvd_v7_0_sw_init(void *handle)
                                 * sriov, so set unused location for other 
unused rings.
                                 */
                                if (i == 0)
-                                       ring->doorbell_index = 
adev->doorbell_index.uvd_vce.uvd_ring0_1 * 2;
+                                       ring->doorbell_dw_idx = 
adev->doorbell_index.uvd_vce.uvd_ring0_1 * 2;
                                else
-                                       ring->doorbell_index = 
adev->doorbell_index.uvd_vce.uvd_ring2_3 * 2 + 1;
+                                       ring->doorbell_dw_idx = 
adev->doorbell_index.uvd_vce.uvd_ring2_3 * 2 + 1;
                        }
                        r = amdgpu_ring_init(adev, ring, 512, 
&adev->uvd.inst[j].irq, 0);
                        if (r)
@@ -721,7 +721,7 @@ static int uvd_v7_0_mmsch_start(struct amdgpu_device *adev,
        for (i = 0; i < adev->uvd.num_uvd_inst; ++i) {
                if (adev->uvd.harvest_config & (1 << i))
                        continue;
-               WDOORBELL32(adev->uvd.inst[i].ring_enc[0].doorbell_index, 0);
+               WDOORBELL32(adev->uvd.inst[i].ring_enc[0].doorbell_dw_idx, 0);
                adev->wb.wb[adev->uvd.inst[i].ring_enc[0].wptr_offs] = 0;
                adev->uvd.inst[i].ring_enc[0].wptr = 0;
                adev->uvd.inst[i].ring_enc[0].wptr_old = 0;
diff --git a/drivers/gpu/drm/amd/amdgpu/vce_v4_0.c 
b/drivers/gpu/drm/amd/amdgpu/vce_v4_0.c
index aadc3e66ebd7..fc76132085d9 100644
--- a/drivers/gpu/drm/amd/amdgpu/vce_v4_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/vce_v4_0.c
@@ -106,7 +106,7 @@ static void vce_v4_0_ring_set_wptr(struct amdgpu_ring *ring)
        if (ring->use_doorbell) {
                /* XXX check if swapping is necessary on BE */
                adev->wb.wb[ring->wptr_offs] = lower_32_bits(ring->wptr);
-               WDOORBELL32(ring->doorbell_index, lower_32_bits(ring->wptr));
+               WDOORBELL32(ring->doorbell_dw_idx, lower_32_bits(ring->wptr));
                return;
        }
 
@@ -175,7 +175,7 @@ static int vce_v4_0_mmsch_start(struct amdgpu_device *adev,
        /* 4, set resp to zero */
        WREG32(SOC15_REG_OFFSET(VCE, 0, mmVCE_MMSCH_VF_MAILBOX_RESP), 0);
 
-       WDOORBELL32(adev->vce.ring[0].doorbell_index, 0);
+       WDOORBELL32(adev->vce.ring[0].doorbell_dw_idx, 0);
        adev->wb.wb[adev->vce.ring[0].wptr_offs] = 0;
        adev->vce.ring[0].wptr = 0;
        adev->vce.ring[0].wptr_old = 0;
@@ -466,9 +466,9 @@ static int vce_v4_0_sw_init(void *handle)
                         * so set unused location for other unused rings.
                         */
                        if (i == 0)
-                               ring->doorbell_index = 
adev->doorbell_index.uvd_vce.vce_ring0_1 * 2;
+                               ring->doorbell_dw_idx = 
adev->doorbell_index.uvd_vce.vce_ring0_1 * 2;
                        else
-                               ring->doorbell_index = 
adev->doorbell_index.uvd_vce.vce_ring2_3 * 2 + 1;
+                               ring->doorbell_dw_idx = 
adev->doorbell_index.uvd_vce.vce_ring2_3 * 2 + 1;
                }
                r = amdgpu_ring_init(adev, ring, 512, &adev->vce.irq, 0);
                if (r)
-- 
2.17.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to