diff options
| author |   Ye Guojin <ye.guojin@zte.com.cn> | 2021-11-02 07:01:25 +0000 |
|---|---|---|
| committer |   Alex Deucher <alexander.deucher@amd.com> | 2021-11-17 16:58:04 -0500 |
| commit | b97788e504daf4ed455ec35047977a3016355571 (patch) | |
| tree | e983e702b32f39a12fa3252027a49048e62bfacb /drivers/gpu/drm/amd/display/dc/core/dc_link_ddc.c | |
| parent | drm/amdgpu: return early on error while setting bar0 memtype (diff) | |
| download | linux-dev-b97788e504daf4ed455ec35047977a3016355571.tar.xz linux-dev-b97788e504daf4ed455ec35047977a3016355571.zip | |
drm/amd/display: remove unnecessary conditional operators
Since the variables named is_end_of_payload and hpd_status are already
bool type, the ?: conditional operator is unnecessary any more.
Clean them up here.
Reported-by: Zeal Robot <zealci@zte.com.cn>
Reviewed-by: Simon Ser <contact@emersion.fr>
Signed-off-by: Ye Guojin <ye.guojin@zte.com.cn>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/display/dc/core/dc_link_ddc.c')
| -rw-r--r-- | drivers/gpu/drm/amd/display/dc/core/dc_link_ddc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_ddc.c b/drivers/gpu/drm/amd/display/dc/core/dc_link_ddc.c index 60539b1f2a80..24dc662ec3e4 100644 --- a/drivers/gpu/drm/amd/display/dc/core/dc_link_ddc.c +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_ddc.c @@ -626,7 +626,7 @@ bool dal_ddc_submit_aux_command(struct ddc_service *ddc, do { struct aux_payload current_payload; bool is_end_of_payload = (retrieved + DEFAULT_AUX_MAX_DATA_SIZE) >= - payload->length ? true : false; + payload->length; uint32_t payload_length = is_end_of_payload ? payload->length - retrieved : DEFAULT_AUX_MAX_DATA_SIZE; |