Hi,

On Mon, Aug 22, 2011 at 09:59:46AM +0530, J, KEERTHY wrote:
> >> +static const struct dev_pm_ops omap_temp_sensor_dev_pm_ops = {
> >> +     .suspend = omap_temp_sensor_suspend,
> >> +     .resume = omap_temp_sensor_resume,
> >> +     .runtime_suspend = omap_temp_sensor_runtime_suspend,
> >> +     .runtime_resume = omap_temp_sensor_runtime_resume,
> >> +};
> >
> > this is not enough... you need to populate the other fields. You might
> > want to use something like:
> >
> > static const struct dev_pm_ops omap_temp_sensor_dev_pm_ops = {
> >        SET_SYSTEM_SLEEP_PM_OPS(omap_temp_sensor_suspend,
> >                omap_temp_sensor_resume)
> >        SET_RUNTIME_PM_OPS(omap_temp_sensor_runtime_suspend,
> >                omap_temp_sensor_runtime_resume, omap_temp_sensor_idle)
> > };
> 
> omap_temp_sensor_idle is undefined for temperature sensor as of now. So i will
> populate these fields once the idle function is defined.

make a nop idle function. Those other fields _must_ be populated. Use
something like:

static int omap_temp_sensor_idle(struct device *dev)
{
        /* nothing to do here */
        return 0;
}

-- 
balbi

Attachment: signature.asc
Description: Digital signature

Reply via email to