From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753537AbdBPJq5 (ORCPT ); Thu, 16 Feb 2017 04:46:57 -0500 Received: from 7.mo7.mail-out.ovh.net ([46.105.43.131]:47782 "EHLO 7.mo7.mail-out.ovh.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752926AbdBPJqx (ORCPT ); Thu, 16 Feb 2017 04:46:53 -0500 X-Greylist: delayed 8459 seconds by postgrey-1.27 at vger.kernel.org; Thu, 16 Feb 2017 04:46:53 EST Subject: Re: drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6: error: redefinition of 'brcmf_debugfs_init' To: Kalle Valo , Randy Dunlap , Arend Van Spriel 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: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= Message-ID: Date: Thu, 16 Feb 2017 10:39:40 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <87inoamqh6.fsf@kamboji.qca.qualcomm.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-Ovh-Tracer-Id: 8314770813677309636 X-VR-SPAMSTATE: OK X-VR-SPAMSCORE: -100 X-VR-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrfeelhedrtdeggddthecutefuodetggdotefrodftvfcurfhrohhfihhlvgemucfqggfjpdevjffgvefmvefgnecuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 :) 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);