From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mms3.broadcom.com ([216.31.210.19]:3016 "EHLO mms3.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751341Ab3KRWf2 convert rfc822-to-8bit (ORCPT ); Mon, 18 Nov 2013 17:35:28 -0500 Message-ID: <528A925C.20602@broadcom.com> (sfid-20131118_233532_761327_D448C2D0) Date: Mon, 18 Nov 2013 23:19:08 +0100 From: "Arend van Spriel" MIME-Version: 1.0 To: "Borislav Petkov" , =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= cc: lkml , "linux-wireless@vger.kernel.org" Subject: Re: [PATCH] brcmsmac: Fix build dep on LEDS_CLASS References: <20131117133702.GA30840@pd.tnic> <5289359B.8090506@broadcom.com> <20131118133250.GH24851@pd.tnic> <20131118204003.GK24851@pd.tnic> In-Reply-To: <20131118204003.GK24851@pd.tnic> Content-Type: text/plain; charset=utf-8; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 11/18/2013 09:40 PM, Borislav Petkov wrote: > On Mon, Nov 18, 2013 at 02:32:50PM +0100, Borislav Petkov wrote: >> On Mon, Nov 18, 2013 at 12:58:58PM +0100, Rafał Miłecki wrote: >>> Borislav: I think I didn't get your original patch. >> >> Forget it, I'll send a better one later. > > Ok, let's try another one. I've done it with depends but we > probably could select LEDS_CLASS in BRCMSMAC_LED as it is required > functionality... what about: config BRCMSMAC tristate "Broadcom IEEE802.11n PCIe SoftMAC WLAN driver" depends on MAC80211 depends on BCMA_POSSIBLE + select LEDS_CLASS if BCMA_DRIVER_GPIO select BCMA Regards, Arend > --- > From: Borislav Petkov > Subject: [PATCH -v2] brcmsmac: Fix build dep on LEDS_CLASS > > When building randconfigs with CONFIG_BCMA_DRIVER_GPIO=y, I get > > drivers/built-in.o: In function `brcms_led_unregister': > (.text+0x351aca): undefined reference to `led_classdev_unregister' > drivers/built-in.o: In function `brcms_led_register': > (.text+0x351c65): undefined reference to `led_classdev_register' > > during final linking stage because brcmsmac/led.c needs LEDS_CLASS for > registering/deregistering the led device. Add a promptless Kconfig > symbol which takes care of that dependency. > > Cc: "Rafał Miłecki" > Cc: > Signed-off-by: Borislav Petkov > --- > drivers/net/wireless/brcm80211/Kconfig | 4 ++++ > drivers/net/wireless/brcm80211/brcmsmac/Makefile | 2 +- > drivers/net/wireless/brcm80211/brcmsmac/led.h | 2 +- > 3 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/brcm80211/Kconfig b/drivers/net/wireless/brcm80211/Kconfig > index b00a7e92225f..c9434b7c7b9b 100644 > --- a/drivers/net/wireless/brcm80211/Kconfig > +++ b/drivers/net/wireless/brcm80211/Kconfig > @@ -16,6 +16,10 @@ config BRCMSMAC > be available if you select BCMA_DRIVER_GPIO. If you choose to build a > module, the driver will be called brcmsmac.ko. > > +config BRCMSMAC_LED > + def_bool y > + depends on BRCMSMAC && BCMA_DRIVER_GPIO && LEDS_CLASS > + > config BRCMFMAC > tristate "Broadcom IEEE802.11n embedded FullMAC WLAN driver" > depends on CFG80211 > diff --git a/drivers/net/wireless/brcm80211/brcmsmac/Makefile b/drivers/net/wireless/brcm80211/brcmsmac/Makefile > index 32464acccd90..9ec2c8b2f26c 100644 > --- a/drivers/net/wireless/brcm80211/brcmsmac/Makefile > +++ b/drivers/net/wireless/brcm80211/brcmsmac/Makefile > @@ -43,6 +43,6 @@ brcmsmac-y := \ > brcms_trace_events.o \ > debug.o > > -brcmsmac-$(CONFIG_BCMA_DRIVER_GPIO) += led.o > +brcmsmac-$(CONFIG_BRCMSMAC_LED) += led.o > > obj-$(CONFIG_BRCMSMAC) += brcmsmac.o > diff --git a/drivers/net/wireless/brcm80211/brcmsmac/led.h b/drivers/net/wireless/brcm80211/brcmsmac/led.h > index 17a0b1f5dbcf..b914466e687b 100644 > --- a/drivers/net/wireless/brcm80211/brcmsmac/led.h > +++ b/drivers/net/wireless/brcm80211/brcmsmac/led.h > @@ -22,7 +22,7 @@ struct brcms_led { > bool active_low; > }; > > -#ifdef CONFIG_BCMA_DRIVER_GPIO > +#ifdef CONFIG_BRCMSMAC_LED > void brcms_led_unregister(struct brcms_info *wl); > int brcms_led_register(struct brcms_info *wl); > #else >