ACPI: LPSS: Replace loop with first entry retrieval

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2176554

commit da13b3361bb609f5e3fde3f57b8e2b42001513a3
Author: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: Mon, 29 Aug 2022 17:11:28 +0300

After the commit 6505e452371d ("ACPI: LPSS: Use the helper
acpi_dev_get_memory_resources()") the list is empty or
contains only resource of IORESOURCE_MEM type. Hence, no
need to check for the type, and since we break after the
first found, no need to iterate over full list. That said,
replace loop with first entry retrieval.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Mark Langsdorf <mlangsdo@redhat.com>
This commit is contained in:
Mark Langsdorf 2023-03-09 12:06:57 -05:00
parent 322c1a5a76
commit b495a98be0
1 changed files with 8 additions and 10 deletions

View File

@ -655,16 +655,14 @@ static int acpi_lpss_create_device(struct acpi_device *adev,
if (ret < 0)
goto err_out;
list_for_each_entry(rentry, &resource_list, node)
if (resource_type(rentry->res) == IORESOURCE_MEM) {
if (dev_desc->prv_size_override)
pdata->mmio_size = dev_desc->prv_size_override;
else
pdata->mmio_size = resource_size(rentry->res);
pdata->mmio_base = ioremap(rentry->res->start,
pdata->mmio_size);
break;
}
rentry = list_first_entry_or_null(&resource_list, struct resource_entry, node);
if (rentry) {
if (dev_desc->prv_size_override)
pdata->mmio_size = dev_desc->prv_size_override;
else
pdata->mmio_size = resource_size(rentry->res);
pdata->mmio_base = ioremap(rentry->res->start, pdata->mmio_size);
}
acpi_dev_free_resource_list(&resource_list);