Zheng Yongjun <zhengyongjun3@xxxxxxxxxx> writes: > Simplify the return expression. > > Signed-off-by: Zheng Yongjun <zhengyongjun3@xxxxxxxxxx> > --- > drivers/net/wireless/ath/ath10k/ahb.c | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath10k/ahb.c b/drivers/net/wireless/ath/ath10k/ahb.c > index 05a61975c83f..0ba31c0bbd24 100644 > --- a/drivers/net/wireless/ath/ath10k/ahb.c > +++ b/drivers/net/wireless/ath/ath10k/ahb.c > @@ -598,16 +598,10 @@ static int ath10k_ahb_prepare_device(struct ath10k *ar) > > static int ath10k_ahb_chip_reset(struct ath10k *ar) > { > - int ret; > - > ath10k_ahb_halt_chip(ar); > ath10k_ahb_clock_disable(ar); > > - ret = ath10k_ahb_prepare_device(ar); > - if (ret) > - return ret; > - > - return 0; > + return ath10k_ahb_prepare_device(ar); > } > > static int ath10k_ahb_wake_target_cpu(struct ath10k *ar) I prefer the original style, easier to add new code to the function. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches