[PATCH 61/66] drm/amdgpu: convert IP version array to include instances
Lazar, Lijo
lijo.lazar at amd.com
Wed Sep 22 09:08:01 UTC 2021
On 9/21/2021 11:37 PM, Alex Deucher wrote:
> Allow us to query instances versions more cleanly.
>
> Signed-off-by: Alex Deucher <alexander.deucher at 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))
>
More information about the amd-gfx
mailing list