Ping-Ke Shih <pkshih@xxxxxxxxxxx> wrote: > The register REG_SYS_CFG1 is used to determine chip basic information > as arguments of following flows, such as download firmware and load PHY > parameters, so driver read the value early (before power-on). > > However, the direct IO is disallowed before power-on, or it causes wrong > values, which driver recognizes a chip as a wrong type RF_1T1R, but > actually RF_2T2R, causing driver warns: > > rtw88_8822cs mmc1:0001:1: unsupported rf path (1) > > Fix it by using indirect IO before power-on. > > Reported-by: Piotr Oniszczuk <piotr.oniszczuk@xxxxxxxxx> > Closes: https://lore.kernel.org/linux-wireless/699C22B4-A3E3-4206-97D0-22AB3348EBF6@xxxxxxxxx/T/#t > Suggested-by: Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> > Signed-off-by: Ping-Ke Shih <pkshih@xxxxxxxxxxx> > Tested-by: Piotr Oniszczuk <piotr.oniszczuk@xxxxxxxxx> > Reviewed-by: Martin Blumenstingl <martin.blumenstingl@xxxxxxxxxxxxxx> 1 patch(es) applied to rtw-next branch of rtw.git, thanks. 58de1f91e033 wifi: rtw88: sdio: use indirect IO for device registers before power-on --- https://github.com/pkshih/rtw.git
