[riot-notifications] [RIOT-OS/RIOT] usbus/hid_io: add missing header file, add RX callback function (#16689)

Nils Ollrogge notifications at github.com
Tue Aug 3 16:15:44 CEST 2021


> Generally, the option of an argument seems like a good idea, although I don't have a specific case in mind either. @Ollrogge what is the typical use of HID IO? Can't you think of a situation where an upper layer needs to pass information to the callback? In worst case, check the API of an other implementation (for example in [KEIL](https://www.keil.com/pack/doc/mw/USB/html/_u_s_b__device.html))

Typical usages of HID IO would be movements of a mouse or keypresses. But since USB HID is a self-explaining protocol it could be anything (e.g. a FIDO2 authenticator as in my [PR](https://github.com/RIOT-OS/RIOT/pull/16489)). For my use case I just need the callback to trigger an event to fetch the data from the USB layer but there surely are use cases that need an arg. I think adding it doesn't hurt.




-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16689#issuecomment-891883963
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210803/a789e3d7/attachment.htm>


More information about the notifications mailing list