drm/amdgpu: Add smc method to register block

Define register access block which consolidates different register access
methods. Add smc method to register access block.

Signed-off-by: Lijo Lazar <lijo.lazar@amd.com>
Reviewed-by: Hawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
index 1e71a03c8bbae93251c42ab5cbd418e3dce0b577..77813928450b6aa7dd1244adac963830294d0055 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
@@ -900,10 +900,8 @@ struct amdgpu_device {
        /* protects concurrent MM_INDEX/DATA based register access */
        spinlock_t mmio_idx_lock;
        struct amdgpu_mmio_remap        rmmio_remap;
-       /* protects concurrent SMC based register access */
-       spinlock_t smc_idx_lock;
-       amdgpu_rreg_t                   smc_rreg;
-       amdgpu_wreg_t                   smc_wreg;
+       /* Indirect register access blocks */
+       struct amdgpu_reg_access reg;
        /* protects concurrent PCIE register access */
        spinlock_t pcie_idx_lock;
        amdgpu_rreg_t                   pcie_rreg;
@@ -1340,8 +1338,8 @@ int emu_soc_asic_init(struct amdgpu_device *adev);
 #define WREG64_PCIE(reg, v) adev->pcie_wreg64(adev, (reg), (v))
 #define RREG64_PCIE_EXT(reg) adev->pcie_rreg64_ext(adev, (reg))
 #define WREG64_PCIE_EXT(reg, v) adev->pcie_wreg64_ext(adev, (reg), (v))
-#define RREG32_SMC(reg) adev->smc_rreg(adev, (reg))
-#define WREG32_SMC(reg, v) adev->smc_wreg(adev, (reg), (v))
+#define RREG32_SMC(reg) amdgpu_reg_smc_rd32(adev, (reg))
+#define WREG32_SMC(reg, v) amdgpu_reg_smc_wr32(adev, (reg), (v))
 #define RREG32_UVD_CTX(reg) adev->uvd_ctx_rreg(adev, (reg))
 #define WREG32_UVD_CTX(reg, v) adev->uvd_ctx_wreg(adev, (reg), (v))
 #define RREG32_DIDT(reg) adev->didt_rreg(adev, (reg))

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
index f7467af2e102e77eb7e836bf974eea46c5c4e2b5..9b751138c418bb4d9a63677e0815c1f1e0ab2718 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
@@ -752,7 +752,7 @@ static ssize_t amdgpu_debugfs_regs_smc_read(struct file *f, char __user *buf,
        ssize_t result = 0;
        int r;
 
-       if (!adev->smc_rreg)
+       if (!adev->reg.smc.rreg)
                return -EOPNOTSUPP;
 
        if (size & 0x3 || *pos & 0x3)
@@ -810,7 +810,7 @@ static ssize_t amdgpu_debugfs_regs_smc_write(struct file *f, const char __user *
        ssize_t result = 0;
        int r;
 
-       if (!adev->smc_wreg)
+       if (!adev->reg.smc.wreg)
                return -EOPNOTSUPP;
 
        if (size & 0x3 || *pos & 0x3)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index c4b86cd7baf46de1b283980caf9b874fd5b8ea1d..f6ca7514a36d436f2e437e63020bd07e333d1e28 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -3830,8 +3830,8 @@ int amdgpu_device_init(struct amdgpu_device *adev,
        adev->fence_context = dma_fence_context_alloc(AMDGPU_MAX_RINGS);
        bitmap_zero(adev->gfx.pipe_reserve_bitmap, AMDGPU_MAX_COMPUTE_QUEUES);
 
-       adev->smc_rreg = &amdgpu_invalid_rreg;
-       adev->smc_wreg = &amdgpu_invalid_wreg;
+       amdgpu_reg_access_init(adev);
+
        adev->pcie_rreg = &amdgpu_invalid_rreg;
        adev->pcie_wreg = &amdgpu_invalid_wreg;
        adev->pcie_rreg_ext = &amdgpu_invalid_rreg_ext;
@@ -3894,7 +3894,6 @@ int amdgpu_device_init(struct amdgpu_device *adev,
                return r;
 
        spin_lock_init(&adev->mmio_idx_lock);
-       spin_lock_init(&adev->smc_idx_lock);
        spin_lock_init(&adev->pcie_idx_lock);
        spin_lock_init(&adev->uvd_ctx_idx_lock);
        spin_lock_init(&adev->didt_idx_lock);

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_reg_access.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_reg_access.c
index 4b8888f82b64bb3a370d3ccdb15a0744832f6011..5debc5c39101be506c1004a33d102d59d9198a81 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_reg_access.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_reg_access.c
@@ -33,6 +33,31 @@
 #define AMDGPU_PCIE_INDEX_HI_FALLBACK (0x44 >> 2)
 #define AMDGPU_PCIE_DATA_FALLBACK (0x3C >> 2)
 
+void amdgpu_reg_access_init(struct amdgpu_device *adev)
+{
+       spin_lock_init(&adev->reg.smc.lock);
+       adev->reg.smc.rreg = NULL;
+       adev->reg.smc.wreg = NULL;
+}
+
+uint32_t amdgpu_reg_smc_rd32(struct amdgpu_device *adev, uint32_t reg)
+{
+       if (!adev->reg.smc.rreg) {
+               dev_err_once(adev->dev, "SMC register read not supported\n");
+               return 0;
+       }
+       return adev->reg.smc.rreg(adev, reg);
+}
+
+void amdgpu_reg_smc_wr32(struct amdgpu_device *adev, uint32_t reg, uint32_t v)
+{
+       if (!adev->reg.smc.wreg) {
+               dev_err_once(adev->dev, "SMC register write not supported\n");
+               return;
+       }
+       adev->reg.smc.wreg(adev, reg, v);
+}
+
 /*
  * register access helper functions.
  */

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_reg_access.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_reg_access.h
index e03865c0c093fbddce7d5a7953c08974280b6b28..225d89eabed5135c490ae64151b6aac98362ed5e 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_reg_access.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_reg_access.h
@@ -25,15 +25,27 @@
 #define __AMDGPU_REG_ACCESS_H__
 
 #include <linux/types.h>
+#include <linux/spinlock.h>
 
 struct amdgpu_device;
 
-/*
- * Registers read & write functions.
- */
 typedef uint32_t (*amdgpu_rreg_t)(struct amdgpu_device *, uint32_t);
 typedef void (*amdgpu_wreg_t)(struct amdgpu_device *, uint32_t, uint32_t);
 
+struct amdgpu_reg_ind {
+       spinlock_t lock;
+       amdgpu_rreg_t rreg;
+       amdgpu_wreg_t wreg;
+};
+
+struct amdgpu_reg_access {
+       struct amdgpu_reg_ind smc;
+};
+
+void amdgpu_reg_access_init(struct amdgpu_device *adev);
+uint32_t amdgpu_reg_smc_rd32(struct amdgpu_device *adev, uint32_t reg);
+void amdgpu_reg_smc_wr32(struct amdgpu_device *adev, uint32_t reg, uint32_t v);
+
 typedef uint32_t (*amdgpu_rreg_ext_t)(struct amdgpu_device *, uint64_t);
 typedef void (*amdgpu_wreg_ext_t)(struct amdgpu_device *, uint64_t, uint32_t);
 

diff --git a/drivers/gpu/drm/amd/amdgpu/cik.c b/drivers/gpu/drm/amd/amdgpu/cik.c
index c081784a19c4f9a35fbf2eaca521e1260a118fe2..cd5cd5da4d47ee03554538e6d00205bfa01afb8d 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/cik.c
+++ b/drivers/gpu/drm/amd/amdgpu/cik.c
@@ -179,10 +179,10 @@ static u32 cik_smc_rreg(struct amdgpu_device *adev, u32 reg)
        unsigned long flags;
        u32 r;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        WREG32(mmSMC_IND_INDEX_0, (reg));
        r = RREG32(mmSMC_IND_DATA_0);
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
        return r;
 }
 
@@ -190,10 +190,10 @@ static void cik_smc_wreg(struct amdgpu_device *adev, u32 reg, u32 v)
 {
        unsigned long flags;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        WREG32(mmSMC_IND_INDEX_0, (reg));
        WREG32(mmSMC_IND_DATA_0, (v));
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 }
 
 static u32 cik_uvd_ctx_rreg(struct amdgpu_device *adev, u32 reg)
@@ -1027,7 +1027,7 @@ static bool cik_read_bios_from_rom(struct amdgpu_device *adev,
        dw_ptr = (u32 *)bios;
        length_dw = ALIGN(length_bytes, 4) / 4;
        /* take the smc lock since we are using the smc index */
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        /* set rom index to 0 */
        WREG32(mmSMC_IND_INDEX_0, ixROM_INDEX);
        WREG32(mmSMC_IND_DATA_0, 0);
@@ -1035,7 +1035,7 @@ static bool cik_read_bios_from_rom(struct amdgpu_device *adev,
        WREG32(mmSMC_IND_INDEX_0, ixROM_DATA);
        for (i = 0; i < length_dw; i++)
                dw_ptr[i] = RREG32(mmSMC_IND_DATA_0);
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 
        return true;
 }
@@ -1984,8 +1984,8 @@ static int cik_common_early_init(struct amdgpu_ip_block *ip_block)
 {
        struct amdgpu_device *adev = ip_block->adev;
 
-       adev->smc_rreg = &cik_smc_rreg;
-       adev->smc_wreg = &cik_smc_wreg;
+       adev->reg.smc.rreg = cik_smc_rreg;
+       adev->reg.smc.wreg = cik_smc_wreg;
        adev->pcie_rreg = &cik_pcie_rreg;
        adev->pcie_wreg = &cik_pcie_wreg;
        adev->uvd_ctx_rreg = &cik_uvd_ctx_rreg;

diff --git a/drivers/gpu/drm/amd/amdgpu/nv.c b/drivers/gpu/drm/amd/amdgpu/nv.c
index f17c3839aea195d33032af2d532f8ae69aa49431..2e7cd27e45c97c6bd7df49bfd3f60be65bffc51e 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/nv.c
+++ b/drivers/gpu/drm/amd/amdgpu/nv.c
@@ -635,8 +635,6 @@ static int nv_common_early_init(struct amdgpu_ip_block *ip_block)
        struct amdgpu_device *adev = ip_block->adev;
 
        adev->nbio.funcs->set_reg_remap(adev);
-       adev->smc_rreg = NULL;
-       adev->smc_wreg = NULL;
        adev->pcie_rreg = &amdgpu_device_indirect_rreg;
        adev->pcie_wreg = &amdgpu_device_indirect_wreg;
        adev->pcie_rreg64 = &amdgpu_device_indirect_rreg64;

diff --git a/drivers/gpu/drm/amd/amdgpu/si.c b/drivers/gpu/drm/amd/amdgpu/si.c
index 509d43b238f3fb5dadb742ceb11c9fa4105a74bc..cbdf8a1c6511b55b9c801433a9a77cdae24b21b1 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/si.c
+++ b/drivers/gpu/drm/amd/amdgpu/si.c
@@ -1077,10 +1077,10 @@ static u32 si_smc_rreg(struct amdgpu_device *adev, u32 reg)
        unsigned long flags;
        u32 r;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        WREG32(mmSMC_IND_INDEX_0, (reg));
        r = RREG32(mmSMC_IND_DATA_0);
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
        return r;
 }
 
@@ -1088,10 +1088,10 @@ static void si_smc_wreg(struct amdgpu_device *adev, u32 reg, u32 v)
 {
        unsigned long flags;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        WREG32(mmSMC_IND_INDEX_0, (reg));
        WREG32(mmSMC_IND_DATA_0, (v));
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 }
 
 static u32 si_uvd_ctx_rreg(struct amdgpu_device *adev, u32 reg)
@@ -2037,8 +2037,8 @@ static int si_common_early_init(struct amdgpu_ip_block *ip_block)
 {
        struct amdgpu_device *adev = ip_block->adev;
 
-       adev->smc_rreg = &si_smc_rreg;
-       adev->smc_wreg = &si_smc_wreg;
+       adev->reg.smc.rreg = si_smc_rreg;
+       adev->reg.smc.wreg = si_smc_wreg;
        adev->pcie_rreg = &si_pcie_rreg;
        adev->pcie_wreg = &si_pcie_wreg;
        adev->pciep_rreg = &si_pciep_rreg;

diff --git a/drivers/gpu/drm/amd/amdgpu/soc15.c b/drivers/gpu/drm/amd/amdgpu/soc15.c
index 4e037a6978f09343578675030dde0a43faf4d5f6..057787ffd19c94ee78357e2729eae5317279f40d 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/soc15.c
+++ b/drivers/gpu/drm/amd/amdgpu/soc15.c
@@ -961,8 +961,6 @@ static int soc15_common_early_init(struct amdgpu_ip_block *ip_block)
        struct amdgpu_device *adev = ip_block->adev;
 
        adev->nbio.funcs->set_reg_remap(adev);
-       adev->smc_rreg = NULL;
-       adev->smc_wreg = NULL;
        adev->pcie_rreg = &amdgpu_device_indirect_rreg;
        adev->pcie_wreg = &amdgpu_device_indirect_wreg;
        adev->pcie_rreg_ext = &amdgpu_device_indirect_rreg_ext;

diff --git a/drivers/gpu/drm/amd/amdgpu/soc21.c b/drivers/gpu/drm/amd/amdgpu/soc21.c
index a0ad1f8a76f06ab49e98b9b500688ff43955ad95..4d4c1adf00d1e8e97d29ee60406dce2a849753d9 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/soc21.c
+++ b/drivers/gpu/drm/amd/amdgpu/soc21.c
@@ -589,8 +589,6 @@ static int soc21_common_early_init(struct amdgpu_ip_block *ip_block)
        struct amdgpu_device *adev = ip_block->adev;
 
        adev->nbio.funcs->set_reg_remap(adev);
-       adev->smc_rreg = NULL;
-       adev->smc_wreg = NULL;
        adev->pcie_rreg = &amdgpu_device_indirect_rreg;
        adev->pcie_wreg = &amdgpu_device_indirect_wreg;
        adev->pcie_rreg64 = &amdgpu_device_indirect_rreg64;

diff --git a/drivers/gpu/drm/amd/amdgpu/soc24.c b/drivers/gpu/drm/amd/amdgpu/soc24.c
index ecb6c3fcfbd15c311bd663161018e025e7166148..867cc4fdc98fe0166e2f69db1eeb65dbf9380732 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/soc24.c
+++ b/drivers/gpu/drm/amd/amdgpu/soc24.c
@@ -362,8 +362,6 @@ static int soc24_common_early_init(struct amdgpu_ip_block *ip_block)
        struct amdgpu_device *adev = ip_block->adev;
 
        adev->nbio.funcs->set_reg_remap(adev);
-       adev->smc_rreg = NULL;
-       adev->smc_wreg = NULL;
        adev->pcie_rreg = &amdgpu_device_indirect_rreg;
        adev->pcie_wreg = &amdgpu_device_indirect_wreg;
        adev->pcie_rreg64 = &amdgpu_device_indirect_rreg64;

diff --git a/drivers/gpu/drm/amd/amdgpu/soc_v1_0.c b/drivers/gpu/drm/amd/amdgpu/soc_v1_0.c
index 59ab952d5cce4ef77d13f81844936261670112c8..f8a49424adeb4caf3e74e9274da0bc3271f8af0c 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/soc_v1_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/soc_v1_0.c
@@ -250,8 +250,6 @@ static int soc_v1_0_common_early_init(struct amdgpu_ip_block *ip_block)
 {
        struct amdgpu_device *adev = ip_block->adev;
 
-       adev->smc_rreg = NULL;
-       adev->smc_wreg = NULL;
        adev->pcie_rreg = &amdgpu_device_indirect_rreg;
        adev->pcie_wreg = &amdgpu_device_indirect_wreg;
        adev->pcie_rreg_ext = &amdgpu_device_indirect_rreg_ext;

diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c
index 6a574b6c8e631a7ec57af09df9ce2cfff577c5a2..925cc275fe849241efc27678d182f25d4581a5e9 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/vi.c
+++ b/drivers/gpu/drm/amd/amdgpu/vi.c
@@ -324,10 +324,10 @@ static u32 vi_smc_rreg(struct amdgpu_device *adev, u32 reg)
        unsigned long flags;
        u32 r;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        WREG32_NO_KIQ(mmSMC_IND_INDEX_11, (reg));
        r = RREG32_NO_KIQ(mmSMC_IND_DATA_11);
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
        return r;
 }
 
@@ -335,10 +335,10 @@ static void vi_smc_wreg(struct amdgpu_device *adev, u32 reg, u32 v)
 {
        unsigned long flags;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        WREG32_NO_KIQ(mmSMC_IND_INDEX_11, (reg));
        WREG32_NO_KIQ(mmSMC_IND_DATA_11, (v));
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 }
 
 /* smu_8_0_d.h */
@@ -350,10 +350,10 @@ static u32 cz_smc_rreg(struct amdgpu_device *adev, u32 reg)
        unsigned long flags;
        u32 r;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        WREG32(mmMP0PUB_IND_INDEX, (reg));
        r = RREG32(mmMP0PUB_IND_DATA);
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
        return r;
 }
 
@@ -361,10 +361,10 @@ static void cz_smc_wreg(struct amdgpu_device *adev, u32 reg, u32 v)
 {
        unsigned long flags;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        WREG32(mmMP0PUB_IND_INDEX, (reg));
        WREG32(mmMP0PUB_IND_DATA, (v));
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 }
 
 static u32 vi_uvd_ctx_rreg(struct amdgpu_device *adev, u32 reg)
@@ -649,7 +649,7 @@ static bool vi_read_bios_from_rom(struct amdgpu_device *adev,
        dw_ptr = (u32 *)bios;
        length_dw = ALIGN(length_bytes, 4) / 4;
        /* take the smc lock since we are using the smc index */
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        /* set rom index to 0 */
        WREG32(mmSMC_IND_INDEX_11, ixROM_INDEX);
        WREG32(mmSMC_IND_DATA_11, 0);
@@ -657,7 +657,7 @@ static bool vi_read_bios_from_rom(struct amdgpu_device *adev,
        WREG32(mmSMC_IND_INDEX_11, ixROM_DATA);
        for (i = 0; i < length_dw; i++)
                dw_ptr[i] = RREG32(mmSMC_IND_DATA_11);
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 
        return true;
 }
@@ -1454,11 +1454,11 @@ static int vi_common_early_init(struct amdgpu_ip_block *ip_block)
        struct amdgpu_device *adev = ip_block->adev;
 
        if (adev->flags & AMD_IS_APU) {
-               adev->smc_rreg = &cz_smc_rreg;
-               adev->smc_wreg = &cz_smc_wreg;
+               adev->reg.smc.rreg = cz_smc_rreg;
+               adev->reg.smc.wreg = cz_smc_wreg;
        } else {
-               adev->smc_rreg = &vi_smc_rreg;
-               adev->smc_wreg = &vi_smc_wreg;
+               adev->reg.smc.rreg = vi_smc_rreg;
+               adev->reg.smc.wreg = vi_smc_wreg;
        }
        adev->pcie_rreg = &vi_pcie_rreg;
        adev->pcie_wreg = &vi_pcie_wreg;

diff --git a/drivers/gpu/drm/amd/pm/legacy-dpm/si_smc.c b/drivers/gpu/drm/amd/pm/legacy-dpm/si_smc.c
index 281a5e377aee416c7fc141a117fb113577c5da90..e1c509bfc39020291e9f79e130d03c39f525cb18 100644 (file)
--- a/drivers/gpu/drm/amd/pm/legacy-dpm/si_smc.c
+++ b/drivers/gpu/drm/amd/pm/legacy-dpm/si_smc.c
@@ -65,7 +65,7 @@ int amdgpu_si_copy_bytes_to_smc(struct amdgpu_device *adev,
 
        addr = smc_start_address;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        while (byte_count >= 4) {
                /* SMC address space is BE */
                data = (src[0] << 24) | (src[1] << 16) | (src[2] << 8) | src[3];
@@ -109,7 +109,7 @@ int amdgpu_si_copy_bytes_to_smc(struct amdgpu_device *adev,
        }
 
 done:
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 
        return ret;
 }
@@ -252,7 +252,7 @@ int amdgpu_si_load_smc_ucode(struct amdgpu_device *adev, u32 limit)
        if (ucode_size & 3)
                return -EINVAL;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        WREG32(mmSMC_IND_INDEX_0, ucode_start_address);
        WREG32_P(mmSMC_IND_ACCESS_CNTL, SMC_IND_ACCESS_CNTL__AUTO_INCREMENT_IND_0_MASK, ~SMC_IND_ACCESS_CNTL__AUTO_INCREMENT_IND_0_MASK);
        while (ucode_size >= 4) {
@@ -265,7 +265,7 @@ int amdgpu_si_load_smc_ucode(struct amdgpu_device *adev, u32 limit)
                ucode_size -= 4;
        }
        WREG32_P(mmSMC_IND_ACCESS_CNTL, 0, ~SMC_IND_ACCESS_CNTL__AUTO_INCREMENT_IND_0_MASK);
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 
        return 0;
 }
@@ -276,11 +276,11 @@ int amdgpu_si_read_smc_sram_dword(struct amdgpu_device *adev, u32 smc_address,
        unsigned long flags;
        int ret;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        ret = si_set_smc_sram_address(adev, smc_address, limit);
        if (ret == 0)
                *value = RREG32(mmSMC_IND_DATA_0);
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 
        return ret;
 }
@@ -291,11 +291,11 @@ int amdgpu_si_write_smc_sram_dword(struct amdgpu_device *adev, u32 smc_address,
        unsigned long flags;
        int ret;
 
-       spin_lock_irqsave(&adev->smc_idx_lock, flags);
+       spin_lock_irqsave(&adev->reg.smc.lock, flags);
        ret = si_set_smc_sram_address(adev, smc_address, limit);
        if (ret == 0)
                WREG32(mmSMC_IND_DATA_0, value);
-       spin_unlock_irqrestore(&adev->smc_idx_lock, flags);
+       spin_unlock_irqrestore(&adev->reg.smc.lock, flags);
 
        return ret;
 }