Subject: Re: [PATCH 2/2] powerpc: Add power management support to VIO bus

From: Brian King <[email-masked]>
Date: Mon, 10 May 2010 08:40:37 -0500

On 05/09/2010 09:35 PM, Michael Ellerman wrote:
> On Fri, 2010-05-07 at 13:58 -0500, Brian King wrote:
>> Adds support for suspend/resume for VIO devices. This is needed for
>> support for HMC initiated hibernation.
>>
>> Signed-off-by: Brian King <[email-masked]>
>> ---
>>
>>  arch/powerpc/kernel/vio.c |   24 ++++++++++++++++++++++++
>>  1 file changed, 24 insertions(+)
>>
>> diff -puN arch/powerpc/kernel/vio.c~powerpc_vio_bus_pm arch/powerpc/kernel/vio.c
>> --- linux-2.6/arch/powerpc/kernel/vio.c~powerpc_vio_bus_pm	2010-05-07 13:49:16.000000000 -0500
>> +++ linux-2.6-bjking1/arch/powerpc/kernel/vio.c	2010-05-07 13:49:16.000000000 -0500
>> @@ -1358,6 +1358,29 @@ static int vio_hotplug(struct device *de
>>  	return 0;
>>  }
>>  
>> +static int vio_pm_suspend(struct device *dev)
>> +{
>> +	const struct dev_pm_ops *pm = dev->driver ? dev->driver->pm : NULL;
>> +
>> +	if (pm && pm->suspend)
>> +		return pm->suspend(dev);
>> +	return 0;
>> +}
>> +
>> +static int vio_pm_resume(struct device *dev)
>> +{
>> +	const struct dev_pm_ops *pm = dev->driver ? dev->driver->pm : NULL;
>> +
>> +	if (pm && pm->resume)
>> +		return pm->resume(dev);
>> +	return 0;
>> +}
>> +
>> +const struct dev_pm_ops vio_dev_pm_ops = {
>> +	.suspend = vio_pm_suspend,
>> +	.resume = vio_pm_resume,
>> +};
> 
> That just looks like boiler plate, is there not a generic version
> somewhere?

Looks there is. I'll update the patch to use the generic helpers.

Thanks,

Brian

-- 
Brian King
Linux on Power Virtualization
IBM Linux Technology Center