platformio / platform-atmelavr

Atmel AVR: development platform for PlatformIO
https://registry.platformio.org/platforms/platformio/atmelavr
Apache License 2.0
140 stars 105 forks source link

Added attiny84_cw_pinmap board option #336

Open nickyonge opened 1 month ago

nickyonge commented 1 month ago

Created a board for ATtiny84 using the reverse (clockwise) pin mappinig, as opposed to the default CCW pinmapping. Reads from the "tinyX4_reverse" board variant.

Why? Tldr: the mapping has changed over time, and newer projects generally use CW mapping. It's odd that pio defaults to CCW. Adding this to the list would at least make it easily (and EXPLICITLY, no ambiguity) selectable, without breaking pre-existing projects by modifying the original "attiny84" board.

See https://github.com/SpenceKonde/ATTinyCore/blob/v2.0.0-devThis-is-the-head-submit-PRs-against-this/avr/extras/ATtiny_x4.md for details on why this pinmapping exists.

Found it exhausting to manually add the variant every time despite it pre-existing in the boards list.

valeros commented 1 month ago

Hi @nickyonge, thanks for the PR. I'd recommend renaming the board manifest to a more concise variant (e.g. attiny84_cw), while the name field in the manifest should have a more explicit name with pinout details (e.g. Generic ATtiny84 (clockwise pinout). Finally, the PR should be submitted to the develop branch.