Dear "Gupta, Ajay Kumar",

In message <19f8576c6e063c45be387c64729e7394044a775...@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: w...@denx.de
panic: kernel trap (ignored)
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to