CRSF frame sanity check incorrect.

[hydra]

From the TBS spec, "Frame Length: ... Valid range is between 2 and 62 ...".

But:

CRServoF/lib/CrsfSerial/CrsfSerial.cpp

Line 86 in d5e739f

if (len < 3 || len > CRSF_MAX_PACKET_LEN)

and:

CRServoF/lib/CrsfSerial/crsf_protocol.h

Line 15 in d5e739f

#define CRSF_MAX_PACKET_LEN 64

CRSF 'FrameLength' field != CRSF BUFFER SIZE / CRSF_MAX_PACKET_LEN.

[CapnBry]

🤣 It is so ambiguous everywhere what the max was, especially when I wrote this and had a lot less experience with everyone's implementation (which is the only spec I know). I've updated the code to clarify. I didn't know if the original define (64) was supposed to represent:

• The max total packet size. The size of the buffer need to hold an entire CRSF packet including the start byte, len, type, payload and CRC. The payload data is limited to 60 bytes. TRUE
• The max total size of everything following the length. FALSE
• The max total size of the payload portion, including TYPE, excluding the CRC. FALSE
• The max total size of the data portion of the payload, excluding the TYPE and CRC. FALSE