On 02/03/15 20:20, Ard Biesheuvel wrote:
> This adds a XenIoMmioLib declaration and implementation that can
> be invoked to install the XENIO_PROTOCOL and a corresponding
> grant table address on a EFI handle.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Ard Biesheuvel <[email protected]>
> ---
>  OvmfPkg/Include/Library/XenIoMmioLib.h        |  64 ++++++++++
>  OvmfPkg/Library/XenIoMmioLib/XenIoMmioLib.c   | 166 
> ++++++++++++++++++++++++++
>  OvmfPkg/Library/XenIoMmioLib/XenIoMmioLib.inf |  39 ++++++
>  OvmfPkg/OvmfPkg.dec                           |   4 +
>  4 files changed, 273 insertions(+)
>  create mode 100644 OvmfPkg/Include/Library/XenIoMmioLib.h
>  create mode 100644 OvmfPkg/Library/XenIoMmioLib/XenIoMmioLib.c
>  create mode 100644 OvmfPkg/Library/XenIoMmioLib/XenIoMmioLib.inf

There are two occurrences of the typo "if if" in this patch. In case you
send a v4, please fix that.

Reviewed-by: Laszlo Ersek <[email protected]>

I'm done with the v3 review; you addressed all my v2 comments.

Thanks!
Laszlo

------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to