From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Brownell Subject: [patch/UPDATE 2.6.18-rc5-mm] PM: platform_bus and late_suspend/early_resume Date: Sun, 3 Sep 2006 13:16:45 -0700 Message-ID: <200609031316.45905.david-b@pacbell.net> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.osdl.org Errors-To: linux-pm-bounces@lists.osdl.org To: Greg KH Cc: Linus Torvalds , Linux-pm mailing list List-Id: linux-pm@vger.kernel.org Teach platform_bus about the new suspend_late/resume_early PM calls, issued with IRQs off. Do we really need sysdev and friends any more, or can janitors start switching its users over to platform_device so we can do a minor code-ectomy? Signed-off-by: David Brownell --- bugfixed from previous version: platform_bus didn't assign the intended methods. Index: g26/drivers/base/platform.c =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D --- g26.orig/drivers/base/platform.c 2006-06-28 22:02:04.000000000 -0700 +++ g26/drivers/base/platform.c 2006-08-31 19:36:07.000000000 -0700 @@ -505,12 +505,36 @@ static int platform_match(struct device = return (strncmp(pdev->name, drv->name, BUS_ID_SIZE) =3D=3D 0); } = -static int platform_suspend(struct device * dev, pm_message_t state) +static int platform_suspend(struct device *dev, pm_message_t mesg) { int ret =3D 0; = if (dev->driver && dev->driver->suspend) - ret =3D dev->driver->suspend(dev, state); + ret =3D dev->driver->suspend(dev, mesg); + + return ret; +} + +static int platform_suspend_late(struct device *dev, pm_message_t mesg) +{ + struct platform_driver *drv =3D to_platform_driver(dev->driver); + struct platform_device *pdev =3D container_of(dev, struct platform_device= , dev); + int ret =3D 0; + + if (dev->driver && drv->suspend_late) + ret =3D drv->suspend_late(pdev, mesg); + + return ret; +} + +static int platform_resume_early(struct device *dev) +{ + struct platform_driver *drv =3D to_platform_driver(dev->driver); + struct platform_device *pdev =3D container_of(dev, struct platform_device= , dev); + int ret =3D 0; + + if (dev->driver && drv->resume_early) + ret =3D drv->resume_early(pdev); = return ret; } @@ -531,6 +555,8 @@ struct bus_type platform_bus_type =3D { .match =3D platform_match, .uevent =3D platform_uevent, .suspend =3D platform_suspend, + .suspend_late =3D platform_suspend_late, + .resume_early =3D platform_resume_early, .resume =3D platform_resume, }; EXPORT_SYMBOL_GPL(platform_bus_type); Index: g26/include/linux/platform_device.h =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D --- g26.orig/include/linux/platform_device.h 2006-08-31 19:30:33.000000000 = -0700 +++ g26/include/linux/platform_device.h 2006-08-31 19:30:46.000000000 -0700 @@ -49,6 +49,8 @@ struct platform_driver { int (*remove)(struct platform_device *); void (*shutdown)(struct platform_device *); int (*suspend)(struct platform_device *, pm_message_t state); + int (*suspend_late)(struct platform_device *, pm_message_t state); + int (*resume_early)(struct platform_device *); int (*resume)(struct platform_device *); struct device_driver driver; };