Ping-Ke Shih <pkshih@xxxxxxxxxxx> wrote: > For normal (successful) flow, it must return 0. The original code uses > 'return ret', and then we need to backward reference to initial value to > know 'ret = 0'. Changing them to 'return 0', because it will be more > readable and intuitive. This patch doesn't change logic at all. > > Reported-by: kernel test robot <lkp@xxxxxxxxx> > Reported-by: Dan Carpenter <error27@xxxxxxxxx> > Link: https://lore.kernel.org/r/202302101023.ctlih5q0-lkp@xxxxxxxxx/ > Signed-off-by: Ping-Ke Shih <pkshih@xxxxxxxxxxx> Patch applied to wireless-next.git, thanks. 210871887208 wifi: rtw89: use readable return 0 in rtw89_mac_cfg_ppdu_status() -- https://patchwork.kernel.org/project/linux-wireless/patch/20230213091328.25481-1-pkshih@xxxxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches