Toke Høiland-Jørgensen <toke@xxxxxxxxxx> wrote: > When the error check in ath9k_hw_read_revisions() was added, it checked for > -EIO which is what ath9k_regread() in the ath9k_htc driver uses. However, > for plain ath9k, the register read function uses ioread32(), which just > returns -1 on error. So if such a read fails, it still gets passed through > and ends up as a weird mac revision in the log output. > > Fix this by changing ath9k_regread() to return -1 on error like ioread32() > does, and fix the error check to look for that instead of -EIO. > > Fixes: 2f90c7e5d094 ("ath9k: Check for errors when reading SREV register") > Signed-off-by: Toke Høiland-Jørgensen <toke@xxxxxxxxxx> > Reviewed-by: Lorenzo Bianconi <lorenzo@xxxxxxxxxx> > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> Patch applied to ath-next branch of ath.git, thanks. 7dd9a40fd6e0 ath9k: Fix error check in ath9k_hw_read_revisions() for PCI devices -- https://patchwork.kernel.org/project/linux-wireless/patch/20210326180819.142480-1-toke@xxxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches