[U-Boot] [PATCH 2/3] pxa25x: Add UDC registers definitions
Łukasz Dałek
luk0104 at gmail.com
Fri Aug 17 23:44:38 CEST 2012
On 17.08.2012 22:50, Marek Vasut wrote:
> Dear Łukasz Dałek,
>
>> + /* UDC Endpoint Control/Status Registers */
>> + uint32_t udccs0; /* 0x010 */
>> + uint32_t udccs1; /* 0x014 */
>> + uint32_t udccs2; /* 0x018 */
>> + uint32_t udccs3; /* 0x01c */
>> + uint32_t udccs4; /* 0x020 */
>> + uint32_t udccs5; /* 0x024 */
>> + uint32_t udccs6; /* 0x028 */
>> + uint32_t udccs7; /* 0x02c */
>> + uint32_t udccs8; /* 0x030 */
>> + uint32_t udccs9; /* 0x034 */
>> + uint32_t udccs10; /* 0x038 */
>> + uint32_t udccs11; /* 0x03c */
>> + uint32_t udccs12; /* 0x040 */
>> + uint32_t udccs13; /* 0x044 */
>> + uint32_t udccs14; /* 0x048 */
>> + uint32_t udccs15; /* 0x04c */
> Use array here maybe? ;-)
Yes, array is a good idea.
>> + uint32_t uddr9; /* 0x900 */
>> + uint8_t reserved14[508];
>> + uint32_t uddr11; /* 0xb00 */
>> + uint8_t reserved15[124];
>> + uint32_t uddr12; /* 0xb80 */
>> + uint8_t reserved16[124];
>> + uint32_t uddr13; /* 0xc00 */
>> + uint8_t reserved17[508];
>> + uint32_t uddr14; /* 0xe00 */
>> +
>> +} __packed;
> You don't need this.
Why? GCC on ARM doesn't align structures?
>> +#define UDCCFR_AREN (1<< 7) /* ACK response enable (now) */
>> +#define UDCCFR_ACM (1<< 2) /* ACK control mode (wait for AREN) */
>> +/* latest pxa255 errata define new "must be one" bits in UDCCFR */
>> +#define UDCCFR_MB1 (0xff& ~(UDCCFR_AREN | UDCCFR_ACM))
> What errata and where? Please document it in the comment. Also, is the register
> only 8 bit wide?
Ok, I will document. All registers are 32 bits wide. But some you have to
access like they are 8 bits wide.
Łukasz Dałek
More information about the U-Boot
mailing list