Commit 5af47fb3 authored by Arend van Spriel's avatar Arend van Spriel Committed by John W. Linville

brcmfmac: fix sparse error 'bad constant expression'

The definition of BRCMF_EVENTING_MASK_LEN results in a sparse
error message

.../fweh.c:331:22: error: bad constant expression
.../fweh.c:388:22: error: bad constant expression
.../dhd_common.c:256:22: error: bad constant expression

This is caused by the use of roundup() in BRCMF_EVENTING_MASK_LEN
and it is used to allocate an array variable on the stack. Better
use DIV_ROUND_UP() macro.
Reviewed-by: default avatarFranky (Zhenhui) Lin <frankyl@broadcom.com>
Reviewed-by: default avatarHante Meuleman <meuleman@broadcom.com>
Reviewed-by: default avatarPieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: default avatarArend van Spriel <arend@broadcom.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 5491c11c
...@@ -115,7 +115,7 @@ enum brcmf_fweh_event_code { ...@@ -115,7 +115,7 @@ enum brcmf_fweh_event_code {
}; };
#undef BRCMF_ENUM_DEF #undef BRCMF_ENUM_DEF
#define BRCMF_EVENTING_MASK_LEN (roundup(BRCMF_E_LAST, 8)/8) #define BRCMF_EVENTING_MASK_LEN DIV_ROUND_UP(BRCMF_E_LAST, 8)
/* flags field values in struct brcmf_event_msg */ /* flags field values in struct brcmf_event_msg */
#define BRCMF_EVENT_MSG_LINK 0x01 #define BRCMF_EVENT_MSG_LINK 0x01
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment