crypto: qat - use BIT_ULL() - 1 pattern for masks
authorGiovanni Cabiddu <giovanni.cabiddu@intel.com>
Mon, 12 Oct 2020 20:38:37 +0000 (21:38 +0100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 30 Oct 2020 06:34:56 +0000 (17:34 +1100)
Replace occurrences of the pattern GENMASK_ULL(var - 1, 0)) with
BIT_ULL(var) - 1 since it produces better code and it is easier to read.

Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Giovanni Cabiddu <giovanni.cabiddu@intel.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/qat/qat_common/adf_sriov.c
drivers/crypto/qat/qat_dh895xcc/adf_dh895xcc_hw_data.c

index 0e8eab0..9a0f6db 100644 (file)
@@ -65,7 +65,7 @@ static int adf_enable_sriov(struct adf_accel_dev *accel_dev)
        hw_data->configure_iov_threads(accel_dev, true);
 
        /* Enable VF to PF interrupts for all VFs */
-       adf_enable_vf2pf_interrupts(accel_dev, GENMASK_ULL(totalvfs - 1, 0));
+       adf_enable_vf2pf_interrupts(accel_dev, BIT_ULL(totalvfs) - 1);
 
        /*
         * Due to the hardware design, when SR-IOV and the ring arbiter
index 7970ebb..1e83d93 100644 (file)
@@ -195,7 +195,7 @@ static void adf_enable_ints(struct adf_accel_dev *accel_dev)
        /* Enable bundle and misc interrupts */
        ADF_CSR_WR(addr, ADF_DH895XCC_SMIAPF0_MASK_OFFSET,
                   accel_dev->pf.vf_info ? 0 :
-                       GENMASK_ULL(GET_MAX_BANKS(accel_dev) - 1, 0));
+                       BIT_ULL(GET_MAX_BANKS(accel_dev)) - 1);
        ADF_CSR_WR(addr, ADF_DH895XCC_SMIAPF1_MASK_OFFSET,
                   ADF_DH895XCC_SMIA1_MASK);
 }