the following patch was just integrated into master:
commit 89461d88a0e163ab5fee96908a4195035fe800f5
Author: Steven A. Falco <[email protected]>
Date:   Fri Jul 15 21:44:35 2011 -0400

    port_enable and port_reset must change atomically.
    
    I have observed two separate EHCI host bridges that do not tolerate
    using C bit-fields to directly manipulate the portsc_t register.  The
    reason for this is that the EHCI spec says that port_enable must go
    to 0 at the time that port_reset goes to 1.  Naturally this cannot be
    done using direct bit-field manipulation.  Instead, we use a temporary
    variable, change the bit-fields there, then atomically write the new
    value back to the hardware.
    
    Signed-off-by: Steven A. Falco <[email protected]>
    Change-Id: If138faee43e0293efa203b86f7893fdf1e811269

See http://review.coreboot.org/101 for details.

-gerrit

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

Reply via email to