Re: [PATCH wireless-next] wifi: brcmsmac: use FAM for debug code
Johannes Berg <[email protected]> Mon, 09 Mar 2026 11:40:12 +0100
| Newsgroups | dev.linux.lists.brcm80211,org.kernel.vger.linux-kernel,org.kernel.vger.linux-wireless |
|---|---|
| Message-ID | <[email protected]> |
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