pci: Fix showing bars

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 <pali@kernel.org>
Reviewed-by: Stefan Roese <sr@denx.de>

diff --git a/cmd/pci.c b/cmd/pci.c
index cfabdc0f3012fbcb3818f08e6e777de6563181da..4a82854db7fac82f3acd52b8d27e05cdd9dd330e 100644 (file)
--- 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;