From: Antonio Ospite <ospite@studenti.unina.it> To: linux-input@vger.kernel.org Cc: linux-bluetooth@vger.kernel.org, Bastien Nocera <hadess@hadess.net>, Marcel Holtmann <marcel@holtmann.org>, Jiri Kosina <jkosina@suse.cz>, Alan Ott <alan@signal11.us> Subject: hidp_output_raw_report, HID_OUTPUT_REPORT and Sixaxis Date: Tue, 30 Nov 2010 14:54:05 +0100 [thread overview] Message-ID: <20101130145405.d8142bc3.ospite@studenti.unina.it> (raw) [-- Attachment #1: Type: text/plain, Size: 1404 bytes --] Hi, another piece in the Sixaxis jigsaw: in commit d4bfa033ed84e0ae446eff445d107ffd5ee78df3 support for setting different report types was added to hidp, however in my Sixaxis experiments setting leds (sending and output report) was not working until I made this change: diff --git a/net/bluetooth/hidp/core.c b/net/bluetooth/hidp/core.c index b68a608..0c443b7 100644 --- a/net/bluetooth/hidp/core.c +++ b/net/bluetooth/hidp/core.c @@ -402,7 +402,7 @@ static int hidp_output_raw_report(struct hid_device *hid, unsigned char *data, s report_type = HIDP_TRANS_SET_REPORT | HIDP_DATA_RTYPE_FEATURE; break; case HID_OUTPUT_REPORT: - report_type = HIDP_TRANS_DATA | HIDP_DATA_RTYPE_OUPUT; + report_type = HIDP_TRANS_SET_REPORT | HIDP_DATA_RTYPE_OUPUT; break; default: return -EINVAL; Is it only the Sixaxis which needs the output report as a SET_REPORT operation, or the change above is an actual fix? I don't know bluetooth at all, sorry. In case this is a sixaxis specific behavior then I guess I'll be overriding hidp_output_raw_report() in hid-sony.c just like I did for the usbhid counterpart. Thanks, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top-posting such a bad thing? [-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Antonio Ospite <ospite-aNJ+ML1ZbiP93QAQaVx+gl6hYfS7NtTn@public.gmane.org> To: linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: linux-bluetooth-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Bastien Nocera <hadess-0MeiytkfxGOsTnJN9+BGXg@public.gmane.org>, Marcel Holtmann <marcel-kz+m5ild9QBg9hUCZPvPmw@public.gmane.org>, Jiri Kosina <jkosina-AlSwsSmVLrQ@public.gmane.org>, Alan Ott <alan-yzvJWuRpmD1zbRFIqnYvSA@public.gmane.org> Subject: hidp_output_raw_report, HID_OUTPUT_REPORT and Sixaxis Date: Tue, 30 Nov 2010 14:54:05 +0100 [thread overview] Message-ID: <20101130145405.d8142bc3.ospite@studenti.unina.it> (raw) [-- Attachment #1: Type: text/plain, Size: 1404 bytes --] Hi, another piece in the Sixaxis jigsaw: in commit d4bfa033ed84e0ae446eff445d107ffd5ee78df3 support for setting different report types was added to hidp, however in my Sixaxis experiments setting leds (sending and output report) was not working until I made this change: diff --git a/net/bluetooth/hidp/core.c b/net/bluetooth/hidp/core.c index b68a608..0c443b7 100644 --- a/net/bluetooth/hidp/core.c +++ b/net/bluetooth/hidp/core.c @@ -402,7 +402,7 @@ static int hidp_output_raw_report(struct hid_device *hid, unsigned char *data, s report_type = HIDP_TRANS_SET_REPORT | HIDP_DATA_RTYPE_FEATURE; break; case HID_OUTPUT_REPORT: - report_type = HIDP_TRANS_DATA | HIDP_DATA_RTYPE_OUPUT; + report_type = HIDP_TRANS_SET_REPORT | HIDP_DATA_RTYPE_OUPUT; break; default: return -EINVAL; Is it only the Sixaxis which needs the output report as a SET_REPORT operation, or the change above is an actual fix? I don't know bluetooth at all, sorry. In case this is a sixaxis specific behavior then I guess I'll be overriding hidp_output_raw_report() in hid-sony.c just like I did for the usbhid counterpart. Thanks, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top-posting such a bad thing? [-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]
next reply other threads:[~2010-11-30 13:54 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2010-11-30 13:54 Antonio Ospite [this message] 2010-11-30 13:54 ` hidp_output_raw_report, HID_OUTPUT_REPORT and Sixaxis Antonio Ospite 2010-11-30 14:06 ` Antonio Ospite 2010-11-30 17:40 ` pascal 2010-12-01 21:06 ` Antonio Ospite 2010-12-01 22:40 ` pascal 2011-02-17 14:19 ` Antonio Ospite 2011-02-17 14:19 ` Antonio Ospite 2011-02-18 8:45 ` Jiri Kosina 2011-02-18 8:45 ` Jiri Kosina 2011-02-20 17:26 ` [PATCH 0/2] Fix sending Output reports to the Sony Sixaxis Antonio Ospite 2011-02-20 17:26 ` [PATCH 1/2] hid-sony.c: Fix sending Output reports to the Sixaxis Antonio Ospite 2011-02-21 12:49 ` Jiri Kosina 2011-02-20 17:26 ` [PATCH 2/2] bt hidp: send Output reports using SET_REPORT on the Control channel Antonio Ospite 2011-02-21 3:45 ` Alan Ott 2011-02-21 3:45 ` Alan Ott 2011-02-21 12:50 ` Jiri Kosina 2011-02-21 21:09 ` Gustavo F. Padovan 2011-02-22 10:09 ` Jiri Kosina 2011-02-22 10:09 ` Jiri Kosina
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20101130145405.d8142bc3.ospite@studenti.unina.it \ --to=ospite@studenti.unina.it \ --cc=alan@signal11.us \ --cc=hadess@hadess.net \ --cc=jkosina@suse.cz \ --cc=linux-bluetooth@vger.kernel.org \ --cc=linux-input@vger.kernel.org \ --cc=marcel@holtmann.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.