drm/amdgpu: clean up set IP function
authorAlex Deucher <alexander.deucher@amd.com>
Tue, 10 Aug 2021 19:21:10 +0000 (15:21 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 4 Oct 2021 19:23:01 +0000 (15:23 -0400)
Split into several smaller per IP functions to make it
easier to handle ordering issues for things like
SR-IOV in a follow up patch.

Acked-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c

index dd2c7b2..d9c2a72 100644 (file)
@@ -540,336 +540,218 @@ int amdgpu_discovery_get_gfx_info(struct amdgpu_device *adev)
        return 0;
 }
 
-int amdgpu_discovery_set_ip_blocks(struct amdgpu_device *adev)
+static int amdgpu_discovery_set_common_ip_blocks(struct amdgpu_device *adev)
 {
-       int r;
-
-       switch (adev->asic_type) {
-       case CHIP_VEGA10:
-               vega10_reg_base_init(adev);
-               adev->sdma.num_instances = 2;
-               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][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][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][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][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][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][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);
-               if (r)
-                       return -EINVAL;
-
-               amdgpu_discovery_harvest_ip(adev);
-
-               if (!adev->mman.discovery_bin) {
-                       DRM_ERROR("ip discovery uninitialized\n");
-                       return -EINVAL;
-               }
-               break;
-       }
-
+       /* what IP to use for this? */
        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):
+       case IP_VERSION(9, 2, 2):
+       case IP_VERSION(9, 3, 0):
        case IP_VERSION(9, 4, 0):
        case IP_VERSION(9, 4, 1):
        case IP_VERSION(9, 4, 2):
-               adev->family = AMDGPU_FAMILY_AI;
-               break;
-       case IP_VERSION(9, 1, 0):
-       case IP_VERSION(9, 2, 2):
-       case IP_VERSION(9, 3, 0):
-               adev->family = AMDGPU_FAMILY_RV;
+               amdgpu_device_ip_block_add(adev, &vega10_common_ip_block);
                break;
        case IP_VERSION(10, 1, 10):
        case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 2):
        case IP_VERSION(10, 1, 3):
        case IP_VERSION(10, 3, 0):
+       case IP_VERSION(10, 3, 1):
        case IP_VERSION(10, 3, 2):
+       case IP_VERSION(10, 3, 3):
        case IP_VERSION(10, 3, 4):
        case IP_VERSION(10, 3, 5):
-               adev->family = AMDGPU_FAMILY_NV;
-               break;
-       case IP_VERSION(10, 3, 1):
-               adev->family = AMDGPU_FAMILY_VGH;
-               break;
-       case IP_VERSION(10, 3, 3):
-               adev->family = AMDGPU_FAMILY_YC;
+               amdgpu_device_ip_block_add(adev, &nv_common_ip_block);
                break;
        default:
                return -EINVAL;
        }
+       return 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][0]) {
-       case IP_VERSION(6, 1, 0):
-       case IP_VERSION(6, 2, 0):
-               adev->nbio.funcs = &nbio_v6_1_funcs;
-               adev->nbio.hdp_flush_reg = &nbio_v6_1_hdp_flush_reg;
-               break;
-       case IP_VERSION(7, 0, 0):
-       case IP_VERSION(7, 0, 1):
-       case IP_VERSION(2, 5, 0):
-               adev->nbio.funcs = &nbio_v7_0_funcs;
-               adev->nbio.hdp_flush_reg = &nbio_v7_0_hdp_flush_reg;
-               break;
-       case IP_VERSION(7, 4, 0):
-       case IP_VERSION(7, 4, 1):
-       case IP_VERSION(7, 4, 4):
-               adev->nbio.funcs = &nbio_v7_4_funcs;
-               adev->nbio.hdp_flush_reg = &nbio_v7_4_hdp_flush_reg;
-               break;
-       case IP_VERSION(7, 2, 0):
-       case IP_VERSION(7, 2, 1):
-       case IP_VERSION(7, 5, 0):
-               adev->nbio.funcs = &nbio_v7_2_funcs;
-               adev->nbio.hdp_flush_reg = &nbio_v7_2_hdp_flush_reg;
+static int amdgpu_discovery_set_gmc_ip_blocks(struct amdgpu_device *adev)
+{
+       /* use GC or MMHUB IP version */
+       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):
+       case IP_VERSION(9, 2, 2):
+       case IP_VERSION(9, 3, 0):
+       case IP_VERSION(9, 4, 0):
+       case IP_VERSION(9, 4, 1):
+       case IP_VERSION(9, 4, 2):
+               amdgpu_device_ip_block_add(adev, &gmc_v9_0_ip_block);
                break;
-       case IP_VERSION(2, 1, 1):
-       case IP_VERSION(2, 3, 0):
-       case IP_VERSION(2, 3, 1):
-       case IP_VERSION(2, 3, 2):
-       case IP_VERSION(3, 3, 0):
-       case IP_VERSION(3, 3, 1):
-       case IP_VERSION(3, 3, 2):
-       case IP_VERSION(3, 3, 3):
-               adev->nbio.funcs = &nbio_v2_3_funcs;
-               adev->nbio.hdp_flush_reg = &nbio_v2_3_hdp_flush_reg;
+       case IP_VERSION(10, 1, 10):
+       case IP_VERSION(10, 1, 1):
+       case IP_VERSION(10, 1, 2):
+       case IP_VERSION(10, 1, 3):
+       case IP_VERSION(10, 3, 0):
+       case IP_VERSION(10, 3, 1):
+       case IP_VERSION(10, 3, 2):
+       case IP_VERSION(10, 3, 3):
+       case IP_VERSION(10, 3, 4):
+       case IP_VERSION(10, 3, 5):
+               amdgpu_device_ip_block_add(adev, &gmc_v10_0_ip_block);
                break;
        default:
-               break;
+               return -EINVAL;
        }
+       return 0;
+}
 
-       switch (adev->ip_versions[HDP_HWIP][0]) {
+static int amdgpu_discovery_set_ih_ip_blocks(struct amdgpu_device *adev)
+{
+       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):
        case IP_VERSION(4, 1, 1):
+       case IP_VERSION(4, 3, 0):
+               amdgpu_device_ip_block_add(adev, &vega10_ih_ip_block);
+               break;
        case IP_VERSION(4, 2, 0):
        case IP_VERSION(4, 2, 1):
        case IP_VERSION(4, 4, 0):
-               adev->hdp.funcs = &hdp_v4_0_funcs;
+               amdgpu_device_ip_block_add(adev, &vega20_ih_ip_block);
                break;
        case IP_VERSION(5, 0, 0):
        case IP_VERSION(5, 0, 1):
        case IP_VERSION(5, 0, 2):
        case IP_VERSION(5, 0, 3):
-       case IP_VERSION(5, 0, 4):
        case IP_VERSION(5, 2, 0):
-               adev->hdp.funcs = &hdp_v5_0_funcs;
-               break;
-       default:
-               break;
-       }
-
-       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):
-               adev->df.funcs = &df_v3_6_funcs;
-               break;
-       case IP_VERSION(2, 1, 0):
-       case IP_VERSION(2, 1, 1):
-       case IP_VERSION(2, 5, 0):
-       case IP_VERSION(3, 5, 1):
-       case IP_VERSION(3, 5, 2):
-               adev->df.funcs = &df_v1_7_funcs;
+       case IP_VERSION(5, 2, 1):
+               amdgpu_device_ip_block_add(adev, &navi10_ih_ip_block);
                break;
        default:
-               break;
+               return -EINVAL;
        }
+       return 0;
+}
 
-       switch (adev->ip_versions[SMUIO_HWIP][0]) {
+static int amdgpu_discovery_set_psp_ip_blocks(struct amdgpu_device *adev)
+{
+       switch (adev->ip_versions[MP0_HWIP][0]) {
        case IP_VERSION(9, 0, 0):
-       case IP_VERSION(9, 0, 1):
+               amdgpu_device_ip_block_add(adev, &psp_v3_1_ip_block);
+               break;
        case IP_VERSION(10, 0, 0):
        case IP_VERSION(10, 0, 1):
-       case IP_VERSION(10, 0, 2):
-               adev->smuio.funcs = &smuio_v9_0_funcs;
+               amdgpu_device_ip_block_add(adev, &psp_v10_0_ip_block);
                break;
        case IP_VERSION(11, 0, 0):
        case IP_VERSION(11, 0, 2):
-       case IP_VERSION(11, 0, 3):
        case IP_VERSION(11, 0, 4):
+       case IP_VERSION(11, 0, 5):
+       case IP_VERSION(11, 0, 9):
        case IP_VERSION(11, 0, 7):
-       case IP_VERSION(11, 0, 8):
-               adev->smuio.funcs = &smuio_v11_0_funcs;
-               break;
-       case IP_VERSION(11, 0, 6):
-       case IP_VERSION(11, 0, 10):
        case IP_VERSION(11, 0, 11):
+       case IP_VERSION(11, 0, 12):
+       case IP_VERSION(11, 0, 13):
        case IP_VERSION(11, 5, 0):
-       case IP_VERSION(13, 0, 1):
-               adev->smuio.funcs = &smuio_v11_0_6_funcs;
+               amdgpu_device_ip_block_add(adev, &psp_v11_0_ip_block);
+               break;
+       case IP_VERSION(11, 0, 8):
+               amdgpu_device_ip_block_add(adev, &psp_v11_0_8_ip_block);
                break;
+       case IP_VERSION(11, 0, 3):
+       case IP_VERSION(12, 0, 1):
+               amdgpu_device_ip_block_add(adev, &psp_v12_0_ip_block);
+               break;
+       case IP_VERSION(13, 0, 1):
        case IP_VERSION(13, 0, 2):
-               adev->smuio.funcs = &smuio_v13_0_funcs;
+       case IP_VERSION(13, 0, 3):
+               amdgpu_device_ip_block_add(adev, &psp_v13_0_ip_block);
                break;
        default:
-               break;
+               return -EINVAL;
        }
+       return 0;
+}
 
-       /* what IP to use for this? */
-       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):
-       case IP_VERSION(9, 2, 2):
-       case IP_VERSION(9, 3, 0):
-       case IP_VERSION(9, 4, 0):
-       case IP_VERSION(9, 4, 1):
-       case IP_VERSION(9, 4, 2):
-               amdgpu_device_ip_block_add(adev, &vega10_common_ip_block);
+static int amdgpu_discovery_set_smu_ip_blocks(struct amdgpu_device *adev)
+{
+       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):
+       case IP_VERSION(11, 0, 2):
+               if (adev->asic_type == CHIP_ARCTURUS)
+                       amdgpu_device_ip_block_add(adev, &smu_v11_0_ip_block);
+               else
+                       amdgpu_device_ip_block_add(adev, &pp_smu_ip_block);
                break;
-       case IP_VERSION(10, 1, 10):
-       case IP_VERSION(10, 1, 1):
-       case IP_VERSION(10, 1, 2):
-       case IP_VERSION(10, 1, 3):
-       case IP_VERSION(10, 3, 0):
-       case IP_VERSION(10, 3, 1):
-       case IP_VERSION(10, 3, 2):
-       case IP_VERSION(10, 3, 3):
-       case IP_VERSION(10, 3, 4):
-       case IP_VERSION(10, 3, 5):
-               amdgpu_device_ip_block_add(adev, &nv_common_ip_block);
+       case IP_VERSION(11, 0, 0):
+       case IP_VERSION(11, 0, 9):
+       case IP_VERSION(11, 0, 7):
+       case IP_VERSION(11, 0, 8):
+       case IP_VERSION(11, 0, 11):
+       case IP_VERSION(11, 0, 12):
+       case IP_VERSION(11, 0, 13):
+       case IP_VERSION(11, 5, 0):
+               amdgpu_device_ip_block_add(adev, &smu_v11_0_ip_block);
+               break;
+       case IP_VERSION(12, 0, 0):
+       case IP_VERSION(12, 0, 1):
+               amdgpu_device_ip_block_add(adev, &smu_v12_0_ip_block);
+               break;
+       case IP_VERSION(13, 0, 1):
+       case IP_VERSION(13, 0, 2):
+       case IP_VERSION(13, 0, 3):
+               amdgpu_device_ip_block_add(adev, &smu_v13_0_ip_block);
                break;
        default:
                return -EINVAL;
        }
+       return 0;
+}
+
+static int amdgpu_discovery_set_display_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][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):
+               case IP_VERSION(2, 0, 0):
+               case IP_VERSION(2, 1, 0):
+               case IP_VERSION(3, 0, 0):
+               case IP_VERSION(3, 0, 2):
+               case IP_VERSION(3, 0, 3):
+               case IP_VERSION(3, 0, 1):
+               case IP_VERSION(3, 1, 2):
+               case IP_VERSION(3, 1, 3):
+                       amdgpu_device_ip_block_add(adev, &dm_ip_block);
+                       break;
+               case IP_VERSION(2, 0, 3):
+                       break;
+               default:
+                       return -EINVAL;
+               }
+       } 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):
+                       amdgpu_device_ip_block_add(adev, &dm_ip_block);
+                       break;
+               default:
+                       return -EINVAL;
+               }
+#endif
+       }
+       return 0;
+}
 
-       /* use GC or MMHUB IP version */
+static int amdgpu_discovery_set_gc_ip_blocks(struct amdgpu_device *adev)
+{
        switch (adev->ip_versions[GC_HWIP][0]) {
        case IP_VERSION(9, 0, 1):
        case IP_VERSION(9, 1, 0):
@@ -879,319 +761,489 @@ int amdgpu_discovery_set_ip_blocks(struct amdgpu_device *adev)
        case IP_VERSION(9, 4, 0):
        case IP_VERSION(9, 4, 1):
        case IP_VERSION(9, 4, 2):
-               amdgpu_device_ip_block_add(adev, &gmc_v9_0_ip_block);
+               amdgpu_device_ip_block_add(adev, &gfx_v9_0_ip_block);
                break;
        case IP_VERSION(10, 1, 10):
-       case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 2):
+       case IP_VERSION(10, 1, 1):
        case IP_VERSION(10, 1, 3):
        case IP_VERSION(10, 3, 0):
-       case IP_VERSION(10, 3, 1):
        case IP_VERSION(10, 3, 2):
-       case IP_VERSION(10, 3, 3):
+       case IP_VERSION(10, 3, 1):
        case IP_VERSION(10, 3, 4):
        case IP_VERSION(10, 3, 5):
-               amdgpu_device_ip_block_add(adev, &gmc_v10_0_ip_block);
+       case IP_VERSION(10, 3, 3):
+               amdgpu_device_ip_block_add(adev, &gfx_v10_0_ip_block);
                break;
        default:
                return -EINVAL;
        }
+       return 0;
+}
 
-       switch (adev->ip_versions[OSSSYS_HWIP][0]) {
+static int amdgpu_discovery_set_sdma_ip_blocks(struct amdgpu_device *adev)
+{
+       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):
        case IP_VERSION(4, 1, 1):
-       case IP_VERSION(4, 3, 0):
-               amdgpu_device_ip_block_add(adev, &vega10_ih_ip_block);
-               break;
+       case IP_VERSION(4, 1, 2):
        case IP_VERSION(4, 2, 0):
-       case IP_VERSION(4, 2, 1):
+       case IP_VERSION(4, 2, 2):
        case IP_VERSION(4, 4, 0):
-               amdgpu_device_ip_block_add(adev, &vega20_ih_ip_block);
+               amdgpu_device_ip_block_add(adev, &sdma_v4_0_ip_block);
                break;
        case IP_VERSION(5, 0, 0):
        case IP_VERSION(5, 0, 1):
        case IP_VERSION(5, 0, 2):
-       case IP_VERSION(5, 0, 3):
+       case IP_VERSION(5, 0, 5):
+               amdgpu_device_ip_block_add(adev, &sdma_v5_0_ip_block);
+               break;
        case IP_VERSION(5, 2, 0):
+       case IP_VERSION(5, 2, 2):
+       case IP_VERSION(5, 2, 4):
+       case IP_VERSION(5, 2, 5):
+       case IP_VERSION(5, 2, 3):
        case IP_VERSION(5, 2, 1):
-               amdgpu_device_ip_block_add(adev, &navi10_ih_ip_block);
+               amdgpu_device_ip_block_add(adev, &sdma_v5_2_ip_block);
                break;
        default:
                return -EINVAL;
        }
+       return 0;
+}
 
-       if (likely(adev->firmware.load_type == AMDGPU_FW_LOAD_PSP)) {
-               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;
-               case IP_VERSION(10, 0, 0):
-               case IP_VERSION(10, 0, 1):
-                       amdgpu_device_ip_block_add(adev, &psp_v10_0_ip_block);
-                       break;
-               case IP_VERSION(11, 0, 0):
-               case IP_VERSION(11, 0, 2):
-               case IP_VERSION(11, 0, 4):
-               case IP_VERSION(11, 0, 5):
-               case IP_VERSION(11, 0, 9):
-               case IP_VERSION(11, 0, 7):
-               case IP_VERSION(11, 0, 11):
-               case IP_VERSION(11, 0, 12):
-               case IP_VERSION(11, 0, 13):
-               case IP_VERSION(11, 5, 0):
-                       amdgpu_device_ip_block_add(adev, &psp_v11_0_ip_block);
-                       break;
-               case IP_VERSION(11, 0, 8):
-                       amdgpu_device_ip_block_add(adev, &psp_v11_0_8_ip_block);
-                       break;
-               case IP_VERSION(11, 0, 3):
-               case IP_VERSION(12, 0, 1):
-                       amdgpu_device_ip_block_add(adev, &psp_v12_0_ip_block);
-                       break;
-               case IP_VERSION(13, 0, 1):
-               case IP_VERSION(13, 0, 2):
-               case IP_VERSION(13, 0, 3):
-                       amdgpu_device_ip_block_add(adev, &psp_v13_0_ip_block);
+static int amdgpu_discovery_set_mm_ip_blocks(struct amdgpu_device *adev)
+{
+       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);
                        break;
                default:
                        return -EINVAL;
                }
-       }
-
-       if (likely(adev->firmware.load_type == AMDGPU_FW_LOAD_PSP)) {
-               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):
-               case IP_VERSION(11, 0, 2):
-                       if (adev->asic_type == CHIP_ARCTURUS)
-                               amdgpu_device_ip_block_add(adev, &smu_v11_0_ip_block);
-                       else
-                               amdgpu_device_ip_block_add(adev, &pp_smu_ip_block);
-                       break;
-               case IP_VERSION(11, 0, 0):
-               case IP_VERSION(11, 0, 9):
-               case IP_VERSION(11, 0, 7):
-               case IP_VERSION(11, 0, 8):
-               case IP_VERSION(11, 0, 11):
-               case IP_VERSION(11, 0, 12):
-               case IP_VERSION(11, 0, 13):
-               case IP_VERSION(11, 5, 0):
-                       amdgpu_device_ip_block_add(adev, &smu_v11_0_ip_block);
-                       break;
-               case IP_VERSION(12, 0, 0):
-               case IP_VERSION(12, 0, 1):
-                       amdgpu_device_ip_block_add(adev, &smu_v12_0_ip_block);
-                       break;
-               case IP_VERSION(13, 0, 1):
-               case IP_VERSION(13, 0, 2):
-               case IP_VERSION(13, 0, 3):
-                       amdgpu_device_ip_block_add(adev, &smu_v13_0_ip_block);
+               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);
                        break;
                default:
                        return -EINVAL;
                }
-       }
-
-       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][0]) {
-               switch (adev->ip_versions[DCE_HWIP][0]) {
+       } else {
+               switch (adev->ip_versions[UVD_HWIP][0]) {
                case IP_VERSION(1, 0, 0):
                case IP_VERSION(1, 0, 1):
-               case IP_VERSION(2, 0, 2):
+                       amdgpu_device_ip_block_add(adev, &vcn_v1_0_ip_block);
+                       break;
                case IP_VERSION(2, 0, 0):
-               case IP_VERSION(2, 1, 0):
+               case IP_VERSION(2, 0, 2):
+               case IP_VERSION(2, 2, 0):
+                       amdgpu_device_ip_block_add(adev, &vcn_v2_0_ip_block);
+                       amdgpu_device_ip_block_add(adev, &jpeg_v2_0_ip_block);
+                       break;
+               case IP_VERSION(2, 0, 3):
+                       break;
+               case IP_VERSION(2, 5, 0):
+                       amdgpu_device_ip_block_add(adev, &vcn_v2_5_ip_block);
+                       amdgpu_device_ip_block_add(adev, &jpeg_v2_5_ip_block);
+                       break;
+               case IP_VERSION(2, 6, 0):
+                       amdgpu_device_ip_block_add(adev, &vcn_v2_6_ip_block);
+                       amdgpu_device_ip_block_add(adev, &jpeg_v2_6_ip_block);
+                       break;
                case IP_VERSION(3, 0, 0):
+               case IP_VERSION(3, 0, 16):
+               case IP_VERSION(3, 1, 1):
                case IP_VERSION(3, 0, 2):
-               case IP_VERSION(3, 0, 3):
-               case IP_VERSION(3, 0, 1):
-               case IP_VERSION(3, 1, 2):
-               case IP_VERSION(3, 1, 3):
-                       amdgpu_device_ip_block_add(adev, &dm_ip_block);
+                       amdgpu_device_ip_block_add(adev, &vcn_v3_0_ip_block);
+                       amdgpu_device_ip_block_add(adev, &jpeg_v3_0_ip_block);
                        break;
-               case IP_VERSION(2, 0, 3):
+               case IP_VERSION(3, 0, 33):
+                       amdgpu_device_ip_block_add(adev, &vcn_v3_0_ip_block);
                        break;
                default:
                        return -EINVAL;
-               }
-       } 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):
-                       amdgpu_device_ip_block_add(adev, &dm_ip_block);
-                       break;
-               default:
+               }
+       }
+       return 0;
+}
+
+static int amdgpu_discovery_set_mes_ip_blocks(struct amdgpu_device *adev)
+{
+       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):
+       case IP_VERSION(10, 1, 3):
+       case IP_VERSION(10, 3, 0):
+       case IP_VERSION(10, 3, 1):
+       case IP_VERSION(10, 3, 2):
+       case IP_VERSION(10, 3, 3):
+       case IP_VERSION(10, 3, 4):
+       case IP_VERSION(10, 3, 5):
+               amdgpu_device_ip_block_add(adev, &mes_v10_1_ip_block);
+               break;
+       default:
+               break;;
+       }
+       return 0;
+}
+
+int amdgpu_discovery_set_ip_blocks(struct amdgpu_device *adev)
+{
+       int r;
+
+       switch (adev->asic_type) {
+       case CHIP_VEGA10:
+               vega10_reg_base_init(adev);
+               adev->sdma.num_instances = 2;
+               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][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][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][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][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][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][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);
+               if (r)
+                       return -EINVAL;
+
+               amdgpu_discovery_harvest_ip(adev);
+
+               if (!adev->mman.discovery_bin) {
+                       DRM_ERROR("ip discovery uninitialized\n");
                        return -EINVAL;
                }
+               break;
        }
-#endif
+
        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):
-       case IP_VERSION(9, 2, 2):
-       case IP_VERSION(9, 3, 0):
        case IP_VERSION(9, 4, 0):
        case IP_VERSION(9, 4, 1):
        case IP_VERSION(9, 4, 2):
-               amdgpu_device_ip_block_add(adev, &gfx_v9_0_ip_block);
+               adev->family = AMDGPU_FAMILY_AI;
+               break;
+       case IP_VERSION(9, 1, 0):
+       case IP_VERSION(9, 2, 2):
+       case IP_VERSION(9, 3, 0):
+               adev->family = AMDGPU_FAMILY_RV;
                break;
        case IP_VERSION(10, 1, 10):
-       case IP_VERSION(10, 1, 2):
        case IP_VERSION(10, 1, 1):
+       case IP_VERSION(10, 1, 2):
        case IP_VERSION(10, 1, 3):
        case IP_VERSION(10, 3, 0):
        case IP_VERSION(10, 3, 2):
-       case IP_VERSION(10, 3, 1):
        case IP_VERSION(10, 3, 4):
        case IP_VERSION(10, 3, 5):
+               adev->family = AMDGPU_FAMILY_NV;
+               break;
+       case IP_VERSION(10, 3, 1):
+               adev->family = AMDGPU_FAMILY_VGH;
+               break;
        case IP_VERSION(10, 3, 3):
-               amdgpu_device_ip_block_add(adev, &gfx_v10_0_ip_block);
+               adev->family = AMDGPU_FAMILY_YC;
                break;
        default:
                return -EINVAL;
        }
 
-       switch (adev->ip_versions[SDMA0_HWIP][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][0]) {
+       case IP_VERSION(6, 1, 0):
+       case IP_VERSION(6, 2, 0):
+               adev->nbio.funcs = &nbio_v6_1_funcs;
+               adev->nbio.hdp_flush_reg = &nbio_v6_1_hdp_flush_reg;
+               break;
+       case IP_VERSION(7, 0, 0):
+       case IP_VERSION(7, 0, 1):
+       case IP_VERSION(2, 5, 0):
+               adev->nbio.funcs = &nbio_v7_0_funcs;
+               adev->nbio.hdp_flush_reg = &nbio_v7_0_hdp_flush_reg;
+               break;
+       case IP_VERSION(7, 4, 0):
+       case IP_VERSION(7, 4, 1):
+       case IP_VERSION(7, 4, 4):
+               adev->nbio.funcs = &nbio_v7_4_funcs;
+               adev->nbio.hdp_flush_reg = &nbio_v7_4_hdp_flush_reg;
+               break;
+       case IP_VERSION(7, 2, 0):
+       case IP_VERSION(7, 2, 1):
+       case IP_VERSION(7, 5, 0):
+               adev->nbio.funcs = &nbio_v7_2_funcs;
+               adev->nbio.hdp_flush_reg = &nbio_v7_2_hdp_flush_reg;
+               break;
+       case IP_VERSION(2, 1, 1):
+       case IP_VERSION(2, 3, 0):
+       case IP_VERSION(2, 3, 1):
+       case IP_VERSION(2, 3, 2):
+       case IP_VERSION(3, 3, 0):
+       case IP_VERSION(3, 3, 1):
+       case IP_VERSION(3, 3, 2):
+       case IP_VERSION(3, 3, 3):
+               adev->nbio.funcs = &nbio_v2_3_funcs;
+               adev->nbio.hdp_flush_reg = &nbio_v2_3_hdp_flush_reg;
+               break;
+       default:
+               break;
+       }
+
+       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):
        case IP_VERSION(4, 1, 1):
-       case IP_VERSION(4, 1, 2):
        case IP_VERSION(4, 2, 0):
-       case IP_VERSION(4, 2, 2):
+       case IP_VERSION(4, 2, 1):
        case IP_VERSION(4, 4, 0):
-               amdgpu_device_ip_block_add(adev, &sdma_v4_0_ip_block);
+               adev->hdp.funcs = &hdp_v4_0_funcs;
                break;
        case IP_VERSION(5, 0, 0):
        case IP_VERSION(5, 0, 1):
        case IP_VERSION(5, 0, 2):
-       case IP_VERSION(5, 0, 5):
-               amdgpu_device_ip_block_add(adev, &sdma_v5_0_ip_block);
-               break;
+       case IP_VERSION(5, 0, 3):
+       case IP_VERSION(5, 0, 4):
        case IP_VERSION(5, 2, 0):
-       case IP_VERSION(5, 2, 2):
-       case IP_VERSION(5, 2, 4):
-       case IP_VERSION(5, 2, 5):
-       case IP_VERSION(5, 2, 3):
-       case IP_VERSION(5, 2, 1):
-               amdgpu_device_ip_block_add(adev, &sdma_v5_2_ip_block);
+               adev->hdp.funcs = &hdp_v5_0_funcs;
                break;
        default:
-               return -EINVAL;
+               break;
        }
 
-       if (adev->firmware.load_type == AMDGPU_FW_LOAD_DIRECT) {
-               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):
-               case IP_VERSION(11, 0, 2):
-                       if (adev->asic_type == CHIP_ARCTURUS)
-                               amdgpu_device_ip_block_add(adev, &smu_v11_0_ip_block);
-                       else
-                               amdgpu_device_ip_block_add(adev, &pp_smu_ip_block);
-                       break;
-               case IP_VERSION(11, 0, 0):
-               case IP_VERSION(11, 0, 9):
-               case IP_VERSION(11, 0, 7):
-               case IP_VERSION(11, 0, 8):
-               case IP_VERSION(11, 0, 11):
-               case IP_VERSION(11, 0, 12):
-               case IP_VERSION(11, 0, 13):
-               case IP_VERSION(11, 5, 0):
-                       amdgpu_device_ip_block_add(adev, &smu_v11_0_ip_block);
-                       break;
-               case IP_VERSION(12, 0, 0):
-               case IP_VERSION(12, 0, 1):
-                       amdgpu_device_ip_block_add(adev, &smu_v12_0_ip_block);
-                       break;
-               case IP_VERSION(13, 0, 1):
-               case IP_VERSION(13, 0, 2):
-               case IP_VERSION(13, 0, 3):
-                       amdgpu_device_ip_block_add(adev, &smu_v13_0_ip_block);
-                       break;
-               default:
-                       return -EINVAL;
-               }
+       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):
+               adev->df.funcs = &df_v3_6_funcs;
+               break;
+       case IP_VERSION(2, 1, 0):
+       case IP_VERSION(2, 1, 1):
+       case IP_VERSION(2, 5, 0):
+       case IP_VERSION(3, 5, 1):
+       case IP_VERSION(3, 5, 2):
+               adev->df.funcs = &df_v1_7_funcs;
+               break;
+       default:
+               break;
        }
 
-       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);
-                       break;
-               default:
-                       return -EINVAL;
-               }
-               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);
-                       break;
-               default:
-                       return -EINVAL;
-               }
-       } else {
-               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);
-                       break;
-               case IP_VERSION(2, 0, 0):
-               case IP_VERSION(2, 0, 2):
-               case IP_VERSION(2, 2, 0):
-                       amdgpu_device_ip_block_add(adev, &vcn_v2_0_ip_block);
-                       amdgpu_device_ip_block_add(adev, &jpeg_v2_0_ip_block);
-                       break;
-               case IP_VERSION(2, 0, 3):
-                       break;
-               case IP_VERSION(2, 5, 0):
-                       amdgpu_device_ip_block_add(adev, &vcn_v2_5_ip_block);
-                       amdgpu_device_ip_block_add(adev, &jpeg_v2_5_ip_block);
-                       break;
-               case IP_VERSION(2, 6, 0):
-                       amdgpu_device_ip_block_add(adev, &vcn_v2_6_ip_block);
-                       amdgpu_device_ip_block_add(adev, &jpeg_v2_6_ip_block);
-                       break;
-               case IP_VERSION(3, 0, 0):
-               case IP_VERSION(3, 0, 16):
-               case IP_VERSION(3, 1, 1):
-               case IP_VERSION(3, 0, 2):
-                       amdgpu_device_ip_block_add(adev, &vcn_v3_0_ip_block);
-                       amdgpu_device_ip_block_add(adev, &jpeg_v3_0_ip_block);
-                       break;
-               case IP_VERSION(3, 0, 33):
-                       amdgpu_device_ip_block_add(adev, &vcn_v3_0_ip_block);
-                       break;
-               default:
-                       return -EINVAL;
-               }
+       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):
+       case IP_VERSION(10, 0, 1):
+       case IP_VERSION(10, 0, 2):
+               adev->smuio.funcs = &smuio_v9_0_funcs;
+               break;
+       case IP_VERSION(11, 0, 0):
+       case IP_VERSION(11, 0, 2):
+       case IP_VERSION(11, 0, 3):
+       case IP_VERSION(11, 0, 4):
+       case IP_VERSION(11, 0, 7):
+       case IP_VERSION(11, 0, 8):
+               adev->smuio.funcs = &smuio_v11_0_funcs;
+               break;
+       case IP_VERSION(11, 0, 6):
+       case IP_VERSION(11, 0, 10):
+       case IP_VERSION(11, 0, 11):
+       case IP_VERSION(11, 5, 0):
+       case IP_VERSION(13, 0, 1):
+               adev->smuio.funcs = &smuio_v11_0_6_funcs;
+               break;
+       case IP_VERSION(13, 0, 2):
+               adev->smuio.funcs = &smuio_v13_0_funcs;
+               break;
+       default:
+               break;
+       }
+
+       r = amdgpu_discovery_set_common_ip_blocks(adev);
+       if (r)
+               return r;
+
+       r = amdgpu_discovery_set_gmc_ip_blocks(adev);
+       if (r)
+               return r;
+
+       r = amdgpu_discovery_set_ih_ip_blocks(adev);
+       if (r)
+               return r;
+
+       if (likely(adev->firmware.load_type == AMDGPU_FW_LOAD_PSP)) {
+               r = amdgpu_discovery_set_psp_ip_blocks(adev);
+               if (r)
+                       return r;
+       }
+
+       if (likely(adev->firmware.load_type == AMDGPU_FW_LOAD_PSP)) {
+               r = amdgpu_discovery_set_smu_ip_blocks(adev);
+               if (r)
+                       return r;
        }
 
+       r = amdgpu_discovery_set_display_ip_blocks(adev);
+       if (r)
+               return r;
+
+       r = amdgpu_discovery_set_gc_ip_blocks(adev);
+       if (r)
+               return r;
+
+       r = amdgpu_discovery_set_sdma_ip_blocks(adev);
+       if (r)
+               return r;
+
+       if (adev->firmware.load_type == AMDGPU_FW_LOAD_DIRECT) {
+               r = amdgpu_discovery_set_smu_ip_blocks(adev);
+               if (r)
+                       return r;
+       }
+
+       r = amdgpu_discovery_set_mm_ip_blocks(adev);
+       if (r)
+               return r;
+
        if (adev->enable_mes) {
-               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):
-               case IP_VERSION(10, 1, 3):
-               case IP_VERSION(10, 3, 0):
-               case IP_VERSION(10, 3, 1):
-               case IP_VERSION(10, 3, 2):
-               case IP_VERSION(10, 3, 3):
-               case IP_VERSION(10, 3, 4):
-               case IP_VERSION(10, 3, 5):
-                       amdgpu_device_ip_block_add(adev, &mes_v10_1_ip_block);
-                       break;
-               default:
-                       break;;
-               }
+               r = amdgpu_discovery_set_mes_ip_blocks(adev);
+               if (r)
+                       return r;
        }
 
        return 0;