| Message ID | 20170716151653.GA2109@lkp-wsx02 (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Kalle Valo |
| Headers | show |
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c @@ -280,7 +280,7 @@ static bool brcmf_chip_axi_iscoreup(stru val = brcmf_readl(ci, core->wrapbase + BCMA_IOCTL); if((val & BCMA_IOCTL_FGC) || !(val & BCMA_IOCTL_CLK)) - return 0; + return false; /* Is core in reset? */ val = brcmf_readl(ci, core->wrapbase + BCMA_RESET_CTL);
drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c:283:9-10: WARNING: return of 0/1 in function 'brcmf_chip_axi_iscoreup' with return type bool Return statements in functions returning bool should use true/false instead of 1/0. Generated by: scripts/coccinelle/misc/boolreturn.cocci Fixes: f9996a7ac81f ("brcmfmac: HUGE cleanup of IO access functions.") CC: Ian Molton <ian@mnementh.co.uk> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com> --- chip.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)