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

fix delayMicroseconds() implementation #4

Merged
merged 2 commits into from
Mar 22, 2024
Merged

Conversation

2bndy5
Copy link
Owner

@2bndy5 2bndy5 commented Mar 22, 2024

This affects linux_kernel, mraa, and pigpio drivers (on linux).

It was a simple math operator mistake that resulted in no delay if the specified microseconds was less than 1000000.

@2bndy5 2bndy5 added the bug Something isn't working label Mar 22, 2024
This affects linux_kernel, mraa, and pigpio drivers (on linux)

It was a simple math operator mistake that resulted in no delay if the specified microseconds was less than 1000000.
@2bndy5 2bndy5 force-pushed the fix-delayMicroseconds branch from b94a767 to 36e17e0 Compare March 22, 2024 05:05
Copy link

Memory usage change @ 0393d83

Board flash % RAM for global variables %
arduino:samd:mkrzero 0 - 0 0.0 - 0.0 0 - 0 0.0 - 0.0
Click for full report table
Board examples/absolute_mode
flash
% examples/absolute_mode
RAM for global variables
% examples/anymeas_mode
flash
% examples/anymeas_mode
RAM for global variables
% examples/relative_mode
flash
% examples/relative_mode
RAM for global variables
%
arduino:samd:mkrzero 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0 0 0.0
Click for full report CSV
Board,examples/absolute_mode<br>flash,%,examples/absolute_mode<br>RAM for global variables,%,examples/anymeas_mode<br>flash,%,examples/anymeas_mode<br>RAM for global variables,%,examples/relative_mode<br>flash,%,examples/relative_mode<br>RAM for global variables,%
arduino:samd:mkrzero,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0

@2bndy5 2bndy5 merged commit 59a61a5 into master Mar 22, 2024
59 checks passed
@2bndy5 2bndy5 deleted the fix-delayMicroseconds branch March 22, 2024 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant