PCI: Fix missing bridge dma_ranges resource list cleanup

robherring · bjorn-helgaas · commit 7608158df3ed · 2019-10-14T10:22:26.000-05:00
Commit e80a91a ("PCI: Add dma_ranges window list") added a dma_ranges resource list, but failed to correctly free the list when devm_pci_alloc_host_bridge() is used. Only the iproc host bridge driver is using the dma_ranges list. Fixes: e80a91a ("PCI: Add dma_ranges window list") Link: https://lore.kernel.org/r/20191008012325.25700-1-robh@kernel.org Signed-off-by: Rob Herring <robh@kernel.org> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> Cc: Srinath Mannam <srinath.mannam@broadcom.com>
diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c
@@ -572,6 +572,7 @@ static void devm_pci_release_host_bridge_dev(struct device *dev)
 		bridge->release_fn(bridge);
 
 	pci_free_resource_list(&bridge->windows);
+	pci_free_resource_list(&bridge->dma_ranges);
 }
 
 static void pci_release_host_bridge_dev(struct device *dev)

Original file line number	Diff line number	Diff line change
`@@ -572,6 +572,7 @@ static void devm_pci_release_host_bridge_dev(struct device *dev)`
`572`	`572`	`bridge->release_fn(bridge);`
`573`	`573`
`574`	`574`	`pci_free_resource_list(&bridge->windows);`
	`575`	`+ pci_free_resource_list(&bridge->dma_ranges);`
`575`	`576`	`}`
`576`	`577`
`577`	`578`	`static void pci_release_host_bridge_dev(struct device *dev)`