Dmitry Antipov <dmantipov@xxxxxxxxx> wrote: > Convert 'brcmf_configure_wowl()' to return 's32' which may be > an error raised by 'brcmf_fil_iovar_data_set()', pass the > value to 'brcmf_cfg80211_suspend()' and adjust related code. > > Found by Linux Verification Center (linuxtesting.org) with SVACE. > > Signed-off-by: Dmitry Antipov <dmantipov@xxxxxxxxx> Using s32 for an error value is strange for me, usually we use int. This patchset feels like random cleanup which makes me wary. How are these 3 patches tested? -- https://patchwork.kernel.org/project/linux-wireless/patch/20230607161611.85106-1-dmantipov@xxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
