I am happy with all three revisions and echo the comments from rgrimes and jtl on a single review and the impact being documented.

This seems ok to me, if it doesn't smoke and you can still do vht then its a good step forward

right after the start of stab week seems like a good time to see if iwx explodes with these

From a glance I am pretty sure this is all incorporated in the udbc patchset.

Unless someone shouts at me I will land this on Monday so it can make ALPHA3 on Thursday

• fix a typo in man page
• provide full year in copyright
• move devmethod_end onto its own line
• use interface define rather than device define

I am pretty sure I've addressed all outstanding comments, but I could easily have missed some. Could interested people check again?

• move tr_setup label to start of line
• remove body of ioctl function

• take hrs's text for the man page

• remove null check for sc
• Reference latest standard
• Remove XXX comment, this isn't that unclear
• Don't hide reserved and vendor print behind print verbose
• Add extended MSI and usb tunnel capabilities
• don't check first in loop
• spell tun as usb3_tun (and correctly)

• don't check first in loop

• Reference latest standard
• Remove XXX comment, this isn't that unclear
• Don't hide reserved and vendor print behind print verbose
• Add extended MSI and usb tunnel capabilities

• Don't split subdir += lines
• rearrange headers to build (I am sure this built in this tree before)
• Move DRIVER_MODULE to the bottom of the file
• Address review comments on makefile

I think starting wpa_supplicant will result in the interface being brought up, but I still think this is a good action to do explicitly.

• remove null check for sc

As far as I can tell this is nop, but good ground work for future stuff.

This looks to do what is advertised in some simple tests. ddb is pretty fragile for me to use on the best of days so I'm happy enough

I've done a little testing and this does what I would expect on my test laptop.

Setting the regdomain on iwlwifi cleared this out - sorry for the testing delay.

I've sent test output via email. With 160MHz configured on my ap and it beaconing iwlwifi doesn't pick a vht rate for the band, just 80211a

I'm not able to convince an ap to actually do 160MHz channels - despite support in the radios.

Please commit this with "Tested by: thj".

Please add

Tested by: thj

• iwx: unlock on error path
• create a diff that can be applied on main (sorry)

No regressions I can see, iwlwifi has vht160 enabled without any configuration.

@adrian said in irc that this should have been caught with other changes. Could you test again on main?

In D49773#1153069, @bz wrote:

I've had a few people try this at least with no ill effects (though I am not sure they could test VHT160). @thj can you test the net80211 part?
I don't know otherwise what to do with it as it's been sitting here for a month now.

I unintentionally created another review with my in progress change please see D50462

• Use iwx_init on resume

Meant to add: