Skip to content

Commit c700216

Browse files
gcabidduherbertx
authored andcommitted
crypto: qat - remove unneeded assignment
The function adf_gen4_get_vf2pf_sources() computes a mask which is stored in a variable which is returned and not used. Remove superfluous assignment of variable. This is to fix the following warning when compiling the QAT driver with clang scan-build: drivers/crypto/qat/qat_common/adf_gen4_pfvf.c:46:9: warning: Although the value stored to 'sou' is used in the enclosing expression, the value is never actually read from 'sou' [deadcode.DeadStores] return sou &= ~mask; ^ ~~~~~ Fixes: 5901b4a ("crypto: qat - fix access to PFVF interrupt registers for GEN4") Signed-off-by: Giovanni Cabiddu <giovanni.cabiddu@intel.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
1 parent 8893d27 commit c700216

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/crypto/qat/qat_common/adf_gen4_pfvf.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ static u32 adf_gen4_get_vf2pf_sources(void __iomem *pmisc_addr)
4343
sou = ADF_CSR_RD(pmisc_addr, ADF_4XXX_VM2PF_SOU);
4444
mask = ADF_CSR_RD(pmisc_addr, ADF_4XXX_VM2PF_MSK);
4545

46-
return sou &= ~mask;
46+
return sou & ~mask;
4747
}
4848

4949
static void adf_gen4_enable_vf2pf_interrupts(void __iomem *pmisc_addr,

0 commit comments

Comments
 (0)