[PATCH 2/3] hab: ahab: cosmetic change

Sascha Hauer s.hauer at pengutronix.de
Mon Jul 15 05:36:13 PDT 2024


We have the SRK_HASH_SIZE define, so use it instead of hardcoded 32.
Also, use sizeof(u32) instead of sizeof(uint32_t) as that is the type
we are operating on.

Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
---
 drivers/hab/hab.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/hab/hab.c b/drivers/hab/hab.c
index 288f440bd2..5701e5feb5 100644
--- a/drivers/hab/hab.c
+++ b/drivers/hab/hab.c
@@ -278,7 +278,7 @@ static int imx_ahab_write_srk_hash(const u8 *__newsrk, unsigned flags)
 		return -EPERM;
 	}
 
-	for (i = 0; i < 32 / sizeof(u32); i++) {
+	for (i = 0; i < SRK_HASH_SIZE / sizeof(u32); i++) {
 		ret = ele_write_fuse(0x80 + i, newsrk[i], false, &resp);
 		if (ret) {
 			pr_err("Writing fuse index 0x%02x failed with %d, response 0x%08x\n",
@@ -296,7 +296,7 @@ static int imx_ahab_read_srk_hash(u8 *__srk)
 	u32 resp;
 	int ret, i;
 
-	for (i = 0; i < SRK_HASH_SIZE / sizeof(uint32_t); i++) {
+	for (i = 0; i < SRK_HASH_SIZE / sizeof(u32); i++) {
 		ret = ele_read_common_fuse(0x80 + i, &srk[i], &resp);
 		if (ret < 0)
 			return ret;
-- 
2.39.2




More information about the barebox mailing list