Commit Graph

789 Commits

Author SHA1 Message Date
Martino Facchin b9bb55584c [USB] use plugged modules name to create iSerial field 2015-10-21 15:23:56 +02:00
NicoHood 281546bccb [PHID] Fixes protocol on reenumeration 2015-10-12 18:05:34 +02:00
Cristian Maglie b8eeeff166 [HID] renamed HIDDescriptorListNode to HIDSubDescriptor 2015-10-12 12:24:07 +02:00
Cristian Maglie 80e228483d [PUSB] Renamed PUSBListNode to PluggableUSBModule 2015-10-12 12:14:07 +02:00
Cristian Maglie 84b3d7ac2b [HID] made HIDDescriptorListNode.length constant 2015-10-12 11:59:08 +02:00
NicoHood 7cef3b3410 [PHID] Added missing GET_IDLE Request 2015-10-11 14:39:36 +02:00
NicoHood 31f93d2c4e [PHID] Added some hints for further optional implementations 2015-10-10 13:11:31 +02:00
NicoHood 2c38941e90 Added official HID definitions
Since the HID library adds some general HID definitions and structs, it is very useful to also include this in a custom library.
If you dupe the information you have problems of redeclaration.
Having those defintions inside the official HID library is essential for a good working additional  HID API.
2015-10-10 12:49:45 +02:00
Nico 4cbaa304fb [PUSB] Minor Style change 2015-10-09 22:10:45 +02:00
Nico a5b1f68aae Remove unnecessary include
Its included by HID.h anyways.
2015-10-08 17:24:09 +02:00
Cristian Maglie 8b6b796f94 [HID] Improved checks in getDescriptor() method 2015-10-08 16:38:57 +02:00
NicoHood ec2cf46acf [PUSB] Removed unnecessary endpoint and interface function 2015-10-07 20:39:50 +02:00
NicoHood 1f5fc56429 [PHID] Added SetReport function
This commit just shows other that this option is available and not implemented.
You may use this to determine the Led Lights state of a keyboard or transmit data via RAWHID from the PC.

Quick usage guide:
int length = ((setup.wValueH << 8) | setup.wLength);
USB_RecvControl((uint8_t* data, USB_EP_SIZE); // Needs to be splitted into USB_EP_SIZE packets, not shown here. See HID Project.
2015-10-07 20:37:05 +02:00
NicoHood fdb1fd384b [PUSB] Made getDescriptor() and setup() more flexible
Alternatively we can only pass the wIndex to getDescriptor but I suggest to just pass the pointer aka reference of the whole setup.
In guess (havent tested this) that this results in more or less the code size but its a) idential with the other functions and b) we late have more flexibility here.
The Code got a quick SerialKeyboard.ino test
2015-10-07 19:02:40 +02:00
NicoHood b75da2a1e5 [PUSB] Changed Interface + Endpoint to unsigned variables
The iterations in the for loop also use unsigned and the setup struct etc as well.
There was no change in HID required since we just init the inherited variables via constructor and the type is never mentioned.
2015-10-07 18:45:10 +02:00
Cristian Maglie f0f10cb290 [PUSB] renamed some parameters 2015-10-07 13:21:21 +02:00
NicoHood b2e48cd627 Small return value error check correction 2015-10-07 12:39:18 +02:00
NicoHood 9196eaed87 Removed not needed public statement for root node 2015-10-03 07:56:48 +02:00
Martino Facchin a6c4a6395e [PUSB] Fix static initialization order fiasco
For details see:
https://isocpp.org/wiki/faq/ctors#static-init-order-on-first-use
2015-10-02 11:59:24 +02:00
Cristian Maglie a185073969 [HID] removed HIDDescriptor field
The field is now built on-the-fly on the stack and sent over USB.
This change increase Flash usage and decrease SRAM usage:

before: 6114 / 241
after:  6152 / 216 (removed HIDDescriptor field)

delta:   +38 / -25

SRAM is a much more scarse resource and this change free up to
about 10% of the "base" usage.
2015-10-02 11:59:24 +02:00
Cristian Maglie f9e6ef1913 [HID] Code cleanup (no semantic changes) 2015-10-02 11:59:23 +02:00
Cristian Maglie 9c93e1d446 [HID] removed unused modules_count field 2015-10-02 11:59:23 +02:00
Cristian Maglie efe6bc0b7b [PUSB] Fixed checks on return values 2015-10-02 11:59:23 +02:00
Cristian Maglie 631e056806 [USB] Fixed some compiler warnings 2015-10-02 11:59:23 +02:00
Cristian Maglie 3cce436602 [PUSB] Fixed the correct number of endpoints 2015-10-02 11:59:23 +02:00
Cristian Maglie 4d6ab7431b [PUSB] Fixed check for available endpoints
The check for available slot in PluggableUSB is done on the endpoint
and not on the number of plugged modules.

The modulesCount field is no longer useful and it has been removed.
2015-10-02 11:59:23 +02:00
Cristian Maglie c5fd3853e6 [PUSB] No more static fields in PluggableUSB class 2015-10-02 11:59:23 +02:00
Cristian Maglie 58b3755853 [PUSB] epType array is now part of HID class 2015-10-02 11:59:23 +02:00
Cristian Maglie 1e7f1fa35e [HID] Removed static fields in HID class
There is still the epType[] array to be handled in some way.
2015-10-02 11:59:23 +02:00
Cristian Maglie a044f89f01 [PUSB] The latest fields are now set via constructor 2015-10-02 11:59:23 +02:00
Cristian Maglie 93e7f0b91b [PUSB] callbacks are now pure virtual methods
This change allows the compiler to handle callbacks resolution.

Callbacks now must be implemented on the class that extends
PUSBListNode and this is forced by compiler by means of pure
virtual methods.

Also the calls to HID.interface() and HID.endpoint() can now
be simplified to interface() and endpoint() respectively since
the methods are no more static.
2015-10-02 11:59:23 +02:00
Cristian Maglie ac0847f257 [PUSB] Moved static members inside HID_ class
This commit prepares for the next refactoring
2015-10-02 11:59:22 +02:00
Cristian Maglie 4245515325 [PUSB] replaced u8 with uint8_t 2015-10-02 11:59:22 +02:00
Cristian Maglie 6d4cbd0ee5 [PUSB] Selected interface and endpoint are now part of PUSBListNode
The method

   int8_t PluggableUSB::addFunction(PUSBListNode *, uint8_t *)

has been changed to

   bool PluggableUSB::plug(PUSBListNode *node)

since both EP and Interfaces are now saved directly into node
2015-10-02 11:59:22 +02:00
Cristian Maglie 6dbfff69e2 [HID] Now HID extends directly PluggableUSBListNode
This avoid duplicate instatiation of callback and save a
considerable amount of flash.
2015-10-02 11:59:22 +02:00
Cristian Maglie 9b4c249090 [PUSB] Global functions PUSB_* are now methods of PluggableUSB class 2015-10-02 11:59:22 +02:00
Cristian Maglie c25a5eefc7 [PUSB] PUSBCallback struct has been merged into PUSBListNode
This slightly simplifies PluggableUSB API.
2015-10-02 11:59:22 +02:00
Arturo Guadalupi 88d546d92a Added LED_BUILTIN to GEMMA
as asked in #3907
2015-10-01 17:09:59 +02:00
Nico 219c9ac945 Removed not used PUSB_Begin() 2015-09-29 17:02:07 +02:00
Cristian Maglie 0034e4b5a3 [HID] Added missing return in HID.begin() 2015-09-28 17:05:36 +02:00
Cristian Maglie 9baff248c4 [HID] Removed unused PUSBReturn structure 2015-09-28 17:05:35 +02:00
Cristian Maglie 7287020aa0 [HID] Changed 'u8' to 'uint8_t' in definitions 2015-09-28 17:05:35 +02:00
Nico e4a5f61c53 Removed not needed Timer warning
Timer is declared above, so nothing is missing here.
2015-09-28 16:30:40 +02:00
Nico 7410817148 Removed Timer warnings when Timer 2 is not present 2015-09-28 16:30:40 +02:00
Nico ab06fbcbf4 Remove Compiler Warning in Tone.cpp 2015-09-28 16:30:39 +02:00
Nico a9005c7627 Fix compiler warning in CDC.cpp 2015-09-28 16:30:39 +02:00
Nico 021c79b372 Fix compiler warnings in USBCore.cpp 2015-09-28 16:30:39 +02:00
Cristian Maglie 1a1419577c [HID] added 'const' qualifier in HID initialization
See https://github.com/arduino/Arduino/pull/3840#discussion_r40438845
2015-09-28 13:32:01 +02:00
Cristian Maglie e22c0bb22a Merge branch 'pluggable_hid_impr' of https://github.com/cmaglie/Arduino 2015-09-28 12:06:35 +02:00
Arturo Guadalupi 4c3de5372d Added Arduino Zero exclusion to README 2015-09-28 11:29:18 +02:00