linux-kernel - Re: [PATCH] brcmsmac: Fix build dep on LEDS

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <528B2B2A.2030504@broadcom.com>
Date:	Tue, 19 Nov 2013 10:11:06 +0100
From:	"Arend van Spriel" <arend@...adcom.com>
To:	"Borislav Petkov" <bp@...en8.de>
cc:	Rafał Miłecki <zajec5@...il.com>,
	lkml <linux-kernel@...r.kernel.org>,
	"linux-wireless@...r.kernel.org" <linux-wireless@...r.kernel.org>
Subject: Re: [PATCH] brcmsmac: Fix build dep on LEDS_CLASS

On 11/19/2013 12:32 AM, Borislav Petkov wrote:
> On Mon, Nov 18, 2013 at 11:19:08PM +0100, Arend van Spriel wrote:
>> 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
>
> I know, nice and short but too good to be true. LEDS_CLASS needs to pull
> in NEW_LEDS unfortunately:
>
> drivers/built-in.o: In function `set_brightness_delayed':
> led-class.c:(.text+0x4596e5): undefined reference to `led_stop_software_blink'
> drivers/built-in.o: In function `led_classdev_register':
> (.text+0x4598db): undefined reference to `leds_list_lock'
> drivers/built-in.o: In function `led_classdev_register':
> (.text+0x4598e7): undefined reference to `leds_list'
> drivers/built-in.o: In function `led_classdev_register':
> (.text+0x4598f3): undefined reference to `leds_list'
> drivers/built-in.o: In function `led_classdev_register':
> (.text+0x4598fa): undefined reference to `leds_list_lock'
> drivers/built-in.o: In function `led_classdev_register':
> (.text+0x459901): undefined reference to `leds_list'
> drivers/built-in.o: In function `led_classdev_unregister':
> (.text+0x4599a8): undefined reference to `led_stop_software_blink'
> drivers/built-in.o: In function `led_classdev_unregister':
> (.text+0x4599b2): undefined reference to `led_set_brightness'
> drivers/built-in.o: In function `led_classdev_unregister':
> (.text+0x4599c2): undefined reference to `leds_list_lock'
> drivers/built-in.o: In function `led_classdev_unregister':
> (.text+0x4599d6): undefined reference to `leds_list_lock'
> make: *** [vmlinux] Error 1
>
> What works, instead, is below. I think my "depends" solution from
> earlier takes care of this without you having to select other stuff
> explicitly but in the end of the day, I don't really care all that much
> which one you guys take.

The problem with 'select' is indeed that you need to cover all 
dependencies. It looks fine to me so why don't you just submit this to 
John Linville (and Cc: linux-wireless@...r.kernel.org).

Regards,
Arend

> --
> diff --git a/drivers/net/wireless/brcm80211/Kconfig b/drivers/net/wireless/brcm80211/Kconfig
> index b00a7e92225f..54e36fcb3954 100644
> --- a/drivers/net/wireless/brcm80211/Kconfig
> +++ b/drivers/net/wireless/brcm80211/Kconfig
> @@ -5,6 +5,8 @@ config BRCMSMAC
>          tristate "Broadcom IEEE802.11n PCIe SoftMAC WLAN driver"
>          depends on MAC80211
>          depends on BCMA
> +       select NEW_LEDS if BCMA_DRIVER_GPIO
> +       select LEDS_CLASS if BCMA_DRIVER_GPIO
>          select BRCMUTIL
>          select FW_LOADER
>          select CRC_CCITT
>


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/