Commit b97788e5 authored by Ye Guojin's avatar Ye Guojin Committed by Alex Deucher

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: default avatarZeal Robot <zealci@zte.com.cn>
Reviewed-by: default avatarSimon Ser <contact@emersion.fr>
Signed-off-by: default avatarYe Guojin <ye.guojin@zte.com.cn>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 26db557e
...@@ -789,8 +789,7 @@ static void dm_dmub_outbox1_low_irq(void *interrupt_params) ...@@ -789,8 +789,7 @@ static void dm_dmub_outbox1_low_irq(void *interrupt_params)
plink = adev->dm.dc->links[notify.link_index]; plink = adev->dm.dc->links[notify.link_index];
if (plink) { if (plink) {
plink->hpd_status = plink->hpd_status =
notify.hpd_status == notify.hpd_status == DP_HPD_PLUG;
DP_HPD_PLUG ? true : false;
} }
} }
queue_work(adev->dm.delayed_hpd_wq, &dmub_hpd_wrk->handle_hpd_work); queue_work(adev->dm.delayed_hpd_wq, &dmub_hpd_wrk->handle_hpd_work);
......
...@@ -626,7 +626,7 @@ bool dal_ddc_submit_aux_command(struct ddc_service *ddc, ...@@ -626,7 +626,7 @@ bool dal_ddc_submit_aux_command(struct ddc_service *ddc,
do { do {
struct aux_payload current_payload; struct aux_payload current_payload;
bool is_end_of_payload = (retrieved + DEFAULT_AUX_MAX_DATA_SIZE) >= 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 ? uint32_t payload_length = is_end_of_payload ?
payload->length - retrieved : DEFAULT_AUX_MAX_DATA_SIZE; payload->length - retrieved : DEFAULT_AUX_MAX_DATA_SIZE;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment