commit 101afa02b71d1a09bb20ee84451cbbfa7ad6c5d0
author Haojian Zhuang <haojian.zhuang@linaro.org> Mon Jun 12 22:18:15 2017 +0800
committer Haojian Zhuang <haojian.zhuang@linaro.org> Tue Jun 13 14:33:49 2017 +0800
tree 26a259f96730c1922a4f0be796b2992156a74162
parent c396b7368a04e561e1f684797f541bfce8037241

ufs: fix the and operator

Should use AND (&), not &&.

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>

drivers/ufs/ufs.c

diff --git a/drivers/ufs/ufs.c b/drivers/ufs/ufs.c
index dbfcd0f..d513d0a 100644
--- a/drivers/ufs/ufs.c
+++ b/drivers/ufs/ufs.c
@@ -45,7 +45,7 @@
 		data = mmio_read_32(base + IS);
 	} while ((data & UFS_INT_UCCS) == 0);
 	mmio_write_32(base + IS, UFS_INT_UCCS);
-	return mmio_read_32(base + UCMDARG2) && CONFIG_RESULT_CODE_MASK;
+	return mmio_read_32(base + UCMDARG2) & CONFIG_RESULT_CODE_MASK;
 }
 
 int ufshc_dme_get(unsigned int attr, unsigned int idx, unsigned int *val)
@@ -77,7 +77,7 @@
 			return -EINVAL;
 	} while ((data & UFS_INT_UCCS) == 0);
 	mmio_write_32(base + IS, UFS_INT_UCCS);
-	data = mmio_read_32(base + UCMDARG2) && CONFIG_RESULT_CODE_MASK;
+	data = mmio_read_32(base + UCMDARG2) & CONFIG_RESULT_CODE_MASK;
 	assert(data == 0);
 
 	*val = mmio_read_32(base + UCMDARG3);
@@ -106,7 +106,7 @@
 			return -EINVAL;
 	} while ((data & UFS_INT_UCCS) == 0);
 	mmio_write_32(base + IS, UFS_INT_UCCS);
-	data = mmio_read_32(base + UCMDARG2) && CONFIG_RESULT_CODE_MASK;
+	data = mmio_read_32(base + UCMDARG2) & CONFIG_RESULT_CODE_MASK;
 	assert(data == 0);
 	return 0;
 }