diff options
Diffstat (limited to 'meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2004-drm-amdgpu-Need-to-set-the-baco-cap-before-baco-rese.patch')
-rw-r--r-- | meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2004-drm-amdgpu-Need-to-set-the-baco-cap-before-baco-rese.patch | 160 |
1 files changed, 160 insertions, 0 deletions
diff --git a/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2004-drm-amdgpu-Need-to-set-the-baco-cap-before-baco-rese.patch b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2004-drm-amdgpu-Need-to-set-the-baco-cap-before-baco-rese.patch new file mode 100644 index 00000000..5aadff23 --- /dev/null +++ b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2004-drm-amdgpu-Need-to-set-the-baco-cap-before-baco-rese.patch @@ -0,0 +1,160 @@ +From 8558a2a42d62eea6fbfae218ab06f13e92925061 Mon Sep 17 00:00:00 2001 +From: Emily Deng <Emily.Deng@amd.com> +Date: Thu, 23 May 2019 11:31:19 +0800 +Subject: [PATCH 2004/2940] drm/amdgpu: Need to set the baco cap before baco + reset + +For passthrough, after rebooted the VM, driver will do +a baco reset before doing other driver initialization during loading + driver. For doing the baco reset, it will first +check the baco reset capability. So first need to set the +cap from the vbios information or baco reset won't be +enabled. + +Signed-off-by: Emily Deng <Emily.Deng@amd.com> +Reviewed-by: Evan Quan <evan.quan@amd.com> +--- + drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 10 ++++++++- + .../gpu/drm/amd/include/kgd_pp_interface.h | 1 + + drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 16 ++++++++++++++ + .../drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 1 + + .../powerplay/hwmgr/vega10_processpptables.c | 22 +++++++++++++++++++ + .../powerplay/hwmgr/vega10_processpptables.h | 1 + + drivers/gpu/drm/amd/powerplay/inc/hwmgr.h | 1 + + 7 files changed, 51 insertions(+), 1 deletion(-) + +diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c +index c10519a1330e..1228adcfab49 100644 +--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c ++++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c +@@ -2657,7 +2657,15 @@ int amdgpu_device_init(struct amdgpu_device *adev, + /* check if we need to reset the asic + * E.g., driver was not cleanly unloaded previously, etc. + */ +- if (!amdgpu_sriov_vf(adev) && amdgpu_asic_need_reset_on_init(adev)) { ++ if (amdgpu_passthrough(adev) && amdgpu_asic_need_reset_on_init(adev)) { ++ if (adev->powerplay.pp_funcs && adev->powerplay.pp_funcs->set_asic_baco_cap) { ++ r = adev->powerplay.pp_funcs->set_asic_baco_cap(adev->powerplay.pp_handle); ++ if (r) { ++ dev_err(adev->dev, "set baco capability failed\n"); ++ goto failed; ++ } ++ } ++ + r = amdgpu_asic_reset(adev); + if (r) { + dev_err(adev->dev, "asic reset on init failed\n"); +diff --git a/drivers/gpu/drm/amd/include/kgd_pp_interface.h b/drivers/gpu/drm/amd/include/kgd_pp_interface.h +index 9f661bf96ed0..c6e2a51370b3 100644 +--- a/drivers/gpu/drm/amd/include/kgd_pp_interface.h ++++ b/drivers/gpu/drm/amd/include/kgd_pp_interface.h +@@ -296,6 +296,7 @@ struct amd_pm_funcs { + int (*set_hard_min_fclk_by_freq)(void *handle, uint32_t clock); + int (*set_min_deep_sleep_dcefclk)(void *handle, uint32_t clock); + int (*get_asic_baco_capability)(void *handle, bool *cap); ++ int (*set_asic_baco_cap)(void *handle); + int (*get_asic_baco_state)(void *handle, int *state); + int (*set_asic_baco_state)(void *handle, int state); + int (*get_ppfeature_status)(void *handle, char *buf); +diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c +index bea1587d352d..9856760fceb5 100644 +--- a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c ++++ b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c +@@ -1404,6 +1404,21 @@ static int pp_set_active_display_count(void *handle, uint32_t count) + return ret; + } + ++static int pp_set_asic_baco_cap(void *handle) ++{ ++ struct pp_hwmgr *hwmgr = handle; ++ ++ if (!hwmgr) ++ return -EINVAL; ++ ++ if (!hwmgr->pm_en || !hwmgr->hwmgr_func->set_asic_baco_cap) ++ return 0; ++ ++ hwmgr->hwmgr_func->set_asic_baco_cap(hwmgr); ++ ++ return 0; ++} ++ + static int pp_get_asic_baco_capability(void *handle, bool *cap) + { + struct pp_hwmgr *hwmgr = handle; +@@ -1546,6 +1561,7 @@ static const struct amd_pm_funcs pp_dpm_funcs = { + .set_hard_min_dcefclk_by_freq = pp_set_hard_min_dcefclk_by_freq, + .set_hard_min_fclk_by_freq = pp_set_hard_min_fclk_by_freq, + .get_asic_baco_capability = pp_get_asic_baco_capability, ++ .set_asic_baco_cap = pp_set_asic_baco_cap, + .get_asic_baco_state = pp_get_asic_baco_state, + .set_asic_baco_state = pp_set_asic_baco_state, + .get_ppfeature_status = pp_get_ppfeature_status, +diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c +index bc904d1977e3..11b82c294d91 100644 +--- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c ++++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c +@@ -5302,6 +5302,7 @@ static const struct pp_hwmgr_func vega10_hwmgr_funcs = { + .odn_edit_dpm_table = vega10_odn_edit_dpm_table, + .get_performance_level = vega10_get_performance_level, + .get_asic_baco_capability = smu9_baco_get_capability, ++ .set_asic_baco_cap = vega10_baco_set_cap, + .get_asic_baco_state = smu9_baco_get_state, + .set_asic_baco_state = vega10_baco_set_state, + .get_ppfeature_status = vega10_get_ppfeature_status, +diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c +index b6767d74dc85..8fdeb23d2480 100644 +--- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c ++++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c +@@ -1371,3 +1371,25 @@ int vega10_get_powerplay_table_entry(struct pp_hwmgr *hwmgr, + + return result; + } ++ ++int vega10_baco_set_cap(struct pp_hwmgr *hwmgr) ++{ ++ int result = 0; ++ const ATOM_Vega10_POWERPLAYTABLE *powerplay_table; ++ ++ powerplay_table = get_powerplay_table(hwmgr); ++ ++ PP_ASSERT_WITH_CODE((powerplay_table != NULL), ++ "Missing PowerPlay Table!", return -1); ++ ++ result = check_powerplay_tables(hwmgr, powerplay_table); ++ ++ PP_ASSERT_WITH_CODE((result == 0), ++ "check_powerplay_tables failed", return result); ++ ++ set_hw_cap( ++ hwmgr, ++ 0 != (le32_to_cpu(powerplay_table->ulPlatformCaps) & ATOM_VEGA10_PP_PLATFORM_CAP_BACO), ++ PHM_PlatformCaps_BACO); ++ return result; ++} +\ No newline at end of file +diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.h b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.h +index d83ed2af7aa3..da5fbec9b0cd 100644 +--- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.h ++++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.h +@@ -59,4 +59,5 @@ extern int vega10_get_number_of_powerplay_table_entries(struct pp_hwmgr *hwmgr); + extern int vega10_get_powerplay_table_entry(struct pp_hwmgr *hwmgr, uint32_t entry_index, + struct pp_power_state *power_state, int (*call_back_func)(struct pp_hwmgr *, void *, + struct pp_power_state *, void *, uint32_t)); ++extern int vega10_baco_set_cap(struct pp_hwmgr *hwmgr); + #endif +diff --git a/drivers/gpu/drm/amd/powerplay/inc/hwmgr.h b/drivers/gpu/drm/amd/powerplay/inc/hwmgr.h +index bac3d85e3b82..14480aeeace0 100644 +--- a/drivers/gpu/drm/amd/powerplay/inc/hwmgr.h ++++ b/drivers/gpu/drm/amd/powerplay/inc/hwmgr.h +@@ -339,6 +339,7 @@ struct pp_hwmgr_func { + int (*set_hard_min_dcefclk_by_freq)(struct pp_hwmgr *hwmgr, uint32_t clock); + int (*set_hard_min_fclk_by_freq)(struct pp_hwmgr *hwmgr, uint32_t clock); + int (*get_asic_baco_capability)(struct pp_hwmgr *hwmgr, bool *cap); ++ int (*set_asic_baco_cap)(struct pp_hwmgr *hwmgr); + int (*get_asic_baco_state)(struct pp_hwmgr *hwmgr, enum BACO_STATE *state); + int (*set_asic_baco_state)(struct pp_hwmgr *hwmgr, enum BACO_STATE state); + int (*get_ppfeature_status)(struct pp_hwmgr *hwmgr, char *buf); +-- +2.17.1 + |