Page MenuHomeFreeBSD
Differential D5369

net80211: hide subtype mask & shift in function call
ClosedPublic
Actions

Authored by avos on Feb 20 2016, 8:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 21, 1:12 AM
Unknown Object (File)
Fri, Nov 21, 1:10 AM
Unknown Object (File)
Fri, Nov 21, 1:09 AM
Unknown Object (File)
Fri, Nov 21, 1:09 AM
Unknown Object (File)
Fri, Nov 21, 1:07 AM
Unknown Object (File)
Tue, Nov 18, 9:02 PM
Unknown Object (File)
Sat, Nov 8, 11:46 PM
Unknown Object (File)
Sat, Nov 8, 6:20 AM
View All Files
Subscribers
imp

Details

Reviewers
adrian
Commits
rS298376: net80211: hide subtype mask & shift in function call.
Summary

Hide subtype mask/shift (which is used for index calculation in ieee80211_mgt_subtype_name[] array) in function call.

Test Plan

Tested with RTL8188CUS, STA mode.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

avos updated this revision to Diff 13554.Feb 20 2016, 8:51 PM
avos retitled this revision from to net80211: hide subtype shift in function call.
avos updated this object.
avos edited the test plan for this revision. (Show Details)
avos added a reviewer: adrian.
avos set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added a subscriber: imp. · View Herald TranscriptFeb 20 2016, 8:51 PM
adrian accepted this revision.Feb 24 2016, 6:24 AM
adrian edited edge metadata.

let's make the two subtype functions also do the mask, and reduce code duplication there. Then yeah, commit away!

sys/net80211/ieee80211_input.c
931 ↗(On Diff #13554)

may as well have the function also do the mask, right?

This revision is now accepted and ready to land.Feb 24 2016, 6:24 AM
avos updated this revision to Diff 13695.Feb 24 2016, 10:02 PM
avos retitled this revision from net80211: hide subtype shift in function call to net80211: hide subtype mask & shift in function call.
avos updated this object.
avos edited edge metadata.

Mask subtype before shifting.

This revision now requires review to proceed.Feb 24 2016, 10:02 PM
avos marked an inline comment as done.Feb 24 2016, 10:02 PM
adrian accepted this revision.Feb 24 2016, 10:26 PM
adrian edited edge metadata.

yay!

This revision is now accepted and ready to land.Feb 24 2016, 10:26 PM
Closed by commit rS298376: net80211: hide subtype mask & shift in function call. (authored by avos). · Explain WhyApr 20 2016, 9:16 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 15406

View Options

head/sys/net80211/ieee80211_adhoc.c

Loading...
View Options

head/sys/net80211/ieee80211_hostap.c

Loading...
View Options

head/sys/net80211/ieee80211_input.h

Loading...
View Options

head/sys/net80211/ieee80211_input.c

Loading...
View Options

head/sys/net80211/ieee80211_mesh.c

Loading...
View Options

head/sys/net80211/ieee80211_output.c

Loading...
View Options

head/sys/net80211/ieee80211_proto.h

Loading...
View Options

head/sys/net80211/ieee80211_proto.c

Loading...
View Options

head/sys/net80211/ieee80211_scan.c

Loading...
View Options

head/sys/net80211/ieee80211_sta.c

Loading...
View Options

head/sys/net80211/ieee80211_tdma.c

Loading...
View Options

head/sys/net80211/ieee80211_wds.c

Loading...