crypto: qat - fix simple_return.cocci warnings
authorWu Fengguang <fengguang.wu@intel.com>
Wed, 12 Aug 2015 04:50:17 +0000 (12:50 +0800)
committerHerbert Xu <herbert@gondor.apana.org.au>
Thu, 13 Aug 2015 07:13:28 +0000 (15:13 +0800)
drivers/crypto/qat/qat_common/adf_sriov.c:258:1-4: WARNING: end returns can be simpified and declaration on line 212 can be dropped

 Simplify a trivial if-return sequence.  Possibly combine with a
 preceding function call.
Generated by: scripts/coccinelle/misc/simple_return.cocci

CC: Tadeusz Struk <tadeusz.struk@intel.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/qat/qat_common/adf_sriov.c

index a766fb59e71baf9a1aff6cc9bced36e67c884a3c..069b9ea55cfa2a7aee5f96d32298743457db4241 100644 (file)
@@ -209,7 +209,7 @@ static int adf_enable_sriov(struct adf_accel_dev *accel_dev)
                        &GET_BARS(accel_dev)[hw_data->get_misc_bar_id(hw_data)];
        void __iomem *pmisc_addr = pmisc->virt_addr;
        struct adf_accel_vf_info *vf_info;
-       int i, ret;
+       int i;
        u32 reg;
 
        /* Workqueue for PF2VF responses */
@@ -255,11 +255,7 @@ static int adf_enable_sriov(struct adf_accel_dev *accel_dev)
         * order for all the hardware resources (i.e. bundles) to be usable.
         * When SR-IOV is enabled, each of the VFs will own one bundle.
         */
-       ret = pci_enable_sriov(pdev, totalvfs);
-       if (ret)
-               return ret;
-
-       return 0;
+       return pci_enable_sriov(pdev, totalvfs);
 }
 
 /**