public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH] MdeModulePkg Xhci: Correct description of Timeout param in XhciReg.h
@ 2017-09-07 11:13 Star Zeng
  2017-09-08  9:08 ` Ni, Ruiyu
  0 siblings, 1 reply; 2+ messages in thread
From: Star Zeng @ 2017-09-07 11:13 UTC (permalink / raw)
  To: edk2-devel; +Cc: Star Zeng, Alexei Fedorov, Ruiyu Ni

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=653

Correct description of Timeout param in XhciReg.h to be matched with
XhciReg.c.

Cc: Alexei Fedorov <Alexei.Fedorov@arm.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Star Zeng <star.zeng@intel.com>
---
 MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h | 10 +++++-----
 MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h |  4 ++--
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h b/MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h
index b748c8d39739..838a44628c27 100644
--- a/MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h
+++ b/MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h
@@ -2,7 +2,7 @@
 
   This file contains the register definition of XHCI host controller.
 
-Copyright (c) 2011 - 2013, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2011 - 2017, 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
@@ -399,7 +399,7 @@ XhcClearOpRegBit (
   @param  Offset       The offset of the operational register.
   @param  Bit          The bit of the register to wait for.
   @param  WaitToSet    Wait the bit to set or clear.
-  @param  Timeout      The time to wait before abort (in microsecond, us).
+  @param  Timeout      The time to wait before abort (in millisecond, ms).
 
   @retval EFI_SUCCESS  The bit successfully changed by host controller.
   @retval EFI_TIMEOUT  The time out occurred.
@@ -521,7 +521,7 @@ XhcIsSysError (
   Reset the XHCI host controller.
 
   @param  Xhc          The XHCI Instance.
-  @param  Timeout      Time to wait before abort (in microsecond, us).
+  @param  Timeout      Time to wait before abort (in millisecond, ms).
 
   @retval EFI_SUCCESS  The XHCI host controller is reset.
   @return Others       Failed to reset the XHCI before Timeout.
@@ -537,7 +537,7 @@ XhcResetHC (
   Halt the XHCI host controller.
 
   @param  Xhc          The XHCI Instance.
-  @param  Timeout      Time to wait before abort (in microsecond, us).
+  @param  Timeout      Time to wait before abort (in millisecond, ms).
 
   @return EFI_SUCCESS  The XHCI host controller is halt.
   @return EFI_TIMEOUT  Failed to halt the XHCI before Timeout.
@@ -553,7 +553,7 @@ XhcHaltHC (
   Set the XHCI host controller to run.
 
   @param  Xhc          The XHCI Instance.
-  @param  Timeout      Time to wait before abort (in microsecond, us).
+  @param  Timeout      Time to wait before abort (in millisecond, ms).
 
   @return EFI_SUCCESS  The XHCI host controller is running.
   @return EFI_TIMEOUT  Failed to set the XHCI to run before Timeout.
diff --git a/MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h b/MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h
index 1a6256066599..0297072ffd6b 100644
--- a/MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h
+++ b/MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h
@@ -1,7 +1,7 @@
 /** @file
 Private Header file for Usb Host Controller PEIM
 
-Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2014 - 2017, Intel Corporation. All rights reserved.<BR>
 
 This program and the accompanying materials
 are licensed and made available under the terms and conditions
@@ -287,7 +287,7 @@ XhcPeiClearOpRegBit (
   @param  Offset        The offset of the operational register.
   @param  Bit           The bit of the register to wait for.
   @param  WaitToSet     Wait the bit to set or clear.
-  @param  Timeout       The time to wait before abort (in microsecond, us).
+  @param  Timeout       The time to wait before abort (in millisecond, ms).
 
   @retval EFI_SUCCESS   The bit successfully changed by host controller.
   @retval EFI_TIMEOUT   The time out occurred.
-- 
2.7.0.windows.1



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] MdeModulePkg Xhci: Correct description of Timeout param in XhciReg.h
  2017-09-07 11:13 [PATCH] MdeModulePkg Xhci: Correct description of Timeout param in XhciReg.h Star Zeng
@ 2017-09-08  9:08 ` Ni, Ruiyu
  0 siblings, 0 replies; 2+ messages in thread
From: Ni, Ruiyu @ 2017-09-08  9:08 UTC (permalink / raw)
  To: Zeng, Star, edk2-devel@lists.01.org; +Cc: Alexei Fedorov, Zeng, Star

Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

Thanks/Ray

> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Star Zeng
> Sent: Thursday, September 7, 2017 7:13 PM
> To: edk2-devel@lists.01.org
> Cc: Alexei Fedorov <Alexei.Fedorov@arm.com>; Ni, Ruiyu
> <ruiyu.ni@intel.com>; Zeng, Star <star.zeng@intel.com>
> Subject: [edk2] [PATCH] MdeModulePkg Xhci: Correct description of Timeout
> param in XhciReg.h
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=653
> 
> Correct description of Timeout param in XhciReg.h to be matched with
> XhciReg.c.
> 
> Cc: Alexei Fedorov <Alexei.Fedorov@arm.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Star Zeng <star.zeng@intel.com>
> ---
>  MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h | 10 +++++-----
> MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h |  4 ++--
>  2 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h
> b/MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h
> index b748c8d39739..838a44628c27 100644
> --- a/MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h
> +++ b/MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h
> @@ -2,7 +2,7 @@
> 
>    This file contains the register definition of XHCI host controller.
> 
> -Copyright (c) 2011 - 2013, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2011 - 2017, 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
> @@ -399,7 +399,7 @@ XhcClearOpRegBit (
>    @param  Offset       The offset of the operational register.
>    @param  Bit          The bit of the register to wait for.
>    @param  WaitToSet    Wait the bit to set or clear.
> -  @param  Timeout      The time to wait before abort (in microsecond, us).
> +  @param  Timeout      The time to wait before abort (in millisecond, ms).
> 
>    @retval EFI_SUCCESS  The bit successfully changed by host controller.
>    @retval EFI_TIMEOUT  The time out occurred.
> @@ -521,7 +521,7 @@ XhcIsSysError (
>    Reset the XHCI host controller.
> 
>    @param  Xhc          The XHCI Instance.
> -  @param  Timeout      Time to wait before abort (in microsecond, us).
> +  @param  Timeout      Time to wait before abort (in millisecond, ms).
> 
>    @retval EFI_SUCCESS  The XHCI host controller is reset.
>    @return Others       Failed to reset the XHCI before Timeout.
> @@ -537,7 +537,7 @@ XhcResetHC (
>    Halt the XHCI host controller.
> 
>    @param  Xhc          The XHCI Instance.
> -  @param  Timeout      Time to wait before abort (in microsecond, us).
> +  @param  Timeout      Time to wait before abort (in millisecond, ms).
> 
>    @return EFI_SUCCESS  The XHCI host controller is halt.
>    @return EFI_TIMEOUT  Failed to halt the XHCI before Timeout.
> @@ -553,7 +553,7 @@ XhcHaltHC (
>    Set the XHCI host controller to run.
> 
>    @param  Xhc          The XHCI Instance.
> -  @param  Timeout      Time to wait before abort (in microsecond, us).
> +  @param  Timeout      Time to wait before abort (in millisecond, ms).
> 
>    @return EFI_SUCCESS  The XHCI host controller is running.
>    @return EFI_TIMEOUT  Failed to set the XHCI to run before Timeout.
> diff --git a/MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h
> b/MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h
> index 1a6256066599..0297072ffd6b 100644
> --- a/MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h
> +++ b/MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h
> @@ -1,7 +1,7 @@
>  /** @file
>  Private Header file for Usb Host Controller PEIM
> 
> -Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2014 - 2017, Intel Corporation. All rights reserved.<BR>
> 
>  This program and the accompanying materials  are licensed and made
> available under the terms and conditions @@ -287,7 +287,7 @@
> XhcPeiClearOpRegBit (
>    @param  Offset        The offset of the operational register.
>    @param  Bit           The bit of the register to wait for.
>    @param  WaitToSet     Wait the bit to set or clear.
> -  @param  Timeout       The time to wait before abort (in microsecond, us).
> +  @param  Timeout       The time to wait before abort (in millisecond, ms).
> 
>    @retval EFI_SUCCESS   The bit successfully changed by host controller.
>    @retval EFI_TIMEOUT   The time out occurred.
> --
> 2.7.0.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-09-08  9:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-07 11:13 [PATCH] MdeModulePkg Xhci: Correct description of Timeout param in XhciReg.h Star Zeng
2017-09-08  9:08 ` Ni, Ruiyu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox