bugAVR C Runtime Library - Bugs: bug #40069, Missing Pin Change Interrupt...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #40069: Missing Pin Change Interrupt Vector for Attiny 2313a

Submitter:  None
Submitted:  Thu 19 Sep 2013 03:00:40 PM UTC
   
 
Category:  Library Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Header files
Status:  Fixed Assigned to:  pitchumani
Percent Complete:  0% Originator Email:  -email is unavailable-
Open/Closed:  Closed Release:  1.8.0
Fixed Release:  1.8.1

Mon 11 Aug 2014 10:25:20 PM UTC, comment #2: 

Has already been fixed in SVN r2412.

Joerg Wunsch <joerg_wunsch>
Group administrator
Thu 19 Sep 2013 03:13:15 PM UTC, comment #1: 

This is obviously due them being named in the XML file that
way:


$ fgrep PCINT ATtiny2313A.xml
        <interrupt index="11" name="PCINT_B" caption="Pin Change Interrupt Request B"/>
        <interrupt index="19" name="PCINT_A" caption="Pin Change Interrupt Request A"/>
        <interrupt index="20" name="PCINT_D" caption="Pin Change Interrupt Request D"/>
          <bitfield caption="Pin Change Interrupt Masks" mask="0x7F" name="PCINT" lsb="11"/>
          <bitfield caption="Pin Change Interrupt Masks" mask="0x07" name="PCINT" lsb="8"/>


But I agree, the datasheet mentions them as PCINT0, PCINT1,
and PCINT2, respectively, so our header file should follow
that.

Please file a bug report at Atmel for the discrepancy between
the datasheet and the XML file.

The compiler cannot generate an error for a misspelled vector
name, but it is supposed to generate a warning.

Joerg Wunsch <joerg_wunsch>
Group administrator
Thu 19 Sep 2013 03:00:40 PM UTC, original submission:  

The om Interrupt Vectors are missing from the libc documentation. This would be important, as the pcints are defined in an inconsistent way! Rather than PCINT1_vec oder PCInt0_vec they have been defined as PCInt_A_vec, PCINT_B_vec and PCINT_D_vec corresponding to the ports involved. This is different to all other implementations, and, sadly, calling PCInt0_vec will be accepted by the compiler and cause a mc reset - strange!

I think this behaviour should be reflected in the libc documentation!

Kind regards,

S. Levedag

Anonymous

 

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

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by joerg_wunsch (Posted a comment)
  • -email is unavailable- added by None (Submitted the item)
  •  

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-08-11 joerg_wunsch StatusConfirmed Fixed
        Assigned toNone pitchumani
        Open/ClosedOpen Closed
        Fixed ReleaseNone 1.8.1
    2013-09-19 joerg_wunsch Item GroupDocumentation Header files
    2013-09-19 joerg_wunsch StatusNone Confirmed

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code