Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1667312 > unrolled thread

linux-next: build warning after merge of the wireless-drivers tree

Started byStephen Rothwell <sfr@canb.auug.org.au>
First post2017-06-16 00:30 +0200
Last post2017-06-16 07:10 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  linux-next: build warning after merge of the wireless-drivers tree Stephen Rothwell <sfr@canb.auug.org.au> - 2017-06-16 00:30 +0200
    Re: linux-next: build warning after merge of the wireless-drivers tree Kalle Valo <kvalo@codeaurora.org> - 2017-06-16 07:10 +0200

#1667312 — linux-next: build warning after merge of the wireless-drivers tree

FromStephen Rothwell <sfr@canb.auug.org.au>
Date2017-06-16 00:30 +0200
Subjectlinux-next: build warning after merge of the wireless-drivers tree
Message-ID<tSLr4-8li-11@gated-at.bofh.it>
Hi all,

After merging the wireless-drivers tree, today's linux-next build
(x86_64 allmodconfig) produced this warning:

drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c: In function 'brcmf_usb_probe_phase2':
drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c:1198:2: warning: 'devinfo' may be used uninitialized in this function [-Wmaybe-uninitialized]
  mutex_unlock(&devinfo->dev_init_lock);
  ^

Introduced by commit

  6d0507a777fb ("brcmfmac: add parameter to pass error code in firmware callback")

-- 
Cheers,
Stephen Rothwell

[toc] | [next] | [standalone]


#1667422

FromKalle Valo <kvalo@codeaurora.org>
Date2017-06-16 07:10 +0200
Message-ID<tSRG9-460-5@gated-at.bofh.it>
In reply to#1667312
Stephen Rothwell <sfr@canb.auug.org.au> writes:

> After merging the wireless-drivers tree, today's linux-next build
> (x86_64 allmodconfig) produced this warning:
>
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c: In function
> 'brcmf_usb_probe_phase2':
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c:1198:2:
> warning: 'devinfo' may be used uninitialized in this function
> [-Wmaybe-uninitialized]
>   mutex_unlock(&devinfo->dev_init_lock);
>   ^
>
> Introduced by commit
>
>   6d0507a777fb ("brcmfmac: add parameter to pass error code in firmware callback")

Thanks, for the report. I also noticed this, only after I had applied
the patch, but hopefully Arend sends a patch to fix this soon.

-- 
Kalle Valo

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web