netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ath9k_htc: Do not select MAC80211_LEDS by default
@ 2020-08-20 12:04 Krzysztof Kozlowski
  2020-08-27 10:17 ` Kalle Valo
  0 siblings, 1 reply; 2+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-20 12:04 UTC (permalink / raw)
  To: QCA ath9k Development, Kalle Valo, David S. Miller,
	Jakub Kicinski, linux-wireless, netdev, linux-kernel
  Cc: Krzysztof Kozlowski

The ath9k_htc driver hides all LEDs related code behind
CONFIG_MAC80211_LEDS ifdefs so it does not really require the
MAC80211_LEDS.  The code builds and works just fine.  Convert the
"select" into "imply" to allow disabling LED trigger when not needed.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/net/wireless/ath/ath9k/Kconfig | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/ath/ath9k/Kconfig b/drivers/net/wireless/ath/ath9k/Kconfig
index d5e9af2dddd8..5effb3886631 100644
--- a/drivers/net/wireless/ath/ath9k/Kconfig
+++ b/drivers/net/wireless/ath/ath9k/Kconfig
@@ -177,10 +177,10 @@ config ATH9K_HTC
 	tristate "Atheros HTC based wireless cards support"
 	depends on USB && MAC80211
 	select ATH9K_HW
-	select MAC80211_LEDS
-	select LEDS_CLASS
-	select NEW_LEDS
 	select ATH9K_COMMON
+	imply NEW_LEDS
+	imply LEDS_CLASS
+	imply MAC80211_LEDS
 	help
 	  Support for Atheros HTC based cards.
 	  Chipsets supported: AR9271
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] ath9k_htc: Do not select MAC80211_LEDS by default
  2020-08-20 12:04 [PATCH] ath9k_htc: Do not select MAC80211_LEDS by default Krzysztof Kozlowski
@ 2020-08-27 10:17 ` Kalle Valo
  0 siblings, 0 replies; 2+ messages in thread
From: Kalle Valo @ 2020-08-27 10:17 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: QCA ath9k Development, David S. Miller, Jakub Kicinski,
	linux-wireless, netdev, linux-kernel, Krzysztof Kozlowski

Krzysztof Kozlowski <krzk@kernel.org> wrote:

> The ath9k_htc driver hides all LEDs related code behind
> CONFIG_MAC80211_LEDS ifdefs so it does not really require the
> MAC80211_LEDS.  The code builds and works just fine.  Convert the
> "select" into "imply" to allow disabling LED trigger when not needed.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

Patch applied to ath-next branch of ath.git, thanks.

197f466e93f5 ath9k_htc: Do not select MAC80211_LEDS by default

-- 
https://patchwork.kernel.org/patch/11726211/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-08-27 10:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-08-20 12:04 [PATCH] ath9k_htc: Do not select MAC80211_LEDS by default Krzysztof Kozlowski
2020-08-27 10:17 ` Kalle Valo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).