Re: [PATCH 1/2] usb_gigaset: suspend support

Am 29.10.2007 20:10 schrieb Alan Stern:
> On Mon, 29 Oct 2007, Tilman Schmidt wrote:
> 
>> From: Tilman Schmidt <[email protected]>
>>
>> Add basic suspend/resume support to the usb_gigaset driver.
> 
>> @@ -117,6 +122,11 @@ static struct usb_driver gigaset_usb_dri
>>  	.probe =	gigaset_probe,
>>  	.disconnect =	gigaset_disconnect,
>>  	.id_table =	gigaset_table,
>> +	.suspend =	gigaset_suspend,
>> +	.resume =	gigaset_resume,
>> +	.reset_resume =	gigaset_resume,
>> +	.pre_reset =	gigaset_suspend,
>> +	.post_reset =	gigaset_resume,
>>  };
> 
> Does this really compile?  The .pre_reset and .suspend members are 
> pointers to functions with different prototypes; I don't see how you 
> can assign the same function to both pointers.

Oops, you're right. Yes, it does compile, but with a warning
"initialization from incompatible pointer type" which I
foolishly ignored.

I'll produce a corrected version promptly.

Thanks,
Tilman

-- 
Tilman Schmidt                          E-Mail: [email protected]
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Ungeöffnet mindestens haltbar bis: (siehe Rückseite)

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/

_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3rc1 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFHJk6hMdB4Whm86/kRAh9UAJ42pW5+NVJVE3g7Ej1zEg3WUhXmCACbB/4+
eKAwKRpnw/fQWyi7D9EfevY=
=ZFlx
-----END PGP SIGNATURE-----