(not near my nbg6716 right now but)
without the pre-cal files the ath10k modules refuse to load.
without the 11-ath10k-caldata no (pre-)cal file
ergo no 5Ghz?
or put differently how to check mac etc, without the ath10k modules loaded?
It doesn't load because the (nonexistent) calibration data from flash is bogus and crashes the firmware. If you remove the files I listed, the on-board calibration data is tried next, and it just might work after reboot.
Well... with the bogus calibration data in the pre-cal file the 5Ghz works just fine.
without that file present the firmware crashes. cfr. the logs i posted earlier. e.g.
you are probably correct.
so preferably we figure out where/how to get proper calibration data.
or figure out how to get the hotplug script to create the pre-cal file instead (but let it run uncalibrated)
or maybe use this option with ath10k: skip_otp=y (haven't tried it yet, but i'm assuming it'll run uncalibrated as well)
There are basically four ways to store calibration data:
On the PCI-E card itself
On the PCI-E card itself, but with the MAC address coming from elsewhere
On the flash in a separate partition
On the flash in the file system of the original firmware
If you buy a QCA988x card from Compex (I have one), option 1 is used. NBG6616 and Archer C7 v2 (both of which I also have) use option 3. NBG6716 obviously uses some other option, I'd guess 1 or 2, since 4 is quite rare. Following my previous instructions will tell us which it is. Pre-cal is only used with board-2.bin file, so that is not the correct way to do this.
The patch only got applied to master and is included in the openwrt-18.06 branch but is still missing in lede-17.01 branch which will be used for the 17.01.5 release. So yes, this issue still exists in the lede-17.01 branch, which is the reason i asked for a cherry-pick to fix this issue for the 17.01.5 release.
Never said i need it
The thought behind that is the following: there is a bug which affects the 17.01-branch and there is a planned maintenance release coming up. In my opinion thats enough reason to backport a known working patch even though i am not in the need for it to be applied onto 17.01. Maybe there are projects out there that use 17.01 as their base and therefore benefit from this fix.