Arend Van Spriel <arend.vanspr...@broadcom.com> writes: > On 16-2-2017 10:39, Rafał Miłecki wrote: >> On 02/16/2017 10:31 AM, Kalle Valo wrote: >>> (Adding linux-wireless) >>> >>> Randy Dunlap <rdun...@infradead.org> writes: >>> >>>> On 02/07/17 02:02, kbuild test robot wrote: >>>>> Hi Kalle, >>>>> >>>>> FYI, the error/warning still remains. >>>>> >>>>> tree: >>>>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git >>>>> master >>>>> head: 8b1b41ee74f9712c355d66dc105bbea663ae0afd >>>>> commit: 05491d2ccf20b20a1375303441fbbfbd12b24a4f brcm80211: move >>>>> under broadcom vendor directory >>>>> date: 1 year, 3 months ago >>>>> config: mips-allyesconfig (attached as .config) >>>>> compiler: mips-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 >>>>> reproduce: >>>>> wget >>>>> https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross >>>>> >>>>> -O ~/bin/make.cross >>>>> chmod +x ~/bin/make.cross >>>>> git checkout 05491d2ccf20b20a1375303441fbbfbd12b24a4f >>>>> # save the attached .config to linux build tree >>>>> make.cross ARCH=mips >>>>> >>>>> All errors (new ones prefixed by >>): n>>>> >>>> when both: >>>> >>>> CONFIG_BRCM_TRACING=y >>>> CONFIG_BRCMDBG=y >>>> >>>> but DEBUG is not defined. >>>> >>>> I think it would help if CONFIG_BRCMDBG set DEBUG >>>> or if some of the tests for DEBUG used CONFIG_BRCMDBG instead. >>> >>> Arend or Rafał, would you be able to look at this build problem? Kbuild >>> found the build errors below, apparently a very old issue. It just >>> blames me now because I moved the driver to a new directory :) > > It is a very old issue. The thing is that the issue can not really occur > because we have this: > > # common flags > subdir-ccflags-$(CONFIG_BRCMDBG) += -DDEBUG > > in drivers/net/wireless/broadcom/brcm80211/Makefile. So no clue what > happens in the "kbuild test robot" scenario.
I actually tried to reproduce this with kbuild bot's instructions and was surprised to see that the build worked ok for me. So the issue is not that simple. -- Kalle Valo
