Skip to content

Adjust wiring_analog.c for Attiny841/441 #6643

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.

Sign up for GitHub

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

Jump to bottom
Closed
mschlenker opened this issue Aug 21, 2017 · 2 comments
Closed

Adjust wiring_analog.c for Attiny841/441 #6643

mschlenker opened this issue Aug 21, 2017 · 2 comments
Labels
Component: Core Related to the code for the standard Arduino API feature request A request to make an enhancement (not a bug fix)

Comments

@mschlenker
Copy link
Contributor

Attiny841/44 with 12+5 ADCs use ADMUXA and ADMUXB instead of ADMUX to select analog reference and channel. Just look at hardware/tools/avr/avr/include/avr/iotn841.h and hardware/tools/avr/avr/include/avr/iotn441.h.

The patch attached properly initializes ADMUXA and ADMUXB to make ADC usable, tested with Attiny841, thanks @awatterott for the hint.

wiring_analog_attiny841.txt
@per1234
Copy link
Collaborator

per1234 commented Aug 21, 2017

If you're proposing a code change it would be better to submit a pull request.

@per1234 per1234 added Component: Core Related to the code for the standard Arduino API feature request A request to make an enhancement (not a bug fix) labels Aug 21, 2017
@mschlenker
Copy link
Contributor Author

mschlenker commented Aug 21, 2017 •
edited
Loading

@per1234 per1234, thank you for the hint:
#6644

@per1234 per1234 closed this as completed Aug 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component: Core Related to the code for the standard Arduino API feature request A request to make an enhancement (not a bug fix)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mschlenker @per1234