[PATCH] LoongArch/crypto: Clean up useless assignment operations

From: Yuli Wang
Date: Tue Mar 19 2024 - 03:50:34 EST


The LoongArch CRC32 hw acceleration is based on arch/mips/crypto/
crc32-mips.c. While the MIPS code supports both MIPS32 and MIPS64,
but LoongArch32 lacks the CRC instruction. As a result, the line
"len -= sizeof(u32)" is unnecessary.

Removing it can make context code style more unified and improve
code readability.

Cc: stable@xxxxxxxxxxxxxxx
Reviewed-by: WANG Xuerui <git@xxxxxxxxxx>
Suggested-by: Wentao Guan <guanwentao@xxxxxxxxxxxxx>
Signed-off-by: Yuli Wang <wangyuli@xxxxxxxxxxxxx>
Signed-off-by: Huacai Chen <chenhuacai@xxxxxxxxxxx>
---
arch/loongarch/crypto/crc32-loongarch.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/arch/loongarch/crypto/crc32-loongarch.c b/arch/loongarch/crypto/crc32-loongarch.c
index a49e507af38c0..3eebea3a7b478 100644
--- a/arch/loongarch/crypto/crc32-loongarch.c
+++ b/arch/loongarch/crypto/crc32-loongarch.c
@@ -44,7 +44,6 @@ static u32 crc32_loongarch_hw(u32 crc_, const u8 *p, unsigned int len)

CRC32(crc, value, w);
p += sizeof(u32);
- len -= sizeof(u32);
}

if (len & sizeof(u16)) {
@@ -80,7 +79,6 @@ static u32 crc32c_loongarch_hw(u32 crc_, const u8 *p, unsigned int len)

CRC32C(crc, value, w);
p += sizeof(u32);
- len -= sizeof(u32);
}

if (len & sizeof(u16)) {
--
2.43.0