Re: What happened to SUSPEND_SAVE_STATE?

[Pavel Machek <pavel@ucw.cz>]

Hi!

> > > What happened to SUSPEND_SAVE_STATE?
> > 
> > SUSPEND_NOTIFY seems dead, too. Should I simply ignore level parameter
> > in pcmcia_socket_dev_suspend?
> 
> No.  Apply this patch (it's cut down from the stuff which is pending
> for Linus - I hope I didn't make any mistakes doing that 8))

Thanks, but:

> diff -Nru a/drivers/pcmcia/i82365.c b/drivers/pcmcia/i82365.c
> --- a/drivers/pcmcia/i82365.c	Wed Sep 10 23:18:34 2003
> +++ b/drivers/pcmcia/i82365.c	Wed Sep 10 23:18:34 2003
> @@ -1351,11 +1351,27 @@
>  
>  /*====================================================================*/
>  
> +static int i82365_suspend(struct device *dev, u32 state, u32 level)
> +{
> +	int ret = 0;
> +	if (level == SUSPEND_SAVE_STATE)
> +		ret = pcmcia_socket_dev_suspend(dev, state);
> +	return ret;
> +}
> +
> +static int i82365_resume(struct device *dev, u32 level)
> +{
> +	int ret = 0;
> +	if (level == RESUME_RESTORE_STATE)
> +		ret = pcmcia_socket_dev_resume(dev);
> +	return ret;
> +}
> +
>  static struct device_driver i82365_driver = {
>  	.name = "i82365",
>  	.bus = &platform_bus_type,
> -	.suspend = pcmcia_socket_dev_suspend,
> -	.resume = pcmcia_socket_dev_resume,
> +	.suspend = i82365_suspend,
> +	.resume = i82365_resume,
>  };
>  
>  static struct platform_device i82365_device = {

I was not able to find *any* place in the tree that would call suspend
with SUSPEND_SAVE_STATE as level. Maybe just my grep was wrong?

								Pavel
-- 
When do you have a heart between your knees?
[Johanka's followup: and *two* hearts?]