Commit 751be5c6 authored by Liu Shixin's avatar Liu Shixin Committed by Mauro Carvalho Chehab

media: venus: simplify the return expression of venus_sys_set_* function

Simplify the return expression.
Signed-off-by: default avatarLiu Shixin <liushixin2@huawei.com>
Reviewed-by: default avatarBjorn Andersson <bjorn.andersson@linaro.org>
Signed-off-by: default avatarStanimir Varbanov <stanimir.k.varbanov@gmail.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@kernel.org>
parent 98d43518
...@@ -835,34 +835,24 @@ static int venus_sys_set_debug(struct venus_hfi_device *hdev, u32 debug) ...@@ -835,34 +835,24 @@ static int venus_sys_set_debug(struct venus_hfi_device *hdev, u32 debug)
{ {
struct hfi_sys_set_property_pkt *pkt; struct hfi_sys_set_property_pkt *pkt;
u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE]; u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE];
int ret;
pkt = (struct hfi_sys_set_property_pkt *)packet; pkt = (struct hfi_sys_set_property_pkt *)packet;
pkt_sys_debug_config(pkt, HFI_DEBUG_MODE_QUEUE, debug); pkt_sys_debug_config(pkt, HFI_DEBUG_MODE_QUEUE, debug);
ret = venus_iface_cmdq_write(hdev, pkt, false); return venus_iface_cmdq_write(hdev, pkt, false);
if (ret)
return ret;
return 0;
} }
static int venus_sys_set_coverage(struct venus_hfi_device *hdev, u32 mode) static int venus_sys_set_coverage(struct venus_hfi_device *hdev, u32 mode)
{ {
struct hfi_sys_set_property_pkt *pkt; struct hfi_sys_set_property_pkt *pkt;
u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE]; u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE];
int ret;
pkt = (struct hfi_sys_set_property_pkt *)packet; pkt = (struct hfi_sys_set_property_pkt *)packet;
pkt_sys_coverage_config(pkt, mode); pkt_sys_coverage_config(pkt, mode);
ret = venus_iface_cmdq_write(hdev, pkt, false); return venus_iface_cmdq_write(hdev, pkt, false);
if (ret)
return ret;
return 0;
} }
static int venus_sys_set_idle_message(struct venus_hfi_device *hdev, static int venus_sys_set_idle_message(struct venus_hfi_device *hdev,
...@@ -870,7 +860,6 @@ static int venus_sys_set_idle_message(struct venus_hfi_device *hdev, ...@@ -870,7 +860,6 @@ static int venus_sys_set_idle_message(struct venus_hfi_device *hdev,
{ {
struct hfi_sys_set_property_pkt *pkt; struct hfi_sys_set_property_pkt *pkt;
u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE]; u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE];
int ret;
if (!enable) if (!enable)
return 0; return 0;
...@@ -879,11 +868,7 @@ static int venus_sys_set_idle_message(struct venus_hfi_device *hdev, ...@@ -879,11 +868,7 @@ static int venus_sys_set_idle_message(struct venus_hfi_device *hdev,
pkt_sys_idle_indicator(pkt, enable); pkt_sys_idle_indicator(pkt, enable);
ret = venus_iface_cmdq_write(hdev, pkt, false); return venus_iface_cmdq_write(hdev, pkt, false);
if (ret)
return ret;
return 0;
} }
static int venus_sys_set_power_control(struct venus_hfi_device *hdev, static int venus_sys_set_power_control(struct venus_hfi_device *hdev,
...@@ -891,17 +876,12 @@ static int venus_sys_set_power_control(struct venus_hfi_device *hdev, ...@@ -891,17 +876,12 @@ static int venus_sys_set_power_control(struct venus_hfi_device *hdev,
{ {
struct hfi_sys_set_property_pkt *pkt; struct hfi_sys_set_property_pkt *pkt;
u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE]; u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE];
int ret;
pkt = (struct hfi_sys_set_property_pkt *)packet; pkt = (struct hfi_sys_set_property_pkt *)packet;
pkt_sys_power_control(pkt, enable); pkt_sys_power_control(pkt, enable);
ret = venus_iface_cmdq_write(hdev, pkt, false); return venus_iface_cmdq_write(hdev, pkt, false);
if (ret)
return ret;
return 0;
} }
static int venus_sys_set_ubwc_config(struct venus_hfi_device *hdev) static int venus_sys_set_ubwc_config(struct venus_hfi_device *hdev)
......
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