Skip to content

Commit ccab5cd

Browse files
cujomalaineylgirdwood
authored andcommitted
ipc: fix signed int overflow in macro
shifting anything into the sign bit location is technically undefined, add unsigned marker to satisfy cppcheck Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
1 parent 773bfc0 commit ccab5cd

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/include/ipc/header.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@
154154

155155
/** Shift-left bits to extract the global cmd type */
156156
#define SOF_GLB_TYPE_SHIFT 28
157-
#define SOF_GLB_TYPE_MASK (0xfL << SOF_GLB_TYPE_SHIFT)
157+
#define SOF_GLB_TYPE_MASK (0xfUL << SOF_GLB_TYPE_SHIFT)
158158
#define SOF_GLB_TYPE(x) ((x) << SOF_GLB_TYPE_SHIFT)
159159

160160
/** @} */

src/include/kernel/abi.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
/** \brief SOF ABI version major, minor and patch numbers */
3131
#define SOF_ABI_MAJOR 3
3232
#define SOF_ABI_MINOR 20
33-
#define SOF_ABI_PATCH 0
33+
#define SOF_ABI_PATCH 1
3434

3535
/** \brief SOF ABI version number. Format within 32bit word is MMmmmppp */
3636
#define SOF_ABI_MAJOR_SHIFT 24

0 commit comments

Comments
 (0)