Conversation
Change index check offset
|
@vertexodessa What was the purpose of changing EEPROM_TX_PWR_INX_8812 index from 0x10 to 0x20 in (PR#16)[https://github.com//pull/16]? The same question about it for @KennyPlus but this time changing it to 0x22? |
|
for ignore the 2G detection of the driver initialize, which causes NIC driver loading crashes. |
|
We cannot change it in every single PR back and forth. Let’s figure out how to change it from a constant to a configurable setting instead. |
This offset is constant and it's a fixed address on HW. |
|
hi @josephnef sorry about the confusion and late reply. The point of the fix for Bonnet is to switch from checking presence of power calibration values in 2.4GHz range, to 5.8GHz range, since 2.4GHz range is not calibrated in Bonnet (values are all 0xFF). While my fix also works, fix from @KennyPlus is more correct because it's using correct offset for 5.8GHz range calibration values |
|
it doesn't need to be configurable, the correct value is 0x22 as suggested by Kenny |
|
Now it’s clear, thanks a lot! |
Change index check offset