AVR C Runtime Library - Bugs: bug #30363, _delay_xx() functions in...
You are not allowed to post comments on this tracker with your current authentication level.
bug #30363: _delay_xx() functions in <util/delay.h> are broken
Submitter: | Bill Perry <bperrybap> | ||
Submitted: | Mon 05 Jul 2010 10:34:10 PM UTC | ||
Category: | Header | Severity: | 3 - Normal |
Priority: | 5 - Normal | Item Group: | Header files |
Status: | Fixed | Percent Complete: | 100% |
Assigned to: | aboyapati | Open/Closed: | Closed |
Release: | 1.7.* | Fixed Release: | None |
Jump to the original submission
Wed 13 Oct 2010 09:38:06 AM UTC, comment #15: |
Anitha Boyapati <aboyapati>![]() |
Mon 11 Oct 2010 02:26:18 PM UTC, comment #14: Updated patch attached. Please review. |
Anitha Boyapati <aboyapati>![]() |
Thu 07 Oct 2010 10:15:49 PM UTC, comment #13: My guess is that many people either don't use these
|
Bill Perry <bperrybap> |
Thu 07 Oct 2010 01:34:32 PM UTC, comment #12: Bill Perry:
...
|
Anitha Boyapati <aboyapati>![]() |
Thu 07 Oct 2010 09:45:06 AM UTC, comment #11:
|
Anitha Boyapati <aboyapati>![]() |
Thu 07 Oct 2010 12:14:26 AM UTC, comment #10: Other than loop/cycle count truncation was the previous way of doing things, what is the reasoning for preferring truncation to be the new default vs "round up" as the default going forward?
|
Bill Perry <bperrybap> |
Wed 06 Oct 2010 09:43:57 AM UTC, comment #9:
|
Anitha Boyapati <aboyapati>![]() |
Tue 05 Oct 2010 06:57:01 PM UTC, comment #8: While the fix is "ok". it doesn't solve some of the
|
Bill Perry <bperrybap> |
Tue 05 Oct 2010 10:53:36 AM UTC, comment #7: Patch attached. Build verified. Please review. |
Anitha Boyapati <aboyapati>![]() |
Mon 04 Oct 2010 07:13:58 AM UTC, comment #6: Bill Perry's patch doesn't define the following cases:
|
Anitha Boyapati <aboyapati>![]() |
Wed 21 Jul 2010 06:25:31 PM UTC, comment #5: Eric,
|
Bill Perry <bperrybap> |
Mon 19 Jul 2010 05:27:35 AM UTC, comment #4:
|
Joerg Wunsch <joerg_wunsch>![]() |
Sun 18 Jul 2010 11:19:27 PM UTC, comment #3: But should <util/delah.h> be using autonconfig information directly to determine if __built_avr_delay_cycles() is available?
|
Bill Perry <bperrybap> |
Sun 18 Jul 2010 09:28:04 PM UTC, comment #2:
|
Joerg Wunsch <joerg_wunsch>![]() |
Fri 16 Jul 2010 07:07:18 PM UTC, comment #1: As a followup comment
|
Bill Perry <bperrybap> |
Mon 05 Jul 2010 10:34:10 PM UTC, original submission:
WIth the conversion to using __builtin_delay_cycles() the _delay_us() and _delay_ms() functions in <util/delay.h> were broken.
|
Bill Perry <bperrybap> |
Depends on the following items: None found
Items that depend on this one: None found
Follow 15 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2011-02-15 | aboyapati | Discussion Lock | Locked | ![]() |
None |
2011-02-15 | aboyapati | Status | In Progress | ![]() |
Fixed |
Discussion Lock | None | ![]() |
Locked | ||
2010-10-13 | aboyapati | Percent Complete | 90% | ![]() |
100% |
Open/Closed | Open | ![]() |
Closed | ||
2010-10-13 | aboyapati | Attached File | - | ![]() |
Added bug30363.v3.patch, #21669 |
2010-10-11 | aboyapati | Attached File | - | ![]() |
Added bug30363.v2.patch, #21657 |
2010-10-06 | aboyapati | Status | Fixed | ![]() |
In Progress |
2010-10-05 | aboyapati | Attached File | - | ![]() |
Added bug30363.patch, #21625 |
Status | Confirmed | ![]() |
Fixed | ||
Percent Complete | 0% | ![]() |
90% | ||
2010-09-16 | aboyapati | Status | None | ![]() |
Confirmed |
Assigned to | None | ![]() |
aboyapati | ||
2010-07-18 | joerg_wunsch | Item Group | libc code | ![]() |
Header files |
2010-07-05 | bperrybap | Attached File | - | ![]() |
Added delay.h.in, #20904 |
Final patch, version 3 attached.
Previous patch has the issue that if no optimization is used when building the application with the compiler that has builtin_avr_delay_cycles() support, no code is produced. This patch fixes the issue. Patch tested.
(file #21669)