status is always zero at the last return in ath6kl_init_upload(). Explicitly return 0 on the success path instead of returning status. No functional change. Signed-off-by: Sang-Heon Jeon --- Changes from v2 [2] - Return 0 instead of removing the check, as suggested by Jeff - Update the subject Changes from v1 [1] - Split the wifi patch per driver - Change base to ath/for-next - Filter only ath6kl changes [1] https://lore.kernel.org/all/20260723184538.3888637-1-ekffu200098@gmail.com/ [2] https://lore.kernel.org/all/20260726133851.1382095-1-ekffu200098@gmail.com/ --- drivers/net/wireless/ath/ath6kl/init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath6kl/init.c b/drivers/net/wireless/ath/ath6kl/init.c index 782209dcb782..6481da4c1991 100644 --- a/drivers/net/wireless/ath/ath6kl/init.c +++ b/drivers/net/wireless/ath/ath6kl/init.c @@ -1570,7 +1570,7 @@ static int ath6kl_init_upload(struct ath6kl *ar) if (status) return status; - return status; + return 0; } int ath6kl_init_hw_params(struct ath6kl *ar) -- 2.43.0