Skip to content

Commit 5d31bce

Browse files
aescolarkartben
authored andcommitted
drivers/i2c/i2c_ambiq: Fix code compliance issues
Fix 4 issues detected by checkpatch Signed-off-by: Alberto Escolar Piedras <alberto.escolar.piedras@nordicsemi.no>
1 parent 448eac1 commit 5d31bce

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

drivers/i2c/i2c_ambiq.c

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -124,9 +124,11 @@ static int i2c_ambiq_read(const struct device *dev, struct i2c_msg *hdr_msg,
124124
return -E2BIG;
125125
}
126126
#if defined(CONFIG_SOC_SERIES_APOLLO3X)
127-
trans.ui32Instr = (*(uint32_t *)hdr_msg->buf) & (0xFFFFFFFF >> (32 - (hdr_msg->len * 8)));
127+
trans.ui32Instr = (*(uint32_t *)hdr_msg->buf)
128+
& (0xFFFFFFFFUL >> (32 - (hdr_msg->len * 8)));
128129
#else
129-
trans.ui64Instr = (*(uint64_t *)hdr_msg->buf) & (0xFFFFFFFFFFFFFFFF >> (64 - (hdr_msg->len * 8)));
130+
trans.ui64Instr = (*(uint64_t *)hdr_msg->buf)
131+
& (0xFFFFFFFFFFFFFFFFULL >> (64 - (hdr_msg->len * 8)));
130132
#endif
131133
trans.ui32InstrLen = hdr_msg->len;
132134
}
@@ -177,9 +179,11 @@ static int i2c_ambiq_write(const struct device *dev, struct i2c_msg *hdr_msg,
177179
return -E2BIG;
178180
}
179181
#if defined(CONFIG_SOC_SERIES_APOLLO3X)
180-
trans.ui32Instr = (*(uint32_t *)hdr_msg->buf) & (0xFFFFFFFF >> (32 - (hdr_msg->len * 8)));
182+
trans.ui32Instr = (*(uint32_t *)hdr_msg->buf)
183+
& (0xFFFFFFFFUL >> (32 - (hdr_msg->len * 8)));
181184
#else
182-
trans.ui64Instr = (*(uint64_t *)hdr_msg->buf) & (0xFFFFFFFFFFFFFFFF >> (64 - (hdr_msg->len * 8)));
185+
trans.ui64Instr = (*(uint64_t *)hdr_msg->buf)
186+
& (0xFFFFFFFFFFFFFFFFULL >> (64 - (hdr_msg->len * 8)));
183187
#endif
184188
trans.ui32InstrLen = hdr_msg->len;
185189
}

0 commit comments

Comments
 (0)