Commit b6d2cccb authored by David Brownell's avatar David Brownell Committed by Linus Torvalds

[PATCH] omap_cf: oops-on-suspend fix

Convert omap_cf into a platform_driver ...  this resolves oopsing during
suspend/resume.

Evidently folk haven't tried suspend/resume on an OSK (the main platform
for this driver) since September or so, which is when platform_device
learned about suspend_late()/resume_early() and stopped being able to
suspend/resume without a platform_driver.
Signed-off-by: default avatarDavid Brownell <dbrownell@users.sourceforge.net>
Cc: Dominik Brodowski <linux@dominikbrodowski.net>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 49f19710
...@@ -202,15 +202,14 @@ static struct pccard_operations omap_cf_ops = { ...@@ -202,15 +202,14 @@ static struct pccard_operations omap_cf_ops = {
* "what chipselect is used". Boards could want more. * "what chipselect is used". Boards could want more.
*/ */
static int __devinit omap_cf_probe(struct device *dev) static int __init omap_cf_probe(struct platform_device *pdev)
{ {
unsigned seg; unsigned seg;
struct omap_cf_socket *cf; struct omap_cf_socket *cf;
struct platform_device *pdev = to_platform_device(dev);
int irq; int irq;
int status; int status;
seg = (int) dev->platform_data; seg = (int) pdev->dev.platform_data;
if (seg == 0 || seg > 3) if (seg == 0 || seg > 3)
return -ENODEV; return -ENODEV;
...@@ -227,7 +226,7 @@ static int __devinit omap_cf_probe(struct device *dev) ...@@ -227,7 +226,7 @@ static int __devinit omap_cf_probe(struct device *dev)
cf->timer.data = (unsigned long) cf; cf->timer.data = (unsigned long) cf;
cf->pdev = pdev; cf->pdev = pdev;
dev_set_drvdata(dev, cf); platform_set_drvdata(pdev, cf);
/* this primarily just shuts up irq handling noise */ /* this primarily just shuts up irq handling noise */
status = request_irq(irq, omap_cf_irq, IRQF_SHARED, status = request_irq(irq, omap_cf_irq, IRQF_SHARED,
...@@ -291,7 +290,7 @@ static int __devinit omap_cf_probe(struct device *dev) ...@@ -291,7 +290,7 @@ static int __devinit omap_cf_probe(struct device *dev)
omap_cf_present() ? "present" : "(not present)"); omap_cf_present() ? "present" : "(not present)");
cf->socket.owner = THIS_MODULE; cf->socket.owner = THIS_MODULE;
cf->socket.dev.parent = dev; cf->socket.dev.parent = &pdev->dev;
cf->socket.ops = &omap_cf_ops; cf->socket.ops = &omap_cf_ops;
cf->socket.resource_ops = &pccard_static_ops; cf->socket.resource_ops = &pccard_static_ops;
cf->socket.features = SS_CAP_PCCARD | SS_CAP_STATIC_MAP cf->socket.features = SS_CAP_PCCARD | SS_CAP_STATIC_MAP
...@@ -318,9 +317,9 @@ static int __devinit omap_cf_probe(struct device *dev) ...@@ -318,9 +317,9 @@ static int __devinit omap_cf_probe(struct device *dev)
return status; return status;
} }
static int __devexit omap_cf_remove(struct device *dev) static int __exit omap_cf_remove(struct platform_device *pdev)
{ {
struct omap_cf_socket *cf = dev_get_drvdata(dev); struct omap_cf_socket *cf = platform_get_drvdata(pdev);
cf->active = 0; cf->active = 0;
pcmcia_unregister_socket(&cf->socket); pcmcia_unregister_socket(&cf->socket);
...@@ -332,26 +331,36 @@ static int __devexit omap_cf_remove(struct device *dev) ...@@ -332,26 +331,36 @@ static int __devexit omap_cf_remove(struct device *dev)
return 0; return 0;
} }
static struct device_driver omap_cf_driver = { static int omap_cf_suspend(struct platform_device *pdev, pm_message_t mesg)
{
return pcmcia_socket_dev_suspend(&pdev->dev, mesg);
}
static int omap_cf_resume(struct platform_device *pdev)
{
return pcmcia_socket_dev_resume(&pdev->dev);
}
static struct platform_driver omap_cf_driver = {
.driver = {
.name = (char *) driver_name, .name = (char *) driver_name,
.bus = &platform_bus_type, },
.probe = omap_cf_probe, .remove = __exit_p(omap_cf_remove),
.remove = __devexit_p(omap_cf_remove), .suspend = omap_cf_suspend,
.suspend = pcmcia_socket_dev_suspend, .resume = omap_cf_resume,
.resume = pcmcia_socket_dev_resume,
}; };
static int __init omap_cf_init(void) static int __init omap_cf_init(void)
{ {
if (cpu_is_omap16xx()) if (cpu_is_omap16xx())
return driver_register(&omap_cf_driver); return platform_driver_probe(&omap_cf_driver, omap_cf_probe);
return -ENODEV; return -ENODEV;
} }
static void __exit omap_cf_exit(void) static void __exit omap_cf_exit(void)
{ {
if (cpu_is_omap16xx()) if (cpu_is_omap16xx())
driver_unregister(&omap_cf_driver); platform_driver_unregister(&omap_cf_driver);
} }
module_init(omap_cf_init); module_init(omap_cf_init);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment