s390/checksum: have consistent calculations
authorHeiko Carstens <hca@linux.ibm.com>
Tue, 11 Aug 2020 14:37:33 +0000 (16:37 +0200)
committerVasily Gorbik <gor@linux.ibm.com>
Wed, 26 Aug 2020 16:47:20 +0000 (18:47 +0200)
Use "|" instead of "+" within csum_fold() for consistency reasons,
like in the rest of the file.

Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
arch/s390/include/asm/checksum.h

index 8bc6bed..de97ae6 100644 (file)
@@ -54,7 +54,7 @@ static inline __sum16 csum_fold(__wsum sum)
 {
        u32 csum = (__force u32) sum;
 
-       csum += (csum >> 16) + (csum << 16);
+       csum += (csum >> 16) | (csum << 16);
        csum >>= 16;
        return (__force __sum16) ~csum;
 }