Skip to content

Only disable timer interrupt instead of all interrutps in millis()/micros() #1442

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

Closed
wants to merge 1 commit into from
Closed

Conversation

chaveiro
Copy link
Contributor

In wiring lib, usualy a global interrupt disabled is issued when not realy needed.
It's very bad for timed interrupt sensitive designs
More info at: http://forum.arduino.cc/index.php?topic=168586.0

In wiring lib, usualy a global interrupt disabled is issued when not realy needed.
It's very bad for timed interrupt sensitive designs
More info at: http://forum.arduino.cc/index.php?topic=168864.0
@chaveiro
Copy link
Contributor Author

It also saves 2 bytes of program code with this fix.

@ffissore ffissore added New and removed New labels Feb 27, 2014
@cmaglie cmaglie added feature request A request to make an enhancement (not a bug fix) Architecture: AVR Applies only to the AVR microcontrollers (Uno, etc.) labels Apr 15, 2015
@matthijskooijman matthijskooijman changed the title Update wiring.c Only disable timer interrupt instead of all interrutps in millis()/micros() Nov 23, 2016
@matthijskooijman
Copy link
Collaborator

Looking through old PR's, I think this one is actually worth merging. It does need to be rebased and needs a better commit message, though.

@facchinm
Copy link
Member

If someone wants to adopt this PR, please open a new one using this link https://github.com/arduino/ArduinoCore-avr/compare/pr_1442?expand=1 , thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Architecture: AVR Applies only to the AVR microcontrollers (Uno, etc.) 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
Development

Successfully merging this pull request may close these issues.

5 participants