diff mbox

crypto: qat - remove incorrect __exit markup

Message ID 20150309202549.GA9684@dtor-ws (mailing list archive)
State Accepted
Delegated to: Herbert Xu
Headers show

Commit Message

Dmitry Torokhov March 9, 2015, 8:25 p.m. UTC
PCI bus is hot-pluggable, and even if it wasn't one can still unbind the
device from driver via sysfs, so we should not make driver's remove
method as __exit.

Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
 drivers/crypto/qat/qat_dh895xcc/adf_drv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Herbert Xu March 11, 2015, 2:13 p.m. UTC | #1
On Mon, Mar 09, 2015 at 01:25:49PM -0700, Dmitry Torokhov wrote:
> PCI bus is hot-pluggable, and even if it wasn't one can still unbind the
> device from driver via sysfs, so we should not make driver's remove
> method as __exit.
> 
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>

Applied.
diff mbox

Patch

diff --git a/drivers/crypto/qat/qat_dh895xcc/adf_drv.c b/drivers/crypto/qat/qat_dh895xcc/adf_drv.c
index 8ffdb95..e7af9d5 100644
--- a/drivers/crypto/qat/qat_dh895xcc/adf_drv.c
+++ b/drivers/crypto/qat/qat_dh895xcc/adf_drv.c
@@ -379,7 +379,7 @@  out_err:
 	return ret;
 }
 
-static void __exit adf_remove(struct pci_dev *pdev)
+static void adf_remove(struct pci_dev *pdev)
 {
 	struct adf_accel_dev *accel_dev = adf_devmgr_pci_to_accel_dev(pdev);