-----Original Message-----
From: <[email protected]> on behalf of Kyle Evans
<[email protected]>
Date: 2022-04-12, Tuesday at 17:34
To: <[email protected]>, <[email protected]>,
<[email protected]>
Subject: git: 454630c72556 - main - stand: libefi: swap /Pci() printing around
The branch main has been updated by kevans:
URL:
https://cgit.FreeBSD.org/src/commit/?id=454630c72556d45e401f29f56b3317c2fb0499a0
commit 454630c72556d45e401f29f56b3317c2fb0499a0
Author: Kyle Evans <[email protected]>
AuthorDate: 2022-04-13 00:29:54 +0000
Commit: Kyle Evans <[email protected]>
CommitDate: 2022-04-13 00:30:30 +0000
stand: libefi: swap /Pci() printing around
Printing device followed by interface matches, e.g., edk2. Note that
this is only a fallback, many firmware implementations will provide the
protocol that we'll use to format device paths.
"(device,function)" makes more sense anyway, since that's the logical
organization (i.e. each unique device may have multiple functions).
What was the rationale for doing "(function,device)" in the first place?
-Ravi (rpokala@)
Reviewed by: imp, tsoome
Sponsored by: Ampere Computing
Submitted by: Klara, Inc.
Differential Revision: https://reviews.freebsd.org/D34539
---
stand/efi/libefi/devpath.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/stand/efi/libefi/devpath.c b/stand/efi/libefi/devpath.c
index b56c583b5fa7..c25ae73e463c 100644
--- a/stand/efi/libefi/devpath.c
+++ b/stand/efi/libefi/devpath.c
@@ -146,8 +146,8 @@ efi_hw_dev_path(EFI_DEVICE_PATH *node, char *suffix)
switch (subtype) {
case HW_PCI_DP:
if (asprintf(&name, "Pci(%x,%x)%s",
- ((PCI_DEVICE_PATH *)node)->Function,
- ((PCI_DEVICE_PATH *)node)->Device, tail) < 0)
+ ((PCI_DEVICE_PATH *)node)->Device,
+ ((PCI_DEVICE_PATH *)node)->Function, tail) < 0)
name = NULL;
break;
case HW_PCCARD_DP:
