[riot-notifications] [RIOT-OS/RIOT] usbus/hid_io: add missing header file, add RX callback function (#16689)
notifications at github.com
Tue Aug 3 12:10:20 CEST 2021
> This is something to add in further PR's I think. Therefore 1 static callback function is actually fine for now I guess.
> Wouldn't it make sense for the callback to at least have a void pointer argument?
> What would you pass as argument from the USB HID layer ?
> It wouldn't be from the USB HID layer, it would be from the upper layer,
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))
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications