bugAVR C Runtime Library - Bugs: bug #25300, Additional i/o port names

 
 

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

bug #25300: Additional i/o port names

Submitter:  None
Submitted:  Sun 11 Jan 2009 11:46:27 AM UTC
   
 
Category:  Feature Request Severity:  1 - Wish
Priority:  6 Item Group:  Header files
Status:  None Assigned to:  None
Percent Complete:  50% Originator Email:  -email is unavailable-
Open/Closed:  Open Release:  Any
Fixed Release:  None

Jump to the original submission

Sat 19 Dec 2009 10:22:40 AM UTC, comment #7: 

Hi, I'm successfully using your patch, thanks for your work! When do you think this can be included in some release?

Grygoriy Fuchedzhy <gry>
Tue 28 Apr 2009 01:06:53 PM UTC, comment #6: 

I reworked the header for ATmega640/1280/2560/641/1281/2561 so that when a 64pin device is use, missing pin definition are not declared. It use the same mechanism that was already in place in that file.

This patch also fix issue where USART Rx 0 and XCLK where missing the '0'.

This patch against HEAD and not against the previously committed patch.

(file #18037)

Frédéric Nadeau <fnadeau>
Fri 17 Apr 2009 05:58:26 PM UTC, comment #5: 

As a side note...

Even if the patch include modification to the script, anyone attempting to regenerate the patch using the script will fail to do so since I modified some AVR Studio XML file that were bogus.

Just so it does not confuse anyone.

Frédéric Nadeau <fnadeau>
Fri 17 Apr 2009 05:53:35 PM UTC, comment #4: 

Hi,

Latest patch against HEAD,
Solve:
- Duplicate issue
- Now Output Compare that are Inverted are named OCXY_N:
#define OC1D_N_DDR  DDRB
#define OC1D_N_PORT PORTB
#define OC1D_N_PIN  PINB
#define OC1D_N_BIT  4

Comments are welcome.

(file #17941)

Frédéric Nadeau <fnadeau>
Mon 30 Mar 2009 06:41:29 PM UTC, comment #3: 

I have attaced a newly generated patch against HEAD

vector now have number in the following format:
XXXXX_vect_num           4
XXXXX_vect              _VECTOR(4)

pin definition looks like:
#define XX_DDR  DDRD
#define XX_PORT PORTD
#define XX_PIN  PIND
#define XX_BIT  5

Patch to the xml folder is also included. This has helped to generate the new io header and I don't beleived it is used for anything else.

(file #17837)

Frédéric Nadeau <fnadeau>
Thu 26 Mar 2009 08:23:03 PM UTC, comment #2: 

I'm including 2 files:
io.patch
xml.patch

added pin definition
added an INTERRUPT_num

TODO:
Some device have no pin definition since XML files lack information.
Some device like ATmega1280/1281 use the same io file, however the 1281 has less pin, should consider using the already existing _ATmegaxx1_ and _ATmegaxx0_ define
Some device may contain errors since the XML are crappy


(file #17813, file #17814)

Frédéric Nadeau <fnadeau>
Tue 13 Jan 2009 09:16:38 AM UTC, comment #1: 

Patches welcome, as doing this change requires modifying a lot of header files.

Eric Weddington <arcanum>
Group administrator
Sun 11 Jan 2009 11:46:27 AM UTC, original submission:  

It's a common thing when single pin on a device represents several features, for example atmega8:
PORTB:
  PB5 can be used as regular i/o port or SCK for SPI interface
  PB4 can be used as regular i/o port or MISO for SPI interface
  PB3 can be used as regular i/o port or MOSI for SPI interface
  PB2 can be used as regular i/o port or SS for SPI interface
PORTD:
  PD0 can be used as regular i/o port or RX for usart
  PD1 can be used as regular i/o port or TX for usart
etc...
Almost every pin have several meanings. So if I'm trying to build software wich communicates using for example SPI interface, i have to use some of i/o ports in another meaning(write logical 1/0 to SS pin to select/deselect slave device) So i think it's a good idea to put such definitions to headers:

for example:
#define SS   PB2
#define MOSI PB3
#define MISO PB4
#define SCK  PB5
#define SPI_PORT PORTB


#define RX PD0
#define TX PD1
#define USART_PORT PORTD

Anonymous

 

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

Attached Files
file #18037:  iomxx0_1.patch added by fnadeau (21KiB - application/octet-stream - iomxx0_1,h patch against HEAD)
file #17941:  io.patch.bz2 added by fnadeau (25KiB - application/x-bzip)
file #17837:  io.patch.bz2 added by fnadeau (25KiB - application/x-bzip)
file #17813:  io.patch added by fnadeau (479KiB - application/octet-stream - patch to the include/avr directory and to the xml(code that help generate the patched io files))
file #17814:  xml.patch added by fnadeau (21KiB - application/octet-stream - patch to the include/avr directory and to the xml(code that help generate the patched io files))

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gry (Posted a comment)
  • -email is unavailable- added by fnadeau (Updated the item)
  • -email is unavailable- added by arcanum (Posted a comment)
  •  

    Follow 13 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-11-16 arcanum Priority9 - Immediate 6
    2012-11-16 arcanum Assigned toarcanum None
        Originator Email -email is unavailable-
    2009-04-28 fnadeau Attached File- Added iomxx0_1.patch, #18037
    2009-04-17 fnadeau Attached File- Added io.patch.bz2, #17941
    2009-03-30 arcanum Percent Complete0% 50%
        Assigned toNone arcanum
    2009-03-30 arcanum Priority1 - Later 9 - Immediate
    2009-03-30 fnadeau Attached File- Added io.patch.bz2, #17837
    2009-03-26 fnadeau Attached File- Added io.patch, #17813
        Attached File- Added xml.patch, #17814
    2009-01-13 arcanum Severity3 - Normal 1 - Wish
        Priority5 - Normal 1 - Later

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code