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

Groups > linux.kernel > #1583024

Re: [kbuild-all] drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6: error: redefinition of 'brcmf_debugfs_init'

Show all headers | View raw

Hi Kalle,

On Thu, Feb 16, 2017 at 03:18:48PM +0200, Kalle Valo wrote:
>Arend Van Spriel <arend.vanspriel@broadcom.com> writes:
>
>> On 16-2-2017 11:01, Kalle Valo wrote:
>>> Arend Van Spriel <arend.vanspriel@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)
>>>>>>
>>>>>> 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.
>>
>> Hi Kalle,
>>
>> Here is the old email exchange we had about this [1]. In my opinion this
>> is a Kbuild issue that needs to be looked into.
>>
>> Regards,
>> Arend
>>
>> [1] http://www.spinics.net/lists/linux-wireless/msg142122.html
>
>Heh, I had completely forgotten that we had discussed this before :)
>
>Fengguang and the kbuild bot team, do you have any ideas why only the
>bot is seeing this?

I guess the root cause is due to the robot's doing partial builds like
these

        make drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.o
        make drivers/net/wireless/broadcom/brcm80211/brcmfmac/

The solution I come about is to climb up to higher level dirs like 

        make drivers/net/wireless/

which is less likely to have Makefile tricks and still maintain fast
enough build tests.

Thanks,
Fengguang

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6: error: redefinition of 'brcmf_debugfs_init' Kalle Valo <kvalo@codeaurora.org> - 2017-02-16 10:40 +0100
  Re: drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6:  error: redefinition of 'brcmf_debugfs_init' Rafał Miłecki <rafal@milecki.pl> - 2017-02-16 10:50 +0100
    Re: drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6:  error: redefinition of 'brcmf_debugfs_init' Arend Van Spriel <arend.vanspriel@broadcom.com> - 2017-02-16 11:00 +0100
      Re: drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6: error: redefinition of 'brcmf_debugfs_init' Kalle Valo <kvalo@codeaurora.org> - 2017-02-16 11:10 +0100
        Re: drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6:  error: redefinition of 'brcmf_debugfs_init' Arend Van Spriel <arend.vanspriel@broadcom.com> - 2017-02-16 11:40 +0100
          Re: [kbuild-all]  drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6: error:  redefinition of 'brcmf_debugfs_init' Fengguang Wu <fengguang.wu@intel.com> - 2017-02-16 14:10 +0100
          Re: drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6: error: redefinition of 'brcmf_debugfs_init' Kalle Valo <kvalo@codeaurora.org> - 2017-02-16 14:20 +0100
            Re: [kbuild-all]  drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c:58:6: error:  redefinition of 'brcmf_debugfs_init' Fengguang Wu <fengguang.wu@intel.com> - 2017-02-17 02:50 +0100

csiph-web