diff options
| author | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2017-03-01 20:50:03 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-03-22 10:14:58 -0300 |
| commit | 733e009c37b6caa219d7057838582d1259c4554f (patch) | |
| tree | 6f57599acda517a437c223e85fc456239378942c /drivers/staging/media/bcm2048 | |
| parent | 78cea55c60fbfbf80f826b8574851de59a661028 (diff) | |
| download | talos-op-linux-733e009c37b6caa219d7057838582d1259c4554f.tar.gz talos-op-linux-733e009c37b6caa219d7057838582d1259c4554f.zip | |
[media] Staging: media: radio-bcm2048: remove incorrect __exit markups
Even if bus is not hot-pluggable, devices can be unbound from the
driver via sysfs, so we should not be using __exit annotations on
remove() methods. The only exception is drivers registered with
platform_driver_probe() which specifically disables sysfs bind/unbind
attributes.
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers/staging/media/bcm2048')
| -rw-r--r-- | drivers/staging/media/bcm2048/radio-bcm2048.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c index ddf7b9dacb50..375c6178032a 100644 --- a/drivers/staging/media/bcm2048/radio-bcm2048.c +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c @@ -2642,7 +2642,7 @@ exit: return err; } -static int __exit bcm2048_i2c_driver_remove(struct i2c_client *client) +static int bcm2048_i2c_driver_remove(struct i2c_client *client) { struct bcm2048_device *bdev = i2c_get_clientdata(client); @@ -2681,7 +2681,7 @@ static struct i2c_driver bcm2048_i2c_driver = { .name = BCM2048_DRIVER_NAME, }, .probe = bcm2048_i2c_driver_probe, - .remove = __exit_p(bcm2048_i2c_driver_remove), + .remove = bcm2048_i2c_driver_remove, .id_table = bcm2048_id, }; |
