diff mbox series

brcm80211: BRCM_TRACING should depend on TRACING

Message ID 81a29b15eaacc1ac1fb421bdace9ac0c3385f40f.1727179742.git.geert@linux-m68k.org (mailing list archive)
State Accepted
Commit b73b2069528f90ec49d5fa1010a759baa2c2be05
Delegated to: Kalle Valo
Headers show
Series brcm80211: BRCM_TRACING should depend on TRACING | expand

Commit Message

Geert Uytterhoeven Sept. 24, 2024, 12:09 p.m. UTC
When tracing is disabled, there is no point in asking the user about
enabling Broadcom wireless device tracing.

Fixes: f5c4f10852d42012 ("brcm80211: Allow trace support to be enabled separately from debug")
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
 drivers/net/wireless/broadcom/brcm80211/Kconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Arend van Spriel Sept. 24, 2024, 12:51 p.m. UTC | #1
On September 24, 2024 2:09:35 PM Geert Uytterhoeven <geert@linux-m68k.org> 
wrote:

> When tracing is disabled, there is no point in asking the user about
> enabling Broadcom wireless device tracing.
>
> Fixes: f5c4f10852d42012 ("brcm80211: Allow trace support to be enabled 
> separately from debug")
Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> ---
> drivers/net/wireless/broadcom/brcm80211/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
Kalle Valo Sept. 24, 2024, 1:35 p.m. UTC | #2
Arend Van Spriel <arend.vanspriel@broadcom.com> writes:

> On September 24, 2024 2:09:35 PM Geert Uytterhoeven
> <geert@linux-m68k.org> wrote:
>
>> When tracing is disabled, there is no point in asking the user about
>> enabling Broadcom wireless device tracing.
>>
>> Fixes: f5c4f10852d42012 ("brcm80211: Allow trace support to be
>> enabled separately from debug")
>
> Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>

I'm planning to take this to wireless and I'll add 'wifi:' to the title.
Kalle Valo Oct. 17, 2024, 2:23 p.m. UTC | #3
Geert Uytterhoeven <geert@linux-m68k.org> wrote:

> When tracing is disabled, there is no point in asking the user about
> enabling Broadcom wireless device tracing.
> 
> Fixes: f5c4f10852d42012 ("brcm80211: Allow trace support to be enabled separately from debug")
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>

Patch applied to wireless.git, thanks.

b73b2069528f wifi: brcm80211: BRCM_TRACING should depend on TRACING
diff mbox series

Patch

diff --git a/drivers/net/wireless/broadcom/brcm80211/Kconfig b/drivers/net/wireless/broadcom/brcm80211/Kconfig
index 3a1a35b5672f1a27..19d0c003f6262675 100644
--- a/drivers/net/wireless/broadcom/brcm80211/Kconfig
+++ b/drivers/net/wireless/broadcom/brcm80211/Kconfig
@@ -27,6 +27,7 @@  source "drivers/net/wireless/broadcom/brcm80211/brcmfmac/Kconfig"
 config BRCM_TRACING
 	bool "Broadcom device tracing"
 	depends on BRCMSMAC || BRCMFMAC
+	depends on TRACING
 	help
 	  If you say Y here, the Broadcom wireless drivers will register
 	  with ftrace to dump event information into the trace ringbuffer.