diff options
| author |   Alex Hung <alex.hung@amd.com> | 2024-10-16 12:23:58 -0600 |
|---|---|---|
| committer |   Alex Deucher <alexander.deucher@amd.com> | 2024-10-28 16:36:26 -0400 |
| commit | d2bf27be839e89c6fd24b3ad3a2b38dcbfbf378a (patch) | |
| tree | c88abd64042fc6ccee8d66dd58a91195cd1b71fd | |
| parent | drm/amd/display: Remove useless assignments and variables (diff) | |
| download | wireguard-linux-d2bf27be839e89c6fd24b3ad3a2b38dcbfbf378a.tar.xz wireguard-linux-d2bf27be839e89c6fd24b3ad3a2b38dcbfbf378a.zip | |
drm/amd/display: Simplify dcn35_is_ips_supported()
[WHAT & HOW]
The variable "ips_supported" is redundant and we can return from
dcn35_smu_get_ips_supported directly.
This fixes 1 UNUSED_VALUE issue reported by Coverity.
Reviewed-by: Rodrigo Siqueira <rodrigo.siqueira@amd.com>
Signed-off-by: Alex Hung <alex.hung@amd.com>
Signed-off-by: Tom Chung <chiahsuan.chung@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
| -rw-r--r-- | drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c b/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c index 90208a5096c0..1d4fe0de0f67 100644 --- a/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c +++ b/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c @@ -978,11 +978,8 @@ static void dcn35_exit_low_power_state(struct clk_mgr *clk_mgr_base) static bool dcn35_is_ips_supported(struct clk_mgr *clk_mgr_base) { struct clk_mgr_internal *clk_mgr = TO_CLK_MGR_INTERNAL(clk_mgr_base); - bool ips_supported = true; - ips_supported = dcn35_smu_get_ips_supported(clk_mgr) ? true : false; - - return ips_supported; + return dcn35_smu_get_ips_supported(clk_mgr) ? true : false; } static void dcn35_init_clocks_fpga(struct clk_mgr *clk_mgr) |