From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruno =?UTF-8?B?UHLDqW1vbnQ=?= Subject: [PATCH 1/2] hid: add suspend/resume hooks for hid drivers Date: Sun, 25 Apr 2010 21:40:03 +0200 Message-ID: <20100425214003.7e87c6a3@neptune.home> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Sender: linux-kernel-owner@vger.kernel.org To: Oliver Neukum , Jiri Kosina , Marcel Holtmann Cc: Dmitry Torokhov , linux-input@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Nicu Pavel , Jaya Kumar , "Rick L. Vinyard Jr." List-Id: linux-input@vger.kernel.org Add suspend/resume hooks for HID drivers so these can do some additional state adjustment when device gets suspended/resumed. v3: - Pass full message to hid driver's suspend hook v2: - Adds auto_suspend parameter to suspend hook - Only calls HID driver's resume hooks when previous code did succeed Signed-off-by: Bruno Pr=C3=A9mont --- Note, I've not added PM parts to bluetooth as I'm not sure how/where to catch the PM events generated by bluetooth. It looks like PM on bluetooth side does just some bluetooth housekeepin= g and sends an event. Marcel, do you have some pointers for wiring up these HID suspend hooks in the bluetooth/hidp code? drivers/hid/usbhid/hid-core.c | 24 +++++++++++++++++++++++- include/linux/hid.h | 8 ++++++++ 2 files changed, 31 insertions(+), 1 deletions(-) diff --git a/drivers/hid/usbhid/hid-core.c b/drivers/hid/usbhid/hid-cor= e.c index 3e7909b..17eb3ec 100644 --- a/drivers/hid/usbhid/hid-core.c +++ b/drivers/hid/usbhid/hid-core.c @@ -1291,6 +1291,11 @@ static int hid_suspend(struct usb_interface *int= f, pm_message_t message) { set_bit(HID_REPORTED_IDLE, &usbhid->iofl); spin_unlock_irq(&usbhid->lock); + if (hid->driver && hid->driver->suspend) { + status =3D hid->driver->suspend(hid, message); + if (status < 0) + return status; + } } else { usbhid_mark_busy(usbhid); spin_unlock_irq(&usbhid->lock); @@ -1298,6 +1303,11 @@ static int hid_suspend(struct usb_interface *int= f, pm_message_t message) } =20 } else { + if (hid->driver && hid->driver->suspend) { + status =3D hid->driver->suspend(hid, message); + if (status < 0) + return status; + } spin_lock_irq(&usbhid->lock); set_bit(HID_REPORTED_IDLE, &usbhid->iofl); spin_unlock_irq(&usbhid->lock); @@ -1352,6 +1362,11 @@ static int hid_resume(struct usb_interface *intf= ) hid_io_error(hid); usbhid_restart_queues(usbhid); =20 + if (status >=3D 0 && hid->driver && hid->driver->resume) { + int ret =3D hid->driver->resume(hid); + if (ret < 0) + status =3D ret; + } dev_dbg(&intf->dev, "resume status %d\n", status); return 0; } @@ -1360,9 +1375,16 @@ static int hid_reset_resume(struct usb_interface= *intf) { struct hid_device *hid =3D usb_get_intfdata(intf); struct usbhid_device *usbhid =3D hid->driver_data; + int status; =20 clear_bit(HID_REPORTED_IDLE, &usbhid->iofl); - return hid_post_reset(intf); + status =3D hid_post_reset(intf); + if (status >=3D 0 && hid->driver && hid->driver->reset_resume) { + int ret =3D hid->driver->reset_resume(hid); + if (ret < 0) + status =3D ret; + } + return status; } =20 #endif /* CONFIG_PM */ diff --git a/include/linux/hid.h b/include/linux/hid.h index b1344ec..069e587 100644 --- a/include/linux/hid.h +++ b/include/linux/hid.h @@ -589,6 +589,9 @@ struct hid_usage_id { * @report_fixup: called before report descriptor parsing (NULL means = nop) * @input_mapping: invoked on input registering before mapping an usag= e * @input_mapped: invoked on input registering after mapping an usage + * @suspend: invoked on suspend (NULL means nop) + * @resume: invoked on resume if device was not reset (NULL means nop) + * @reset_resume: invoked on resume if device was reset (NULL means no= p) * * raw_event and event should return 0 on no action performed, 1 when = no * further processing should be done and negative on error @@ -629,6 +632,11 @@ struct hid_driver { int (*input_mapped)(struct hid_device *hdev, struct hid_input *hidinput, struct hid_field *field, struct hid_usage *usage, unsigned long **bit, int *max); +#ifdef CONFIG_PM + int (*suspend)(struct hid_device *hdev, pm_message_t message); + int (*resume)(struct hid_device *hdev); + int (*reset_resume)(struct hid_device *hdev); +#endif /* private: */ struct device_driver driver; }; --=20 1.6.4.4