Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/usb hid multiple report fix #1091

Merged
merged 2 commits into from Aug 27, 2016

Conversation

avtolstoy
Copy link
Member

@avtolstoy avtolstoy commented Aug 14, 2016

Fixes issue #1090. Consecutive HID reports were overwriting the previous report before it was delivered to the host.

Adds:

  • HAL_USB_HID_Status() function to check status of HID transfer;
  • wiring/usbhid test

Doneness:

  • Contributor has signed CLA
  • Problem and Solution clearly stated
  • Code peer reviewed
  • API tests compiled
  • Run unit/integration/application tests on device
  • Add documentation (n/a)
  • Add to CHANGELOG.md after merging (add links to docs and issues)

BUGFIX

  • Consecutive HID reports were overwriting previous the report before it was delivered to the host. Fixes #1090.

@avtolstoy avtolstoy added this to the 0.6.x milestone Aug 14, 2016
@hramadan
Copy link

@avtolstoy - got it, understood. Tricky timing!

Am very keen to assist you where I can in the implementation of USB HID - but I will unfortunately not be in a situation to assist with testing until the end of August 2016.

@ScruffR
Copy link
Contributor

ScruffR commented Aug 17, 2016

Can #1096 be addressed in one go with this?

@technobly technobly merged commit 6b7576f into develop Aug 27, 2016
@technobly technobly deleted the feature/usb-hid-multiple-report-fix branch October 27, 2016 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants