Forums

MC56F835x.h SCI0 & SCI wrong in CW 6.0.1

Started by Corey, Rick June 15, 2004
Hello

John Wieners, one of our engineers, just found a typo in MC56F835x.h (a list
of register address #defines) in the directory :
C:\Program
Files\Metrowerks\CW6\Stationery\DSP56800x_Examples\MC56F8357\isr_led
and presumably in other .h files. This is in CW 6.0.1.

The .h file doesn't allow for the fact that SCIx_BASE+2 is reserved, at
least for the 56F8357.
SCI1 has the same error as SCI2.

wrong:

// Serial Communications Interface 0 (SCI0)
#define SCI0_BASE 0x00F280
#define SCI0_BR (SCI0_BASE + 0)
#define SCI0_CR (SCI0_BASE + 1)
#define SCI0_SR (SCI0_BASE + 2) // <-- should be + 3
#define SCI0_DR (SCI0_BASE + 3) // <-- should be + 4 Rick Corey
Senior Software Engineer
DPC Instrument Systems Division



Hi Rick and All,

It looks like they may have mixed up the 5683xx SCI stuff with the
56F80x SCI stuff. I had just downloaded the latest 56F80x User's
Manual and it shows the SCISR and SCIDR as Base + $3 and $4
respectively when it should be Base + $2 and $3.

Pete --- In , "Corey, Rick" <rcorey@d...> wrote:
> Hello
>
> John Wieners, one of our engineers, just found a typo in
MC56F835x.h (a list
> of register address #defines) in the directory :
> C:\Program
> Files\Metrowerks\CW6\Stationery\DSP56800x_Examples\MC56F8357\isr_led
> and presumably in other .h files. This is in CW 6.0.1.
>
> The .h file doesn't allow for the fact that SCIx_BASE+2 is
reserved, at
> least for the 56F8357.
> SCI1 has the same error as SCI2.
>
> wrong:
>
> // Serial Communications Interface 0 (SCI0)
> #define SCI0_BASE 0x00F280
> #define SCI0_BR (SCI0_BASE + 0)
> #define SCI0_CR (SCI0_BASE + 1)
> #define SCI0_SR (SCI0_BASE + 2) // <-- should be + 3
> #define SCI0_DR (SCI0_BASE + 3) // <-- should be + 4 > Rick Corey
> Senior Software Engineer
> DPC Instrument Systems Division