OK, it indeed should return the 'result' to caller,
I will make a modification and repost the patch.
Thanks
Chen Wandun
On 2019/11/18 14:50, Quan, Evan wrote:
Thanks. But it's better to return the 'result' out on 'result != 0'.
Regards,
Evan
-Original Message-
From: Chen Wandun
Sent
:
variable result set but not used [-Wunused-but-set-variable]
Signed-off-by: Chen Wandun
---
drivers/gpu/drm/amd/powerplay/smumgr/vegam_smumgr.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/vegam_smumgr.c
b/drivers/gpu/drm/amd/powerplay/smumgr
]
Signed-off-by: Chen Wandun
---
drivers/gpu/drm/amd/powerplay/smumgr/vegam_smumgr.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/vegam_smumgr.c
b/drivers/gpu/drm/amd/powerplay/smumgr/vegam_smumgr.c
index 2068eb0..fad78bf
]
Signed-off-by: Chen Wandun
---
drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr.c
b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr.c
index 32ebb38..6df5af0 100644
From: Chenwandun
drivers/gpu/drm/drm_dp_mst_topology.c: In function __topology_ref_save:
drivers/gpu/drm/drm_dp_mst_topology.c:1424:6: error: implicit declaration of
function stack_trace_save; did you mean stack_depot_save?
[-Werror=implicit-function-declaration]
n =
From: Chenwandun
drivers/gpu/drm/amd/display/dc/dce/dce_aux.c: In function
dce_aux_configure_timeout:
drivers/gpu/drm/amd/display/dc/dce/dce_aux.c: warning: variable timeout set but
not used [-Wunused-but-set-variable]
Signed-off-by: Chenwandun
---
From: Chenwandun
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn20/dcn20_resource.c:1913:48: error:
struct dc_crtc_timing_flags has no member named DSC
if (res_ctx->pipe_ctx[i].stream->timing.flags.DSC)
^
