On 9/21/2021 11:37 PM, Alex Deucher wrote:
Allow us to query instances versions more cleanly.

Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
---
  drivers/gpu/drm/amd/amdgpu/amdgpu.h           |   2 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 262 +++++++++---------
  drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c       |  34 +--
  drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c       |   4 +-
  drivers/gpu/drm/amd/amdgpu/athub_v2_0.c       |   2 +-
  drivers/gpu/drm/amd/amdgpu/athub_v2_1.c       |   2 +-
  drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c        |  80 +++---
  drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c         |  72 ++---
  drivers/gpu/drm/amd/amdgpu/gfxhub_v2_1.c      |   4 +-
  drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c        |  16 +-
  drivers/gpu/drm/amd/amdgpu/hdp_v4_0.c         |  14 +-
  drivers/gpu/drm/amd/amdgpu/mmhub_v2_0.c       |  14 +-
  drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c       |   2 +-
  drivers/gpu/drm/amd/amdgpu/navi10_ih.c        |   4 +-
  drivers/gpu/drm/amd/amdgpu/nv.c               |   8 +-
  drivers/gpu/drm/amd/amdgpu/psp_v11_0.c        |   4 +-
  drivers/gpu/drm/amd/amdgpu/psp_v13_0.c        |   4 +-
  drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c        |  34 +--
  drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c        |   8 +-
  drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c        |  10 +-
  drivers/gpu/drm/amd/amdgpu/soc15.c            |  24 +-
  drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c         |   4 +-
  drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c         |   8 +-
  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |  26 +-
  drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c     |  52 ++--
  .../gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c   |  32 +--
  .../amd/pm/swsmu/smu11/sienna_cichlid_ppt.c   |  24 +-
  .../gpu/drm/amd/pm/swsmu/smu11/smu_v11_0.c    |  28 +-
  .../gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c    |  10 +-
  29 files changed, 394 insertions(+), 394 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h 
b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
index b153c3740307..f4bceb2624fb 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
@@ -1096,7 +1096,7 @@ struct amdgpu_device {
        struct pci_saved_state          *pci_state;
struct amdgpu_reset_control *reset_cntl;
-       uint32_t                        ip_versions[HW_ID_MAX];
+       uint32_t                        
ip_versions[HW_ID_MAX][HWIP_MAX_INSTANCE];

Here is the confusion. "ip_versions" is used like this - adev->ip_versions[MMHUB_HWIP] - which means the first dimension can max out at MAX_HWIP. So why is the array maxed to HW_ID_MAX? HWID looks like a completely different thing.

Got this confusion while reviewing patch 52. Besides, this change doesn't go well with that patch because HWIP is really replicating instances.

Thanks,
Lijo

  };
static inline struct amdgpu_device *drm_to_adev(struct drm_device *ddev)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c
index 9d8aea39f36a..eff348dee9e9 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c
@@ -369,7 +369,7 @@ int amdgpu_discovery_reg_base_init(struct amdgpu_device 
*adev)
                                                        
hw_id_names[le16_to_cpu(ip->hw_id)]);
                                        
adev->reg_offset[hw_ip][ip->number_instance] =
                                                ip->base_address;
-                                       adev->ip_versions[hw_ip + 
ip->number_instance] =
+                                       
adev->ip_versions[hw_ip][ip->number_instance] =
                                                
amdgpu_discovery_convert_version(ip->major,
                                                                                 
ip->minor,
                                                                                 
ip->revision);
@@ -525,139 +525,139 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
        case CHIP_VEGA10:
                vega10_reg_base_init(adev);
                adev->sdma.num_instances = 2;
-               adev->ip_versions[MMHUB_HWIP] = IP_VERSION(9, 0, 0);
-               adev->ip_versions[ATHUB_HWIP] = IP_VERSION(9, 0, 0);
-               adev->ip_versions[OSSSYS_HWIP] = IP_VERSION(4, 0, 0);
-               adev->ip_versions[HDP_HWIP] = IP_VERSION(4, 0, 0);
-               adev->ip_versions[SDMA0_HWIP] = IP_VERSION(4, 0, 0);
-               adev->ip_versions[DF_HWIP] = IP_VERSION(2, 1, 0);
-               adev->ip_versions[NBIO_HWIP] = IP_VERSION(6, 1, 0);
-               adev->ip_versions[UMC_HWIP] = IP_VERSION(6, 0, 0);
-               adev->ip_versions[MP0_HWIP] = IP_VERSION(9, 0, 0);
-               adev->ip_versions[MP1_HWIP] = IP_VERSION(9, 0, 0);
-               adev->ip_versions[THM_HWIP] = IP_VERSION(9, 0, 0);
-               adev->ip_versions[SMUIO_HWIP] = IP_VERSION(9, 0, 0);
-               adev->ip_versions[GC_HWIP] = IP_VERSION(9, 0, 1);
-               adev->ip_versions[UVD_HWIP] = IP_VERSION(7, 0, 0);
-               adev->ip_versions[VCE_HWIP] = IP_VERSION(4, 0, 0);
-               adev->ip_versions[DCI_HWIP] = IP_VERSION(12, 0, 0);
+               adev->ip_versions[MMHUB_HWIP][0] = IP_VERSION(9, 0, 0);
+               adev->ip_versions[ATHUB_HWIP][0] = IP_VERSION(9, 0, 0);
+               adev->ip_versions[OSSSYS_HWIP][0] = IP_VERSION(4, 0, 0);
+               adev->ip_versions[HDP_HWIP][0] = IP_VERSION(4, 0, 0);
+               adev->ip_versions[SDMA0_HWIP][0] = IP_VERSION(4, 0, 0);
+               adev->ip_versions[DF_HWIP][0] = IP_VERSION(2, 1, 0);
+               adev->ip_versions[NBIO_HWIP][0] = IP_VERSION(6, 1, 0);
+               adev->ip_versions[UMC_HWIP][0] = IP_VERSION(6, 0, 0);
+               adev->ip_versions[MP0_HWIP][0] = IP_VERSION(9, 0, 0);
+               adev->ip_versions[MP1_HWIP][0] = IP_VERSION(9, 0, 0);
+               adev->ip_versions[THM_HWIP][0] = IP_VERSION(9, 0, 0);
+               adev->ip_versions[SMUIO_HWIP][0] = IP_VERSION(9, 0, 0);
+               adev->ip_versions[GC_HWIP][0] = IP_VERSION(9, 0, 1);
+               adev->ip_versions[UVD_HWIP][0] = IP_VERSION(7, 0, 0);
+               adev->ip_versions[VCE_HWIP][0] = IP_VERSION(4, 0, 0);
+               adev->ip_versions[DCI_HWIP][0] = IP_VERSION(12, 0, 0);
                break;
        case CHIP_VEGA12:
                vega10_reg_base_init(adev);
                adev->sdma.num_instances = 2;
-               adev->ip_versions[MMHUB_HWIP] = IP_VERSION(9, 3, 0);
-               adev->ip_versions[ATHUB_HWIP] = IP_VERSION(9, 3, 0);
-               adev->ip_versions[OSSSYS_HWIP] = IP_VERSION(4, 0, 1);
-               adev->ip_versions[HDP_HWIP] = IP_VERSION(4, 0, 1);
-               adev->ip_versions[SDMA0_HWIP] = IP_VERSION(4, 0, 1);
-               adev->ip_versions[DF_HWIP] = IP_VERSION(2, 5, 0);
-               adev->ip_versions[NBIO_HWIP] = IP_VERSION(6, 2, 0);
-               adev->ip_versions[UMC_HWIP] = IP_VERSION(6, 1, 0);
-               adev->ip_versions[MP0_HWIP] = IP_VERSION(9, 0, 0);
-               adev->ip_versions[MP1_HWIP] = IP_VERSION(9, 0, 0);
-               adev->ip_versions[THM_HWIP] = IP_VERSION(9, 0, 0);
-               adev->ip_versions[SMUIO_HWIP] = IP_VERSION(9, 0, 1);
-               adev->ip_versions[GC_HWIP] = IP_VERSION(9, 2, 1);
-               adev->ip_versions[UVD_HWIP] = IP_VERSION(7, 0, 0);
-               adev->ip_versions[VCE_HWIP] = IP_VERSION(4, 0, 0);
-               adev->ip_versions[DCI_HWIP] = IP_VERSION(12, 0, 1);
+               adev->ip_versions[MMHUB_HWIP][0] = IP_VERSION(9, 3, 0);
+               adev->ip_versions[ATHUB_HWIP][0] = IP_VERSION(9, 3, 0);
+               adev->ip_versions[OSSSYS_HWIP][0] = IP_VERSION(4, 0, 1);
+               adev->ip_versions[HDP_HWIP][0] = IP_VERSION(4, 0, 1);
+               adev->ip_versions[SDMA0_HWIP][0] = IP_VERSION(4, 0, 1);
+               adev->ip_versions[DF_HWIP][0] = IP_VERSION(2, 5, 0);
+               adev->ip_versions[NBIO_HWIP][0] = IP_VERSION(6, 2, 0);
+               adev->ip_versions[UMC_HWIP][0] = IP_VERSION(6, 1, 0);
+               adev->ip_versions[MP0_HWIP][0] = IP_VERSION(9, 0, 0);
+               adev->ip_versions[MP1_HWIP][0] = IP_VERSION(9, 0, 0);
+               adev->ip_versions[THM_HWIP][0] = IP_VERSION(9, 0, 0);
+               adev->ip_versions[SMUIO_HWIP][0] = IP_VERSION(9, 0, 1);
+               adev->ip_versions[GC_HWIP][0] = IP_VERSION(9, 2, 1);
+               adev->ip_versions[UVD_HWIP][0] = IP_VERSION(7, 0, 0);
+               adev->ip_versions[VCE_HWIP][0] = IP_VERSION(4, 0, 0);
+               adev->ip_versions[DCI_HWIP][0] = IP_VERSION(12, 0, 1);
                break;
        case CHIP_RAVEN:
                vega10_reg_base_init(adev);
                adev->sdma.num_instances = 1;
                adev->vcn.num_vcn_inst = 1;
                if (adev->apu_flags & AMD_APU_IS_RAVEN2) {
-                       adev->ip_versions[MMHUB_HWIP] = IP_VERSION(9, 2, 0);
-                       adev->ip_versions[ATHUB_HWIP] = IP_VERSION(9, 2, 0);
-                       adev->ip_versions[OSSSYS_HWIP] = IP_VERSION(4, 1, 1);
-                       adev->ip_versions[HDP_HWIP] = IP_VERSION(4, 1, 1);
-                       adev->ip_versions[SDMA0_HWIP] = IP_VERSION(4, 1, 1);
-                       adev->ip_versions[DF_HWIP] = IP_VERSION(2, 1, 1);
-                       adev->ip_versions[NBIO_HWIP] = IP_VERSION(7, 0, 1);
-                       adev->ip_versions[UMC_HWIP] = IP_VERSION(7, 5, 0);
-                       adev->ip_versions[MP0_HWIP] = IP_VERSION(10, 0, 1);
-                       adev->ip_versions[MP1_HWIP] = IP_VERSION(10, 0, 1);
-                       adev->ip_versions[THM_HWIP] = IP_VERSION(10, 1, 0);
-                       adev->ip_versions[SMUIO_HWIP] = IP_VERSION(10, 0, 1);
-                       adev->ip_versions[GC_HWIP] = IP_VERSION(9, 2, 2);
-                       adev->ip_versions[UVD_HWIP] = IP_VERSION(1, 0, 1);
-                       adev->ip_versions[DCE_HWIP] = IP_VERSION(1, 0, 1);
+                       adev->ip_versions[MMHUB_HWIP][0] = IP_VERSION(9, 2, 0);
+                       adev->ip_versions[ATHUB_HWIP][0] = IP_VERSION(9, 2, 0);
+                       adev->ip_versions[OSSSYS_HWIP][0] = IP_VERSION(4, 1, 1);
+                       adev->ip_versions[HDP_HWIP][0] = IP_VERSION(4, 1, 1);
+                       adev->ip_versions[SDMA0_HWIP][0] = IP_VERSION(4, 1, 1);
+                       adev->ip_versions[DF_HWIP][0] = IP_VERSION(2, 1, 1);
+                       adev->ip_versions[NBIO_HWIP][0] = IP_VERSION(7, 0, 1);
+                       adev->ip_versions[UMC_HWIP][0] = IP_VERSION(7, 5, 0);
+                       adev->ip_versions[MP0_HWIP][0] = IP_VERSION(10, 0, 1);
+                       adev->ip_versions[MP1_HWIP][0] = IP_VERSION(10, 0, 1);
+                       adev->ip_versions[THM_HWIP][0] = IP_VERSION(10, 1, 0);
+                       adev->ip_versions[SMUIO_HWIP][0] = IP_VERSION(10, 0, 1);
+                       adev->ip_versions[GC_HWIP][0] = IP_VERSION(9, 2, 2);
+                       adev->ip_versions[UVD_HWIP][0] = IP_VERSION(1, 0, 1);
+                       adev->ip_versions[DCE_HWIP][0] = IP_VERSION(1, 0, 1);
                } else {
-                       adev->ip_versions[MMHUB_HWIP] = IP_VERSION(9, 1, 0);
-                       adev->ip_versions[ATHUB_HWIP] = IP_VERSION(9, 1, 0);
-                       adev->ip_versions[OSSSYS_HWIP] = IP_VERSION(4, 1, 0);
-                       adev->ip_versions[HDP_HWIP] = IP_VERSION(4, 1, 0);
-                       adev->ip_versions[SDMA0_HWIP] = IP_VERSION(4, 1, 0);
-                       adev->ip_versions[DF_HWIP] = IP_VERSION(2, 1, 0);
-                       adev->ip_versions[NBIO_HWIP] = IP_VERSION(7, 0, 0);
-                       adev->ip_versions[UMC_HWIP] = IP_VERSION(7, 0, 0);
-                       adev->ip_versions[MP0_HWIP] = IP_VERSION(10, 0, 0);
-                       adev->ip_versions[MP1_HWIP] = IP_VERSION(10, 0, 0);
-                       adev->ip_versions[THM_HWIP] = IP_VERSION(10, 0, 0);
-                       adev->ip_versions[SMUIO_HWIP] = IP_VERSION(10, 0, 0);
-                       adev->ip_versions[GC_HWIP] = IP_VERSION(9, 1, 0);
-                       adev->ip_versions[UVD_HWIP] = IP_VERSION(1, 0, 0);
-                       adev->ip_versions[DCE_HWIP] = IP_VERSION(1, 0, 0);
+                       adev->ip_versions[MMHUB_HWIP][0] = IP_VERSION(9, 1, 0);
+                       adev->ip_versions[ATHUB_HWIP][0] = IP_VERSION(9, 1, 0);
+                       adev->ip_versions[OSSSYS_HWIP][0] = IP_VERSION(4, 1, 0);
+                       adev->ip_versions[HDP_HWIP][0] = IP_VERSION(4, 1, 0);
+                       adev->ip_versions[SDMA0_HWIP][0] = IP_VERSION(4, 1, 0);
+                       adev->ip_versions[DF_HWIP][0] = IP_VERSION(2, 1, 0);
+                       adev->ip_versions[NBIO_HWIP][0] = IP_VERSION(7, 0, 0);
+                       adev->ip_versions[UMC_HWIP][0] = IP_VERSION(7, 0, 0);
+                       adev->ip_versions[MP0_HWIP][0] = IP_VERSION(10, 0, 0);
+                       adev->ip_versions[MP1_HWIP][0] = IP_VERSION(10, 0, 0);
+                       adev->ip_versions[THM_HWIP][0] = IP_VERSION(10, 0, 0);
+                       adev->ip_versions[SMUIO_HWIP][0] = IP_VERSION(10, 0, 0);
+                       adev->ip_versions[GC_HWIP][0] = IP_VERSION(9, 1, 0);
+                       adev->ip_versions[UVD_HWIP][0] = IP_VERSION(1, 0, 0);
+                       adev->ip_versions[DCE_HWIP][0] = IP_VERSION(1, 0, 0);
                }
                break;
        case CHIP_VEGA20:
                vega20_reg_base_init(adev);
                adev->sdma.num_instances = 2;
-               adev->ip_versions[MMHUB_HWIP] = IP_VERSION(9, 4, 0);
-               adev->ip_versions[ATHUB_HWIP] = IP_VERSION(9, 4, 0);
-               adev->ip_versions[OSSSYS_HWIP] = IP_VERSION(4, 2, 0);
-               adev->ip_versions[HDP_HWIP] = IP_VERSION(4, 2, 0);
-               adev->ip_versions[SDMA0_HWIP] = IP_VERSION(4, 2, 0);
-               adev->ip_versions[DF_HWIP] = IP_VERSION(3, 6, 0);
-               adev->ip_versions[NBIO_HWIP] = IP_VERSION(7, 4, 0);
-               adev->ip_versions[UMC_HWIP] = IP_VERSION(6, 1, 1);
-               adev->ip_versions[MP0_HWIP] = IP_VERSION(11, 0, 2);
-               adev->ip_versions[MP1_HWIP] = IP_VERSION(11, 0, 2);
-               adev->ip_versions[THM_HWIP] = IP_VERSION(11, 0, 2);
-               adev->ip_versions[SMUIO_HWIP] = IP_VERSION(11, 0, 2);
-               adev->ip_versions[GC_HWIP] = IP_VERSION(9, 4, 0);
-               adev->ip_versions[UVD_HWIP] = IP_VERSION(7, 2, 0);
-               adev->ip_versions[VCE_HWIP] = IP_VERSION(4, 1, 0);
-               adev->ip_versions[DCI_HWIP] = IP_VERSION(12, 1, 0);
+               adev->ip_versions[MMHUB_HWIP][0] = IP_VERSION(9, 4, 0);
+               adev->ip_versions[ATHUB_HWIP][0] = IP_VERSION(9, 4, 0);
+               adev->ip_versions[OSSSYS_HWIP][0] = IP_VERSION(4, 2, 0);
+               adev->ip_versions[HDP_HWIP][0] = IP_VERSION(4, 2, 0);
+               adev->ip_versions[SDMA0_HWIP][0] = IP_VERSION(4, 2, 0);
+               adev->ip_versions[DF_HWIP][0] = IP_VERSION(3, 6, 0);
+               adev->ip_versions[NBIO_HWIP][0] = IP_VERSION(7, 4, 0);
+               adev->ip_versions[UMC_HWIP][0] = IP_VERSION(6, 1, 1);
+               adev->ip_versions[MP0_HWIP][0] = IP_VERSION(11, 0, 2);
+               adev->ip_versions[MP1_HWIP][0] = IP_VERSION(11, 0, 2);
+               adev->ip_versions[THM_HWIP][0] = IP_VERSION(11, 0, 2);
+               adev->ip_versions[SMUIO_HWIP][0] = IP_VERSION(11, 0, 2);
+               adev->ip_versions[GC_HWIP][0] = IP_VERSION(9, 4, 0);
+               adev->ip_versions[UVD_HWIP][0] = IP_VERSION(7, 2, 0);
+               adev->ip_versions[VCE_HWIP][0] = IP_VERSION(4, 1, 0);
+               adev->ip_versions[DCI_HWIP][0] = IP_VERSION(12, 1, 0);
                break;
        case CHIP_ARCTURUS:
                arct_reg_base_init(adev);
                adev->sdma.num_instances = 8;
                adev->vcn.num_vcn_inst = 2;
-               adev->ip_versions[MMHUB_HWIP] = IP_VERSION(9, 4, 1);
-               adev->ip_versions[ATHUB_HWIP] = IP_VERSION(9, 4, 1);
-               adev->ip_versions[OSSSYS_HWIP] = IP_VERSION(4, 2, 1);
-               adev->ip_versions[HDP_HWIP] = IP_VERSION(4, 2, 1);
-               adev->ip_versions[SDMA0_HWIP] = IP_VERSION(4, 2, 2);
-               adev->ip_versions[DF_HWIP] = IP_VERSION(3, 6, 1);
-               adev->ip_versions[NBIO_HWIP] = IP_VERSION(7, 4, 1);
-               adev->ip_versions[UMC_HWIP] = IP_VERSION(6, 1, 2);
-               adev->ip_versions[MP0_HWIP] = IP_VERSION(11, 0, 4);
-               adev->ip_versions[MP1_HWIP] = IP_VERSION(11, 0, 2);
-               adev->ip_versions[THM_HWIP] = IP_VERSION(11, 0, 3);
-               adev->ip_versions[SMUIO_HWIP] = IP_VERSION(11, 0, 3);
-               adev->ip_versions[GC_HWIP] = IP_VERSION(9, 4, 1);
-               adev->ip_versions[UVD_HWIP] = IP_VERSION(2, 5, 0);
+               adev->ip_versions[MMHUB_HWIP][0] = IP_VERSION(9, 4, 1);
+               adev->ip_versions[ATHUB_HWIP][0] = IP_VERSION(9, 4, 1);
+               adev->ip_versions[OSSSYS_HWIP][0] = IP_VERSION(4, 2, 1);
+               adev->ip_versions[HDP_HWIP][0] = IP_VERSION(4, 2, 1);
+               adev->ip_versions[SDMA0_HWIP][0] = IP_VERSION(4, 2, 2);
+               adev->ip_versions[DF_HWIP][0] = IP_VERSION(3, 6, 1);
+               adev->ip_versions[NBIO_HWIP][0] = IP_VERSION(7, 4, 1);
+               adev->ip_versions[UMC_HWIP][0] = IP_VERSION(6, 1, 2);
+               adev->ip_versions[MP0_HWIP][0] = IP_VERSION(11, 0, 4);
+               adev->ip_versions[MP1_HWIP][0] = IP_VERSION(11, 0, 2);
+               adev->ip_versions[THM_HWIP][0] = IP_VERSION(11, 0, 3);
+               adev->ip_versions[SMUIO_HWIP][0] = IP_VERSION(11, 0, 3);
+               adev->ip_versions[GC_HWIP][0] = IP_VERSION(9, 4, 1);
+               adev->ip_versions[UVD_HWIP][0] = IP_VERSION(2, 5, 0);
                break;
        case CHIP_ALDEBARAN:
                aldebaran_reg_base_init(adev);
                adev->sdma.num_instances = 5;
                adev->vcn.num_vcn_inst = 2;
-               adev->ip_versions[MMHUB_HWIP] = IP_VERSION(9, 4, 2);
-               adev->ip_versions[ATHUB_HWIP] = IP_VERSION(9, 4, 2);
-               adev->ip_versions[OSSSYS_HWIP] = IP_VERSION(4, 4, 0);
-               adev->ip_versions[HDP_HWIP] = IP_VERSION(4, 4, 0);
-               adev->ip_versions[SDMA0_HWIP] = IP_VERSION(4, 4, 0);
-               adev->ip_versions[DF_HWIP] = IP_VERSION(3, 6, 2);
-               adev->ip_versions[NBIO_HWIP] = IP_VERSION(7, 4, 4);
-               adev->ip_versions[UMC_HWIP] = IP_VERSION(6, 7, 0);
-               adev->ip_versions[MP0_HWIP] = IP_VERSION(13, 0, 2);
-               adev->ip_versions[MP1_HWIP] = IP_VERSION(13, 0, 2);
-               adev->ip_versions[THM_HWIP] = IP_VERSION(13, 0, 2);
-               adev->ip_versions[SMUIO_HWIP] = IP_VERSION(13, 0, 2);
-               adev->ip_versions[GC_HWIP] = IP_VERSION(9, 4, 2);
-               adev->ip_versions[UVD_HWIP] = IP_VERSION(2, 6, 0);
-               adev->ip_versions[XGMI_HWIP] = IP_VERSION(6, 1, 0);
+               adev->ip_versions[MMHUB_HWIP][0] = IP_VERSION(9, 4, 2);
+               adev->ip_versions[ATHUB_HWIP][0] = IP_VERSION(9, 4, 2);
+               adev->ip_versions[OSSSYS_HWIP][0] = IP_VERSION(4, 4, 0);
+               adev->ip_versions[HDP_HWIP][0] = IP_VERSION(4, 4, 0);
+               adev->ip_versions[SDMA0_HWIP][0] = IP_VERSION(4, 4, 0);
+               adev->ip_versions[DF_HWIP][0] = IP_VERSION(3, 6, 2);
+               adev->ip_versions[NBIO_HWIP][0] = IP_VERSION(7, 4, 4);
+               adev->ip_versions[UMC_HWIP][0] = IP_VERSION(6, 7, 0);
+               adev->ip_versions[MP0_HWIP][0] = IP_VERSION(13, 0, 2);
+               adev->ip_versions[MP1_HWIP][0] = IP_VERSION(13, 0, 2);
+               adev->ip_versions[THM_HWIP][0] = IP_VERSION(13, 0, 2);
+               adev->ip_versions[SMUIO_HWIP][0] = IP_VERSION(13, 0, 2);
+               adev->ip_versions[GC_HWIP][0] = IP_VERSION(9, 4, 2);
+               adev->ip_versions[UVD_HWIP][0] = IP_VERSION(2, 6, 0);
+               adev->ip_versions[XGMI_HWIP][0] = IP_VERSION(6, 1, 0);
                break;
        default:
                r = amdgpu_discovery_reg_base_init(adev);
@@ -673,7 +673,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                break;
        }
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 2, 1):
        case IP_VERSION(9, 4, 0):
@@ -706,11 +706,11 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                return -EINVAL;
        }
- if (adev->ip_versions[XGMI_HWIP] == IP_VERSION(4, 8, 0))
+       if (adev->ip_versions[XGMI_HWIP][0] == IP_VERSION(4, 8, 0))
                adev->gmc.xgmi.supported = true;
/* set NBIO version */
-       switch (adev->ip_versions[NBIO_HWIP]) {
+       switch (adev->ip_versions[NBIO_HWIP][0]) {
        case IP_VERSION(6, 1, 0):
        case IP_VERSION(6, 2, 0):
                adev->nbio.funcs = &nbio_v6_1_funcs;
@@ -749,7 +749,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                break;
        }
- switch (adev->ip_versions[HDP_HWIP]) {
+       switch (adev->ip_versions[HDP_HWIP][0]) {
        case IP_VERSION(4, 0, 0):
        case IP_VERSION(4, 0, 1):
        case IP_VERSION(4, 1, 0):
@@ -771,7 +771,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                break;
        }
- switch (adev->ip_versions[DF_HWIP]) {
+       switch (adev->ip_versions[DF_HWIP][0]) {
        case IP_VERSION(3, 6, 0):
        case IP_VERSION(3, 6, 1):
        case IP_VERSION(3, 6, 2):
@@ -788,7 +788,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                break;
        }
- switch (adev->ip_versions[SMUIO_HWIP]) {
+       switch (adev->ip_versions[SMUIO_HWIP][0]) {
        case IP_VERSION(9, 0, 0):
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(10, 0, 0):
@@ -819,7 +819,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
        }
/* what IP to use for this? */
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 1, 0):
        case IP_VERSION(9, 2, 1):
@@ -847,7 +847,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
        }
/* use GC or MMHUB IP version */
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 1, 0):
        case IP_VERSION(9, 2, 1):
@@ -874,7 +874,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                return -EINVAL;
        }
- switch (adev->ip_versions[OSSSYS_HWIP]) {
+       switch (adev->ip_versions[OSSSYS_HWIP][0]) {
        case IP_VERSION(4, 0, 0):
        case IP_VERSION(4, 0, 1):
        case IP_VERSION(4, 1, 0):
@@ -900,7 +900,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
        }
if (likely(adev->firmware.load_type == AMDGPU_FW_LOAD_PSP)) {
-               switch (adev->ip_versions[MP0_HWIP]) {
+               switch (adev->ip_versions[MP0_HWIP][0]) {
                case IP_VERSION(9, 0, 0):
                        amdgpu_device_ip_block_add(adev, &psp_v3_1_ip_block);
                        break;
@@ -938,7 +938,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
        }
if (likely(adev->firmware.load_type == AMDGPU_FW_LOAD_PSP)) {
-               switch (adev->ip_versions[MP1_HWIP]) {
+               switch (adev->ip_versions[MP1_HWIP][0]) {
                case IP_VERSION(9, 0, 0):
                case IP_VERSION(10, 0, 0):
                case IP_VERSION(10, 0, 1):
@@ -975,8 +975,8 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
        if (adev->enable_virtual_display || amdgpu_sriov_vf(adev)) {
                amdgpu_device_ip_block_add(adev, &amdgpu_vkms_ip_block);
  #if defined(CONFIG_DRM_AMD_DC)
-       } else if (adev->ip_versions[DCE_HWIP]) {
-               switch (adev->ip_versions[DCE_HWIP]) {
+       } else if (adev->ip_versions[DCE_HWIP][0]) {
+               switch (adev->ip_versions[DCE_HWIP][0]) {
                case IP_VERSION(1, 0, 0):
                case IP_VERSION(1, 0, 1):
                case IP_VERSION(2, 0, 2):
@@ -995,8 +995,8 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                default:
                        return -EINVAL;
                }
-       } else if (adev->ip_versions[DCI_HWIP]) {
-               switch (adev->ip_versions[DCI_HWIP]) {
+       } else if (adev->ip_versions[DCI_HWIP][0]) {
+               switch (adev->ip_versions[DCI_HWIP][0]) {
                case IP_VERSION(12, 0, 0):
                case IP_VERSION(12, 0, 1):
                case IP_VERSION(12, 1, 0):
@@ -1007,7 +1007,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                }
        }
  #endif
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 1, 0):
        case IP_VERSION(9, 2, 1):
@@ -1034,7 +1034,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                return -EINVAL;
        }
- switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(4, 0, 0):
        case IP_VERSION(4, 0, 1):
        case IP_VERSION(4, 1, 0):
@@ -1064,7 +1064,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
        }
if (adev->firmware.load_type == AMDGPU_FW_LOAD_DIRECT) {
-               switch (adev->ip_versions[MP1_HWIP]) {
+               switch (adev->ip_versions[MP1_HWIP][0]) {
                case IP_VERSION(9, 0, 0):
                case IP_VERSION(10, 0, 0):
                case IP_VERSION(10, 0, 1):
@@ -1098,8 +1098,8 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                }
        }
- if (adev->ip_versions[VCE_HWIP]) {
-               switch (adev->ip_versions[UVD_HWIP]) {
+       if (adev->ip_versions[VCE_HWIP][0]) {
+               switch (adev->ip_versions[UVD_HWIP][0]) {
                case IP_VERSION(7, 0, 0):
                case IP_VERSION(7, 2, 0):
                        amdgpu_device_ip_block_add(adev, &uvd_v7_0_ip_block);
@@ -1107,7 +1107,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                default:
                        return -EINVAL;
                }
-               switch (adev->ip_versions[VCE_HWIP]) {
+               switch (adev->ip_versions[VCE_HWIP][0]) {
                case IP_VERSION(4, 0, 0):
                case IP_VERSION(4, 1, 0):
                        amdgpu_device_ip_block_add(adev, &vce_v4_0_ip_block);
@@ -1116,7 +1116,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
                        return -EINVAL;
                }
        } else {
-               switch (adev->ip_versions[UVD_HWIP]) {
+               switch (adev->ip_versions[UVD_HWIP][0]) {
                case IP_VERSION(1, 0, 0):
                case IP_VERSION(1, 0, 1):
                        amdgpu_device_ip_block_add(adev, &vcn_v1_0_ip_block);
@@ -1153,7 +1153,7 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device 
*adev)
        }
if (adev->enable_mes) {
-               switch (adev->ip_versions[GC_HWIP]) {
+               switch (adev->ip_versions[GC_HWIP][0]) {
                case IP_VERSION(10, 1, 10):
                case IP_VERSION(10, 1, 1):
                case IP_VERSION(10, 1, 2):
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
index c47d29689be4..7d414f1606b9 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
@@ -76,7 +76,7 @@ static void 
psp_check_pmfw_centralized_cstate_management(struct psp_context *psp
                return;
        }
- switch (adev->ip_versions[MP0_HWIP]) {
+       switch (adev->ip_versions[MP0_HWIP][0]) {
        case IP_VERSION(11, 0, 4):
        case IP_VERSION(11, 0, 7):
        case IP_VERSION(11, 0, 9):
@@ -97,7 +97,7 @@ static int psp_early_init(void *handle)
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
        struct psp_context *psp = &adev->psp;
- switch (adev->ip_versions[MP0_HWIP]) {
+       switch (adev->ip_versions[MP0_HWIP][0]) {
        case IP_VERSION(9, 0, 0):
                psp_v3_1_set_psp_funcs(psp);
                psp->autoload_supported = false;
@@ -279,7 +279,7 @@ static int psp_sw_init(void *handle)
                        return ret;
                }
        } else if (amdgpu_sriov_vf(adev) &&
-                  adev->ip_versions[MP0_HWIP] == IP_VERSION(13, 0, 2)) {
+                  adev->ip_versions[MP0_HWIP][0] == IP_VERSION(13, 0, 2)) {
                ret = psp_init_ta_microcode(psp, "aldebaran");
                if (ret) {
                        DRM_ERROR("Failed to initialize ta microcode!\n");
@@ -322,8 +322,8 @@ static int psp_sw_init(void *handle)
                }
        }
- if (adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 0) ||
-           adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 7)) {
+       if (adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 0) ||
+           adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 7)) {
                ret= psp_sysfs_init(adev);
                if (ret) {
                        return ret;
@@ -353,8 +353,8 @@ static int psp_sw_fini(void *handle)
                psp->ta_fw = NULL;
        }
- if (adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 0) ||
-           adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 7))
+       if (adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 0) ||
+           adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 7))
                psp_sysfs_fini(adev);
kfree(cmd);
@@ -613,7 +613,7 @@ static int psp_tmr_init(struct psp_context *psp)
static bool psp_skip_tmr(struct psp_context *psp)
  {
-       switch (psp->adev->ip_versions[MP0_HWIP]) {
+       switch (psp->adev->ip_versions[MP0_HWIP][0]) {
        case IP_VERSION(11, 0, 9):
        case IP_VERSION(11, 0, 7):
        case IP_VERSION(13, 0, 2):
@@ -1010,8 +1010,8 @@ int psp_xgmi_terminate(struct psp_context *psp)
        struct amdgpu_device *adev = psp->adev;
/* XGMI TA unload currently is not supported on Arcturus/Aldebaran A+A */
-       if (adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 4) ||
-           (adev->ip_versions[MP0_HWIP] == IP_VERSION(13, 0, 2) &&
+       if (adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 4) ||
+           (adev->ip_versions[MP0_HWIP][0] == IP_VERSION(13, 0, 2) &&
             adev->gmc.xgmi.connected_to_cpu))
                return 0;
@@ -1113,7 +1113,7 @@ int psp_xgmi_get_node_id(struct psp_context *psp, uint64_t *node_id) static bool psp_xgmi_peer_link_info_supported(struct psp_context *psp)
  {
-       return psp->adev->ip_versions[MP0_HWIP] == IP_VERSION(13, 0, 2) &&
+       return psp->adev->ip_versions[MP0_HWIP][0] == IP_VERSION(13, 0, 2) &&
                psp->xgmi_context.context.bin_desc.feature_version >= 
0x2000000b;
  }
@@ -2232,8 +2232,8 @@ static int psp_load_smu_fw(struct psp_context *psp) if ((amdgpu_in_reset(adev) &&
             ras && adev->ras_enabled &&
-            (adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 4) ||
-             adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 2)))) {
+            (adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 4) ||
+             adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 2)))) {
                ret = amdgpu_dpm_set_mp1_state(adev, PP_MP1_STATE_UNLOAD);
                if (ret) {
                        DRM_WARN("Failed to set MP1 state prepare for 
reload\n");
@@ -2330,9 +2330,9 @@ static int psp_load_non_psp_fw(struct psp_context *psp)
                        continue;
if (psp->autoload_supported &&
-                   (adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 7) ||
-                    adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 11) ||
-                    adev->ip_versions[MP0_HWIP] == IP_VERSION(11, 0, 12)) &&
+                   (adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 7) ||
+                    adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 11) ||
+                    adev->ip_versions[MP0_HWIP][0] == IP_VERSION(11, 0, 12)) &&
                    (ucode->ucode_id == AMDGPU_UCODE_ID_SDMA1 ||
                     ucode->ucode_id == AMDGPU_UCODE_ID_SDMA2 ||
                     ucode->ucode_id == AMDGPU_UCODE_ID_SDMA3))
@@ -2920,7 +2920,7 @@ static int psp_init_sos_base_fw(struct amdgpu_device 
*adev)
                le32_to_cpu(sos_hdr->header.ucode_array_offset_bytes);
if (adev->gmc.xgmi.connected_to_cpu ||
-           (adev->ip_versions[MP0_HWIP] != IP_VERSION(13, 0, 2))) {
+           (adev->ip_versions[MP0_HWIP][0] != IP_VERSION(13, 0, 2))) {
                adev->psp.sos.fw_version = 
le32_to_cpu(sos_hdr->header.ucode_version);
                adev->psp.sos.feature_version = 
le32_to_cpu(sos_hdr->sos.fw_version);
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
index 9724f5f5f702..4d3f2386ef53 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
@@ -86,7 +86,7 @@ int amdgpu_vcn_sw_init(struct amdgpu_device *adev)
        for (i = 0; i < adev->vcn.num_vcn_inst; i++)
                atomic_set(&adev->vcn.inst[i].dpg_enc_submission_cnt, 0);
- switch (adev->ip_versions[UVD_HWIP]) {
+       switch (adev->ip_versions[UVD_HWIP][0]) {
        case IP_VERSION(1, 0, 0):
        case IP_VERSION(1, 0, 1):
                if (adev->apu_flags & AMD_APU_IS_RAVEN2)
@@ -134,7 +134,7 @@ int amdgpu_vcn_sw_init(struct amdgpu_device *adev)
                        adev->vcn.indirect_sram = true;
                break;
        case IP_VERSION(3, 0, 0):
-               if (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 3, 0))
+               if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 3, 0))
                        fw_name = FIRMWARE_SIENNA_CICHLID;
                else
                        fw_name = FIRMWARE_NAVY_FLOUNDER;
diff --git a/drivers/gpu/drm/amd/amdgpu/athub_v2_0.c 
b/drivers/gpu/drm/amd/amdgpu/athub_v2_0.c
index d3f634eb40fb..ab6a07e5e8c4 100644
--- a/drivers/gpu/drm/amd/amdgpu/athub_v2_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/athub_v2_0.c
@@ -77,7 +77,7 @@ int athub_v2_0_set_clockgating(struct amdgpu_device *adev,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[ATHUB_HWIP]) {
+       switch (adev->ip_versions[ATHUB_HWIP][0]) {
        case IP_VERSION(2, 0, 0):
        case IP_VERSION(2, 0, 2):
                athub_v2_0_update_medium_grain_clock_gating(adev,
diff --git a/drivers/gpu/drm/amd/amdgpu/athub_v2_1.c 
b/drivers/gpu/drm/amd/amdgpu/athub_v2_1.c
index 0fbeaf05204e..2edefd10e56c 100644
--- a/drivers/gpu/drm/amd/amdgpu/athub_v2_1.c
+++ b/drivers/gpu/drm/amd/amdgpu/athub_v2_1.c
@@ -70,7 +70,7 @@ int athub_v2_1_set_clockgating(struct amdgpu_device *adev,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[ATHUB_HWIP]) {
+       switch (adev->ip_versions[ATHUB_HWIP][0]) {
        case IP_VERSION(2, 1, 0):
        case IP_VERSION(2, 1, 1):
        case IP_VERSION(2, 1, 2):
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
index 4c632306ed70..71bb3c0dc1da 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
@@ -1537,7 +1537,7 @@ static u32 gfx_v10_rlcg_rw(struct amdgpu_device *adev, 
u32 offset, u32 v, uint32
        scratch_reg3 = adev->rmmio +
                       (adev->reg_offset[GC_HWIP][0][mmSCRATCH_REG1_BASE_IDX] + 
mmSCRATCH_REG3) * 4;
- if (adev->ip_versions[GC_HWIP] >= IP_VERSION(10, 3, 0)) {
+       if (adev->ip_versions[GC_HWIP][0] >= IP_VERSION(10, 3, 0)) {
                spare_int = adev->rmmio +
                            
(adev->reg_offset[GC_HWIP][0][mmRLC_SPARE_INT_0_Sienna_Cichlid_BASE_IDX]
                             + mmRLC_SPARE_INT_0_Sienna_Cichlid) * 4;
@@ -3727,7 +3727,7 @@ static void gfx_v10_0_set_kiq_pm4_funcs(struct 
amdgpu_device *adev)
static void gfx_v10_0_init_spm_golden_registers(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
                soc15_program_register_sequence(adev,
                                                
golden_settings_gc_rlc_spm_10_0_nv10,
@@ -3750,7 +3750,7 @@ static void gfx_v10_0_init_spm_golden_registers(struct 
amdgpu_device *adev)
static void gfx_v10_0_init_golden_registers(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
                soc15_program_register_sequence(adev,
                                                golden_settings_gc_10_1,
@@ -3985,7 +3985,7 @@ static void gfx_v10_0_check_fw_write_wait(struct 
amdgpu_device *adev)
  {
        adev->gfx.cp_fw_write_wait = false;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
        case IP_VERSION(10, 1, 2):
        case IP_VERSION(10, 1, 1):
@@ -4066,7 +4066,7 @@ static bool gfx_v10_0_navi10_gfxoff_should_enable(struct 
amdgpu_device *adev)
static void gfx_v10_0_check_gfxoff_flag(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
                if (!gfx_v10_0_navi10_gfxoff_should_enable(adev))
                        adev->pm.pp_feature &= ~PP_GFXOFF_MASK;
@@ -4093,7 +4093,7 @@ static int gfx_v10_0_init_microcode(struct amdgpu_device 
*adev)
DRM_DEBUG("\n"); - switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
                chip_name = "navi10";
                break;
@@ -4684,7 +4684,7 @@ static void gfx_v10_0_gpu_early_init(struct amdgpu_device 
*adev)
adev->gfx.funcs = &gfx_v10_0_gfx_funcs; - switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
        case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 2):
@@ -4818,7 +4818,7 @@ static int gfx_v10_0_sw_init(void *handle)
        struct amdgpu_kiq *kiq;
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
        case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 2):
@@ -5068,8 +5068,8 @@ static void gfx_v10_0_setup_rb(struct amdgpu_device *adev)
        for (i = 0; i < adev->gfx.config.max_shader_engines; i++) {
                for (j = 0; j < adev->gfx.config.max_sh_per_se; j++) {
                        bitmap = i * adev->gfx.config.max_sh_per_se + j;
-                       if (((adev->ip_versions[GC_HWIP] == IP_VERSION(10, 3, 
0)) ||
-                               (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 3, 3))) 
&&
+                       if (((adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 
3, 0)) ||
+                               (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 3, 
3))) &&
                            ((gfx_v10_3_get_disabled_sa(adev) >> bitmap) & 1))
                                continue;
                        gfx_v10_0_select_se_sh(adev, i, j, 0xffffffff);
@@ -5096,7 +5096,7 @@ static u32 
gfx_v10_0_init_pa_sc_tile_steering_override(struct amdgpu_device *ade
/* for ASICs that integrates GFX v10.3
         * pa_sc_tile_steering_override should be set to 0 */
-       if (adev->ip_versions[GC_HWIP] >= IP_VERSION(10, 3, 0))
+       if (adev->ip_versions[GC_HWIP][0] >= IP_VERSION(10, 3, 0))
                return 0;
/* init num_sc */
@@ -5249,7 +5249,7 @@ static void gfx_v10_0_get_tcc_info(struct amdgpu_device 
*adev)
        /* TCCs are global (not instanced). */
        uint32_t tcc_disable;
- if (adev->ip_versions[GC_HWIP] >= IP_VERSION(10, 3, 0)) {
+       if (adev->ip_versions[GC_HWIP][0] >= IP_VERSION(10, 3, 0)) {
                tcc_disable = RREG32_SOC15(GC, 0, mmCGTS_TCC_DISABLE_gc_10_3) |
                              RREG32_SOC15(GC, 0, 
mmCGTS_USER_TCC_DISABLE_gc_10_3);
        } else {
@@ -5326,7 +5326,7 @@ static int gfx_v10_0_init_csb(struct amdgpu_device *adev)
        adev->gfx.rlc.funcs->get_csb_buffer(adev, adev->gfx.rlc.cs_ptr);
/* csib */
-       if (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 1, 2)) {
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 1, 2)) {
                WREG32_SOC15_RLC(GC, 0, mmRLC_CSIB_ADDR_HI,
                                adev->gfx.rlc.clear_state_gpu_addr >> 32);
                WREG32_SOC15_RLC(GC, 0, mmRLC_CSIB_ADDR_LO,
@@ -5948,7 +5948,7 @@ static int gfx_v10_0_cp_gfx_enable(struct amdgpu_device 
*adev, bool enable)
        tmp = REG_SET_FIELD(tmp, CP_ME_CNTL, PFP_HALT, enable ? 0 : 1);
        tmp = REG_SET_FIELD(tmp, CP_ME_CNTL, CE_HALT, enable ? 0 : 1);
- if (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 1, 2)) {
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 1, 2)) {
                WREG32_SOC15_RLC(GC, 0, mmCP_ME_CNTL, tmp);
        } else {
                WREG32_SOC15(GC, 0, mmCP_ME_CNTL, tmp);
@@ -6337,7 +6337,7 @@ static void gfx_v10_0_cp_gfx_set_doorbell(struct 
amdgpu_device *adev,
                }
                WREG32_SOC15(GC, 0, mmCP_RB_DOORBELL_CONTROL, tmp);
        }
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 3, 0):
        case IP_VERSION(10, 3, 2):
        case IP_VERSION(10, 3, 1):
@@ -6474,7 +6474,7 @@ static int gfx_v10_0_cp_gfx_resume(struct amdgpu_device 
*adev)
  static void gfx_v10_0_cp_compute_enable(struct amdgpu_device *adev, bool 
enable)
  {
        if (enable) {
-               switch (adev->ip_versions[GC_HWIP]) {
+               switch (adev->ip_versions[GC_HWIP][0]) {
                case IP_VERSION(10, 3, 0):
                case IP_VERSION(10, 3, 2):
                case IP_VERSION(10, 3, 1):
@@ -6488,7 +6488,7 @@ static void gfx_v10_0_cp_compute_enable(struct 
amdgpu_device *adev, bool enable)
                        break;
                }
        } else {
-               switch (adev->ip_versions[GC_HWIP]) {
+               switch (adev->ip_versions[GC_HWIP][0]) {
                case IP_VERSION(10, 3, 0):
                case IP_VERSION(10, 3, 2):
                case IP_VERSION(10, 3, 1):
@@ -6586,7 +6586,7 @@ static void gfx_v10_0_kiq_setting(struct amdgpu_ring 
*ring)
        struct amdgpu_device *adev = ring->adev;
/* tell RLC which is KIQ queue */
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 3, 0):
        case IP_VERSION(10, 3, 2):
        case IP_VERSION(10, 3, 1):
@@ -7303,7 +7303,7 @@ static bool gfx_v10_0_check_grbm_cam_remapping(struct 
amdgpu_device *adev)
/* check if mmVGT_ESGS_RING_SIZE_UMD
         * has been remapped to mmVGT_ESGS_RING_SIZE */
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 3, 0):
        case IP_VERSION(10, 3, 2):
        case IP_VERSION(10, 3, 4):
@@ -7350,7 +7350,7 @@ static void gfx_v10_0_setup_grbm_cam_remapping(struct 
amdgpu_device *adev)
         * index will auto-inc after each data writting */
        WREG32_SOC15(GC, 0, mmGRBM_CAM_INDEX, 0);
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 3, 0):
        case IP_VERSION(10, 3, 2):
        case IP_VERSION(10, 3, 1):
@@ -7520,19 +7520,19 @@ static int gfx_v10_0_hw_init(void *handle)
         * init golden registers and rlc resume may override some registers,
         * reconfig them here
         */
-       if (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 1, 10) ||
-           adev->ip_versions[GC_HWIP] == IP_VERSION(10, 1, 1) ||
-           adev->ip_versions[GC_HWIP] == IP_VERSION(10, 1, 2))
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 1, 10) ||
+           adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 1, 1) ||
+           adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 1, 2))
                gfx_v10_0_tcp_harvest(adev);
r = gfx_v10_0_cp_resume(adev);
        if (r)
                return r;
- if (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 3, 0))
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 3, 0))
                gfx_v10_3_program_pbb_mode(adev);
- if (adev->ip_versions[GC_HWIP] >= IP_VERSION(10, 3, 0))
+       if (adev->ip_versions[GC_HWIP][0] >= IP_VERSION(10, 3, 0))
                gfx_v10_3_set_power_brake_sequence(adev);
return r;
@@ -7584,7 +7584,7 @@ static int gfx_v10_0_hw_fini(void *handle)
        if (amdgpu_sriov_vf(adev)) {
                gfx_v10_0_cp_gfx_enable(adev, false);
                /* Program KIQ position of RLC_CP_SCHEDULERS during destroy */
-               if (adev->ip_versions[GC_HWIP] >= IP_VERSION(10, 3, 0)) {
+               if (adev->ip_versions[GC_HWIP][0] >= IP_VERSION(10, 3, 0)) {
                        tmp = RREG32_SOC15(GC, 0, 
mmRLC_CP_SCHEDULERS_Sienna_Cichlid);
                        tmp &= 0xffffff00;
                        WREG32_SOC15(GC, 0, mmRLC_CP_SCHEDULERS_Sienna_Cichlid, 
tmp);
@@ -7670,7 +7670,7 @@ static int gfx_v10_0_soft_reset(void *handle)
/* GRBM_STATUS2 */
        tmp = RREG32_SOC15(GC, 0, mmGRBM_STATUS2);
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 3, 0):
        case IP_VERSION(10, 3, 2):
        case IP_VERSION(10, 3, 1):
@@ -7726,7 +7726,7 @@ static uint64_t gfx_v10_0_get_gpu_clock_counter(struct 
amdgpu_device *adev)
  {
        uint64_t clock, clock_lo, clock_hi, hi_check;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 3, 1):
        case IP_VERSION(10, 3, 3):
                clock = (uint64_t)RREG32_SOC15(SMUIO, 0, 
mmGOLDEN_TSC_COUNT_LOWER_Vangogh) |
@@ -7784,7 +7784,7 @@ static int gfx_v10_0_early_init(void *handle)
  {
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
        case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 2):
@@ -7848,7 +7848,7 @@ static void gfx_v10_0_set_safe_mode(struct amdgpu_device 
*adev)
        data = RLC_SAFE_MODE__CMD_MASK;
        data |= (1 << RLC_SAFE_MODE__MESSAGE__SHIFT);
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 3, 0):
        case IP_VERSION(10, 3, 2):
        case IP_VERSION(10, 3, 1):
@@ -7884,7 +7884,7 @@ static void gfx_v10_0_unset_safe_mode(struct 
amdgpu_device *adev)
        uint32_t data;
data = RLC_SAFE_MODE__CMD_MASK;
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 3, 0):
        case IP_VERSION(10, 3, 2):
        case IP_VERSION(10, 3, 1):
@@ -8193,7 +8193,7 @@ static void 
gfx_v10_0_apply_medium_grain_clock_gating_workaround(struct amdgpu_d
                mmCGTS_SA1_QUAD1_SM_CTRL_REG
        };
- if (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 1, 2)) {
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 1, 2)) {
                for (i = 0; i < ARRAY_SIZE(tcp_ctrl_regs_nv12); i++) {
                        reg_idx = 
adev->reg_offset[GC_HWIP][0][mmCGTS_SA0_WGP00_CU0_TCP_CTRL_REG_BASE_IDX] +
                                  tcp_ctrl_regs_nv12[i];
@@ -8238,8 +8238,8 @@ static int gfx_v10_0_update_gfx_clock_gating(struct 
amdgpu_device *adev,
                /* ===  CGCG + CGLS === */
                gfx_v10_0_update_coarse_grain_clock_gating(adev, enable);
- if ((adev->ip_versions[GC_HWIP] >= IP_VERSION(10, 1, 10)) &&
-                    (adev->ip_versions[GC_HWIP] <= IP_VERSION(10, 1, 2)))
+               if ((adev->ip_versions[GC_HWIP][0] >= IP_VERSION(10, 1, 10)) &&
+                    (adev->ip_versions[GC_HWIP][0] <= IP_VERSION(10, 1, 2)))
                        
gfx_v10_0_apply_medium_grain_clock_gating_workaround(adev);
        } else {
                /* CGCG/CGLS should be disabled before MGCG/MGLS
@@ -8335,7 +8335,7 @@ static void gfx_v10_cntl_power_gating(struct 
amdgpu_device *adev, bool enable)
         * Power/performance team will optimize it and might give a new value 
later.
         */
        if (enable && (adev->pg_flags & AMD_PG_SUPPORT_GFX_PG)) {
-               switch (adev->ip_versions[GC_HWIP]) {
+               switch (adev->ip_versions[GC_HWIP][0]) {
                case IP_VERSION(10, 3, 1):
                        data = 0x4E20 & 
RLC_PG_DELAY_3__CGCG_ACTIVE_BEFORE_CGPG_MASK_Vangogh;
                        WREG32_SOC15(GC, 0, mmRLC_PG_DELAY_3, data);
@@ -8399,7 +8399,7 @@ static int gfx_v10_0_set_powergating_state(void *handle,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
        case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 2):
@@ -8428,7 +8428,7 @@ static int gfx_v10_0_set_clockgating_state(void *handle,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
        case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 2):
@@ -9541,7 +9541,7 @@ static void gfx_v10_0_set_irq_funcs(struct amdgpu_device 
*adev)
static void gfx_v10_0_set_rlc_funcs(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
        case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 3):
@@ -9641,8 +9641,8 @@ static int gfx_v10_0_get_cu_info(struct amdgpu_device 
*adev,
        for (i = 0; i < adev->gfx.config.max_shader_engines; i++) {
                for (j = 0; j < adev->gfx.config.max_sh_per_se; j++) {
                        bitmap = i * adev->gfx.config.max_sh_per_se + j;
-                       if (((adev->ip_versions[GC_HWIP] == IP_VERSION(10, 3, 
0)) ||
-                               (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 3, 3))) 
&&
+                       if (((adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 
3, 0)) ||
+                               (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 3, 
3))) &&
                            ((gfx_v10_3_get_disabled_sa(adev) >> bitmap) & 1))
                                continue;
                        mask = 1;
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
index 3a160d78c98f..8f2966bcf5a4 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
@@ -953,7 +953,7 @@ static void gfx_v9_0_set_kiq_pm4_funcs(struct amdgpu_device 
*adev)
static void gfx_v9_0_init_golden_registers(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
                soc15_program_register_sequence(adev,
                                                golden_settings_gc_9_0,
@@ -1009,8 +1009,8 @@ static void gfx_v9_0_init_golden_registers(struct 
amdgpu_device *adev)
                break;
        }
- if ((adev->ip_versions[GC_HWIP] != IP_VERSION(9, 4, 1)) &&
-           (adev->ip_versions[GC_HWIP] != IP_VERSION(9, 4, 2)))
+       if ((adev->ip_versions[GC_HWIP][0] != IP_VERSION(9, 4, 1)) &&
+           (adev->ip_versions[GC_HWIP][0] != IP_VERSION(9, 4, 2)))
                soc15_program_register_sequence(adev, 
golden_settings_gc_9_x_common,
                                                (const 
u32)ARRAY_SIZE(golden_settings_gc_9_x_common));
  }
@@ -1194,14 +1194,14 @@ static void gfx_v9_0_check_fw_write_wait(struct 
amdgpu_device *adev)
        adev->gfx.me_fw_write_wait = false;
        adev->gfx.mec_fw_write_wait = false;
- if ((adev->ip_versions[GC_HWIP] != IP_VERSION(9, 4, 1)) &&
+       if ((adev->ip_versions[GC_HWIP][0] != IP_VERSION(9, 4, 1)) &&
            ((adev->gfx.mec_fw_version < 0x000001a5) ||
            (adev->gfx.mec_feature_version < 46) ||
            (adev->gfx.pfp_fw_version < 0x000000b7) ||
            (adev->gfx.pfp_feature_version < 46)))
                DRM_WARN_ONCE("CP firmware version too old, please update!");
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
                if ((adev->gfx.me_fw_version >= 0x0000009c) &&
                    (adev->gfx.me_feature_version >= 42) &&
@@ -1301,7 +1301,7 @@ static bool is_raven_kicker(struct amdgpu_device *adev)
static bool check_if_enlarge_doorbell_range(struct amdgpu_device *adev)
  {
-       if ((adev->ip_versions[GC_HWIP] == IP_VERSION(9, 3, 0)) &&
+       if ((adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 3, 0)) &&
            (adev->gfx.me_fw_version >= 0x000000a5) &&
            (adev->gfx.me_feature_version >= 52))
                return true;
@@ -1314,7 +1314,7 @@ static void gfx_v9_0_check_if_need_gfxoff(struct 
amdgpu_device *adev)
        if (gfx_v9_0_should_disable_gfxoff(adev->pdev))
                adev->pm.pp_feature &= ~PP_GFXOFF_MASK;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 2, 1):
        case IP_VERSION(9, 4, 0):
@@ -1558,9 +1558,9 @@ static int gfx_v9_0_init_rlc_microcode(struct 
amdgpu_device *adev,
static bool gfx_v9_0_load_mec2_fw_bin_support(struct amdgpu_device *adev)
  {
-       if (adev->ip_versions[GC_HWIP] == IP_VERSION(9, 4, 2) ||
-           adev->ip_versions[GC_HWIP] == IP_VERSION(9, 4, 1) ||
-           adev->ip_versions[GC_HWIP] == IP_VERSION(9, 3, 0))
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 4, 2) ||
+           adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 4, 1) ||
+           adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 3, 0))
                return false;
return true;
@@ -1668,7 +1668,7 @@ static int gfx_v9_0_init_microcode(struct amdgpu_device 
*adev)
DRM_DEBUG("\n"); - switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
                chip_name = "vega10";
                break;
@@ -1800,7 +1800,7 @@ static void gfx_v9_0_init_always_on_cu_mask(struct 
amdgpu_device *adev)
if (adev->flags & AMD_IS_APU)
                always_on_cu_num = 4;
-       else if (adev->ip_versions[GC_HWIP] == IP_VERSION(9, 2, 1))
+       else if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 2, 1))
                always_on_cu_num = 8;
        else
                always_on_cu_num = 12;
@@ -1969,7 +1969,7 @@ static int gfx_v9_0_rlc_init(struct amdgpu_device *adev)
                        return r;
        }
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 2, 2):
        case IP_VERSION(9, 1, 0):
                gfx_v9_0_init_lbpw(adev);
@@ -2149,7 +2149,7 @@ static int gfx_v9_0_gpu_early_init(struct amdgpu_device 
*adev)
adev->gfx.funcs = &gfx_v9_0_gfx_funcs; - switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
                adev->gfx.config.max_hw_contexts = 8;
                adev->gfx.config.sc_prim_fifo_size_frontend = 0x20;
@@ -2313,7 +2313,7 @@ static int gfx_v9_0_sw_init(void *handle)
        struct amdgpu_kiq *kiq;
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 2, 1):
        case IP_VERSION(9, 4, 0):
@@ -2605,7 +2605,7 @@ static void gfx_v9_0_init_sq_config(struct amdgpu_device 
*adev)
  {
        uint32_t tmp;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 4, 1):
                tmp = RREG32_SOC15(GC, 0, mmSQ_CONFIG);
                tmp = REG_SET_FIELD(tmp, SQ_CONFIG,
@@ -2941,7 +2941,7 @@ static void gfx_v9_0_init_gfx_power_gating(struct 
amdgpu_device *adev)
                /* program GRBM_REG_SAVE_GFX_IDLE_THRESHOLD to 0x55f0 */
                data |= (0x55f0 << 
RLC_AUTO_PG_CTRL__GRBM_REG_SAVE_GFX_IDLE_THRESHOLD__SHIFT);
                WREG32(SOC15_REG_OFFSET(GC, 0, mmRLC_AUTO_PG_CTRL), data);
-               if (adev->ip_versions[GC_HWIP] != IP_VERSION(9, 3, 0))
+               if (adev->ip_versions[GC_HWIP][0] != IP_VERSION(9, 3, 0))
                        pwr_10_0_gfxip_control_over_cgpg(adev, true);
        }
  }
@@ -3053,7 +3053,7 @@ static void gfx_v9_0_init_pg(struct amdgpu_device *adev)
         * And it's needed by gfxoff feature.
         */
        if (adev->gfx.rlc.is_rlc_v2_1) {
-               if (adev->ip_versions[GC_HWIP] == IP_VERSION(9, 2, 1) ||
+               if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 2, 1) ||
                    (adev->apu_flags & AMD_APU_IS_RAVEN2))
                        gfx_v9_1_init_rlc_save_restore_list(adev);
                gfx_v9_0_enable_save_restore_machine(adev);
@@ -3166,7 +3166,7 @@ static int gfx_v9_0_rlc_resume(struct amdgpu_device *adev)
                        return r;
        }
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 2, 2):
        case IP_VERSION(9, 1, 0):
                if (amdgpu_lbpw == 0)
@@ -3969,8 +3969,8 @@ static void gfx_v9_0_init_tcp_config(struct amdgpu_device 
*adev)
  {
        u32 tmp;
- if (adev->ip_versions[GC_HWIP] != IP_VERSION(9, 4, 1) &&
-           adev->ip_versions[GC_HWIP] != IP_VERSION(9, 4, 2))
+       if (adev->ip_versions[GC_HWIP][0] != IP_VERSION(9, 4, 1) &&
+           adev->ip_versions[GC_HWIP][0] != IP_VERSION(9, 4, 2))
                return;
tmp = RREG32_SOC15(GC, 0, mmTCP_ADDR_CONFIG);
@@ -4010,7 +4010,7 @@ static int gfx_v9_0_hw_init(void *handle)
        if (r)
                return r;
- if (adev->ip_versions[GC_HWIP] == IP_VERSION(9, 4, 2))
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 4, 2))
                gfx_v9_4_2_set_power_brake_sequence(adev);
return r;
@@ -4242,7 +4242,7 @@ static uint64_t gfx_v9_0_get_gpu_clock_counter(struct 
amdgpu_device *adev)
amdgpu_gfx_off_ctrl(adev, false);
        mutex_lock(&adev->gfx.gpu_clock_mutex);
-       if (adev->ip_versions[GC_HWIP] == IP_VERSION(9, 0, 1) && 
amdgpu_sriov_runtime(adev)) {
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 0, 1) && 
amdgpu_sriov_runtime(adev)) {
                clock = gfx_v9_0_kiq_read_clock(adev);
        } else {
                WREG32_SOC15(GC, 0, mmRLC_CAPTURE_GPU_CLOCK_COUNT, 1);
@@ -4592,7 +4592,7 @@ static int gfx_v9_0_do_edc_gpr_workarounds(struct 
amdgpu_device *adev)
        if (!ring->sched.ready)
                return 0;
- if (adev->ip_versions[GC_HWIP] == IP_VERSION(9, 4, 1)) {
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 4, 1)) {
                vgpr_init_shader_ptr = vgpr_init_compute_shader_arcturus;
                vgpr_init_shader_size = 
sizeof(vgpr_init_compute_shader_arcturus);
                vgpr_init_regs_ptr = vgpr_init_regs_arcturus;
@@ -4742,8 +4742,8 @@ static int gfx_v9_0_early_init(void *handle)
  {
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
- if (adev->ip_versions[GC_HWIP] == IP_VERSION(9, 4, 1) ||
-           adev->ip_versions[GC_HWIP] == IP_VERSION(9, 4, 2))
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 4, 1) ||
+           adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 4, 2))
                adev->gfx.num_gfx_rings = 0;
        else
                adev->gfx.num_gfx_rings = GFX9_NUM_GFX_RINGS;
@@ -4777,7 +4777,7 @@ static int gfx_v9_0_ecc_late_init(void *handle)
        }
/* requires IBs so do in late init after IB pool is initialized */
-       if (adev->ip_versions[GC_HWIP] == IP_VERSION(9, 4, 2))
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 4, 2))
                r = gfx_v9_4_2_do_edc_gpr_workarounds(adev);
        else
                r = gfx_v9_0_do_edc_gpr_workarounds(adev);
@@ -4905,7 +4905,7 @@ static void 
gfx_v9_0_update_medium_grain_clock_gating(struct amdgpu_device *adev
                /* 1 - RLC_CGTT_MGCG_OVERRIDE */
                def = data = RREG32_SOC15(GC, 0, mmRLC_CGTT_MGCG_OVERRIDE);
- if (adev->ip_versions[GC_HWIP] != IP_VERSION(9, 2, 1))
+               if (adev->ip_versions[GC_HWIP][0] != IP_VERSION(9, 2, 1))
                        data &= 
~RLC_CGTT_MGCG_OVERRIDE__CPF_CGTT_SCLK_OVERRIDE_MASK;
data &= ~(RLC_CGTT_MGCG_OVERRIDE__GRBM_CGTT_SCLK_OVERRIDE_MASK |
@@ -4939,7 +4939,7 @@ static void 
gfx_v9_0_update_medium_grain_clock_gating(struct amdgpu_device *adev
                /* 1 - MGCG_OVERRIDE */
                def = data = RREG32_SOC15(GC, 0, mmRLC_CGTT_MGCG_OVERRIDE);
- if (adev->ip_versions[GC_HWIP] != IP_VERSION(9, 2, 1))
+               if (adev->ip_versions[GC_HWIP][0] != IP_VERSION(9, 2, 1))
                        data |= 
RLC_CGTT_MGCG_OVERRIDE__CPF_CGTT_SCLK_OVERRIDE_MASK;
data |= (RLC_CGTT_MGCG_OVERRIDE__RLC_CGTT_SCLK_OVERRIDE_MASK |
@@ -5045,7 +5045,7 @@ static void 
gfx_v9_0_update_coarse_grain_clock_gating(struct amdgpu_device *adev
                /* enable cgcg FSM(0x0000363F) */
                def = RREG32_SOC15(GC, 0, mmRLC_CGCG_CGLS_CTRL);
- if (adev->ip_versions[GC_HWIP] == IP_VERSION(9, 4, 1))
+               if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(9, 4, 1))
                        data = (0x2000 << 
RLC_CGCG_CGLS_CTRL__CGCG_GFX_IDLE_THRESHOLD__SHIFT) |
                                RLC_CGCG_CGLS_CTRL__CGCG_EN_MASK;
                else
@@ -5171,7 +5171,7 @@ static int gfx_v9_0_set_powergating_state(void *handle,
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
        bool enable = (state == AMD_PG_STATE_GATE);
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 2, 2):
        case IP_VERSION(9, 1, 0):
        case IP_VERSION(9, 3, 0):
@@ -5218,7 +5218,7 @@ static int gfx_v9_0_set_clockgating_state(void *handle,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 2, 1):
        case IP_VERSION(9, 4, 0):
@@ -5268,7 +5268,7 @@ static void gfx_v9_0_get_clockgating_state(void *handle, 
u32 *flags)
        if (data & CP_MEM_SLP_CNTL__CP_MEM_LS_EN_MASK)
                *flags |= AMD_CG_SUPPORT_GFX_CP_LS | AMD_CG_SUPPORT_GFX_MGLS;
- if (adev->ip_versions[GC_HWIP] != IP_VERSION(9, 4, 1)) {
+       if (adev->ip_versions[GC_HWIP][0] != IP_VERSION(9, 4, 1)) {
                /* AMD_CG_SUPPORT_GFX_3D_CGCG */
                data = RREG32_KIQ(SOC15_REG_OFFSET(GC, 0, 
mmRLC_CGCG_CGLS_CTRL_3D));
                if (data & RLC_CGCG_CGLS_CTRL_3D__CGCG_EN_MASK)
@@ -7039,7 +7039,7 @@ static void gfx_v9_0_set_irq_funcs(struct amdgpu_device 
*adev)
static void gfx_v9_0_set_rlc_funcs(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 2, 1):
        case IP_VERSION(9, 4, 0):
@@ -7058,7 +7058,7 @@ static void gfx_v9_0_set_rlc_funcs(struct amdgpu_device 
*adev)
  static void gfx_v9_0_set_gds_init(struct amdgpu_device *adev)
  {
        /* init asci gds info */
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 2, 1):
        case IP_VERSION(9, 4, 0):
@@ -7080,7 +7080,7 @@ static void gfx_v9_0_set_gds_init(struct amdgpu_device 
*adev)
                break;
        }
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 4, 0):
                adev->gds.gds_compute_max_wave_id = 0x7ff;
diff --git a/drivers/gpu/drm/amd/amdgpu/gfxhub_v2_1.c 
b/drivers/gpu/drm/amd/amdgpu/gfxhub_v2_1.c
index 149f3843e0e7..e80d1dc43079 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfxhub_v2_1.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfxhub_v2_1.c
@@ -506,7 +506,7 @@ static int gfxhub_v2_1_get_xgmi_info(struct amdgpu_device 
*adev)
        u32 max_num_physical_nodes   = 0;
        u32 max_physical_node_id     = 0;
- switch (adev->ip_versions[XGMI_HWIP]) {
+       switch (adev->ip_versions[XGMI_HWIP][0]) {
        case IP_VERSION(4, 8, 0):
                max_num_physical_nodes   = 4;
                max_physical_node_id     = 3;
@@ -544,7 +544,7 @@ static void gfxhub_v2_1_utcl2_harvest(struct amdgpu_device 
*adev)
                adev->gfx.config.max_sh_per_se *
                adev->gfx.config.max_shader_engines);
- if (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 3, 3)) {
+       if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 3, 3)) {
                /* Get SA disabled bitmap from eFuse setting */
                efuse_setting = RREG32_SOC15(GC, 0, mmCC_GC_SA_UNIT_DISABLE);
                efuse_setting &= CC_GC_SA_UNIT_DISABLE__SA_DISABLE_MASK;
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c 
b/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
index b488257ff201..6d0586009de3 100644
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
@@ -133,7 +133,7 @@ static int gmc_v10_0_process_interrupt(struct amdgpu_device 
*adev,
                 * the new fast GRBM interface.
                 */
                if ((entry->vmid_src == AMDGPU_GFXHUB_0) &&
-                   (adev->ip_versions[GC_HWIP] < IP_VERSION(10, 3, 0)))
+                   (adev->ip_versions[GC_HWIP][0] < IP_VERSION(10, 3, 0)))
                        RREG32(hub->vm_l2_pro_fault_status);
status = RREG32(hub->vm_l2_pro_fault_status);
@@ -268,7 +268,7 @@ static void gmc_v10_0_flush_vm_hub(struct amdgpu_device 
*adev, uint32_t vmid,
         * to avoid a false ACK due to the new fast GRBM interface.
         */
        if ((vmhub == AMDGPU_GFXHUB_0) &&
-           (adev->ip_versions[GC_HWIP] < IP_VERSION(10, 3, 0)))
+           (adev->ip_versions[GC_HWIP][0] < IP_VERSION(10, 3, 0)))
                RREG32_RLC_NO_KIQ(hub->vm_inv_eng0_req +
                                  hub->eng_distance * eng, hub_ip);
@@ -657,7 +657,7 @@ static void gmc_v10_0_set_gmc_funcs(struct amdgpu_device *adev) static void gmc_v10_0_set_umc_funcs(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[UMC_HWIP]) {
+       switch (adev->ip_versions[UMC_HWIP][0]) {
        case IP_VERSION(8, 7, 0):
                adev->umc.max_ras_err_cnt_per_query = 
UMC_V8_7_TOTAL_CHANNEL_NUM;
                adev->umc.channel_inst_num = UMC_V8_7_CHANNEL_INSTANCE_NUM;
@@ -674,7 +674,7 @@ static void gmc_v10_0_set_umc_funcs(struct amdgpu_device 
*adev)
static void gmc_v10_0_set_mmhub_funcs(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[MMHUB_HWIP]) {
+       switch (adev->ip_versions[MMHUB_HWIP][0]) {
        case IP_VERSION(2, 3, 0):
        case IP_VERSION(2, 4, 0):
                adev->mmhub.funcs = &mmhub_v2_3_funcs;
@@ -687,7 +687,7 @@ static void gmc_v10_0_set_mmhub_funcs(struct amdgpu_device 
*adev)
static void gmc_v10_0_set_gfxhub_funcs(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 3, 0):
        case IP_VERSION(10, 3, 2):
        case IP_VERSION(10, 3, 1):
@@ -857,7 +857,7 @@ static int gmc_v10_0_sw_init(void *handle)
                adev->gmc.vram_vendor = vram_vendor;
        }
- switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
        case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 2):
@@ -1132,7 +1132,7 @@ static int gmc_v10_0_set_clockgating_state(void *handle,
        if (r)
                return r;
- if (adev->ip_versions[ATHUB_HWIP] >= IP_VERSION(2, 1, 0))
+       if (adev->ip_versions[ATHUB_HWIP][0] >= IP_VERSION(2, 1, 0))
                return athub_v2_1_set_clockgating(adev, state);
        else
                return athub_v2_0_set_clockgating(adev, state);
@@ -1144,7 +1144,7 @@ static void gmc_v10_0_get_clockgating_state(void *handle, 
u32 *flags)
adev->mmhub.funcs->get_clockgating(adev, flags); - if (adev->ip_versions[ATHUB_HWIP] >= IP_VERSION(2, 1, 0))
+       if (adev->ip_versions[ATHUB_HWIP][0] >= IP_VERSION(2, 1, 0))
                athub_v2_1_get_clockgating(adev, flags);
        else
                athub_v2_0_get_clockgating(adev, flags);
diff --git a/drivers/gpu/drm/amd/amdgpu/hdp_v4_0.c 
b/drivers/gpu/drm/amd/amdgpu/hdp_v4_0.c
index 88938115557b..eecfb1545c1e 100644
--- a/drivers/gpu/drm/amd/amdgpu/hdp_v4_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/hdp_v4_0.c
@@ -49,7 +49,7 @@ static void hdp_v4_0_flush_hdp(struct amdgpu_device *adev,
  static void hdp_v4_0_invalidate_hdp(struct amdgpu_device *adev,
                                    struct amdgpu_ring *ring)
  {
-       if (adev->ip_versions[HDP_HWIP] == IP_VERSION(4, 4, 0))
+       if (adev->ip_versions[HDP_HWIP][0] == IP_VERSION(4, 4, 0))
                return;
if (!ring || !ring->funcs->emit_wreg)
@@ -79,7 +79,7 @@ static void hdp_v4_0_reset_ras_error_count(struct 
amdgpu_device *adev)
        if (!amdgpu_ras_is_supported(adev, AMDGPU_RAS_BLOCK__HDP))
                return;
- if (adev->ip_versions[HDP_HWIP] >= IP_VERSION(4, 4, 0))
+       if (adev->ip_versions[HDP_HWIP][0] >= IP_VERSION(4, 4, 0))
                WREG32_SOC15(HDP, 0, mmHDP_EDC_CNT, 0);
        else
                /*read back hdp ras counter to reset it to 0 */
@@ -91,10 +91,10 @@ static void hdp_v4_0_update_clock_gating(struct 
amdgpu_device *adev,
  {
        uint32_t def, data;
- if (adev->ip_versions[HDP_HWIP] == IP_VERSION(4, 0, 0) ||
-           adev->ip_versions[HDP_HWIP] == IP_VERSION(4, 0, 1) ||
-           adev->ip_versions[HDP_HWIP] == IP_VERSION(4, 1, 1) ||
-           adev->ip_versions[HDP_HWIP] == IP_VERSION(4, 1, 0)) {
+       if (adev->ip_versions[HDP_HWIP][0] == IP_VERSION(4, 0, 0) ||
+           adev->ip_versions[HDP_HWIP][0] == IP_VERSION(4, 0, 1) ||
+           adev->ip_versions[HDP_HWIP][0] == IP_VERSION(4, 1, 1) ||
+           adev->ip_versions[HDP_HWIP][0] == IP_VERSION(4, 1, 0)) {
                def = data = RREG32(SOC15_REG_OFFSET(HDP, 0, 
mmHDP_MEM_POWER_LS));
if (enable && (adev->cg_flags & AMD_CG_SUPPORT_HDP_LS))
@@ -136,7 +136,7 @@ static void hdp_v4_0_get_clockgating_state(struct 
amdgpu_device *adev,
static void hdp_v4_0_init_registers(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[HDP_HWIP]) {
+       switch (adev->ip_versions[HDP_HWIP][0]) {
        case IP_VERSION(4, 2, 1):
                WREG32_FIELD15(HDP, 0, HDP_MMHUB_CNTL, HDP_MMHUB_GCC, 1);
                break;
diff --git a/drivers/gpu/drm/amd/amdgpu/mmhub_v2_0.c 
b/drivers/gpu/drm/amd/amdgpu/mmhub_v2_0.c
index e0cb919b4814..25f8e93e5ec3 100644
--- a/drivers/gpu/drm/amd/amdgpu/mmhub_v2_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/mmhub_v2_0.c
@@ -153,7 +153,7 @@ mmhub_v2_0_print_l2_protection_fault_status(struct 
amdgpu_device *adev,
        dev_err(adev->dev,
                "MMVM_L2_PROTECTION_FAULT_STATUS:0x%08X\n",
                status);
-       switch (adev->ip_versions[MMHUB_HWIP]) {
+       switch (adev->ip_versions[MMHUB_HWIP][0]) {
        case IP_VERSION(2, 0, 0):
        case IP_VERSION(2, 0, 2):
                mmhub_cid = mmhub_client_ids_navi1x[cid][rw];
@@ -569,7 +569,7 @@ static void 
mmhub_v2_0_update_medium_grain_clock_gating(struct amdgpu_device *ad
        if (!(adev->cg_flags & AMD_CG_SUPPORT_MC_MGCG))
                return;
- switch (adev->ip_versions[MMHUB_HWIP]) {
+       switch (adev->ip_versions[MMHUB_HWIP][0]) {
        case IP_VERSION(2, 1, 0):
        case IP_VERSION(2, 1, 1):
        case IP_VERSION(2, 1, 2):
@@ -603,7 +603,7 @@ static void 
mmhub_v2_0_update_medium_grain_clock_gating(struct amdgpu_device *ad
                          DAGB0_CNTL_MISC2__DISABLE_TLBRD_CG_MASK);
        }
- switch (adev->ip_versions[MMHUB_HWIP]) {
+       switch (adev->ip_versions[MMHUB_HWIP][0]) {
        case IP_VERSION(2, 1, 0):
        case IP_VERSION(2, 1, 1):
        case IP_VERSION(2, 1, 2):
@@ -629,7 +629,7 @@ static void 
mmhub_v2_0_update_medium_grain_light_sleep(struct amdgpu_device *ade
        if (!(adev->cg_flags & AMD_CG_SUPPORT_MC_LS))
                return;
- switch (adev->ip_versions[MMHUB_HWIP]) {
+       switch (adev->ip_versions[MMHUB_HWIP][0]) {
        case IP_VERSION(2, 1, 0):
        case IP_VERSION(2, 1, 1):
        case IP_VERSION(2, 1, 2):
@@ -646,7 +646,7 @@ static void 
mmhub_v2_0_update_medium_grain_light_sleep(struct amdgpu_device *ade
                data &= ~MM_ATC_L2_MISC_CG__MEM_LS_ENABLE_MASK;
if (def != data) {
-               switch (adev->ip_versions[MMHUB_HWIP]) {
+               switch (adev->ip_versions[MMHUB_HWIP][0]) {
                case IP_VERSION(2, 1, 0):
                case IP_VERSION(2, 1, 1):
                case IP_VERSION(2, 1, 2):
@@ -665,7 +665,7 @@ static int mmhub_v2_0_set_clockgating(struct amdgpu_device 
*adev,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[MMHUB_HWIP]) {
+       switch (adev->ip_versions[MMHUB_HWIP][0]) {
        case IP_VERSION(2, 0, 0):
        case IP_VERSION(2, 0, 2):
        case IP_VERSION(2, 1, 0):
@@ -690,7 +690,7 @@ static void mmhub_v2_0_get_clockgating(struct amdgpu_device 
*adev, u32 *flags)
        if (amdgpu_sriov_vf(adev))
                *flags = 0;
- switch (adev->ip_versions[MMHUB_HWIP]) {
+       switch (adev->ip_versions[MMHUB_HWIP][0]) {
        case IP_VERSION(2, 1, 0):
        case IP_VERSION(2, 1, 1):
        case IP_VERSION(2, 1, 2):
diff --git a/drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c 
b/drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c
index 4cf5d0595bbe..a11d60ec6321 100644
--- a/drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c
+++ b/drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c
@@ -90,7 +90,7 @@ mmhub_v2_3_print_l2_protection_fault_status(struct 
amdgpu_device *adev,
        dev_err(adev->dev,
                "MMVM_L2_PROTECTION_FAULT_STATUS:0x%08X\n",
                status);
-       switch (adev->ip_versions[MMHUB_HWIP]) {
+       switch (adev->ip_versions[MMHUB_HWIP][0]) {
        case IP_VERSION(2, 3, 0):
        case IP_VERSION(2, 4, 0):
                mmhub_cid = mmhub_client_ids_vangogh[cid][rw];
diff --git a/drivers/gpu/drm/amd/amdgpu/navi10_ih.c 
b/drivers/gpu/drm/amd/amdgpu/navi10_ih.c
index c3f968d9cfcb..1d8414c3fadb 100644
--- a/drivers/gpu/drm/amd/amdgpu/navi10_ih.c
+++ b/drivers/gpu/drm/amd/amdgpu/navi10_ih.c
@@ -107,7 +107,7 @@ force_update_wptr_for_self_int(struct amdgpu_device *adev,
  {
        u32 ih_cntl, ih_rb_cntl;
- if (adev->ip_versions[OSSSYS_HWIP] < IP_VERSION(5, 0, 3))
+       if (adev->ip_versions[OSSSYS_HWIP][0] < IP_VERSION(5, 0, 3))
                return;
ih_cntl = RREG32_SOC15(OSSSYS, 0, mmIH_CNTL2);
@@ -332,7 +332,7 @@ static int navi10_ih_irq_init(struct amdgpu_device *adev)
if (unlikely(adev->firmware.load_type == AMDGPU_FW_LOAD_DIRECT)) {
                if (ih[0]->use_bus_addr) {
-                       switch (adev->ip_versions[OSSSYS_HWIP]) {
+                       switch (adev->ip_versions[OSSSYS_HWIP][0]) {
                        case IP_VERSION(5, 0, 3):
                        case IP_VERSION(5, 2, 0):
                        case IP_VERSION(5, 2, 1):
diff --git a/drivers/gpu/drm/amd/amdgpu/nv.c b/drivers/gpu/drm/amd/amdgpu/nv.c
index 57be517d70bf..89e32d6e2c10 100644
--- a/drivers/gpu/drm/amd/amdgpu/nv.c
+++ b/drivers/gpu/drm/amd/amdgpu/nv.c
@@ -180,7 +180,7 @@ static const struct amdgpu_video_codecs 
yc_video_codecs_decode = {
  static int nv_query_video_codecs(struct amdgpu_device *adev, bool encode,
                                 const struct amdgpu_video_codecs **codecs)
  {
-       switch (adev->ip_versions[UVD_HWIP]) {
+       switch (adev->ip_versions[UVD_HWIP][0]) {
        case IP_VERSION(3, 0, 0):
                if (amdgpu_sriov_vf(adev)) {
                        if (encode)
@@ -509,7 +509,7 @@ nv_asic_reset_method(struct amdgpu_device *adev)
                dev_warn(adev->dev, "Specified reset method:%d isn't supported, 
using AUTO instead.\n",
                                  amdgpu_reset_method);
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(11, 5, 0):
        case IP_VERSION(13, 0, 1):
        case IP_VERSION(13, 0, 3):
@@ -1044,7 +1044,7 @@ static int nv_common_early_init(void *handle)
        /* TODO: split the GC and PG flags based on the relevant IP version for 
which
         * they are relevant.
         */
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(10, 1, 10):
                adev->cg_flags = AMD_CG_SUPPORT_GFX_MGCG |
                        AMD_CG_SUPPORT_GFX_CGCG |
@@ -1375,7 +1375,7 @@ static int nv_common_set_clockgating_state(void *handle,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[NBIO_HWIP]) {
+       switch (adev->ip_versions[NBIO_HWIP][0]) {
        case IP_VERSION(2, 3, 0):
        case IP_VERSION(2, 3, 1):
        case IP_VERSION(2, 3, 2):
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c
index 13dace5d2855..382cebfc2069 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c
@@ -93,7 +93,7 @@ static int psp_v11_0_init_microcode(struct psp_context *psp)
DRM_DEBUG("\n"); - switch (adev->ip_versions[MP0_HWIP]) {
+       switch (adev->ip_versions[MP0_HWIP][0]) {
        case IP_VERSION(11, 0, 2):
                chip_name = "vega20";
                break;
@@ -129,7 +129,7 @@ static int psp_v11_0_init_microcode(struct psp_context *psp)
        }
- switch (adev->ip_versions[MP0_HWIP]) {
+       switch (adev->ip_versions[MP0_HWIP][0]) {
        case IP_VERSION(11, 0, 2):
        case IP_VERSION(11, 0, 4):
                err = psp_init_sos_microcode(psp, chip_name);
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c 
b/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c
index 64b52c5ea981..17655bc6d2f1 100644
--- a/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c
@@ -47,7 +47,7 @@ static int psp_v13_0_init_microcode(struct psp_context *psp)
        const char *chip_name;
        int err = 0;
- switch (adev->ip_versions[MP0_HWIP]) {
+       switch (adev->ip_versions[MP0_HWIP][0]) {
        case IP_VERSION(13, 0, 2):
                chip_name = "aldebaran";
                break;
@@ -58,7 +58,7 @@ static int psp_v13_0_init_microcode(struct psp_context *psp)
        default:
                BUG();
        }
-       switch (adev->ip_versions[MP0_HWIP]) {
+       switch (adev->ip_versions[MP0_HWIP][0]) {
        case IP_VERSION(13, 0, 2):
                err = psp_init_sos_microcode(psp, chip_name);
                if (err)
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c 
b/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
index cb74f8c35d0a..e8e4749e9c79 100644
--- a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
@@ -469,7 +469,7 @@ static int sdma_v4_0_irq_id_to_seq(unsigned client_id)
static void sdma_v4_0_init_golden_registers(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(4, 0, 0):
                soc15_program_register_sequence(adev,
                                                golden_settings_sdma_4,
@@ -539,7 +539,7 @@ static void sdma_v4_0_setup_ulv(struct amdgpu_device *adev)
         * The only chips with SDMAv4 and ULV are VG10 and VG20.
         * Server SKUs take a different hysteresis setting from other SKUs.
         */
-       switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(4, 0, 0):
                if (adev->pdev->device == 0x6860)
                        break;
@@ -590,8 +590,8 @@ static void sdma_v4_0_destroy_inst_ctx(struct amdgpu_device 
*adev)
/* arcturus shares the same FW memory across
                   all SDMA isntances */
-               if (adev->ip_versions[SDMA0_HWIP] == IP_VERSION(4, 2, 2) ||
-                   adev->ip_versions[SDMA0_HWIP] == IP_VERSION(4, 4, 0))
+               if (adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(4, 2, 2) ||
+                   adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(4, 4, 0))
                        break;
        }
@@ -621,7 +621,7 @@ static int sdma_v4_0_init_microcode(struct amdgpu_device *adev) DRM_DEBUG("\n"); - switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(4, 0, 0):
                chip_name = "vega10";
                break;
@@ -667,8 +667,8 @@ static int sdma_v4_0_init_microcode(struct amdgpu_device 
*adev)
                goto out;
for (i = 1; i < adev->sdma.num_instances; i++) {
-               if (adev->ip_versions[SDMA0_HWIP] == IP_VERSION(4, 2, 2) ||
-                    adev->ip_versions[SDMA0_HWIP] == IP_VERSION(4, 4, 0)) {
+               if (adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(4, 2, 2) ||
+                    adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(4, 4, 0)) {
                        /* Acturus & Aldebaran will leverage the same FW memory
                           for every SDMA instance */
                        memcpy((void *)&adev->sdma.instance[i],
@@ -1108,7 +1108,7 @@ static void sdma_v4_0_ctx_switch_enable(struct 
amdgpu_device *adev, bool enable)
                 * Arcturus for the moment and firmware version 14
                 * and above.
                 */
-               if (adev->ip_versions[SDMA0_HWIP] == IP_VERSION(4, 2, 2) &&
+               if (adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(4, 2, 2) &&
                    adev->sdma.instance[i].fw_version >= 14)
                        WREG32_SDMA(i, mmSDMA0_PUB_DUMMY_REG2, enable);
                /* Extend page fault timeout to avoid interrupt storm */
@@ -1395,7 +1395,7 @@ static void sdma_v4_0_init_pg(struct amdgpu_device *adev)
        if (!(adev->pg_flags & AMD_PG_SUPPORT_SDMA))
                return;
- switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(4, 1, 0):
          case IP_VERSION(4, 1, 1):
        case IP_VERSION(4, 1, 2):
@@ -1838,7 +1838,7 @@ static bool sdma_v4_0_fw_support_paging_queue(struct 
amdgpu_device *adev)
  {
        uint fw_version = adev->sdma.instance[0].fw_version;
- switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(4, 0, 0):
                return fw_version >= 430;
        case IP_VERSION(4, 0, 1):
@@ -1863,7 +1863,7 @@ static int sdma_v4_0_early_init(void *handle)
        }
/* TODO: Page queue breaks driver reload under SRIOV */
-       if ((adev->ip_versions[SDMA0_HWIP] == IP_VERSION(4, 0, 0)) &&
+       if ((adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(4, 0, 0)) &&
            amdgpu_sriov_vf((adev)))
                adev->sdma.has_page_queue = false;
        else if (sdma_v4_0_fw_support_paging_queue(adev))
@@ -2136,14 +2136,14 @@ static int sdma_v4_0_process_trap_irq(struct 
amdgpu_device *adev,
                amdgpu_fence_process(&adev->sdma.instance[instance].ring);
                break;
        case 1:
-               if (adev->ip_versions[SDMA0_HWIP] == IP_VERSION(4, 2, 0))
+               if (adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(4, 2, 0))
                        
amdgpu_fence_process(&adev->sdma.instance[instance].page);
                break;
        case 2:
                /* XXX compute */
                break;
        case 3:
-               if (adev->ip_versions[SDMA0_HWIP] != IP_VERSION(4, 2, 0))
+               if (adev->ip_versions[SDMA0_HWIP][0] != IP_VERSION(4, 2, 0))
                        
amdgpu_fence_process(&adev->sdma.instance[instance].page);
                break;
        }
@@ -2359,7 +2359,7 @@ static int sdma_v4_0_set_powergating_state(void *handle,
  {
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
- switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(4, 1, 0):
        case IP_VERSION(4, 1, 1):
        case IP_VERSION(4, 1, 2):
@@ -2547,7 +2547,7 @@ static void sdma_v4_0_set_ring_funcs(struct amdgpu_device 
*adev)
        int i;
for (i = 0; i < adev->sdma.num_instances; i++) {
-               if (adev->ip_versions[SDMA0_HWIP] == IP_VERSION(4, 2, 2) && i 
>= 5)
+               if (adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(4, 2, 2) && i 
>= 5)
                        adev->sdma.instance[i].ring.funcs =
                                        &sdma_v4_0_ring_funcs_2nd_mmhub;
                else
@@ -2555,7 +2555,7 @@ static void sdma_v4_0_set_ring_funcs(struct amdgpu_device 
*adev)
                                        &sdma_v4_0_ring_funcs;
                adev->sdma.instance[i].ring.me = i;
                if (adev->sdma.has_page_queue) {
-                       if (adev->ip_versions[SDMA0_HWIP] == IP_VERSION(4, 2, 2) 
&& i >= 5)
+                       if (adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(4, 2, 2) 
&& i >= 5)
                                adev->sdma.instance[i].page.funcs =
                                        &sdma_v4_0_page_ring_funcs_2nd_mmhub;
                        else
@@ -2782,7 +2782,7 @@ static const struct amdgpu_sdma_ras_funcs 
sdma_v4_0_ras_funcs = {
static void sdma_v4_0_set_ras_funcs(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(4, 2, 0):
        case IP_VERSION(4, 2, 2):
                adev->sdma.funcs = &sdma_v4_0_ras_funcs;
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c 
b/drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c
index d3b01d2977c0..853d1511b889 100644
--- a/drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c
@@ -187,7 +187,7 @@ static u32 sdma_v5_0_get_reg_offset(struct amdgpu_device 
*adev, u32 instance, u3
static void sdma_v5_0_init_golden_registers(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(5, 0, 0):
                soc15_program_register_sequence(adev,
                                                golden_settings_sdma_5,
@@ -248,12 +248,12 @@ static int sdma_v5_0_init_microcode(struct amdgpu_device 
*adev)
        const struct common_firmware_header *header = NULL;
        const struct sdma_firmware_header_v1_0 *hdr;
- if (amdgpu_sriov_vf(adev) && (adev->ip_versions[SDMA0_HWIP] == IP_VERSION(5, 0, 5)))
+       if (amdgpu_sriov_vf(adev) && (adev->ip_versions[SDMA0_HWIP][0] == 
IP_VERSION(5, 0, 5)))
                return 0;
DRM_DEBUG("\n"); - switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(5, 0, 0):
                chip_name = "navi10";
                break;
@@ -1634,7 +1634,7 @@ static int sdma_v5_0_set_clockgating_state(void *handle,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(5, 0, 0):
        case IP_VERSION(5, 0, 2):
        case IP_VERSION(5, 0, 5):
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c 
b/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
index 92f29beda3b2..0fec1d9dd638 100644
--- a/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
+++ b/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
@@ -136,7 +136,7 @@ static int sdma_v5_2_init_microcode(struct amdgpu_device 
*adev)
DRM_DEBUG("\n"); - switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(5, 2, 0):
                chip_name = "sienna_cichlid";
                break;
@@ -174,7 +174,7 @@ static int sdma_v5_2_init_microcode(struct amdgpu_device 
*adev)
                       (void *)&adev->sdma.instance[0],
                       sizeof(struct amdgpu_sdma_instance));
- if (amdgpu_sriov_vf(adev) && (adev->ip_versions[SDMA0_HWIP] == IP_VERSION(5, 2, 0)))
+       if (amdgpu_sriov_vf(adev) && (adev->ip_versions[SDMA0_HWIP][0] == 
IP_VERSION(5, 2, 0)))
                return 0;
DRM_DEBUG("psp_load == '%s'\n",
@@ -1530,7 +1530,7 @@ static void 
sdma_v5_2_update_medium_grain_clock_gating(struct amdgpu_device *ade
for (i = 0; i < adev->sdma.num_instances; i++) { - if (adev->sdma.instance[i].fw_version < 70 && adev->ip_versions[SDMA0_HWIP] == IP_VERSION(5, 2, 1))
+               if (adev->sdma.instance[i].fw_version < 70 && 
adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(5, 2, 1))
                        adev->cg_flags &= ~AMD_CG_SUPPORT_SDMA_MGCG;
if (enable && (adev->cg_flags & AMD_CG_SUPPORT_SDMA_MGCG)) {
@@ -1567,7 +1567,7 @@ static void 
sdma_v5_2_update_medium_grain_light_sleep(struct amdgpu_device *adev
for (i = 0; i < adev->sdma.num_instances; i++) { - if (adev->sdma.instance[i].fw_version < 70 && adev->ip_versions[SDMA0_HWIP] == IP_VERSION(5, 2, 1))
+               if (adev->sdma.instance[i].fw_version < 70 && 
adev->ip_versions[SDMA0_HWIP][0] == IP_VERSION(5, 2, 1))
                        adev->cg_flags &= ~AMD_CG_SUPPORT_SDMA_LS;
if (enable && (adev->cg_flags & AMD_CG_SUPPORT_SDMA_LS)) {
@@ -1596,7 +1596,7 @@ static int sdma_v5_2_set_clockgating_state(void *handle,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[SDMA0_HWIP]) {
+       switch (adev->ip_versions[SDMA0_HWIP][0]) {
        case IP_VERSION(5, 2, 0):
        case IP_VERSION(5, 2, 2):
        case IP_VERSION(5, 2, 1):
diff --git a/drivers/gpu/drm/amd/amdgpu/soc15.c 
b/drivers/gpu/drm/amd/amdgpu/soc15.c
index 7f4cd1a4f6cb..74310bb4216a 100644
--- a/drivers/gpu/drm/amd/amdgpu/soc15.c
+++ b/drivers/gpu/drm/amd/amdgpu/soc15.c
@@ -156,8 +156,8 @@ static const struct amdgpu_video_codecs 
rn_video_codecs_decode =
  static int soc15_query_video_codecs(struct amdgpu_device *adev, bool encode,
                                    const struct amdgpu_video_codecs **codecs)
  {
-       if (adev->ip_versions[VCE_HWIP]) {
-               switch (adev->ip_versions[VCE_HWIP]) {
+       if (adev->ip_versions[VCE_HWIP][0]) {
+               switch (adev->ip_versions[VCE_HWIP][0]) {
                case IP_VERSION(4, 0, 0):
                case IP_VERSION(4, 1, 0):
                        if (encode)
@@ -169,7 +169,7 @@ static int soc15_query_video_codecs(struct amdgpu_device 
*adev, bool encode,
                        return -EINVAL;
                }
        } else {
-               switch (adev->ip_versions[UVD_HWIP]) {
+               switch (adev->ip_versions[UVD_HWIP][0]) {
                case IP_VERSION(1, 0, 0):
                case IP_VERSION(1, 0, 1):
                        if (encode)
@@ -341,11 +341,11 @@ static u32 soc15_get_xclk(struct amdgpu_device *adev)
  {
        u32 reference_clock = adev->clock.spll.reference_freq;
- if (adev->ip_versions[MP1_HWIP] == IP_VERSION(12, 0, 0) ||
-           adev->ip_versions[MP1_HWIP] == IP_VERSION(12, 0, 1))
+       if (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(12, 0, 0) ||
+           adev->ip_versions[MP1_HWIP][0] == IP_VERSION(12, 0, 1))
                return 10000;
-       if (adev->ip_versions[MP1_HWIP] == IP_VERSION(10, 0, 0) ||
-           adev->ip_versions[MP1_HWIP] == IP_VERSION(10, 0, 1))
+       if (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(10, 0, 0) ||
+           adev->ip_versions[MP1_HWIP][0] == IP_VERSION(10, 0, 1))
                return reference_clock / 4;
return reference_clock;
@@ -576,7 +576,7 @@ soc15_asic_reset_method(struct amdgpu_device *adev)
                dev_warn(adev->dev, "Specified reset method:%d isn't supported, 
using AUTO instead.\n",
                                  amdgpu_reset_method);
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(10, 0, 0):
        case IP_VERSION(10, 0, 1):
        case IP_VERSION(12, 0, 0):
@@ -641,7 +641,7 @@ static int soc15_asic_reset(struct amdgpu_device *adev)
static bool soc15_supports_baco(struct amdgpu_device *adev)
  {
-       switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(9, 0, 0):
        case IP_VERSION(11, 0, 2):
                if (adev->asic_type == CHIP_VEGA20) {
@@ -1172,7 +1172,7 @@ static int soc15_common_early_init(void *handle)
        /* TODO: split the GC and PG flags based on the relevant IP version for 
which
         * they are relevant.
         */
-       switch (adev->ip_versions[GC_HWIP]) {
+       switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
                adev->asic_funcs = &soc15_asic_funcs;
                adev->cg_flags = AMD_CG_SUPPORT_GFX_MGCG |
@@ -1583,7 +1583,7 @@ static int soc15_common_set_clockgating_state(void 
*handle,
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[NBIO_HWIP]) {
+       switch (adev->ip_versions[NBIO_HWIP][0]) {
        case IP_VERSION(6, 1, 0):
        case IP_VERSION(6, 2, 0):
        case IP_VERSION(7, 4, 0):
@@ -1639,7 +1639,7 @@ static void soc15_common_get_clockgating_state(void 
*handle, u32 *flags)
adev->hdp.funcs->get_clock_gating_state(adev, flags); - if (adev->ip_versions[MP0_HWIP] != IP_VERSION(13, 0, 2)) {
+       if (adev->ip_versions[MP0_HWIP][0] != IP_VERSION(13, 0, 2)) {
/* AMD_CG_SUPPORT_DRM_MGCG */
                data = RREG32(SOC15_REG_OFFSET(MP0, 0, mmMP0_MISC_CGTT_CTRL0));
diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c 
b/drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c
index e9758969fbe6..f4a7a30ecebd 100644
--- a/drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c
+++ b/drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c
@@ -1720,7 +1720,7 @@ static void vcn_v2_5_set_dec_ring_funcs(struct 
amdgpu_device *adev)
        for (i = 0; i < adev->vcn.num_vcn_inst; ++i) {
                if (adev->vcn.harvest_config & (1 << i))
                        continue;
-               if (adev->ip_versions[UVD_HWIP] == IP_VERSION(2, 5, 0))
+               if (adev->ip_versions[UVD_HWIP][0] == IP_VERSION(2, 5, 0))
                        adev->vcn.inst[i].ring_dec.funcs = 
&vcn_v2_5_dec_ring_vm_funcs;
                else /* CHIP_ALDEBARAN */
                        adev->vcn.inst[i].ring_dec.funcs = 
&vcn_v2_6_dec_ring_vm_funcs;
@@ -1737,7 +1737,7 @@ static void vcn_v2_5_set_enc_ring_funcs(struct 
amdgpu_device *adev)
                if (adev->vcn.harvest_config & (1 << j))
                        continue;
                for (i = 0; i < adev->vcn.num_enc_rings; ++i) {
-                       if (adev->ip_versions[UVD_HWIP] == IP_VERSION(2, 5, 0))
+                       if (adev->ip_versions[UVD_HWIP][0] == IP_VERSION(2, 5, 
0))
                                adev->vcn.inst[j].ring_enc[i].funcs = 
&vcn_v2_5_enc_ring_vm_funcs;
                        else /* CHIP_ALDEBARAN */
                                adev->vcn.inst[j].ring_enc[i].funcs = 
&vcn_v2_6_enc_ring_vm_funcs;
diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c 
b/drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c
index a81d834ea0d3..6940716db1df 100644
--- a/drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c
@@ -95,7 +95,7 @@ static int vcn_v3_0_early_init(void *handle)
                adev->vcn.num_enc_rings = 1;
} else {
-               if (adev->ip_versions[UVD_HWIP] == IP_VERSION(3, 0, 0)) {
+               if (adev->ip_versions[UVD_HWIP][0] == IP_VERSION(3, 0, 0)) {
                        u32 harvest;
for (i = 0; i < adev->vcn.num_vcn_inst; i++) {
@@ -110,7 +110,7 @@ static int vcn_v3_0_early_init(void *handle)
                                return -ENOENT;
                }
- if (adev->ip_versions[UVD_HWIP] == IP_VERSION(3, 0, 33))
+               if (adev->ip_versions[UVD_HWIP][0] == IP_VERSION(3, 0, 33))
                        adev->vcn.num_enc_rings = 0;
                else
                        adev->vcn.num_enc_rings = 2;
@@ -1270,7 +1270,7 @@ static int vcn_v3_0_start(struct amdgpu_device *adev)
                fw_shared->rb.wptr = lower_32_bits(ring->wptr);
                fw_shared->multi_queue.decode_queue_mode &= 
cpu_to_le32(~FW_QUEUE_RING_RESET);
- if (adev->ip_versions[UVD_HWIP] != IP_VERSION(3, 0, 33)) {
+               if (adev->ip_versions[UVD_HWIP][0] != IP_VERSION(3, 0, 33)) {
                        fw_shared->multi_queue.encode_generalpurpose_queue_mode 
|= cpu_to_le32(FW_QUEUE_RING_RESET);
                        ring = &adev->vcn.inst[i].ring_enc[0];
                        WREG32_SOC15(VCN, i, mmUVD_RB_RPTR, 
lower_32_bits(ring->wptr));
@@ -1642,7 +1642,7 @@ static int vcn_v3_0_pause_dpg_mode(struct amdgpu_device 
*adev,
                                        
UVD_POWER_STATUS__STALL_DPG_POWER_UP_MASK,
                                        
~UVD_POWER_STATUS__STALL_DPG_POWER_UP_MASK);
- if (adev->ip_versions[UVD_HWIP] != IP_VERSION(3, 0, 33)) {
+                               if (adev->ip_versions[UVD_HWIP][0] != 
IP_VERSION(3, 0, 33)) {
                                        /* Restore */
                                        fw_shared = 
adev->vcn.inst[inst_idx].fw_shared_cpu_addr;
                                        
fw_shared->multi_queue.encode_generalpurpose_queue_mode |= 
cpu_to_le32(FW_QUEUE_RING_RESET);
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index fc8a30251fa4..fb89ce4dd28c 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -1344,7 +1344,7 @@ static int amdgpu_dm_init(struct amdgpu_device *adev)
                init_data.flags.gpu_vm_support = true;
                break;
        default:
-               switch (adev->ip_versions[DCE_HWIP]) {
+               switch (adev->ip_versions[DCE_HWIP][0]) {
                case IP_VERSION(2, 1, 0):
                        init_data.flags.gpu_vm_support = true;
                        if (ASICREV_IS_GREEN_SARDINE(adev->external_rev_id))
@@ -1658,7 +1658,7 @@ static int load_dmcu_fw(struct amdgpu_device *adev)
                        return 0;
                break;
        default:
-               switch (adev->ip_versions[DCE_HWIP]) {
+               switch (adev->ip_versions[DCE_HWIP][0]) {
                case IP_VERSION(2, 0, 2):
                case IP_VERSION(2, 0, 0):
                case IP_VERSION(2, 1, 0):
@@ -1750,7 +1750,7 @@ static int dm_dmub_sw_init(struct amdgpu_device *adev)
        enum dmub_status status;
        int r;
- switch (adev->ip_versions[DCE_HWIP]) {
+       switch (adev->ip_versions[DCE_HWIP][0]) {
        case IP_VERSION(2, 1, 0):
                dmub_asic = DMUB_ASIC_DCN21;
                fw_name_dmub = FIRMWARE_RENOIR_DMUB;
@@ -1758,7 +1758,7 @@ static int dm_dmub_sw_init(struct amdgpu_device *adev)
                        fw_name_dmub = FIRMWARE_GREEN_SARDINE_DMUB;
                break;
        case IP_VERSION(3, 0, 0):
-               if (adev->ip_versions[GC_HWIP] == IP_VERSION(10, 3, 0)) {
+               if (adev->ip_versions[GC_HWIP][0] == IP_VERSION(10, 3, 0)) {
                        dmub_asic = DMUB_ASIC_DCN30;
                        fw_name_dmub = FIRMWARE_SIENNA_CICHLID_DMUB;
                } else {
@@ -2078,7 +2078,7 @@ static int amdgpu_dm_smu_write_watermarks_table(struct 
amdgpu_device *adev)
         * therefore, this function apply to navi10/12/14 but not Renoir
         * *
         */
-       switch (adev->ip_versions[DCE_HWIP]) {
+       switch (adev->ip_versions[DCE_HWIP][0]) {
        case IP_VERSION(2, 0, 2):
        case IP_VERSION(2, 0, 0):
                break;
@@ -4086,7 +4086,7 @@ static int amdgpu_dm_initialize_drm_device(struct 
amdgpu_device *adev)
#if defined(CONFIG_DRM_AMD_DC_DCN)
        /* Use Outbox interrupt */
-       switch (adev->ip_versions[DCE_HWIP]) {
+       switch (adev->ip_versions[DCE_HWIP][0]) {
        case IP_VERSION(3, 0, 0):
        case IP_VERSION(3, 1, 2):
        case IP_VERSION(3, 1, 3):
@@ -4098,7 +4098,7 @@ static int amdgpu_dm_initialize_drm_device(struct 
amdgpu_device *adev)
                break;
        default:
                DRM_DEBUG_KMS("Unsupported DCN IP version for outbox: 0x%X\n",
-                             adev->ip_versions[DCE_HWIP]);
+                             adev->ip_versions[DCE_HWIP][0]);
        }
  #endif
@@ -4186,7 +4186,7 @@ static int amdgpu_dm_initialize_drm_device(struct amdgpu_device *adev)
                break;
        default:
  #if defined(CONFIG_DRM_AMD_DC_DCN)
-               switch (adev->ip_versions[DCE_HWIP]) {
+               switch (adev->ip_versions[DCE_HWIP][0]) {
                case IP_VERSION(1, 0, 0):
                case IP_VERSION(1, 0, 1):
                case IP_VERSION(2, 0, 2):
@@ -4205,7 +4205,7 @@ static int amdgpu_dm_initialize_drm_device(struct 
amdgpu_device *adev)
                        break;
                default:
                        DRM_ERROR("Unsupported DCE IP versions: 0x%X\n",
-                                       adev->ip_versions[DCE_HWIP]);
+                                       adev->ip_versions[DCE_HWIP][0]);
                        goto fail;
                }
  #endif
@@ -4357,7 +4357,7 @@ static int dm_early_init(void *handle)
                break;
        default:
  #if defined(CONFIG_DRM_AMD_DC_DCN)
-               switch (adev->ip_versions[DCE_HWIP]) {
+               switch (adev->ip_versions[DCE_HWIP][0]) {
                case IP_VERSION(2, 0, 2):
                case IP_VERSION(3, 0, 0):
                        adev->mode_info.num_crtc = 6;
@@ -4387,7 +4387,7 @@ static int dm_early_init(void *handle)
                        break;
                default:
                        DRM_ERROR("Unsupported DCE IP versions: 0x%x\n",
-                                       adev->ip_versions[DCE_HWIP]);
+                                       adev->ip_versions[DCE_HWIP][0]);
                        return -EINVAL;
                }
  #endif
@@ -4609,7 +4609,7 @@ fill_gfx9_tiling_info_from_device(const struct 
amdgpu_device *adev,
        tiling_info->gfx9.num_rb_per_se =
                adev->gfx.config.gb_addr_config_fields.num_rb_per_se;
        tiling_info->gfx9.shaderEnable = 1;
-       if (adev->ip_versions[GC_HWIP] >= IP_VERSION(10, 3, 0))
+       if (adev->ip_versions[GC_HWIP][0] >= IP_VERSION(10, 3, 0))
                tiling_info->gfx9.num_pkrs = 
adev->gfx.config.gb_addr_config_fields.num_pkrs;
  }
@@ -5050,7 +5050,7 @@ get_plane_modifiers(const struct amdgpu_device *adev, unsigned int plane_type, u
        case AMDGPU_FAMILY_NV:
        case AMDGPU_FAMILY_VGH:
        case AMDGPU_FAMILY_YC:
-               if (adev->ip_versions[GC_HWIP] >= IP_VERSION(10, 3, 0))
+               if (adev->ip_versions[GC_HWIP][0] >= IP_VERSION(10, 3, 0))
                        add_gfx10_3_modifiers(adev, mods, &size, &capacity);
                else
                        add_gfx10_1_modifiers(adev, mods, &size, &capacity);
diff --git a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c 
b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
index 150cac4ea75c..9c601997a96a 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
@@ -455,7 +455,7 @@ static int smu_get_power_num_states(void *handle,
bool is_support_sw_smu(struct amdgpu_device *adev)
  {
-       if (adev->ip_versions[MP1_HWIP] >= IP_VERSION(11, 0, 0))
+       if (adev->ip_versions[MP1_HWIP][0] >= IP_VERSION(11, 0, 0))
                return true;
return false;
@@ -575,7 +575,7 @@ static int smu_set_funcs(struct amdgpu_device *adev)
        if (adev->pm.pp_feature & PP_OVERDRIVE_MASK)
                smu->od_enabled = true;
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(11, 0, 0):
        case IP_VERSION(11, 0, 5):
        case IP_VERSION(11, 0, 9):
@@ -694,7 +694,7 @@ static int smu_late_init(void *handle)
                return ret;
        }
- if (adev->ip_versions[MP1_HWIP] == IP_VERSION(13, 0, 1))
+       if (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(13, 0, 1))
                return 0;
if (!amdgpu_sriov_vf(adev) || smu->od_enabled) {
@@ -1140,10 +1140,10 @@ static int smu_smc_hw_setup(struct smu_context *smu)
        if (adev->in_suspend && smu_is_dpm_running(smu)) {
                dev_info(adev->dev, "dpm has been enabled\n");
                /* this is needed specifically */
-               if ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7)) ||
-                   (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 11)) ||
-                   (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 5, 0)) ||
-                   (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 12)))
+               if ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7)) ||
+                   (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 11)) ||
+                   (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 5, 0)) ||
+                   (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 12)))
                        ret = smu_system_features_control(smu, true);
                return ret;
        }
@@ -1286,7 +1286,7 @@ static int smu_start_smc_engine(struct smu_context *smu)
        int ret = 0;
if (adev->firmware.load_type != AMDGPU_FW_LOAD_PSP) {
-               if (adev->ip_versions[MP1_HWIP] < IP_VERSION(11, 0, 0)) {
+               if (adev->ip_versions[MP1_HWIP][0] < IP_VERSION(11, 0, 0)) {
                        if (smu->ppt_funcs->load_microcode) {
                                ret = smu->ppt_funcs->load_microcode(smu);
                                if (ret)
@@ -1405,14 +1405,14 @@ static int smu_disable_dpms(struct smu_context *smu)
         *     properly.
         */
        if (smu->uploading_custom_pp_table &&
-           ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 5)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 9)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 11)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 5, 0)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 12)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 13))))
+           ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 0)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 5)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 9)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 11)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 5, 0)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 12)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 13))))
                return smu_disable_all_features_with_exception(smu,
                                                               true,
                                                               
SMU_FEATURE_COUNT);
@@ -1421,10 +1421,10 @@ static int smu_disable_dpms(struct smu_context *smu)
         * For Sienna_Cichlid, PMFW will handle the features disablement 
properly
         * on BACO in. Driver involvement is unnecessary.
         */
-       if (((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 5)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 9))) &&
+       if (((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 0)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 5)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 9))) &&
            use_baco)
                return smu_disable_all_features_with_exception(smu,
                                                               true,
@@ -1446,7 +1446,7 @@ static int smu_disable_dpms(struct smu_context *smu)
                        dev_err(adev->dev, "Failed to disable smu features.\n");
        }
- if (adev->ip_versions[MP1_HWIP] >= IP_VERSION(11, 0, 0) &&
+       if (adev->ip_versions[MP1_HWIP][0] >= IP_VERSION(11, 0, 0) &&
            adev->gfx.rlc.funcs->stop)
                adev->gfx.rlc.funcs->stop(adev);
@@ -2283,11 +2283,11 @@ int smu_get_power_limit(void *handle,
        } else {
                switch (limit_level) {
                case SMU_PPT_LIMIT_CURRENT:
-                       if ((adev->ip_versions[MP1_HWIP] == IP_VERSION(13, 0, 
2)) ||
-                           (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 
7)) ||
-                           (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 
11)) ||
-                           (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 
12)) ||
-                           (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 
13)))
+                       if ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(13, 
0, 2)) ||
+                           (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 
0, 7)) ||
+                           (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 
0, 11)) ||
+                           (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 
0, 12)) ||
+                           (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 
0, 13)))
                                ret = smu_get_asic_power_limits(smu,
                                                                
&smu->current_power_limit,
                                                                NULL,
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c
index d96f904a765b..71161f6b78fe 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c
@@ -345,7 +345,7 @@ navi10_get_allowed_feature_mask(struct smu_context *smu,
/* DPM UCLK enablement should be skipped for navi10 A0 secure board */
        if (!(is_asic_secure(smu) &&
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0)) &&
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 0)) &&
             (adev->rev_id == 0)) &&
            (adev->pm.pp_feature & PP_MCLK_DPM_MASK))
                *(uint64_t *)feature_mask |= FEATURE_MASK(FEATURE_DPM_UCLK_BIT)
@@ -354,7 +354,7 @@ navi10_get_allowed_feature_mask(struct smu_context *smu,
/* DS SOCCLK enablement should be skipped for navi10 A0 secure board */
        if (is_asic_secure(smu) &&
-           (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0)) &&
+           (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 0)) &&
            (adev->rev_id == 0))
                *(uint64_t *)feature_mask &=
                                ~FEATURE_MASK(FEATURE_DS_SOCCLK_BIT);
@@ -925,7 +925,7 @@ static int navi1x_get_smu_metrics_data(struct smu_context 
*smu,
                return ret;
        }
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(11, 0, 9):
                if (smu_version > 0x00341C00)
                        ret = navi12_get_smu_metrics_data(smu, member, value);
@@ -935,8 +935,8 @@ static int navi1x_get_smu_metrics_data(struct smu_context 
*smu,
        case IP_VERSION(11, 0, 0):
        case IP_VERSION(11, 0, 5):
        default:
-               if (((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 5)) && 
smu_version > 0x00351F00) ||
-                     ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0)) && 
smu_version > 0x002A3B00))
+               if (((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 5)) && 
smu_version > 0x00351F00) ||
+                     ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 0)) && 
smu_version > 0x002A3B00))
                        ret = navi10_get_smu_metrics_data(smu, member, value);
                else
                        ret = navi10_get_legacy_smu_metrics_data(smu, member, 
value);
@@ -1509,7 +1509,7 @@ static int navi10_populate_umd_state_clk(struct 
smu_context *smu)
        uint32_t sclk_freq;
pstate_table->gfxclk_pstate.min = gfx_table->min;
-       switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(11, 0, 0):
                switch (adev->pdev->revision) {
                case 0xf0: /* XTX */
@@ -2562,8 +2562,8 @@ static bool navi10_need_umc_cdr_workaround(struct 
smu_context *smu)
        if (!smu_cmn_feature_is_enabled(smu, SMU_FEATURE_DPM_UCLK_BIT))
                return false;
- if (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0) ||
-           adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 5))
+       if (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 0) ||
+           adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 5))
                return true;
return false;
@@ -2671,8 +2671,8 @@ static int navi10_run_umc_cdr_workaround(struct 
smu_context *smu)
         * - PPSMC_MSG_SetDriverDummyTableDramAddrLow
         * - PPSMC_MSG_GetUMCFWWA
         */
-       if (((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0)) && 
(pmfw_version >= 0x2a3500)) ||
-           ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 5)) && 
(pmfw_version >= 0x351D00))) {
+       if (((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 0)) && 
(pmfw_version >= 0x2a3500)) ||
+           ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 5)) && 
(pmfw_version >= 0x351D00))) {
                ret = smu_cmn_send_smc_msg_with_param(smu,
                                                      SMU_MSG_GET_UMC_FW_WA,
                                                      0,
@@ -2691,13 +2691,13 @@ static int navi10_run_umc_cdr_workaround(struct 
smu_context *smu)
                        return 0;
if (umc_fw_disable_cdr) {
-                       if (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0))
+                       if (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 
0))
                                return navi10_umc_hybrid_cdr_workaround(smu);
                } else {
                        return navi10_set_dummy_pstates_table_location(smu);
                }
        } else {
-               if (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0))
+               if (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 0))
                        return navi10_umc_hybrid_cdr_workaround(smu);
        }
@@ -3151,7 +3151,7 @@ static ssize_t navi1x_get_gpu_metrics(struct smu_context *smu,
                return ret;
        }
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(11, 0, 9):
                if (smu_version > 0x00341C00)
                        ret = navi12_get_gpu_metrics(smu, table);
@@ -3161,8 +3161,8 @@ static ssize_t navi1x_get_gpu_metrics(struct smu_context 
*smu,
        case IP_VERSION(11, 0, 0):
        case IP_VERSION(11, 0, 5):
        default:
-               if (((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 5)) && 
smu_version > 0x00351F00) ||
-                     ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 0)) && 
smu_version > 0x002A3B00))
+               if (((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 5)) && 
smu_version > 0x00351F00) ||
+                     ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 0)) && 
smu_version > 0x002A3B00))
                        ret = navi10_get_gpu_metrics(smu, table);
                else
                        ret =navi10_get_legacy_gpu_metrics(smu, table);
@@ -3180,7 +3180,7 @@ static int navi10_enable_mgpu_fan_boost(struct 
smu_context *smu)
        uint32_t param = 0;
/* Navi12 does not support this */
-       if (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 9))
+       if (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 9))
                return 0;
/*
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
index 6a0fed5e33d9..15e66e1912de 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
@@ -74,7 +74,7 @@
  #define SMU_11_0_7_GFX_BUSY_THRESHOLD 15
#define GET_PPTABLE_MEMBER(field, member) do {\
-       if (smu->adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 13))\
+       if (smu->adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 13))\
                (*member) = (smu->smu_table.driver_pptable + 
offsetof(PPTable_beige_goby_t, field));\
        else\
                (*member) = (smu->smu_table.driver_pptable + 
offsetof(PPTable_t, field));\
@@ -82,7 +82,7 @@
static int get_table_size(struct smu_context *smu)
  {
-       if (smu->adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 13))
+       if (smu->adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 13))
                return sizeof(PPTable_beige_goby_t);
        else
                return sizeof(PPTable_t);
@@ -298,7 +298,7 @@ sienna_cichlid_get_allowed_feature_mask(struct smu_context 
*smu,
        }
if ((adev->pm.pp_feature & PP_GFX_DCS_MASK) &&
-           (adev->ip_versions[MP1_HWIP] > IP_VERSION(11, 0, 7)) &&
+           (adev->ip_versions[MP1_HWIP][0] > IP_VERSION(11, 0, 7)) &&
            !(adev->flags & AMD_IS_APU))
                *(uint64_t *)feature_mask |= FEATURE_MASK(FEATURE_GFX_DCS_BIT);
@@ -496,7 +496,7 @@ static uint32_t sienna_cichlid_get_throttler_status_locked(struct smu_context *s
        uint32_t throttler_status = 0;
        int i;
- if ((smu->adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7)) &&
+       if ((smu->adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7)) &&
             (smu->smc_fw_version >= 0x3A4300)) {
                for (i = 0; i < THROTTLER_COUNT; i++)
                        throttler_status |=
@@ -517,7 +517,7 @@ static int sienna_cichlid_get_smu_metrics_data(struct 
smu_context *smu,
                &(((SmuMetricsExternal_t 
*)(smu_table->metrics_table))->SmuMetrics);
        SmuMetrics_V2_t *metrics_v2 =
                &(((SmuMetricsExternal_t 
*)(smu_table->metrics_table))->SmuMetrics_V2);
-       bool use_metrics_v2 = ((smu->adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 
7)) &&
+       bool use_metrics_v2 = ((smu->adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 
0, 7)) &&
                (smu->smc_fw_version >= 0x3A4300)) ? true : false;
        uint16_t average_gfx_activity;
        int ret = 0;
@@ -1170,7 +1170,7 @@ static int sienna_cichlid_print_clk_levels(struct 
smu_context *smu,
                 * and onwards SMU firmwares.
                 */
                smu_cmn_get_smc_version(smu, NULL, &smu_version);
-               if ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7)) &&
+               if ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7)) &&
                     (smu_version < 0x003a2900))
                        break;
@@ -1937,7 +1937,7 @@ static void sienna_cichlid_dump_od_table(struct smu_context *smu,
                                                        od_table->UclkFmax);
smu_cmn_get_smc_version(smu, NULL, &smu_version);
-       if (!((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7)) &&
+       if (!((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7)) &&
               (smu_version < 0x003a2900)))
                dev_dbg(smu->adev->dev, "OD: VddGfxOffset: %d\n", 
od_table->VddGfxOffset);
  }
@@ -2161,7 +2161,7 @@ static int sienna_cichlid_od_edit_dpm_table(struct 
smu_context *smu,
                 * and onwards SMU firmwares.
                 */
                smu_cmn_get_smc_version(smu, NULL, &smu_version);
-               if ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7)) &&
+               if ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7)) &&
                     (smu_version < 0x003a2900)) {
                        dev_err(smu->adev->dev, "OD GFX Voltage offset functionality 
is supported "
                                                "only by 58.41.0 and onwards SMU 
firmwares!\n");
@@ -2865,7 +2865,7 @@ static void sienna_cichlid_dump_pptable(struct 
smu_context *smu)
        PPTable_t *pptable = table_context->driver_pptable;
        int i;
- if (smu->adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 13)) {
+       if (smu->adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 13)) {
                beige_goby_dump_pptable(smu);
                return;
        }
@@ -3625,7 +3625,7 @@ static ssize_t sienna_cichlid_get_gpu_metrics(struct 
smu_context *smu,
        SmuMetrics_V2_t *metrics_v2 =
                &(metrics_external.SmuMetrics_V2);
        struct amdgpu_device *adev = smu->adev;
-       bool use_metrics_v2 = ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7)) 
&&
+       bool use_metrics_v2 = ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7)) 
&&
                (smu->smc_fw_version >= 0x3A4300)) ? true : false;
        uint16_t average_gfx_activity;
        int ret = 0;
@@ -3706,8 +3706,8 @@ static ssize_t sienna_cichlid_get_gpu_metrics(struct 
smu_context *smu,
gpu_metrics->current_fan_speed = use_metrics_v2 ? metrics_v2->CurrFanSpeed : metrics->CurrFanSpeed; - if (((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7)) && smu->smc_fw_version > 0x003A1E00) ||
-             ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 11)) && 
smu->smc_fw_version > 0x00410400)) {
+       if (((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7)) && 
smu->smc_fw_version > 0x003A1E00) ||
+             ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 11)) && 
smu->smc_fw_version > 0x00410400)) {
                gpu_metrics->pcie_link_width = use_metrics_v2 ? 
metrics_v2->PcieWidth : metrics->PcieWidth;
                gpu_metrics->pcie_link_speed = link_speed[use_metrics_v2 ? 
metrics_v2->PcieRate : metrics->PcieRate];
        } else {
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/smu_v11_0.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu11/smu_v11_0.c
index 1ae8b8dc3bd3..3470c33ee09d 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu11/smu_v11_0.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/smu_v11_0.c
@@ -90,11 +90,11 @@ int smu_v11_0_init_microcode(struct smu_context *smu)
        struct amdgpu_firmware_info *ucode = NULL;
if (amdgpu_sriov_vf(adev) &&
-           ((adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 9)) ||
-            (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 7))))
+           ((adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 9)) ||
+            (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 7))))
                return 0;
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(11, 0, 0):
                chip_name = "navi10";
                break;
@@ -121,7 +121,7 @@ int smu_v11_0_init_microcode(struct smu_context *smu)
                break;
        default:
                dev_err(adev->dev, "Unsupported IP version 0x%x\n",
-                       adev->ip_versions[MP1_HWIP]);
+                       adev->ip_versions[MP1_HWIP][0]);
                return -EINVAL;
        }
@@ -239,7 +239,7 @@ int smu_v11_0_check_fw_version(struct smu_context *smu)
        if (smu->is_apu)
                adev->pm.fw_version = smu_version;
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(11, 0, 0):
                smu->smc_driver_if_version = SMU11_DRIVER_IF_VERSION_NV10;
                break;
@@ -272,7 +272,7 @@ int smu_v11_0_check_fw_version(struct smu_context *smu)
                break;
        default:
                dev_err(smu->adev->dev, "smu unsupported IP version: 0x%x.\n",
-                       adev->ip_versions[MP1_HWIP]);
+                       adev->ip_versions[MP1_HWIP][0]);
                smu->smc_driver_if_version = SMU11_DRIVER_IF_VERSION_INV;
                break;
        }
@@ -496,7 +496,7 @@ int smu_v11_0_init_power(struct smu_context *smu)
  {
        struct amdgpu_device *adev = smu->adev;
        struct smu_power_context *smu_power = &smu->smu_power;
-       size_t size = adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 5, 0) ?
+       size_t size = adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 5, 0) ?
                        sizeof(struct smu_11_5_power_context) :
                        sizeof(struct smu_11_0_power_context);
@@ -753,9 +753,9 @@ int smu_v11_0_init_display_count(struct smu_context *smu, uint32_t count)
        /* Navy_Flounder/Dimgrey_Cavefish do not support to change
         * display num currently
         */
-       if (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 11) ||
-           adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 5, 0) ||
-           adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 13))
+       if (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 11) ||
+           adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 5, 0) ||
+           adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 13))
                return 0;
return smu_cmn_send_smc_msg_with_param(smu,
@@ -1140,7 +1140,7 @@ int smu_v11_0_gfx_off_control(struct smu_context *smu, 
bool enable)
        int ret = 0;
        struct amdgpu_device *adev = smu->adev;
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(11, 0, 0):
        case IP_VERSION(11, 0, 5):
        case IP_VERSION(11, 0, 9):
@@ -1634,7 +1634,7 @@ int smu_v11_0_baco_set_state(struct smu_context *smu, 
enum smu_baco_state state)
        mutex_lock(&smu_baco->mutex);
if (state == SMU_BACO_STATE_ENTER) {
-               switch (adev->ip_versions[MP1_HWIP]) {
+               switch (adev->ip_versions[MP1_HWIP][0]) {
                case IP_VERSION(11, 0, 7):
                case IP_VERSION(11, 0, 11):
                case IP_VERSION(11, 0, 12):
@@ -1653,7 +1653,7 @@ int smu_v11_0_baco_set_state(struct smu_context *smu, 
enum smu_baco_state state)
                default:
                        if (!ras || !adev->ras_enabled ||
                            adev->gmc.xgmi.pending_reset) {
-                               if (adev->ip_versions[MP1_HWIP] == 
IP_VERSION(11, 0, 2)) {
+                               if (adev->ip_versions[MP1_HWIP][0] == 
IP_VERSION(11, 0, 2)) {
                                        data = RREG32_SOC15(THM, 0, 
mmTHM_BACO_CNTL_ARCT);
                                        data |= 0x80000000;
                                        WREG32_SOC15(THM, 0, 
mmTHM_BACO_CNTL_ARCT, data);
@@ -1935,7 +1935,7 @@ int smu_v11_0_set_performance_level(struct smu_context 
*smu,
         * Separate MCLK and SOCCLK soft min/max settings are not allowed
         * on Arcturus.
         */
-       if (adev->ip_versions[MP1_HWIP] == IP_VERSION(11, 0, 2)) {
+       if (adev->ip_versions[MP1_HWIP][0] == IP_VERSION(11, 0, 2)) {
                mclk_min = mclk_max = 0;
                socclk_min = socclk_max = 0;
        }
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
index 08f3cc809fc6..05c5e61f3506 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0.c
@@ -89,13 +89,13 @@ int smu_v13_0_init_microcode(struct smu_context *smu)
        if (amdgpu_sriov_vf(adev))
                return 0;
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(13, 0, 2):
                chip_name = "aldebaran";
                break;
        default:
                dev_err(adev->dev, "Unsupported IP version 0x%x\n",
-                       adev->ip_versions[MP1_HWIP]);
+                       adev->ip_versions[MP1_HWIP][0]);
                return -EINVAL;
        }
@@ -211,7 +211,7 @@ int smu_v13_0_check_fw_version(struct smu_context *smu)
        smu_minor = (smu_version >> 8) & 0xff;
        smu_debug = (smu_version >> 0) & 0xff;
- switch (smu->adev->ip_versions[MP1_HWIP]) {
+       switch (smu->adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(13, 0, 2):
                smu->smc_driver_if_version = SMU13_DRIVER_IF_VERSION_ALDE;
                break;
@@ -221,7 +221,7 @@ int smu_v13_0_check_fw_version(struct smu_context *smu)
                break;
        default:
                dev_err(smu->adev->dev, "smu unsupported IP version: 0x%x.\n",
-                       smu->adev->ip_versions[MP1_HWIP]);
+                       smu->adev->ip_versions[MP1_HWIP][0]);
                smu->smc_driver_if_version = SMU13_DRIVER_IF_VERSION_INV;
                break;
        }
@@ -743,7 +743,7 @@ int smu_v13_0_gfx_off_control(struct smu_context *smu, bool 
enable)
        int ret = 0;
        struct amdgpu_device *adev = smu->adev;
- switch (adev->ip_versions[MP1_HWIP]) {
+       switch (adev->ip_versions[MP1_HWIP][0]) {
        case IP_VERSION(13, 0, 1):
        case IP_VERSION(13, 0, 3):
                if (!(adev->pm.pp_feature & PP_GFXOFF_MASK))

Reply via email to