On 2020-06-05 15:47:03 [+0800], yhchuang@xxxxxxxxxxx wrote: > From: Yan-Hsuan Chuang <yhchuang@xxxxxxxxxxx> > > Some platforms cannot read the DBI register successfully for the > ASPM settings. After the read failed, the bus could be unstable, > and the device just became unavailable [1]. For those platforms, > the ASPM should be disabled. But as the ASPM can help the driver > to save the power consumption in power save mode, the ASPM is still > needed. So, add a module parameter for them to disable it, then > the device can still work, while others can benefit from the less > power consumption that brings by ASPM enabled. Can you set disable_aspm if rtw_dbi_read8() fails? Or make a test if it is save to use? If someone notices the warning they still have to search for the warning in order to make the link towards loading the module with the disable_aspm=1 paramter. Is it known what causes the failure? > [1] https://bugzilla.kernel.org/show_bug.cgi?id=206411 > [2] Note that my lenovo T430 is the same. Sebastian
