From: Evan Quan Date: Thu, 25 Jul 2019 08:40:51 +0000 (+0800) Subject: drm/amd/powerplay: determine the features to enable by pptable only X-Git-Tag: v5.4-rc1~32^2~19^2~135 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=59de58f84f7f561fe67b2d78a6a2cae6c0b50c24;p=platform%2Fkernel%2Flinux-rpi.git drm/amd/powerplay: determine the features to enable by pptable only Per current logics, the features to enable are determined together by driver and pptable. This is not efficient in co-debug with firmware team. Signed-off-by: Evan Quan Reviewed-by: Kenneth Feng Signed-off-by: Alex Deucher --- diff --git a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c index 9360f5a..215f717 100644 --- a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c +++ b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c @@ -327,7 +327,6 @@ static int arcturus_allocate_dpm_context(struct smu_context *smu) return 0; } -#define FEATURE_MASK(feature) (1ULL << feature) static int arcturus_get_allowed_feature_mask(struct smu_context *smu, uint32_t *feature_mask, uint32_t num) @@ -335,9 +334,8 @@ arcturus_get_allowed_feature_mask(struct smu_context *smu, if (num > 2) return -EINVAL; - memset(feature_mask, 0, sizeof(uint32_t) * num); - - *(uint64_t *)feature_mask |= FEATURE_MASK(FEATURE_DPM_PREFETCHER_BIT); + /* pptable will handle the features to enable */ + memset(feature_mask, 0xFF, sizeof(uint32_t) * num); return 0; }