Re: [PATCH V3 0/4] Remove unnecessary Map/Unmap in XhciDxe/EhciDxe for AsyncInterruptTransfer

["Ni, Ruiyu" <ruiyu.ni@Intel.com>]

On 10/26/2018 9:41 PM, Star Zeng wrote:
> V3: Thanks for Hao's feedback.
> 1. Match function parameter name and description between
> XhciSched.c and XhciSched.h, EhciSched.c and EhciSched.h.
> 2. Call XhcFreeUrb after XhcCreateTransferTrb fails in XhcCreateTrb.
> 
> 
> V2: Thanks for Ray's feedback.
> 1. Add the missing "FreePool (Data);".
> 2. Remove the unnecessary indentation change.
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1274
> 
> Please refer to the log message of each commit for more details.
> 
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Hao Wu <hao.a.wu@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> 
> Star Zeng (4):
>    MdeModulePkg XhciDxe: Extract new XhciInsertAsyncIntTransfer function
>    MdeModulePkg EhciDxe: Extract new EhciInsertAsyncIntTransfer function
>    MdeModulePkg XhciDxe: Use common buffer for AsyncInterruptTransfer
>    MdeModulePkg EhciDxe: Use common buffer for AsyncInterruptTransfer
> 
>   MdeModulePkg/Bus/Pci/EhciDxe/Ehci.c      |  28 +----
>   MdeModulePkg/Bus/Pci/EhciDxe/EhciSched.c | 114 +++++++++----------
>   MdeModulePkg/Bus/Pci/EhciDxe/EhciSched.h |  35 +++++-
>   MdeModulePkg/Bus/Pci/EhciDxe/EhciUrb.c   |  38 ++++++-
>   MdeModulePkg/Bus/Pci/EhciDxe/EhciUrb.h   |  33 +++---
>   MdeModulePkg/Bus/Pci/XhciDxe/Xhci.c      |  19 +---
>   MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c | 185 +++++++++++++++++--------------
>   MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.h |  55 +++++++--
>   8 files changed, 296 insertions(+), 211 deletions(-)
> 
Reviewed-by: Ruiyu Ni <ruiyu.ni@Intel.com>

-- 
Thanks,
Ray