can_suspend_to_{x}

[Danny Kukawka]

On Monday 08 May 2006 19:56, David Zeuthen wrote:
> On Mon, 2006-05-08 at 18:43 +0100, Richard Hughes wrote:
> > On Mon, 2006-05-08 at 12:31 -0400, David Zeuthen wrote:
> > > OK to add a patch that adds
> > >
> > >   power_management.can_hibernate
> > >   power_management.can_suspend
> > >
> > > keys but please keep the old ones
> > >
> > >   power_management.can_suspend_to_disk
> > >   power_management.can_suspend_to_ram
> >
> > Something like the attached?
>
> Sounds good, suggest

I have to say, I don't like this _needless_ API break. The *can_suspend_* keys 
are already in the API, and it's also used already by different apps, and 
this keys also do the right thing atm (providing information what the 
_kernel_ support, not more and no less). IMO there is no need for a change.

>  s/This key will be removed on May 1st 2007./This key will be removed
> but not before May 1st 2007./

If you break the API, you need IMO to provide the key longer than only one 
year. Take e.g. a look at the rules for kernel for such API changes ...

Danny