OvmfPkg/QemuBootOrderLib: clean up translation of virtio-net over MMIO

The "/MAC(" suffix of the translated UEFI devpath prefix is unnecessary
for matching, because the virtio-mmio base address in VenHwString is
unique anyway. Furthermore, the partial string "MAC(" cannot be processed
by ConvertTextToDevicePath(), which will become relevant later in this
series. Remove "/MAC(".

While at it, remove a bogus comment on PCI.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Shannon Zhao <zhaoshenglong@huawei.com>
Cc: Xiang Zheng <xiang.zheng@linaro.org>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Tested-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> # ArmVirtQemu
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Tested-by: Xiang Zheng <xiang.zheng@linaro.org>
This commit is contained in:
Laszlo Ersek 2018-03-13 15:55:12 +01:00
parent 5851b8253a
commit 90f0918846

View File

@ -1217,14 +1217,14 @@ TranslateMmioOfwNodes (
// | fixed // | fixed
// base address of virtio-mmio register block // base address of virtio-mmio register block
// //
// UEFI device path prefix (dependent on presence of nonzero PCI function): // UEFI device path prefix:
// //
// <VenHwString>/MAC( // <VenHwString>
// //
Written = UnicodeSPrintAsciiFormat ( Written = UnicodeSPrintAsciiFormat (
Translated, Translated,
*TranslatedSize * sizeof (*Translated), // BufferSize in bytes *TranslatedSize * sizeof (*Translated), // BufferSize in bytes
"%s/MAC(", "%s",
VenHwString VenHwString
); );
} else { } else {