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>

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index c911b30de6588e707611095c61975d7241652589..06cf674026bfc406b9502fe836931b158ca8b78a 100644 (file)
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -789,8 +789,7 @@ static void dm_dmub_outbox1_low_irq(void *interrupt_params)
                                        plink = adev->dm.dc->links[notify.link_index];
                                        if (plink) {
                                                plink->hpd_status =
-                                                       notify.hpd_status ==
-                                                       DP_HPD_PLUG ? true : false;
+                                                       notify.hpd_status == DP_HPD_PLUG;
                                        }
                                }
                                queue_work(adev->dm.delayed_hpd_wq, &dmub_hpd_wrk->handle_hpd_work);

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 60539b1f2a80a155d2927f0d8693dc8fc823bf1e..24dc662ec3e4523880a000b1625ebf0b6b123c0c 100644 (file)
--- 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;