Re: [PATCH wireless-next] wifi: brcmsmac: use FAM for debug code
From: Johannes Berg
Date: Mon Mar 09 2026 - 06:41:18 EST
On Sun, 2026-02-22 at 19:05 -0800, Rosen Penev wrote:
>
> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/mac80211_if.h
> @@ -43,7 +43,7 @@ struct brcms_timer {
> bool set; /* indicates if timer is active */
> struct brcms_timer *next; /* for freeing on unload */
> #ifdef DEBUG
> - char *name; /* Description of the timer */
> + char name[]; /* Description of the timer */
> #endif
>
Might be better to remove that ifdef too, if only to have cleaner code
and get errors when someone tries to add something after it in non-debug
builds?
johannes