Subject: Re: linux-next: Tree for Dec 19 (nfc)

From: "John W. Linville" <[email-masked]>
Date: Wed, 21 Dec 2011 14:02:11 -0500

On Wed, Dec 21, 2011 at 11:57:19AM -0800, Randy Dunlap wrote:
> On 12/20/2011 08:39 AM, Samuel Ortiz wrote:
> > Hi Randy,
> > 
> > On Mon, 2011-12-19 at 12:44 -0800, Randy Dunlap wrote:
> >> On 12/18/2011 11:55 PM, Stephen Rothwell wrote:
> >>> Hi all,
> >>>
> >>> Changes since 20111216:
> >>
> >>
> >> (not new today, just not reported earlier)
> > John applied my fix for this one. It should be fixed now.
> 
> Hi Samuel,
> 
> Is this supposed to be fixed in linux-next?
> It still fails today (linux-next 20111221).
> 
> 
> data.c:(.text+0x3e0): multiple definition of `nfc_llcp_mac_is_down'
> data.c:(.text+0x400): multiple definition of `nfc_llcp_mac_is_up'
> ntf.c:(.text+0xb60): multiple definition of `nfc_llcp_mac_is_down'
> ntf.c:(.text+0xb80): multiple definition of `nfc_llcp_mac_is_up'
> rsp.c:(.text+0x750): multiple definition of `nfc_llcp_mac_is_down'
> rsp.c:(.text+0x770): multiple definition of `nfc_llcp_mac_is_up'
> netlink.c:(.text+0x14e0): multiple definition of `nfc_llcp_mac_is_down'
> netlink.c:(.text+0x1500): multiple definition of `nfc_llcp_mac_is_up'
> af_nfc.c:(.text+0xe0): multiple definition of `nfc_llcp_mac_is_down'
> af_nfc.c:(.text+0x100): multiple definition of `nfc_llcp_mac_is_up'
> rawsock.c:(.text+0x1000): multiple definition of `nfc_llcp_mac_is_down'
> rawsock.c:(.text+0x1020): multiple definition of `nfc_llcp_mac_is_up'

I have this commit in wireless-next, been there a couple of days:

commit 645d35902c8f05a1b12fa838aa9052d8eeaf161e
Author: Samuel Ortiz <[email-masked]>
Date:   Fri Dec 16 15:03:36 2011 +0100

    NFC: Fix LLCP related build failure
    
    llcp_mac routines should be static and inlined or build will fail with NFC
    selected without LLCP.
    
    This patch fixes:
    
    LD [M]  net/nfc/nfc.o
    net/nfc/netlink.o: In function `nfc_llcp_mac_is_down':
    netlink.c:(.text+0x0): multiple definition of `nfc_llcp_mac_is_down'
    net/nfc/core.o:(.text+0x0): first defined here
    net/nfc/netlink.o: In function `nfc_llcp_mac_is_up':
    netlink.c:(.text+0x10): multiple definition of `nfc_llcp_mac_is_up'
    net/nfc/core.o:(.text+0x10): first defined here
    net/nfc/af_nfc.o: In function `nfc_llcp_mac_is_down':
    (.text+0x0): multiple definition of `nfc_llcp_mac_is_down'
    net/nfc/core.o:(.text+0x0): first defined here
    net/nfc/af_nfc.o: In function `nfc_llcp_mac_is_up':
    (.text+0x10): multiple definition of `nfc_llcp_mac_is_up'
    net/nfc/core.o:(.text+0x10): first defined here
    net/nfc/rawsock.o: In function `nfc_llcp_mac_is_down':
    rawsock.c:(.text+0x0): multiple definition of `nfc_llcp_mac_is_down'
    net/nfc/core.o:(.text+0x0): first defined here
    net/nfc/rawsock.o: In function `nfc_llcp_mac_is_up':
    rawsock.c:(.text+0x10): multiple definition of `nfc_llcp_mac_is_up'
    net/nfc/core.o:(.text+0x10): first defined here
    
    Signed-off-by: Samuel Ortiz <[email-masked]>
    Signed-off-by: John W. Linville <[email-masked]>

-- 
John W. Linville		Someday the world will need a hero, and you
[email-masked]			might be all we have.  Be ready.