platform-drivers: move probe to .devinit.text in drivers/scsi
Uwe Kleine-König [Thu, 4 Feb 2010 19:56:50 +0000 (20:56 +0100)]
A pointer to a probe callback is passed to the core via
platform_driver_register and so the function must not disappear when the
.init sections are discarded.  Otherwise (if also having HOTPLUG=y)
unbinding and binding a device to the driver via sysfs will result in an
oops as does a device being registered late.

An alternative to this patch is using platform_driver_probe instead of
platform_driver_register plus removing the pointer to the probe function
from the struct platform_driver.

Signed-off-by: Uwe Kleine-König <>
Cc: Andrew Morton <>
Cc: David Brownell <>
Cc: Dmitri Vorobiev <>
Cc: Henrik Kretzschmar <>
Cc: James Bottomley <>
Cc: Kay Sievers <>
Cc: peter fuerst <>
Cc: Thomas Bogendoerfer <>
Acked-by: Ralf Baechle <>
Signed-off-by: Greg Kroah-Hartman <>


index 0807b26..fef0e3c 100644 (file)
@@ -226,7 +226,7 @@ static struct scsi_host_template sgiwd93_template = {
        .use_clustering         = DISABLE_CLUSTERING,
-static int __init sgiwd93_probe(struct platform_device *pdev)
+static int __devinit sgiwd93_probe(struct platform_device *pdev)
        struct sgiwd93_platform_data *pd = pdev->dev.platform_data;
        unsigned char *wdregs = pd->wdregs;
index 37b3359..56cf0bb 100644 (file)
@@ -64,7 +64,7 @@ static struct scsi_host_template snirm710_template = {
        .module         = THIS_MODULE,
-static int __init snirm710_probe(struct platform_device *dev)
+static int __devinit snirm710_probe(struct platform_device *dev)
        unsigned long base;
        struct NCR_700_Host_Parameters *hostdata;