[coreboot-gerrit] New patch to review for coreboot: 7e4028b libpayload: EHCI - detach controller in ehci_shutdown()

Mathias Krause (minipli@googlemail.com) gerrit at coreboot.org
Mon Mar 25 12:10:22 CET 2013

Mathias Krause (minipli at googlemail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2899


commit 7e4028b07c143ac0411ac4b5a0b51159076eeda5
Author: Mathias Krause <minipli at googlemail.com>
Date:   Sun Mar 24 19:40:01 2013 +0100

    libpayload: EHCI - detach controller in ehci_shutdown()
    It shouldn't be used any more as we're about to free() the memory behind
    the controller -- therefore detach it.
    Change-Id: I875322a9940570c51d412a7f3bfb6af4ea3b3764
    Signed-off-by: Mathias Krause <minipli at googlemail.com>
 payloads/libpayload/drivers/usb/ehci.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/payloads/libpayload/drivers/usb/ehci.c b/payloads/libpayload/drivers/usb/ehci.c
index 972962c..3e5de1d 100644
--- a/payloads/libpayload/drivers/usb/ehci.c
+++ b/payloads/libpayload/drivers/usb/ehci.c
@@ -170,6 +170,7 @@ static int ehci_set_periodic_schedule(ehci_t *ehcic, int enable)
 static void ehci_shutdown (hci_t *controller)
+	detach_controller(controller);
 	/* Make sure periodic schedule is disabled */
 	ehci_set_periodic_schedule(EHCI_INST(controller), 0);
 	/* Free periodic frame list */

More information about the coreboot-gerrit mailing list