right; we shouldn't need to update rval from the rate table again, as it's already been looked up earlier.

ok, from my reading, they SHOULD be the same thing for CCK and OFDM. This particular bit of code is pretty terrible though - it doesn't support HT or VHT in this path; I have no idea how static rate HT would work for management frames here (ie, the path that isn't going via firmware rate control), and VHT just isn't hooked up.

I tested my setup, and it worked for me (it did what it was supposed to do).

1. I could document the testing if you define a version for me. Let me know if you are OK with the changes (different, more complex ideas were floating around, and I reverted to the most conservative extension), and I'll pick it up.

hm, whats left before we can land this?

more fixes - i needed to change the sta add and sta rm to
also use the DQA queues or else I get firmware assertions!

oops, this actually DOES now work on the 7260/9260.
I had a whole bunch of little stupid things to fix here.

In D60080#1379601, @vexeduxr wrote:

For some more info, the laptop is a dell latitude e7440.

hw rev 0x140, fw ver 17.352738.0

I'm just noticing that it also fails to load the firmware:

iwm7260fw: could not load firmware image, error 8

So maybe my problem is just outdated firmware?

For some more info, the laptop is a dell latitude e7440.

hw rev 0x140, fw ver 17.352738.0

a reported problem on irc:

Yeah, we're working out a work flow for it. chances are when I land them I'll squash them myself and deal with the reviews fallout (manually linking commit hashes and closing things.)

IMO it's not worth including this as a separate commit. D59757 and D59758 can be combined and just note the NOP 1.176 in the commit message.