On Wednesday, February 19, 2014 12:16:48 PM Aaron Lu wrote: > We put FAN to full speed in acpi_fan_suspend but that is only needed > in the hibernation case when we are generating hibernation image, so > this patch removes that when we are to suspend or poweroff. > > Link: http://www.spinics.net/lists/linux-acpi/msg48636.html > Signed-off-by: Aaron Lu <aaron.lu@xxxxxxxxx> Queued up for 3.15, thanks! > --- > drivers/acpi/fan.c | 13 +++++++++---- > 1 file changed, 9 insertions(+), 4 deletions(-) > > diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c > index 09e423f3d8ad..8acf53e62966 100644 > --- a/drivers/acpi/fan.c > +++ b/drivers/acpi/fan.c > @@ -55,11 +55,16 @@ MODULE_DEVICE_TABLE(acpi, fan_device_ids); > #ifdef CONFIG_PM_SLEEP > static int acpi_fan_suspend(struct device *dev); > static int acpi_fan_resume(struct device *dev); > +static struct dev_pm_ops acpi_fan_pm = { > + .resume = acpi_fan_resume, > + .freeze = acpi_fan_suspend, > + .thaw = acpi_fan_resume, > + .restore = acpi_fan_resume, > +}; > +#define FAN_PM_OPS_PTR (&acpi_fan_pm) > #else > -#define acpi_fan_suspend NULL > -#define acpi_fan_resume NULL > +#define FAN_PM_OPS_PTR NULL > #endif > -static SIMPLE_DEV_PM_OPS(acpi_fan_pm, acpi_fan_suspend, acpi_fan_resume); > > static struct acpi_driver acpi_fan_driver = { > .name = "fan", > @@ -69,7 +74,7 @@ static struct acpi_driver acpi_fan_driver = { > .add = acpi_fan_add, > .remove = acpi_fan_remove, > }, > - .drv.pm = &acpi_fan_pm, > + .drv.pm = FAN_PM_OPS_PTR, > }; > > /* thermal cooling device callbacks */ > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
