From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754230AbdBPJxL (ORCPT ); Thu, 16 Feb 2017 04:53:11 -0500 Received: from mail-wr0-f169.google.com ([209.85.128.169]:33362 "EHLO mail-wr0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750982AbdBPJxG (ORCPT ); Thu, 16 Feb 2017 04:53:06 -0500 Subject: Re: drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6: error: redefinition of 'brcmf_debugfs_init' To: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , Kalle Valo , Randy Dunlap References: <201702071839.Ej9MjmlI%fengguang.wu@intel.com> <87inoamqh6.fsf@kamboji.qca.qualcomm.com> Cc: kbuild test robot , kbuild-all@01.org, linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org From: Arend Van Spriel Message-ID: <59013c13-af43-cf9f-8e3e-14d97e48f0dd@broadcom.com> Date: Thu, 16 Feb 2017 10:53:02 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 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 >>): >>> >>> 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. Regards, Arend > Something like this maybe? Note I didn't test it with above config. > > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h > b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h > index 6687812770cc..8c7695fde482 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.h > @@ -108,7 +108,7 @@ do { \ > extern int brcmf_msg_level; > > struct brcmf_pub; > -#ifdef DEBUG > +#ifdef CONFIG_BRCMDBG > void brcmf_debugfs_init(void); > void brcmf_debugfs_exit(void); > int brcmf_debug_attach(struct brcmf_pub *drvr); >