Re: git: 75556c7e999e - main - net80211: fix arguments to IEEE80211_NOTE in ieee80211_fix_rate

From: Ruslan Makhmatkhanov <rm_at_FreeBSD.org>
Date: Wed, 14 Jan 2026 20:44:05 UTC
Bjoern A. Zeeb wrote on 1/14/26 9:51 PM:
> The branch main has been updated by bz:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=75556c7e999e9095ce71558ae61f49c1ba61a192
> 
> commit 75556c7e999e9095ce71558ae61f49c1ba61a192
> Author:     Yichen Chai <yichen.chai@gmail.com>
> AuthorDate: 2026-01-14 18:44:08 +0000
> Commit:     Bjoern A. Zeeb <bz@FreeBSD.org>
> CommitDate: 2026-01-14 18:51:11 +0000
> 
>      net80211: fix arguments to IEEE80211_NOTE in ieee80211_fix_rate
>      
>      Fix the arguments to the debug statement.
>      
>      (slightly adjusted from the original submission by bz)
>      
>      PR:             286448
>      MFC after:      3 days
> ---
>   sys/net80211/ieee80211_proto.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/sys/net80211/ieee80211_proto.c b/sys/net80211/ieee80211_proto.c
> index 4918bf7d025f..ba09cd015a57 100644
> --- a/sys/net80211/ieee80211_proto.c
> +++ b/sys/net80211/ieee80211_proto.c
> @@ -747,8 +747,8 @@ ieee80211_fix_rate(struct ieee80211_node *ni,
>   	    ((flags & (IEEE80211_F_DOFRATE|IEEE80211_F_DOFMCS)) &&
>   	     fixedrate != ucastrate)) {
>   		IEEE80211_NOTE(vap, IEEE80211_MSG_XRATE | IEEE80211_MSG_11N, ni,
> -		    "%s: flags 0x%x okrate %d error %d fixedrate 0x%x "
> -		    "ucastrate %x\n", __func__, fixedrate, ucastrate, flags);
> +		    "%s: flags 0x%x okrate %d error %d fixedrate 0x%x ucastrate 0x%x\n",
> +		    __func__, flags, okrate, error, fixedrate, ucastrate);
                                                                ^^^^^^^^^
           Not sure if it worth nothing, but this looks like an insult :)

>   		return badrate | IEEE80211_RATE_BASIC;
>   	} else
>   		return IEEE80211_RV(okrate);
> 

-- 
Regards,
Ruslan

T.O.S. Of Reality