Fixes some compile errors and warnings

Fix #172

It’s no longer needed with the timeout mechanism in USBCore.

Timeout is checked on the next send attempt. Initial timeout is set to
70ms, after a time out occurs and the previous transfer is still not
complete a send will report as failed. If a send times out, a ZLP will
be in the send queue to check for future timeouts.

This bypass the generic (but inefficient) implementation of Stream::readBytes
and uses an optimized version of readBytes than can do efficient multi-byte reads.

This change improves read performance when massive data is sent
via USB by exploiting the hardware capability to handle multi-packet
transfers autonomously.

It makes no sense to have ::recv calling repeatedly ::read in this
case.

Now the release() function only performs the action that is called for,
i.e. release the endpoint and let it receive data. All the buffers handling
has been inlined in the callers, this also slightly improves performance
because it allows to remove some redundant checks.

The reference to the upper USBDevice class is passed on
the EPHandler constructor.

this is the proper type to encapsulate a byte when data is available
or -1 when there is no data.

Previous 0 was always returned.

Only enabled if PIN_LED_TXL or PIN_LED_RXL is defined

Resolve compiler warning from performing arithmetic on void pointers

To be compliant with the one sent by the AVR core