[U-Boot] TI Davinci and MUSB

[Wolfgang Denk]

Dear "Gupta, Ajay Kumar",

In message <19F8576C6E063C45BE387C64729E7394044A775FE3 at dbde02.ent.ti.com> you wrote:
> 
> >     u32 reserved[0x20];
> >     u32 intclrr;
> >     u32 intmskr;
> >     u32 intmsksetr;
> > };
> > 
> > Shouldn't this be reserved[0x20/4] ?
> 
> Yes, this is a bug. Though the correct fix is,
> - 	u32 reserved[0x20];
> +	u32 reserved[5];

Umm...

> Please refer musb Davinci (DM6446) document (page-74) at,
> http://focus.ti.com/lit/ug/sprue35f/sprue35f.pdf
> 
> -->4h CTRLR Control Register Section 4.1
> 8h STATR Status Register Section 4.2
> 10h RNDISR RNDIS Register Section 4.3
> 14h AUTOREQ Autorequest Register Section 4.4
> 20h INTSRCR USB Interrupt Source Register Section 4.5
> 24h INTSETR USB Interrupt Source Set Register Section 4.6
> -->28h INTCLRR USB Interrupt Source Clear Register

Why don't we add these fields to he struct above if they are not
actually reserved, but have a documented use?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
panic: kernel trap (ignored)