bugAVR C Runtime Library - Bugs: bug #49150, Incorrect bit definitions in...

 
 

bug #49150: Incorrect bit definitions in io90pwm3b.h

Submitted by:  John Kasunich <jmkasunich>
Submitted on:  Wed 21 Sep 2016 04:04:56 PM UTC  
 
Category: HeaderSeverity: 4 - Important
Priority: 5 - NormalItem Group: Header files
Status: ConfirmedPercent Complete: 0%
Assigned to: NoneOpen/Closed: Open
Release: 2.0.0Fixed Release: None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Wed 21 Sep 2016 10:49:56 PM UTC, comment #1:

Yes, it appears that these are incorrect. Thank you for your report.

Chris Pavlina <c4757p>
Project Member
Wed 21 Sep 2016 04:04:56 PM UTC, original submission:

Two incorrect bit definitions in the General Timer/Counter Control Register GTCCR:

According to the Atmel datasheet (document 4317, section 13.4), ICPSEL1 is bit 6, and TSM is bit 7.

According to io90pwm3b.h, lines 261 & 262, those bits are 2 and 3 respectively.

I have confirmed that ICPSEL1 needs to be 6 by testing on actual hardware.

The incorrect definitions are present in version 2.0 and have been present since the file was added. Odd that it hasn't been noticed before, although it does require special circumstances: user must be doing captures with timer 1, and must be using capture input B, pin PB6. (The default is to use input A, pin PD4.)

John Kasunich <jmkasunich>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by c4757p (Posted a comment)
  • -unavailable- added by jmkasunich (Submitted the item)
  •  

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 21 Sep 2016 10:49:56 PM UTCc4757pSeverity3 - Normal=>4 - Important
      StatusNone=>Confirmed
      ReleaseAny=>2.0.0

    Back to the top


    Powered by Savane 3.1-cleanup1