Andrew Morton [off-list ref] wrote:

quoted

quoted

+static int pmac_pm_valid(suspend_state_t state)

 > > +{
 > > +	switch (state) {
 > > +	case PM_SUSPEND_DISK:
 > > +		return 1;
 > > +	/* can't do any other states via generic mechanism yet */
 > > +	default:
 > > +		return 0;
 > > +	}
 > > +}
 > > +
 > >  static struct pm_ops pmac_pm_ops = {
 > >  	.pm_disk_mode	= PM_DISK_SHUTDOWN,
 > >  	.prepare	= pmac_pm_prepare,
 > >  	.enter		= pmac_pm_enter,
 > >  	.finish		= pmac_pm_finish,
 > > +	.valid		= pmac_pm_valid,
 > >  };
 > >  
 > >  #endif /* CONFIG_SOFTWARE_SUSPEND */
 > 

 It looks like a 2.6.17 patch to me.  If someone wants to send it over with
 changelog, signed-off-by, etc I can take care of it.

Please also explain whether this fix is needed in 2.6.16.x.