From patchwork Thu Jan 18 13:14:11 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 10173191 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 8C325603B5 for ; Thu, 18 Jan 2018 13:15:53 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 79C421FF3E for ; Thu, 18 Jan 2018 13:15:53 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 6A28920134; Thu, 18 Jan 2018 13:15:53 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id E3A781FF3E for ; Thu, 18 Jan 2018 13:15:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date: Subject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To: References:List-Owner; bh=WgzYsXcVG6JOproLkHL73ZaC29ngoAZLiovVTTDdoZQ=; b=OMh MqkEJwIsfwlLO+E8C93finSX3N2VeV3Pb66+L1rjSKrA7vUatzkAhp6eqcJc5aRL4zWDIO9vu/3Ly TCxmjewTtZquaU4fIBe01azbD2CeFsYsQxEafWjuR9fVJSRKSuhXzi/FMI4nsKTgaCilCMsz3H2fH tWkj4Wmx5s4BUCh2WYJEg+p/f13hiDXHfaC0OO3Y7jkn8Jlqpmo8dD091uM3CrYfu2637km1sFxWC z14a+sBgtYp0d/E4jmRfK+1pdmZzl7j5LXwCkaVr42OEn7apkyj/S3ZT064l1AFlZc4nb/C2quEYT UX+vyWrd1VKeURd+So46yQJ2jyWJGYA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.89 #1 (Red Hat Linux)) id 1ecA2p-00024R-RK; Thu, 18 Jan 2018 13:15:51 +0000 Received: from mout.kundenserver.de ([212.227.17.13]) by bombadil.infradead.org with esmtps (Exim 4.89 #1 (Red Hat Linux)) id 1ecA2l-00023r-LB; Thu, 18 Jan 2018 13:15:50 +0000 Received: from wuerfel.lan ([95.208.111.237]) by mrelayeu.kundenserver.de (mreue104 [212.227.15.145]) with ESMTPA (Nemesis) id 0Lkxnh-1fCaea1uJJ-00aniz; Thu, 18 Jan 2018 14:15:29 +0100 From: Arnd Bergmann To: Kalle Valo Subject: [PATCH] [wireless-next] mt76: fix building without CONFIG_LEDS_CLASS Date: Thu, 18 Jan 2018 14:14:11 +0100 Message-Id: <20180118131524.817297-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:vdvdCeukQTmG1/7hjQjCAOspxxxE7XN8bcQISMhYdrrSsHF8p77 fbOhwiU/u2StBKAEhh889B5LYTacQ/EgLCy1gUagn9zewUmUSagL7ulKuM2eVCYMdJyUyDa 1kIYSUST+kd9cd2Pn/LRDriYUDhpTW6gceMXEkDx71Lx+uzZCpIeY9dIYkeIq1XnlEm4jHw HvLjrorYQGXjdL909VwGQ== X-UI-Out-Filterresults: notjunk:1; V01:K0:Djlq+l4b3RQ=:R+pggZ/U4lNLFzW/rbzJN5 QLnpohBafLLyx/cZ3LfQXAf6MleDn2RH/4dKSCyYEUuzg8wiXWsAHbUrEnOotfLMnesmBt+eZ sSPvexu+u9WULfkeNvX2zSF8p7yPfsDGho0WCXsW+Gx4WZ8uDc2QpkFEJ58dJqlA9BAiZAAvd sad2oKsJ12m2Mf58rgzCIbScpN1gdjk0kG6pQDhaVee0qa2TDNV9eDGNGeRTGSuWSf+ulYZaa JKF1jtR3B0U0Hutm4PhtuFGDqTY/BOcqEC4IjFPO+FuyR49/mZEas1zwyiMmZo5K6Fq2NOz9I PWQjxGDMbGpVGr5ODXHwnwuKvgjw51wGH2vbehWw4bTJIIbgd0hntwIKdFLD/g9nSvqYLfUxc C+9/xA/9azwShmtR56ZP9INVgwWshKHVe38dtETaf+Ez6IBnJC+jiOAfULN7qVdP0FLHObSM/ 5YXNL09W0zFUsF4FpOSCODTmUFXqNiFV7lNVxKW65WPtrMX2Eb2K7in2kYZrEL3hasRM8uxXy XWivrXmCQQkpgalAL6FbZ8DDX3imkQBK+Pz8LSxkjXgjqvHAOunNgmAS4MDAwlhGnpZMkgbZz sg9XrVy7HETZu8s0lKHp6cLQ6fownbNCZJ/kbaYVhYnDMCflGDKhgGl/Aa8Bh5Jr4DQPL55D7 jqevBXzrRdFbI0XxR8kYhnsI1o5Upm6fkkCPPFvmsqmtJS06JRyqXDWVudOGKb0PpGT8Tu2tS ih+krmTASyQZxw5wU+0cS3+Ej64Ln4ttVFVP3w== X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Arnd Bergmann , netdev@vger.kernel.org, linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, Lorenzo Bianconi , linux-arm-kernel@lists.infradead.org, Felix Fietkau MIME-Version: 1.0 Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+patchwork-linux-mediatek=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP 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 --- 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 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));