Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi @bstiq I have made some minor corrections to the schematics, mostly style stuff and added some explanations why things are needed. I have also added the schematics of the OG blackpill under docs. For reference these are the pin mappings that are DMA channel conflict free.
DMA capabilities are needed for the WS2812 RGB LED driver in PWM mode. I have to check if this is also true for SPI and I2C (which is unused).
For reference the DMA channels used in this config:
Please note that the update event DMA channel is needed for the TIM2 timmer i.e. TIM2_UP not the channel events like TIM2_CH2.
Support for the SOIC-8 EEPROM is a good thing as QMK currently has no EEPROM emulation for the STM32F4 series.
On the support of Full-duplex, the transfer speed gains differences to Half-duplex are rather minor in practice so I wouldn't bother to much, also the the STM32F4 doesn't support TX/RX pin swapping in hardware so this would have to be done with solder jumpers which is a bit cumbersome.