media: venus: simplify the return expression of venus_sys_set_* function
authorLiu Shixin <liushixin2@huawei.com>
Mon, 21 Sep 2020 08:24:40 +0000 (09:24 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 9 Jun 2023 12:41:42 +0000 (13:41 +0100)
Simplify the return expression.

Signed-off-by: Liu Shixin <liushixin2@huawei.com>
Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Signed-off-by: Stanimir Varbanov <stanimir.k.varbanov@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/platform/qcom/venus/hfi_venus.c

index 2ad40b3945b0b315c60b12a9a504289001555809..f0b46389e8d569c8e3e2dfd56ee55d10f45c8f6b 100644 (file)
@@ -835,34 +835,24 @@ static int venus_sys_set_debug(struct venus_hfi_device *hdev, u32 debug)
 {
        struct hfi_sys_set_property_pkt *pkt;
        u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE];
-       int ret;
 
        pkt = (struct hfi_sys_set_property_pkt *)packet;
 
        pkt_sys_debug_config(pkt, HFI_DEBUG_MODE_QUEUE, debug);
 
-       ret = venus_iface_cmdq_write(hdev, pkt, false);
-       if (ret)
-               return ret;
-
-       return 0;
+       return venus_iface_cmdq_write(hdev, pkt, false);
 }
 
 static int venus_sys_set_coverage(struct venus_hfi_device *hdev, u32 mode)
 {
        struct hfi_sys_set_property_pkt *pkt;
        u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE];
-       int ret;
 
        pkt = (struct hfi_sys_set_property_pkt *)packet;
 
        pkt_sys_coverage_config(pkt, mode);
 
-       ret = venus_iface_cmdq_write(hdev, pkt, false);
-       if (ret)
-               return ret;
-
-       return 0;
+       return venus_iface_cmdq_write(hdev, pkt, false);
 }
 
 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;
        u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE];
-       int ret;
 
        if (!enable)
                return 0;
@@ -879,11 +868,7 @@ static int venus_sys_set_idle_message(struct venus_hfi_device *hdev,
 
        pkt_sys_idle_indicator(pkt, enable);
 
-       ret = venus_iface_cmdq_write(hdev, pkt, false);
-       if (ret)
-               return ret;
-
-       return 0;
+       return venus_iface_cmdq_write(hdev, pkt, false);
 }
 
 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;
        u8 packet[IFACEQ_VAR_SMALL_PKT_SIZE];
-       int ret;
 
        pkt = (struct hfi_sys_set_property_pkt *)packet;
 
        pkt_sys_power_control(pkt, enable);
 
-       ret = venus_iface_cmdq_write(hdev, pkt, false);
-       if (ret)
-               return ret;
-
-       return 0;
+       return venus_iface_cmdq_write(hdev, pkt, false);
 }
 
 static int venus_sys_set_ubwc_config(struct venus_hfi_device *hdev)