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

Add support for Atmega32A to pin declarations and universal matrix #4015

Merged
merged 8 commits into from
Nov 2, 2018

Conversation

jackhumbert
Copy link
Member

No description provided.

@jackhumbert
Copy link
Member Author

jackhumbert commented Sep 29, 2018

@mechmerlin can you test this out on your panc60 when you get the chance? I was getting some weird results with my Atmega32a board, and wanted to make sure I wasn't doing something weird. You should be able to define the pins in the config.h file like usual.

Copy link
Contributor

@a-chol a-chol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jackhumbert : I merged your PR locally because we depend on it for #3615 , and also ended up with a weird behaviour : inputs read on A1 would also trigger keys on A2. The highlighted code shows the mistake. Apart from that, everything seems to be working fine. This should allow to remove a whole lot of code duplication amongst keybaords.
edit: btw, i tested on a bminiex, which is a ps2avrGB board using an ATMega32A.

#ifdef PORTA
#define A0 PINDEF(A, 0)
#define A1 PINDEF(A, 1)
#define A2 PINDEF(A, 1)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is copy-pasted from the previous one but should apply 2 as the pin instead of 1

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah! Thanks :)

@drashna
Copy link
Member

drashna commented Oct 22, 2018

Just a note:

Travis appears to be failing solely because of firmware size errors. Rebasing/merging would fix this issue, as this branch doesn't include the changes to fix those.

@jackhumbert jackhumbert requested a review from drashna October 27, 2018 00:27
Copy link
Member

@drashna drashna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@jackhumbert jackhumbert merged commit 15f6278 into master Nov 2, 2018
@jackhumbert jackhumbert deleted the atmega32a_matrix branch November 2, 2018 16:31
3n4rKy pushed a commit to 3n4rKy/qmk_firmware that referenced this pull request Nov 4, 2018
…mk#4015)

* add computed pins from mcu type

* update for atmega32a

* doc typo

* add atmega16 chips, link to references

* remove avr include from config

* exclude assembler in config.h includes

* consolodate options, add 646

* fix typo in pindef
zer09 pushed a commit to zer09/qmk_firmware that referenced this pull request Nov 10, 2018
…mk#4015)

* add computed pins from mcu type

* update for atmega32a

* doc typo

* add atmega16 chips, link to references

* remove avr include from config

* exclude assembler in config.h includes

* consolodate options, add 646

* fix typo in pindef
rseymour pushed a commit to rseymour/qmk_firmware that referenced this pull request Mar 13, 2019
…mk#4015)

* add computed pins from mcu type

* update for atmega32a

* doc typo

* add atmega16 chips, link to references

* remove avr include from config

* exclude assembler in config.h includes

* consolodate options, add 646

* fix typo in pindef
djthread pushed a commit to djthread/qmk_firmware that referenced this pull request Mar 17, 2019
…mk#4015)

* add computed pins from mcu type

* update for atmega32a

* doc typo

* add atmega16 chips, link to references

* remove avr include from config

* exclude assembler in config.h includes

* consolodate options, add 646

* fix typo in pindef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants