[coreboot-gerrit] Patch merged into coreboot/master: 7234d60 libpayload: usb: Fix up usb_shutdown() code paths

gerrit at coreboot.org gerrit at coreboot.org
Thu Nov 13 06:22:46 CET 2014


the following patch was just integrated into master:
commit 7234d60a6e6ee3a3403865ad1469105ff1d92c4e
Author: Julius Werner <jwerner at chromium.org>
Date:   Tue Apr 8 12:54:25 2014 -0700

    libpayload: usb: Fix up usb_shutdown() code paths
    
    This patch combines a few minor fixes and refactoring to the various
    host controller and root hub drivers to ensure they all do the right
    thing on a call to usb_exit(). It puts a usb_detach_device(0) call
    into detach_controller() so that the HCD doesn't need to remember to
    tear down the root hub itself, and makes sure all root hubs properly
    detach the subtree of devices connected to their ports first (as
    generic_hub and by extension XHCI had already been doing).
    
    It also fixes up some missing free() calls and replaces most 'ptr =
    malloc(); if (!ptr) fatal()' idioms with the new x(z)alloc().
    
    BUG=chromium:343415
    TEST=Tested EHCI on Big and OHCI, EHCI, and XHCI on Snow. Could not test
    UHCI (unless anyone volunteers to port coreboot to a ZGB? ;) ), but the
    changes are really tame.
    
    Original-Change-Id: I6eca51ff2685d0946fe4267ad7d3ec48ad7fc510
    Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/193731
    Original-Reviewed-by: Stefan Reinauer <reinauer at chromium.org>
    (cherry picked from commit 5791b546e5a21a360d0c65888a5b92d5f48f8178)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I00138f0aeceb12ed721f7368c7788c9b6bee227d
    Reviewed-on: http://review.coreboot.org/7222
    Tested-by: build bot (Jenkins)
    Reviewed-by: David Hendricks <dhendrix at chromium.org>


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

-gerrit



More information about the coreboot-gerrit mailing list