[Core] STM32_USB_USE_OTG1 => USB_ENDPOINTS_ARE_REORDERABLE

[tzarc]

Description

Creates a new define -- USB_ENDPOINTS_ARE_REORDERABLE -- which decouples bidirectional endpoints from the previously-(ab)used STM32_USB_USE_OTG1.

Also fixes make <kb>:<km>:dump_vars.

SHA1's match when comparing builds:

                           gmmk/gmmk2/p96/ansi:via - 8eaa98c6771c15326f943749d3185465d2d392e8
                            gmmk/gmmk2/p96/iso:via - 22f3d4a0d85e9d9be4d9dcc7426232ae75bb89c4

Types of Changes

• Core
• Bugfix
• New feature
• Enhancement/optimization
• Keyboard (addition or update)
• Keymap/layout/userspace (addition or update)
• Documentation

Checklist

• My code follows the code style of this project: C, Python
• I have read the PR Checklist document and have made the appropriate changes.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• I have tested the changes and verified that they work and don't break anything (as well as I can manage).

[KarlK90]

In the endpoint initialization code usb_main.c:

           .inout_ep_config =                                                                                                  \
                {                                                                                                               \
                    stream##_IN_MODE,      /* Interrupt EP */                                                                   \
                    NULL,                  /* SETUP packet notification callback */                                             \
                    qmkusbDataTransmitted, /* IN notification callback */                                                       \
                    qmkusbDataReceived,    /* OUT notification callback */                                                      \
                    stream##_EPSIZE,       /* IN maximum packet size */                                                         \
                    stream##_EPSIZE,       /* OUT maximum packet size */                                                        \
                    NULL,                  /* IN Endpoint state */                                                              \
                    NULL,                  /* OUT endpoint state */                                                             \
                    2,                     /* IN multiplier */                                                                  \
                    NULL                   /* SETUP buffer (not a SETUP endpoint) */                                            \
                },                                                                                                              \

the vendor usb lld driver specific part

                    2,                     /* IN multiplier */                                                                  \
                    NULL                   /* SETUP buffer (not a SETUP endpoint) */        

should be replaced with usb_lld_endpoint_fields define so it can be reusable for e.g. RP2040. (though I have to confirm that it works on that chip).

[KarlK90]

Curiosity got the better of me: It is just two places usb_patch.txt but RP2040 sadly won't boot up. But doesn't really matter either with 32 endpoints available.

[KarlK90]

Thanks for making it mcu independent.