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

2018-10-28 Thread Ni, Ruiyu
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

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

2018-10-28 Thread Zeng, Star
Thanks for all the reviews, I just pushed the patches at https://github.com/tianocore/edk2/compare/a6eb94e...0cd6452. Star -Original Message- From: Zeng, Star Sent: Friday, October 26, 2018 9:41 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Ni, Ruiyu ; Wu, Hao A ; Wang, Jian J ; Yao,

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

2018-10-26 Thread Star Zeng
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