Commit 4bee4fc0 authored by Ilpo Järvinen's avatar Ilpo Järvinen Committed by Bjorn Helgaas
Browse files

PCI: Use res_to_dev_res() in reassign_resources_sorted() 



reassign_resources_sorted() contains a search loop for a particular
resource in the head list. res_to_dev_res() already implements the same
search so use it instead.

Drop unused found_match and dev_res variables.

Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarBjorn Helgaas <bhelgaas@google.com>
Link: https://patch.msgid.link/20251219174036.16738-9-ilpo.jarvinen@linux.intel.com
parent 5819403a

drivers/pci/setup-bus.c

+1 −10
Original line number Diff line number Diff line
@@ -414,7 +414,6 @@ static void reassign_resources_sorted(struct list_head *realloc_head, 
				      struct list_head *head)

{

	struct pci_dev_resource *add_res, *tmp;

	struct pci_dev_resource *dev_res;

	struct pci_dev *dev;

	struct resource *res;

	const char *res_name;
@@ -422,8 +421,6 @@ static void reassign_resources_sorted(struct list_head *realloc_head, 
	int idx;



	list_for_each_entry_safe(add_res, tmp, realloc_head, list) {

		bool found_match = false;



		res = add_res->res;

		dev = add_res->dev;

		idx = pci_resource_num(dev, res);
@@ -437,13 +434,7 @@ static void reassign_resources_sorted(struct list_head *realloc_head, 
			goto out;



		/* Skip this resource if not found in head list */

		list_for_each_entry(dev_res, head, list) {

			if (dev_res->res == res) {

				found_match = true;

				break;

			}

		}

		if (!found_match) /* Just skip */

		if (!res_to_dev_res(head, res))

			continue;



		res_name = pci_resource_name(dev, idx);