PCI: Disable decoding for BAR sizing only when it was actually enabled
We disable BARs while sizing them so we don't cause conflicts with other devices (see253d2e5498
andbbffe43524
). But if device decoding is already disabled before we size the BAR, we don't need to disable it again. [bhelgaas: changelog, add PCI_COMMAND_DECODING_ENABLE for readability] Signed-off-by: Zoltan Kiss <zoltan.kiss@citrix.com> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
This commit is contained in:
parent
da27f4b3ec
commit
808e34e2cd
|
@ -156,6 +156,8 @@ static inline unsigned long decode_bar(struct pci_dev *dev, u32 bar)
|
||||||
return flags;
|
return flags;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#define PCI_COMMAND_DECODE_ENABLE (PCI_COMMAND_MEMORY | PCI_COMMAND_IO)
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* pci_read_base - read a PCI BAR
|
* pci_read_base - read a PCI BAR
|
||||||
* @dev: the PCI device
|
* @dev: the PCI device
|
||||||
|
@ -178,8 +180,10 @@ int __pci_read_base(struct pci_dev *dev, enum pci_bar_type type,
|
||||||
/* No printks while decoding is disabled! */
|
/* No printks while decoding is disabled! */
|
||||||
if (!dev->mmio_always_on) {
|
if (!dev->mmio_always_on) {
|
||||||
pci_read_config_word(dev, PCI_COMMAND, &orig_cmd);
|
pci_read_config_word(dev, PCI_COMMAND, &orig_cmd);
|
||||||
pci_write_config_word(dev, PCI_COMMAND,
|
if (orig_cmd & PCI_COMMAND_DECODE_ENABLE) {
|
||||||
orig_cmd & ~(PCI_COMMAND_MEMORY | PCI_COMMAND_IO));
|
pci_write_config_word(dev, PCI_COMMAND,
|
||||||
|
orig_cmd & ~PCI_COMMAND_DECODE_ENABLE);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
res->name = pci_name(dev);
|
res->name = pci_name(dev);
|
||||||
|
@ -293,7 +297,8 @@ int __pci_read_base(struct pci_dev *dev, enum pci_bar_type type,
|
||||||
fail:
|
fail:
|
||||||
res->flags = 0;
|
res->flags = 0;
|
||||||
out:
|
out:
|
||||||
if (!dev->mmio_always_on)
|
if (!dev->mmio_always_on &&
|
||||||
|
(orig_cmd & PCI_COMMAND_DECODE_ENABLE))
|
||||||
pci_write_config_word(dev, PCI_COMMAND, orig_cmd);
|
pci_write_config_word(dev, PCI_COMMAND, orig_cmd);
|
||||||
|
|
||||||
if (bar_too_big)
|
if (bar_too_big)
|
||||||
|
|
Loading…
Reference in New Issue