RE: [avr-libc-dev] Re: [bug #25300] Additional i/o port names

[Weddington, Eric]

> -----Original Message-----
> From: 
> address@hidden 
> [mailto:address@hidden
> org] On Behalf Of Frédéric Nadeau
> Sent: Tuesday, March 31, 2009 3:59 PM
> To: address@hidden
> Subject: Re: [avr-libc-dev] Re: [bug #25300] Additional i/o port names
> 
> 2009/3/31 Weddington, Eric <address@hidden>:
> > Ideally, we need both: we need to fix the patch to the 
> header files so that the definitions are in uppercase, but we 
> need to generate a list of what was wrong with the XML files 
> to give to Atmel.
> >
> 
> Making the script to generate all in upper case is a piece of cake.
> However having the script to report all the error is more or less
> complicated. I'll see what I can do about that

Something else I just found out:
I was testing my generator script with the ATmega6490 and it reported back that 
there was duplicate definitions with the new pin definitions:

#define SEG33_DDR   DDRJ
#define SEG33_PORT  PORTJ
#define SEG33_PIN   PINJ
#define SEG33_BIT   1

#define SEG33_DDR   DDRG
#define SEG33_PORT  PORTG
#define SEG33_PIN   PING
#define SEG33_BIT   3

I checked and it seems that the XML file is incorrect. :-P

This duplicated data is in your patch too, unfortunately.

So when you generate the new definitions, you'll need to check to make sure 
that there are no duplicate definitions. If there are, you'll have to verify 
which one is correct and probably fix the header by hand.

Eric Weddington