On 18/07/2024 09:41, Ping-Ke Shih wrote: > Originally in order to read partial registers from large area, we write > a range value stored into a static variable and read registers according > to the static variable. > > However, if we install more than one adapters supported by this driver, > the static variables will be overwritten by latter adapters. To resolve > the problem, move the static variables to struct rtw_dev for each adapter. > > With changes, smatch spends too much time to parse rtw_debugfs_init(): > debug.c:1289 rtw_debugfs_init() parse error: turning off implications > after 60 seconds > Move stuffs of adding debugfs entries to three rtw_debugfs_add_xxx() > functions. > > Reported-by: Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> > Closes: https://lore.kernel.org/linux-wireless/cd6a2acf3c2c36d938b40140b52a779516f446a9.camel@xxxxxxxxxxx/T/#m27662022c70d9f893ba96f6c6a8dd8fce2434dfe > Signed-off-by: Ping-Ke Shih <pkshih@xxxxxxxxxxx> > --- Thank you for fixing it so quickly! It works for me. Tested-by: Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx>