Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> wrote: > There is no point in allocating 'skb' and then freeing it if !channel. > > Make the sanity check first to slightly simplify the code. > > Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> > Reviewed-by: Simon Horman <simon.horman@xxxxxxxxxxxx> Patch applied to wireless-next.git, thanks. 8de7838acfa3 wifi: rsi: Slightly simplify rsi_set_channel() -- https://patchwork.kernel.org/project/linux-wireless/patch/29bf0296bd939e3f6952272bfdcc73b22edbc374.1679328588.git.christophe.jaillet@xxxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
