[coreboot] Patch set updated for coreboot: 2928dbe Revert "libpayload: Add reset function for OHCI USB driver"

Nico Huber (nico.huber@secunet.com) gerrit at coreboot.org
Thu Nov 8 12:50:25 CET 2012

Nico Huber (nico.huber at secunet.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1806


commit 2928dbed8445c436b46635cda57ca77564cbf60d
Author: Nico Huber <nico.huber at secunet.com>
Date:   Thu Nov 8 11:26:09 2012 +0100

    Revert "libpayload: Add reset function for OHCI USB driver"
    This reverts commit db2c400a.
    commit db2c400ab852a2aae09d812b20e520ca371da35b
    Author: Anton Kochkov <a.kochkov at securitycode.ru>
    Date:   Fri Jul 6 11:54:17 2012 +0400
        libpayload: Add reset function for OHCI USB driver
        Implemented OHCI reset function ohci_reset() in ohci.c
        for libpayload's USB driver.
    The given implementation of ohci_reset() doesn't work as expected. It
    leaves the controller in a non-functional state. Since the only place
    I can find where it's called is the end of ohci_init(), this cannot be
    what we want. I guess the original intent of ohci_reset() was to reset
    the controller to a working state while this does a controller reset
    without initializing it afterwards, leaving it unusable.
    Change-Id: I480407b5d5834517a8037ac5fddbdfa946d54927
    Signed-off-by: Nico Huber <nico.huber at secunet.com>
 payloads/libpayload/drivers/usb/ohci.c | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/payloads/libpayload/drivers/usb/ohci.c b/payloads/libpayload/drivers/usb/ohci.c
index 095eec9..d856665 100644
--- a/payloads/libpayload/drivers/usb/ohci.c
+++ b/payloads/libpayload/drivers/usb/ohci.c
@@ -49,13 +49,6 @@ static void ohci_process_done_queue(ohci_t *ohci, int spew_debug);
 static void
 ohci_reset (hci_t *controller)
-	if (controller == NULL)
-		return;
-	OHCI_INST(controller)->opreg->HcCommandStatus = HostControllerReset;
-	mdelay(2); /* wait 2ms */
-	OHCI_INST(controller)->opreg->HcControl = 0;
-	mdelay(10); /* wait 10ms */
 #ifdef USB_DEBUG

More information about the coreboot mailing list