Page MenuHomeFreeBSD

Provide MS() and SM() macros for 80211.
ClosedPublic

Authored by bz on Sep 23 2020, 10:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 23 2024, 9:46 PM
Unknown Object (File)
Feb 23 2024, 9:45 PM
Unknown Object (File)
Feb 23 2024, 12:13 PM
Unknown Object (File)
Feb 21 2024, 3:33 PM
Unknown Object (File)
Feb 19 2024, 5:28 PM
Unknown Object (File)
Dec 20 2023, 3:59 PM
Unknown Object (File)
Dec 20 2023, 8:38 AM
Unknown Object (File)
Dec 3 2023, 1:16 PM

Details

Summary

We have (two versions) of MS() and SM() macros which we use throughout
the wireless code. Change all but three places (ath_hal, rtwn, and rsu)
to the newly provided _IEEE80211_MASKSHIFT() and _IEEE80211_SHIFTMASK()
macros. Also change one internal case using both _S and _M instead of
just _S away from _M (leave others, such as rsu).

This has been done semi-mechanically. No functional changes intended.
TODO: improve line-wrapping after this change before commit, if the
new macro names are generally acceptable.

Requested by:	gnn (D26091)
Sponsored by:   Rubicon Communications, LLC (d/b/a "Netgate")

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 33784
Build 31001: arc lint + arc unit

Event Timeline

bz requested review of this revision.Sep 23 2020, 10:02 PM
bz set the repository for this revision to rS FreeBSD src repository - subversion.

I am mostly interested if the names are okay. I did not want to use _IEEE80211_MS() as that might be confused with some time operation MS_TO_xxx. If the names are okay, I'll wrap the lines and upload a new version for actual review.

This revision is now accepted and ready to land.Sep 24 2020, 3:01 AM

Arg. Sorry, used the wrong alias and pushed the change into head instead of here. Please do a post-commit review in head then.

This landed by accident in head as https://svnweb.freebsd.org/changeset/base/366112 (not sure why the review wasn't automatically closed either).