Reviewed-by: Star Zeng <star.z...@intel.com>

-----Original Message-----
From: Tian, Feng 
Sent: Monday, July 11, 2016 11:22 AM
To: Zeng, Star <star.z...@intel.com>
Cc: edk2-devel@lists.01.org
Subject: [patch 1/3] MdeModulePkg/UsbBus: remove redundant host controller 
reset in UsbBus

The Host Controller reset has been done at EDKII UHCI/EHCI/XHCI, The original 
code will do twice host controller initialization which is unnecessary. It also 
bring issues on some USB HCs.

Cc: Star Zeng <star.z...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Feng Tian <feng.t...@intel.com>
---
 MdeModulePkg/Bus/Usb/UsbBusDxe/UsbBus.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/MdeModulePkg/Bus/Usb/UsbBusDxe/UsbBus.c 
b/MdeModulePkg/Bus/Usb/UsbBusDxe/UsbBus.c
index ef002f5..7822022 100644
--- a/MdeModulePkg/Bus/Usb/UsbBusDxe/UsbBus.c
+++ b/MdeModulePkg/Bus/Usb/UsbBusDxe/UsbBus.c
@@ -2,7 +2,7 @@
 
     Usb Bus Driver Binding and Bus IO Protocol.
 
-Copyright (c) 2004 - 2013, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2004 - 2016, Intel Corporation. All rights reserved.<BR>
 This program and the accompanying materials  are licensed and made available 
under the terms and conditions of the BSD License  which accompanies this 
distribution.  The full text of the license may be found at @@ -976,9 +976,6 @@ 
UsbBusBuildProtocol (
     }
   }
 
-  UsbHcReset (UsbBus, EFI_USB_HC_RESET_GLOBAL);
-  UsbHcSetState (UsbBus, EfiUsbHcStateOperational);
-
   //
   // Install an EFI_USB_BUS_PROTOCOL to host controller to identify it.
   //
--
2.7.1.windows.2

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to