RE: [PATCH] drm/amd/powerplay: Add SMU WMTABLE Validity Check for Renoir
[AMD Official Use Only - Internal Distribution Only] The patch looks good. Reviewed-by: Hersen Wu -Original Message- From: Liu, Zhan Sent: Friday, December 13, 2019 11:50 AM To: amd-gfx@lists.freedesktop.org; Wu, Hersen ; Deucher, Alexander ; Wang, Kevin(Yang) ; Quan, Evan ; Yin, Tianci (Rico) Cc: Liu, Zhan Subject: [PATCH] drm/amd/powerplay: Add SMU WMTABLE Validity Check for Renoir [Why] SMU watermark table (WMTABLE) validity check is missing on Renoir. This validity check is very useful for checking whether WMTABLE is updated successfully. [How] Add SMU watermark validity check. Signed-off-by: Zhan Liu --- drivers/gpu/drm/amd/powerplay/renoir_ppt.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c index 89a54f8e08d3..81520b0fca68 100644 --- a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c +++ b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c @@ -777,9 +777,17 @@ static int renoir_set_watermarks_table( } /* pass data to smu controller */ - ret = smu_write_watermarks_table(smu); + if ((smu->watermarks_bitmap & WATERMARKS_EXIST) && + !(smu->watermarks_bitmap & WATERMARKS_LOADED)) { + ret = smu_write_watermarks_table(smu); + if (ret) { + pr_err("Failed to update WMTABLE!"); + return ret; + } + smu->watermarks_bitmap |= WATERMARKS_LOADED; + } - return ret; + return 0; } static int renoir_get_power_profile_mode(struct smu_context *smu, -- 2.17.1 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
RE: [PATCH] drm/amd/powerplay: Add SMU WMTABLE Validity Check for Renoir
Ping... > -Original Message- > From: Liu, Zhan > Sent: 2019/December/13, Friday 11:50 AM > To: amd-gfx@lists.freedesktop.org; Wu, Hersen ; > Deucher, Alexander ; Wang, Kevin(Yang) > ; Quan, Evan ; Yin, Tianci > (Rico) > Cc: Liu, Zhan > Subject: [PATCH] drm/amd/powerplay: Add SMU WMTABLE Validity Check > for Renoir > > [Why] > SMU watermark table (WMTABLE) validity check is missing on Renoir. This > validity check is very useful for checking whether WMTABLE is updated > successfully. > > [How] > Add SMU watermark validity check. > > Signed-off-by: Zhan Liu > --- > drivers/gpu/drm/amd/powerplay/renoir_ppt.c | 12 ++-- > 1 file changed, 10 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c > b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c > index 89a54f8e08d3..81520b0fca68 100644 > --- a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c > +++ b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c > @@ -777,9 +777,17 @@ static int renoir_set_watermarks_table( > } > > /* pass data to smu controller */ > - ret = smu_write_watermarks_table(smu); > + if ((smu->watermarks_bitmap & WATERMARKS_EXIST) && > + !(smu->watermarks_bitmap & > WATERMARKS_LOADED)) { > + ret = smu_write_watermarks_table(smu); > + if (ret) { > + pr_err("Failed to update WMTABLE!"); > + return ret; > + } > + smu->watermarks_bitmap |= WATERMARKS_LOADED; > + } > > - return ret; > + return 0; > } > > static int renoir_get_power_profile_mode(struct smu_context *smu, > -- > 2.17.1 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx