wifi: ath10k: Convert to platform remove callback returning void
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Thu, 1 Jun 2023 08:25:55 +0000 (10:25 +0200)
committerKalle Valo <quic_kvalo@quicinc.com>
Fri, 9 Jun 2023 12:27:58 +0000 (15:27 +0300)
commitd457bff2763366513bce36c2fc51fcba12a2f912
tree681058e8cf5e86704e5e034ecfb682ed0a7353b6
parentfad5ac80dfa5010d44c24d59d9e8e1552a447911
wifi: ath10k: Convert to platform remove callback returning void

The .remove() callback for a platform driver returns an int which makes
many driver authors wrongly assume it's possible to do error handling by
returning an error code. However the value returned is (mostly) ignored
and this typically results in resource leaks. To improve here there is a
quest to make the remove callback return void. In the first step of this
quest all drivers are converted to .remove_new() which already returns
void.

Both ath10k platform drivers return zero unconditionally in their remove
callback, so they can be trivially converted to use .remove_new().

Also fix on of the more offending whitespace issues in the definition
of ath10k_snoc_driver.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20230601082556.2738446-4-u.kleine-koenig@pengutronix.de
drivers/net/wireless/ath/ath10k/ahb.c
drivers/net/wireless/ath/ath10k/snoc.c