[media] Media: remove incorrect __init/__exit markups
authorDmitry Torokhov <dmitry.torokhov@gmail.com>
Tue, 26 Feb 2013 06:17:27 +0000 (03:17 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 18 Mar 2013 19:59:59 +0000 (16:59 -0300)
Even if bus is not hot-pluggable, the 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.
Similarly probe() methods should not be marked __init unless
platform_driver_probe() is used.

Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Acked-by: Sakari Ailus <sakari.ailus@iki.fi>
Acked-by: Timo Kokkonen <timo.t.kokkonen@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/i2c/adp1653.c
drivers/media/i2c/smiapp/smiapp-core.c
drivers/media/platform/soc_camera/omap1_camera.c
drivers/media/radio/radio-si4713.c
drivers/media/rc/ir-rx51.c

index df163800c8e1e5ec7f7ffe035e8888552da8cc1f..ef75abe5984c38f37c0025e9e38b85a8d45e44a9 100644 (file)
@@ -447,7 +447,7 @@ free_and_quit:
        return ret;
 }
 
-static int __exit adp1653_remove(struct i2c_client *client)
+static int adp1653_remove(struct i2c_client *client)
 {
        struct v4l2_subdev *subdev = i2c_get_clientdata(client);
        struct adp1653_flash *flash = to_adp1653_flash(subdev);
@@ -476,7 +476,7 @@ static struct i2c_driver adp1653_i2c_driver = {
                .pm     = &adp1653_pm_ops,
        },
        .probe          = adp1653_probe,
-       .remove         = __exit_p(adp1653_remove),
+       .remove         = adp1653_remove,
        .id_table       = adp1653_id_table,
 };
 
index 83c7ed7ffcc2c7b7b8d94d82e42a84879410b10a..cae4f46838517472f4f705267ed9b3e93a1999c1 100644 (file)
@@ -2833,7 +2833,7 @@ static int smiapp_probe(struct i2c_client *client,
                                 sensor->src->pads, 0);
 }
 
-static int __exit smiapp_remove(struct i2c_client *client)
+static int smiapp_remove(struct i2c_client *client)
 {
        struct v4l2_subdev *subdev = i2c_get_clientdata(client);
        struct smiapp_sensor *sensor = to_smiapp_sensor(subdev);
@@ -2881,7 +2881,7 @@ static struct i2c_driver smiapp_i2c_driver = {
                .pm = &smiapp_pm_ops,
        },
        .probe  = smiapp_probe,
-       .remove = __exit_p(smiapp_remove),
+       .remove = smiapp_remove,
        .id_table = smiapp_id_table,
 };
 
index 2547bf88f79fe8212a3840f19b0da238be273fa8..9689a6e89b7f9f0fd545556e0facc363823d8b87 100644 (file)
@@ -1546,7 +1546,7 @@ static struct soc_camera_host_ops omap1_host_ops = {
        .poll           = omap1_cam_poll,
 };
 
-static int __init omap1_cam_probe(struct platform_device *pdev)
+static int omap1_cam_probe(struct platform_device *pdev)
 {
        struct omap1_cam_dev *pcdev;
        struct resource *res;
@@ -1677,7 +1677,7 @@ exit:
        return err;
 }
 
-static int __exit omap1_cam_remove(struct platform_device *pdev)
+static int omap1_cam_remove(struct platform_device *pdev)
 {
        struct soc_camera_host *soc_host = to_soc_camera_host(&pdev->dev);
        struct omap1_cam_dev *pcdev = container_of(soc_host,
@@ -1709,7 +1709,7 @@ static struct platform_driver omap1_cam_driver = {
                .name   = DRIVER_NAME,
        },
        .probe          = omap1_cam_probe,
-       .remove         = __exit_p(omap1_cam_remove),
+       .remove         = omap1_cam_remove,
 };
 
 module_platform_driver(omap1_cam_driver);
index 1507c9d508d7c59e98ea0f58cbc017726b3329c5..8ae8442d6f97493f1704a53afc371700d5802a2a 100644 (file)
@@ -328,7 +328,7 @@ exit:
 }
 
 /* radio_si4713_pdriver_remove - remove the device */
-static int __exit radio_si4713_pdriver_remove(struct platform_device *pdev)
+static int radio_si4713_pdriver_remove(struct platform_device *pdev)
 {
        struct v4l2_device *v4l2_dev = platform_get_drvdata(pdev);
        struct radio_si4713_device *rsdev = container_of(v4l2_dev,
@@ -350,7 +350,7 @@ static struct platform_driver radio_si4713_pdriver = {
                .name   = "radio-si4713",
        },
        .probe          = radio_si4713_pdriver_probe,
-       .remove         = __exit_p(radio_si4713_pdriver_remove),
+       .remove         = radio_si4713_pdriver_remove,
 };
 
 module_platform_driver(radio_si4713_pdriver);
index 8ead492d03aac905f2d1007bd22173c57ab3fc1c..31b955bf7664497ff967ab08e55271d99ca70eba 100644 (file)
@@ -464,14 +464,14 @@ static int lirc_rx51_probe(struct platform_device *dev)
        return 0;
 }
 
-static int __exit lirc_rx51_remove(struct platform_device *dev)
+static int lirc_rx51_remove(struct platform_device *dev)
 {
        return lirc_unregister_driver(lirc_rx51_driver.minor);
 }
 
 struct platform_driver lirc_rx51_platform_driver = {
        .probe          = lirc_rx51_probe,
-       .remove         = __exit_p(lirc_rx51_remove),
+       .remove         = lirc_rx51_remove,
        .suspend        = lirc_rx51_suspend,
        .resume         = lirc_rx51_resume,
        .driver         = {