Am 29.10.2007 20:10 schrieb Alan Stern:
quoted text
> On Mon, 29 Oct 2007, Tilman Schmidt wrote:
>=20
>> From: Tilman Schmidt <tilman@imap.cc>
>>
>> Add basic suspend/resume support to the usb_gigaset driver.
>=20
>> @@ -117,6 +122,11 @@ static struct usb_driver gigaset_usb_dri
>>  	.probe =3D	gigaset_probe,
>>  	.disconnect =3D	gigaset_disconnect,
>>  	.id_table =3D	gigaset_table,
>> +	.suspend =3D	gigaset_suspend,
>> +	.resume =3D	gigaset_resume,
>> +	.reset_resume =3D	gigaset_resume,
>> +	.pre_reset =3D	gigaset_suspend,
>> +	.post_reset =3D	gigaset_resume,
>>  };
>=20
> Does this really compile?  The .pre_reset and .suspend members are=20
> pointers to functions with different prototypes; I don't see how you=20
> 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

--=20
Tilman Schmidt                          E-Mail: tilman@imap.cc
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Unge=F6ffnet mindestens haltbar bis: (siehe R=FCckseite)