On Thu, Jun 01, 2023 at 10:25:55AM +0200, Uwe Kleine-König wrote: > 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@xxxxxxxxxxxxxx> Reviewed-by: Simon Horman <simon.horman@xxxxxxxxxxxx>
