diff mbox series

[wireless-next] mt76: fix building without CONFIG_LEDS_CLASS

Message ID 20180118131524.817297-1-arnd@arndb.de
State New
Headers show
Series [wireless-next] mt76: fix building without CONFIG_LEDS_CLASS | expand

Commit Message

Arnd Bergmann Jan. 18, 2018, 1:14 p.m. UTC
When CONFIG_LEDS_CLASS is disabled, or it is a loadable module while
mt76 is built-in, we run into a link error:

drivers/net/wireless/mediatek/mt76/mac80211.o: In function `mt76_register_device':
mac80211.c:(.text+0xb78): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `devm_of_led_classdev_register'

We don't really need a hard dependency here as the driver can presumably
work just fine without LEDs, so this follows the iwlwifi example and
adds a separate Kconfig option for the LED support, this will be available
whenever it will link, and otherwise the respective code gets left out from
the driver object.

Fixes: 17f1de56df05 ("mt76: add common code shared between multiple chipsets")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>

---
 drivers/net/wireless/mediatek/mt76/Kconfig       | 5 +++++
 drivers/net/wireless/mediatek/mt76/mac80211.c    | 8 +++++---
 drivers/net/wireless/mediatek/mt76/mt76x2_init.c | 6 ++++--
 3 files changed, 14 insertions(+), 5 deletions(-)

-- 
2.9.0

Comments

Kalle Valo Jan. 18, 2018, 1:23 p.m. UTC | #1
Arnd Bergmann <arnd@arndb.de> writes:

> When CONFIG_LEDS_CLASS is disabled, or it is a loadable module while

> mt76 is built-in, we run into a link error:

>

> drivers/net/wireless/mediatek/mt76/mac80211.o: In function `mt76_register_device':

> mac80211.c:(.text+0xb78): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `devm_of_led_classdev_register'

>

> We don't really need a hard dependency here as the driver can presumably

> work just fine without LEDs, so this follows the iwlwifi example and

> adds a separate Kconfig option for the LED support, this will be available

> whenever it will link, and otherwise the respective code gets left out from

> the driver object.

>

> Fixes: 17f1de56df05 ("mt76: add common code shared between multiple chipsets")

> Signed-off-by: Arnd Bergmann <arnd@arndb.de>


I'm planning to queue this for 4.16.

-- 
Kalle Valo
Felix Fietkau Jan. 18, 2018, 5:31 p.m. UTC | #2
On 2018-01-18 14:14, Arnd Bergmann wrote:
> When CONFIG_LEDS_CLASS is disabled, or it is a loadable module while

> mt76 is built-in, we run into a link error:

> 

> drivers/net/wireless/mediatek/mt76/mac80211.o: In function `mt76_register_device':

> mac80211.c:(.text+0xb78): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `devm_of_led_classdev_register'

> 

> We don't really need a hard dependency here as the driver can presumably

> work just fine without LEDs, so this follows the iwlwifi example and

> adds a separate Kconfig option for the LED support, this will be available

> whenever it will link, and otherwise the respective code gets left out from

> the driver object.

> 

> Fixes: 17f1de56df05 ("mt76: add common code shared between multiple chipsets")

> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

> ---

>  drivers/net/wireless/mediatek/mt76/Kconfig       | 5 +++++

>  drivers/net/wireless/mediatek/mt76/mac80211.c    | 8 +++++---

>  drivers/net/wireless/mediatek/mt76/mt76x2_init.c | 6 ++++--

>  3 files changed, 14 insertions(+), 5 deletions(-)

> 

> diff --git a/drivers/net/wireless/mediatek/mt76/Kconfig b/drivers/net/wireless/mediatek/mt76/Kconfig

> index fc05d79c80d0..9d12c1f5284e 100644

> --- a/drivers/net/wireless/mediatek/mt76/Kconfig

> +++ b/drivers/net/wireless/mediatek/mt76/Kconfig

> @@ -8,3 +8,8 @@ config MT76x2E

>  	depends on PCI

>  	---help---

>  	  This adds support for MT7612/MT7602/MT7662-based wireless PCIe devices.

> +

> +config MT76_LEDS

> +	bool "LED support for MT76"

> +	depends on MT76_CORE

> +	depends on LEDS_CLASS=y || MT76_CORE=LEDS_CLASS

I think this should have a 'default y'

- Felix
diff mbox series

Patch

diff --git a/drivers/net/wireless/mediatek/mt76/Kconfig b/drivers/net/wireless/mediatek/mt76/Kconfig
index fc05d79c80d0..9d12c1f5284e 100644
--- a/drivers/net/wireless/mediatek/mt76/Kconfig
+++ b/drivers/net/wireless/mediatek/mt76/Kconfig
@@ -8,3 +8,8 @@  config MT76x2E
 	depends on PCI
 	---help---
 	  This adds support for MT7612/MT7602/MT7662-based wireless PCIe devices.
+
+config MT76_LEDS
+	bool "LED support for MT76"
+	depends on MT76_CORE
+	depends on LEDS_CLASS=y || MT76_CORE=LEDS_CLASS
diff --git a/drivers/net/wireless/mediatek/mt76/mac80211.c b/drivers/net/wireless/mediatek/mt76/mac80211.c
index 3acf0e175d71..144e312a9407 100644
--- a/drivers/net/wireless/mediatek/mt76/mac80211.c
+++ b/drivers/net/wireless/mediatek/mt76/mac80211.c
@@ -295,9 +295,11 @@  int mt76_register_device(struct mt76_dev *dev, bool vht,
 	mt76_check_sband(dev, NL80211_BAND_2GHZ);
 	mt76_check_sband(dev, NL80211_BAND_5GHZ);
 
-	ret = mt76_led_init(dev);
-	if (ret)
-		return ret;
+	if (IS_ENABLED(CONFIG_MT76_LEDS)) {
+		ret = mt76_led_init(dev);
+		if (ret)
+			return ret;
+	}
 
 	return ieee80211_register_hw(hw);
 }
diff --git a/drivers/net/wireless/mediatek/mt76/mt76x2_init.c b/drivers/net/wireless/mediatek/mt76/mt76x2_init.c
index 4373a2ba5143..0c9d02af205d 100644
--- a/drivers/net/wireless/mediatek/mt76/mt76x2_init.c
+++ b/drivers/net/wireless/mediatek/mt76/mt76x2_init.c
@@ -849,8 +849,10 @@  int mt76x2_register_device(struct mt76x2_dev *dev)
 	mt76x2_dfs_init_detector(dev);
 
 	/* init led callbacks */
-	dev->mt76.led_cdev.brightness_set = mt76x2_led_set_brightness;
-	dev->mt76.led_cdev.blink_set = mt76x2_led_set_blink;
+	if (IS_ENABLED(CONFIG_MT76_LEDS)) {
+		dev->mt76.led_cdev.brightness_set = mt76x2_led_set_brightness;
+		dev->mt76.led_cdev.blink_set = mt76x2_led_set_blink;
+	}
 
 	ret = mt76_register_device(&dev->mt76, true, mt76x2_rates,
 				   ARRAY_SIZE(mt76x2_rates));