Header type is 7-bit number so properly clear upper 8th bit which
indicates multifunction device.

And do not try to show bars for unsupported header types.

Signed-off-by: Pali Rohár <p...@kernel.org>
---
 cmd/pci.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/cmd/pci.c b/cmd/pci.c
index cfabdc0f3012..4a82854db7fa 100644
--- a/cmd/pci.c
+++ b/cmd/pci.c
@@ -71,10 +71,15 @@ static int pci_bar_show(struct udevice *dev)
        int prefetchable;
 
        dm_pci_read_config8(dev, PCI_HEADER_TYPE, &header_type);
+       header_type &= 0x7f;
 
        if (header_type == PCI_HEADER_TYPE_CARDBUS) {
                printf("CardBus doesn't support BARs\n");
                return -ENOSYS;
+       } else if (header_type != PCI_HEADER_TYPE_NORMAL &&
+                  header_type != PCI_HEADER_TYPE_BRIDGE) {
+               printf("unknown header type\n");
+               return -ENOSYS;
        }
 
        bar_cnt = (header_type == PCI_HEADER_TYPE_NORMAL) ? 6 : 2;
-- 
2.20.1

Reply via email to