83xx immap_qe.h -> SIR type def error?

[Kumar Gala]

On Feb 1, 2008, at 5:47 AM, Russell McGuire wrote:

> All Freescale,
>
> Not sure if this is the place to post this, but I have run across  
> what I
> consider to be a possible type error in the immap_qe.h file, for the
> asm/powerpc branch.
>
> In the file immap_qe.h
>
> /* SI Routing Tables */
> struct sir {
> 	u8  tx[0x400];
> 	u8  rx[0x400];
> 	u8  res0[0x800];
> }
>
> Shouldn't these types be defined as __be16 ?
>
> According to the Freescale manual this is a 16 bit field, not an 8-bit
> field.
>
> Spent an hour trying to figure out why I couldn't fill this field  
> out with
> upper 8 bits last night.
>
> Thoughts?

I'm guessing it was done this way since they are just looked as base  
offsets.  Where in the UM do you see anything about them being 16-bit  
quantities?  (I'm really know little about this).

- k