crypto: qat - remove writes into WQCFG
authorGiovanni Cabiddu <giovanni.cabiddu@intel.com>
Mon, 12 Oct 2020 20:38:29 +0000 (21:38 +0100)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 30 Oct 2020 06:34:52 +0000 (17:34 +1100)
WQCFG registers contain the correct values after reset in all
generations of QAT. No need to write into them.

Signed-off-by: Giovanni Cabiddu <giovanni.cabiddu@intel.com>
Reviewed-by: Fiona Trahe <fiona.trahe@intel.com>
Reviewed-by: Wojciech Ziemba <wojciech.ziemba@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_hw_arbiter.c

index d416278..cbb9f0b 100644 (file)
@@ -11,7 +11,6 @@
 #define ADF_ARB_REG_SLOT 0x1000
 #define ADF_ARB_WTR_OFFSET 0x010
 #define ADF_ARB_RO_EN_OFFSET 0x090
-#define ADF_ARB_WQCFG_OFFSET 0x100
 #define ADF_ARB_WRK_2_SER_MAP_OFFSET 0x180
 #define ADF_ARB_RINGSRVARBEN_OFFSET 0x19C
 
        ADF_ARB_WRK_2_SER_MAP_OFFSET) + \
        (ADF_ARB_REG_SIZE * index), value)
 
-#define WRITE_CSR_ARB_WQCFG(csr_addr, index, value) \
-       ADF_CSR_WR(csr_addr, (ADF_ARB_OFFSET + \
-       ADF_ARB_WQCFG_OFFSET) + (ADF_ARB_REG_SIZE * index), value)
-
 int adf_init_arb(struct adf_accel_dev *accel_dev)
 {
        struct adf_hw_device_data *hw_data = accel_dev->hw_device;
@@ -45,10 +40,6 @@ int adf_init_arb(struct adf_accel_dev *accel_dev)
        for (arb = 0; arb < ADF_ARB_NUM; arb++)
                WRITE_CSR_ARB_SARCONFIG(csr, arb, arb_cfg);
 
-       /* Setup worker queue registers */
-       for (i = 0; i < hw_data->num_engines; i++)
-               WRITE_CSR_ARB_WQCFG(csr, i, i);
-
        /* Map worker threads to service arbiters */
        hw_data->get_arb_mapping(accel_dev, &thd_2_arb_cfg);
 
@@ -84,10 +75,6 @@ void adf_exit_arb(struct adf_accel_dev *accel_dev)
        for (i = 0; i < ADF_ARB_NUM; i++)
                WRITE_CSR_ARB_SARCONFIG(csr, i, 0);
 
-       /* Shutdown work queue */
-       for (i = 0; i < hw_data->num_engines; i++)
-               WRITE_CSR_ARB_WQCFG(csr, i, 0);
-
        /* Unmap worker threads to service arbiters */
        for (i = 0; i < hw_data->num_engines; i++)
                WRITE_CSR_ARB_WRK_2_SER_MAP(csr, i, 0);