<div dir="auto"><div>Ok but out the Coverity Id number in the commit long message <br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 2, 2023, 1:17 AM Christian Mauderer <<a href="mailto:christian.mauderer@embedded-brains.de">christian.mauderer@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Coverity warns that (buf[...] & 0x7) can't be bigger than 7. This patch<br>
removes the unnecessary comparison.<br>
---<br>
 bsps/shared/dev/rtc/mcp7940m.c | 5 ++---<br>
 1 file changed, 2 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/bsps/shared/dev/rtc/mcp7940m.c b/bsps/shared/dev/rtc/mcp7940m.c<br>
index 78a4f21b58..1abc5faaad 100644<br>
--- a/bsps/shared/dev/rtc/mcp7940m.c<br>
+++ b/bsps/shared/dev/rtc/mcp7940m.c<br>
@@ -312,9 +312,8 @@ static int mcp7940m_set_time(int minor, const rtems_time_of_day *time)<br>
   }<br>
<br>
   if (rv == 0) {<br>
-    /* Make sure weekday is in range. Otherwise it's not relevant. */<br>
-    if (RTCWKDAY_WKDAY_GET(buf[REG_RTCWKDAY]) < 1 ||<br>
-        RTCWKDAY_WKDAY_GET(buf[REG_RTCWKDAY]) > 7) {<br>
+    /* Make sure weekday is not 0 (out of range). Otherwise it's not used. */<br>
+    if (RTCWKDAY_WKDAY_GET(buf[REG_RTCWKDAY]) < 1) {<br>
       buf[REG_RTCWKDAY] &= ~RTCWKDAY_WKDAY_MASK;<br>
       buf[REG_RTCWKDAY] |= RTCWKDAY_WKDAY(1);<br>
     }<br>
-- <br>
2.35.3<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>