Skip to content

Commit 6bd6597

Browse files
jpbruckerbjorn-helgaas
authored andcommitted
PCI/ACPI: Don't reset a fwnode set by OF
Commit 375553a ("PCI: Setup ACPI fwnode early and at the same time with OF") added a call to pci_set_acpi_fwnode() in pci_setup_device(), which unconditionally clears any fwnode previously set by pci_set_of_node(). pci_set_acpi_fwnode() looks for ACPI_COMPANION(), which only returns the existing fwnode if it was set by ACPI_COMPANION_SET(). If it was set by OF instead, ACPI_COMPANION() returns NULL and pci_set_acpi_fwnode() accidentally clears the fwnode. To fix this, look for any fwnode instead of just ACPI companions. Fixes a virtio-iommu boot regression in v5.15-rc1. Fixes: 375553a ("PCI: Setup ACPI fwnode early and at the same time with OF") Link: https://lore.kernel.org/r/20210913172358.1775381-1-jean-philippe@linaro.org Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> Acked-by: Rob Herring <robh@kernel.org> Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
1 parent 00e1a5d commit 6bd6597

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/pci/pci-acpi.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -937,7 +937,7 @@ static struct acpi_device *acpi_pci_find_companion(struct device *dev);
937937

938938
void pci_set_acpi_fwnode(struct pci_dev *dev)
939939
{
940-
if (!ACPI_COMPANION(&dev->dev) && !pci_dev_is_added(dev))
940+
if (!dev_fwnode(&dev->dev) && !pci_dev_is_added(dev))
941941
ACPI_COMPANION_SET(&dev->dev,
942942
acpi_pci_find_companion(&dev->dev));
943943
}

0 commit comments

Comments
 (0)