Merge branch 'net-ethernet-convert-to-platform-remove-callback-returning-void'
authorPaolo Abeni <pabeni@redhat.com>
Thu, 30 Nov 2023 12:04:15 +0000 (13:04 +0100)
committerPaolo Abeni <pabeni@redhat.com>
Thu, 30 Nov 2023 12:08:09 +0000 (13:08 +0100)
commit7e0222686316f5506e51182f02c1d83ecc34c471
tree2ff9f07b626c77fb7295eba1975b4edefb58b906
parent04447185dadbb2dfab2b5a3c73c76cbaccd8fbe1
parent7ec1bb2ce64ba9528b1f67bf489dd5f6147d1f64
Merge branch 'net-ethernet-convert-to-platform-remove-callback-returning-void'

Uwe Kleine-König says:

====================
net: ethernet: Convert to platform remove callback returning void

in (implicit) v1 of this series
(https://lore.kernel.org/netdev/20231117091655.872426-1-u.kleine-koenig@pengutronix.de)
I tried to address the resource leaks in the three cpsw drivers. However
this is hard to get right without being able to test the changes. So
here comes a series that just converts all drivers below
drivers/net/ethernet to use .remove_new() and adds a comment about the
potential leaks for someone else to fix the problem.

See commit 5c5a7680e67b ("platform: Provide a remove callback that
returns no value") for an extended explanation and the eventual goal.
The TL;DR; is to prevent bugs like the three noticed here.

Note this series results in no change of behaviour apart from improving
the error message for the three cpsw drivers from

remove callback returned a non-zero value. This will be ignored.

to

Failed to resume device (-ESOMETHING)
====================

Link: https://lore.kernel.org/r/20231128173823.867512-1-u.kleine-koenig@pengutronix.de
Signed-off-by: Paolo Abeni <pabeni@redhat.com>