From: Ilpo Järvinen Date: Mon, 16 Dec 2024 17:56:16 +0000 (+0200) Subject: PCI: Check resource_size() separately X-Git-Tag: block-6.15-20250403~40^2~22^2~27 X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=2bd0c72117841f7fb82aab7f4a0d65e66ef9543e;p=linux-block.git PCI: Check resource_size() separately Instead of chaining logic inside if () condition so that multiple lines are required, make !resource_size() a separate check and use continue. Link: https://lore.kernel.org/r/20241216175632.4175-10-ilpo.jarvinen@linux.intel.com Signed-off-by: Ilpo Järvinen Signed-off-by: Bjorn Helgaas Tested-by: Xiaochun Lee --- diff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c index 5f3215e0904d..eaeaf09cd91d 100644 --- a/drivers/pci/setup-bus.c +++ b/drivers/pci/setup-bus.c @@ -285,8 +285,11 @@ static void assign_requested_resources_sorted(struct list_head *head, list_for_each_entry(dev_res, head, list) { res = dev_res->res; idx = res - &dev_res->dev->resource[0]; - if (resource_size(res) && - pci_assign_resource(dev_res->dev, idx)) { + + if (!resource_size(res)) + continue; + + if (pci_assign_resource(dev_res->dev, idx)) { if (fail_head) { /* * If the failed resource is a ROM BAR and