Patrick Georgi ([email protected]) just uploaded a new patch set to 
gerrit, which you can find at http://review.coreboot.org/155

-gerrit

commit 193878daff4270d483257bea167e324e95b3ddec
Author: Patrick Georgi <[email protected]>
Date:   Tue Aug 16 15:47:15 2011 +0200

    libpayload: EHCI registers are volatile
    
    Some gcc versions seem to honor volatile at different places in a
    struct declaration.
    
    Change-Id: I0df2a3fb2eff4cee8cc1b8ac15d9cd9b86178752
    Signed-off-by: Patrick Georgi <[email protected]>
---
 payloads/libpayload/drivers/usb/ehci_private.h |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/payloads/libpayload/drivers/usb/ehci_private.h 
b/payloads/libpayload/drivers/usb/ehci_private.h
index e63a81d..f4890cc 100644
--- a/payloads/libpayload/drivers/usb/ehci_private.h
+++ b/payloads/libpayload/drivers/usb/ehci_private.h
@@ -37,7 +37,7 @@
 #define FLADJ 0x61
 #define FLADJ_framelength(x) (((x)-59488)/16)
 
-typedef union {
+typedef volatile union {
        u32 val;
        volatile struct {
                unsigned long current_conn_status:1;
@@ -62,7 +62,7 @@ typedef union {
        } __attribute__ ((packed));
 } __attribute__ ((packed)) portsc_t;
 
-typedef struct {
+typedef volatile struct {
        u8 caplength;
        u8 res1;
        u16 hciversion;
@@ -117,7 +117,7 @@ typedef struct {
        };
 } __attribute__ ((packed)) hc_cap_t;
 
-typedef struct {
+typedef volatile struct {
        union {
                u32 usbcmd;
                volatile struct {
@@ -174,7 +174,7 @@ typedef struct {
        portsc_t portsc[0];
 } hc_op_t;
 
-typedef struct {
+typedef volatile struct {
        union {
                u32 next_qtd;
                struct {
@@ -223,7 +223,7 @@ typedef struct {
        u32 bufptrs64[5];
 } __attribute__ ((packed)) qtd_t;
 
-typedef struct {
+typedef volatile struct {
        union {
                u32 horiz_link_ptr;
                struct {

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to