Hi Ray,

One small comment, below files:
\EmulatorPkg\Win\Host\WinGop.h
\EmulatorPkg\Win\Host\WinGopInput.c

have an CR line ending (rather than CRLF) at the last line.

Please help to correct the format.
With the changes, the series is good to me:
Reviewed-by: Hao Wu <hao.a...@intel.com>

Best Regards,
Hao Wu


> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ruiyu
> Ni
> Sent: Thursday, August 23, 2018 5:56 PM
> To: edk2-devel@lists.01.org
> Subject: [edk2] [PATCH v2 00/12] Add WinHost support in EmulatorPkg
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1112
> 
> The patch sets add WinHost support (Nt32) in EmulatorPkg.
> 3 EmulatorPkg common issues were found and fixed.
> Other 9 patches are to step-by-step enable the WinHost.
> 
> v2 sends to correct mail address.
> 
> Ruiyu Ni (12):
>   EmulatorPkg/ThunkProtocolList: Fix VS build failure
>   EmulatorPkg/Win: Add Windows host support
>   EmulatorPkg/Win: Enable source level debugging
>   EmulatorPkg/Win: Enable native OS console as firmware console
>   EmulatorPkg/Win: Add input/output support
>   EmulatorPkg/Win: Add timer and interrupt support
>   EmulatorPkg/Win: Add RTC support
>   EmulatorPkg/Win: Add SimpleFileSystem support
>   EmulatorPkg/Win: Add BlockIo support
>   EmulatorPkg/PlatformBds: Signal EndOfDxe in platform BDS
>   EmulatorPkg/EmuFileSystem: Fix a bug that causes Close() assertion
>   EmulatorPkg/DSC: Remove FS mapping to EDK Shell bin directory
> 
>  .../EmuSimpleFileSystemDxe/EmuSimpleFileSystem.c   |   33 +-
>  EmulatorPkg/EmulatorPkg.dsc                        |   17 +-
>  EmulatorPkg/Library/EmuBdsLib/BdsPlatform.c        |    4 +-
>  EmulatorPkg/Library/EmuBdsLib/BdsPlatform.h        |    4 +-
>  EmulatorPkg/Library/EmuBdsLib/EmuBdsLib.inf        |    5 +-
>  .../Library/ThunkProtocolList/ThunkProtocolList.c  |    4 +-
>  EmulatorPkg/Win/Host/WinBlockIo.c                  |  563 +++++
>  EmulatorPkg/Win/Host/WinFileSystem.c               | 2409
> ++++++++++++++++++++
>  EmulatorPkg/Win/Host/WinGop.h                      |  204 ++
>  EmulatorPkg/Win/Host/WinGopInput.c                 |  417 ++++
>  EmulatorPkg/Win/Host/WinGopScreen.c                |  872 +++++++
>  EmulatorPkg/Win/Host/WinHost.c                     |  947 ++++++++
>  EmulatorPkg/Win/Host/WinHost.h                     |  209 ++
>  EmulatorPkg/Win/Host/WinHost.inf                   |  107 +
>  EmulatorPkg/Win/Host/WinInclude.h                  |   75 +
>  EmulatorPkg/Win/Host/WinMemoryAllocationLib.c      |  178 ++
>  EmulatorPkg/Win/Host/WinThunk.c                    |  577 +++++
>  17 files changed, 6614 insertions(+), 11 deletions(-)
>  create mode 100644 EmulatorPkg/Win/Host/WinBlockIo.c
>  create mode 100644 EmulatorPkg/Win/Host/WinFileSystem.c
>  create mode 100644 EmulatorPkg/Win/Host/WinGop.h
>  create mode 100644 EmulatorPkg/Win/Host/WinGopInput.c
>  create mode 100644 EmulatorPkg/Win/Host/WinGopScreen.c
>  create mode 100644 EmulatorPkg/Win/Host/WinHost.c
>  create mode 100644 EmulatorPkg/Win/Host/WinHost.h
>  create mode 100644 EmulatorPkg/Win/Host/WinHost.inf
>  create mode 100644 EmulatorPkg/Win/Host/WinInclude.h
>  create mode 100644 EmulatorPkg/Win/Host/WinMemoryAllocationLib.c
>  create mode 100644 EmulatorPkg/Win/Host/WinThunk.c
> 
> --
> 2.16.1.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to