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 | Assigned to: | aboyapati |
Percent Complete: | 100% | 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 |
Powered by Savane 3.13-f8d8.
Corresponding source code
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)