On 06/02/21 06:59, Gerd Hoffmann wrote:
> This error was found while compiling VirtioMmioDeviceLib for X64
> with the GCC5 toolchain, where EFIAPI makes a difference.
>
> Signed-off-by: Gerd Hoffmann <kra...@redhat.com>
> ---
>  OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDevice.h          | 1 +
>  OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDeviceFunctions.c | 1 +
>  2 files changed, 2 insertions(+)
>
> diff --git a/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDevice.h 
> b/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDevice.h
> index 785876c75f82..ab53b90d51c9 100644
> --- a/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDevice.h
> +++ b/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDevice.h
> @@ -107,6 +107,7 @@ VirtioMmioSetQueueSel (
>    );
>
>  EFI_STATUS
> +EFIAPI
>  VirtioMmioSetQueueAddress (
>    IN VIRTIO_DEVICE_PROTOCOL  *This,
>    IN VRING                   *Ring,
> diff --git a/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDeviceFunctions.c 
> b/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDeviceFunctions.c
> index 454f008827be..b0d75fb1dd24 100644
> --- a/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDeviceFunctions.c
> +++ b/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDeviceFunctions.c
> @@ -173,6 +173,7 @@ VirtioMmioSetQueueSel (
>  }
>
>  EFI_STATUS
> +EFIAPI
>  VirtioMmioSetQueueAddress (
>    IN VIRTIO_DEVICE_PROTOCOL  *This,
>    IN VRING                   *Ring,
>

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

Merged as commit 75e9154f818a, via 
<https://github.com/tianocore/edk2/pull/1685>, with the following tweak (as 
shown with git-range-diff):

1:  2abeb914c87e ! 1:  f9e70d9c0f8a Add missing EFIAPI to 
VirtioMmioSetQueueAddress
    @@ -1,12 +1,15 @@
     Author: Gerd Hoffmann <kra...@redhat.com>

    -    Add missing EFIAPI to VirtioMmioSetQueueAddress
    +    OvmfPkg/VirtioMmioDeviceLib: Add EFIAPI to VirtioMmioSetQueueAddress

         This error was found while compiling VirtioMmioDeviceLib for X64
         with the GCC5 toolchain, where EFIAPI makes a difference.

         Signed-off-by: Gerd Hoffmann <kra...@redhat.com>
         Message-Id: <20210602045935.762211-1-kra...@redhat.com>
    +    [ler...@redhat.com: prepend module name to subject, trim subject back 
to
    +     allowed length]
    +    Reviewed-by: Laszlo Ersek <ler...@redhat.com>

     diff --git a/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDevice.h 
b/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDevice.h
     --- a/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDevice.h

Thanks!
Laszlo



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#75994): https://edk2.groups.io/g/devel/message/75994
Mute This Topic: https://groups.io/mt/83252055/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to