Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use PD4 for row0 for matrix rather than PC6 #22

Open
josefadamcik opened this issue Jan 5, 2021 · 2 comments
Open

Use PD4 for row0 for matrix rather than PC6 #22

josefadamcik opened this issue Jan 5, 2021 · 2 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@josefadamcik
Copy link
Owner

The pins are next to each other and PD4 is currently unused. But PC6 (which is used for row0) supports PWM output (Timer3 ouput) so it might be usefull if somebody want's to add a speaker.

It might be also nice to have some pad for this pin somwhere in a similar way as for RGB pin PD3.

Datasheet: https://cdn-shop.adafruit.com/datasheets/atmel-7766-8-bit-avr-atmega16u4-32u4_datasheet.pdf

@josefadamcik josefadamcik added the enhancement New feature or request label Jan 5, 2021
@josefadamcik josefadamcik self-assigned this Jan 5, 2021
@harshitgoel96
Copy link

Hi. Let me know if you need help on this. I can make the change and submit a PR.

@josefadamcik
Copy link
Owner Author

@harshitgoel96

Hi. Let me know if you need help on this. I can make the change and submit a PR.

Thank you but no, not really. This is technically a breaking change (would brake firmware) so it's not going to happen for any of the current versions, it's waiting for potential version 3.

@josefadamcik josefadamcik added this to the Version 3 milestone Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants