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

Nico Huber (nico.huber@secunet.com) gerrit at coreboot.org
Thu Nov 8 12:27:27 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

-gerrit

commit 98af66eb4c774fe7e97b3b4907bfa0c12fd6a49c
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
    perform some kind of bus reset while this does a controller reset.
    
    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