On 11/13/14 18:27, Anthony PERARD wrote:
> This patch replace some types in GrantTable and the argument Index of
> XenHypercallHvmGetParam to what the types should be.
> 
> This avoid to have type cast in code.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Anthony PERARD <anthony.per...@citrix.com>
> ---
>  OvmfPkg/XenBusDxe/GrantTable.c   | 4 ++--
>  OvmfPkg/XenBusDxe/XenHypercall.c | 2 +-
>  OvmfPkg/XenBusDxe/XenHypercall.h | 2 +-
>  3 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/OvmfPkg/XenBusDxe/GrantTable.c b/OvmfPkg/XenBusDxe/GrantTable.c
> index b17fef3..f68a854 100644
> --- a/OvmfPkg/XenBusDxe/GrantTable.c
> +++ b/OvmfPkg/XenBusDxe/GrantTable.c
> @@ -74,7 +74,7 @@ XenGrantTableGetFreeEntry (
>    VOID
>    )
>  {
> -  UINTN Ref;
> +  grant_ref_t Ref;
>  
>    EfiAcquireLock (&mGrantListLock);
>    Ref = GrantList[0];
> @@ -97,7 +97,7 @@ XenGrantTableGrantAccess (
>    )
>  {
>    grant_ref_t Ref;
> -  UINT32 Flags;
> +  UINT16 Flags;
>  
>    ASSERT (GrantTable != NULL);
>    Ref = XenGrantTableGetFreeEntry ();
> diff --git a/OvmfPkg/XenBusDxe/XenHypercall.c 
> b/OvmfPkg/XenBusDxe/XenHypercall.c
> index 0f2ba5d..15e9e1c 100644
> --- a/OvmfPkg/XenBusDxe/XenHypercall.c
> +++ b/OvmfPkg/XenBusDxe/XenHypercall.c
> @@ -43,7 +43,7 @@ XenHyperpageInit (
>  UINT64
>  XenHypercallHvmGetParam (
>    IN XENBUS_DEVICE *Dev,
> -  IN INTN          Index
> +  IN UINT32        Index
>    )
>  {
>    xen_hvm_param_t     Parameter;
> diff --git a/OvmfPkg/XenBusDxe/XenHypercall.h 
> b/OvmfPkg/XenBusDxe/XenHypercall.h
> index 088037b..bb282a4 100644
> --- a/OvmfPkg/XenBusDxe/XenHypercall.h
> +++ b/OvmfPkg/XenBusDxe/XenHypercall.h
> @@ -60,7 +60,7 @@ XenHyperpageInit (
>  UINT64
>  XenHypercallHvmGetParam (
>    XENBUS_DEVICE *Dev,
> -  INTN Index
> +  UINT32 Index
>    );
>  
>  /**
> 

Reviewed-by: Laszlo Ersek <ler...@redhat.com>

------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to