From 30140d2be95acb683d74d6a3ac13ce1cf791971b Mon Sep 17 00:00:00 2001
From: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Date: Sat, 12 Nov 2016 13:50:17 +0100
Subject: soc/marvell/mvmap2315: Fix integer arithmetic
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Replace logical with bitwise AND operator.

Change-Id: I712fab61fbbc833fc2b2575948a875e2f07fd5de
Reported-by: Coverity (CID 1362808)
Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Reviewed-on: https://review.coreboot.org/17401
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
---
 src/soc/marvell/mvmap2315/mcu.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/soc/marvell/mvmap2315/mcu.c b/src/soc/marvell/mvmap2315/mcu.c
index c334cda145..e634ff8d56 100644
--- a/src/soc/marvell/mvmap2315/mcu.c
+++ b/src/soc/marvell/mvmap2315/mcu.c
@@ -31,7 +31,7 @@ static u8 mvmap2315_calc_checksum(const void  *data, u32 size)
 	for (i = csum = 0; i < size; i++)
 		csum += bytes[i];
 
-	return (~csum) && 0xFF;
+	return (~csum) & 0xFF;
 }
 
 void mcu_irq(void)
-- 
cgit v1.2.3