crypto: qat - remove unnecessary return codes
authorTadeusz Struk <tadeusz.struk@intel.com>
Fri, 25 Jul 2014 22:55:09 +0000 (15:55 -0700)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 1 Aug 2014 14:36:00 +0000 (22:36 +0800)
Remove unnecessary return code variables and change function types
accordingly.

Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Signed-off-by: Tadeusz Struk <tadeusz.struk@intel.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/qat/qat_common/qat_uclo.c

index dd4e0d3c323a65996a03f432fd37ba36b1aa1cb4..f22b48889d333d397dfdae5657026dce14cc6ff3 100644 (file)
@@ -1088,14 +1088,13 @@ static void qat_uclo_fill_uwords(struct icp_qat_uclo_objhandle *obj_handle,
                *uword = fill;
 }
 
-static int qat_uclo_wr_uimage_raw_page(struct icp_qat_fw_loader_handle *handle,
-                                      struct icp_qat_uclo_encap_page
-                                      *encap_page, unsigned int ae)
+static void qat_uclo_wr_uimage_raw_page(struct icp_qat_fw_loader_handle *handle,
+                                       struct icp_qat_uclo_encap_page
+                                       *encap_page, unsigned int ae)
 {
        unsigned int uw_physical_addr, uw_relative_addr, i, words_num, cpylen;
        struct icp_qat_uclo_objhandle *obj_handle = handle->obj_handle;
        uint64_t fill_pat;
-       int status = 0;
 
        /* load the page starting at appropriate ustore address */
        /* get fill-pattern from an image -- they are all the same */
@@ -1126,18 +1125,15 @@ static int qat_uclo_wr_uimage_raw_page(struct icp_qat_fw_loader_handle *handle,
                uw_relative_addr += cpylen;
                words_num -= cpylen;
        }
-       return status;
 }
 
-static int
-qat_uclo_wr_uimage_pages(struct icp_qat_fw_loader_handle *handle,
-                        struct icp_qat_uof_image *image)
+static void qat_uclo_wr_uimage_pages(struct icp_qat_fw_loader_handle *handle,
+                                    struct icp_qat_uof_image *image)
 {
        struct icp_qat_uclo_objhandle *obj_handle = handle->obj_handle;
        unsigned int ctx_mask, s;
        struct icp_qat_uclo_page *page;
        unsigned char ae;
-       int retval = 0;
        int ctx;
 
        if (ICP_QAT_CTX_MODE(image->ae_mode) == ICP_QAT_UCLO_MAX_CTX)
@@ -1160,8 +1156,7 @@ qat_uclo_wr_uimage_pages(struct icp_qat_fw_loader_handle *handle,
                page = obj_handle->ae_data[ae].ae_slices[s].page;
                if (!page->encap_page->def_page)
                        continue;
-               if (qat_uclo_wr_uimage_raw_page(handle, page->encap_page, ae))
-                       return -EINVAL;
+               qat_uclo_wr_uimage_raw_page(handle, page->encap_page, ae);
 
                page = obj_handle->ae_data[ae].ae_slices[s].page;
                for (ctx = 0; ctx < ICP_QAT_UCLO_MAX_CTX; ctx++)
@@ -1172,7 +1167,6 @@ qat_uclo_wr_uimage_pages(struct icp_qat_fw_loader_handle *handle,
                qat_hal_set_pc(handle, (unsigned char)ae, image->ctx_assigned,
                               image->entry_address);
        }
-       return retval;
 }
 
 int qat_uclo_wr_all_uimage(struct icp_qat_fw_loader_handle *handle)
@@ -1187,9 +1181,8 @@ int qat_uclo_wr_all_uimage(struct icp_qat_fw_loader_handle *handle)
                        return -EINVAL;
                if (qat_uclo_init_ustore(handle, &(obj_handle->ae_uimage[i])))
                        return -EINVAL;
-               if (qat_uclo_wr_uimage_pages(handle,
-                                            obj_handle->ae_uimage[i].img_ptr))
-                       return -EINVAL;
+               qat_uclo_wr_uimage_pages(handle,
+                                        obj_handle->ae_uimage[i].img_ptr);
        }
        return 0;
 }