On Tue, Sep 12, 2023 at 04:17:14PM +0200, Laszlo Ersek wrote:
> "OvmfPkg/Include/IndustryStandard/Virtio095.h" defines the macro
> VIRTIO_SUBSYSTEM_CONSOLE with value 3; other locations in the tree already
> use it (such as ArmVirtPkg/PlatformBootManagerLib,
> OvmfPkg/VirtioSerialDxe). We should use it in
> OvmfPkg/PlatformBootManagerLib too, rather than the naked constant 3.

>    switch (VirtIo->SubSystemDeviceId) {
> -    case 3:
> +    case VIRTIO_SUBSYSTEM_CONSOLE:
>        PrepareVirtioSerialDevicePath (Handle);
>        break;
>      default:

Acked-by: Gerd Hoffmann <kra...@redhat.com>



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


Reply via email to