[PATCH v1] ACPI: scan: Clean up after recent changes

Rafael J. Wysocki posted 1 patch 4 weeks ago
drivers/acpi/acpi_platform.c |    2 +-
drivers/acpi/scan.c          |    3 +--
2 files changed, 2 insertions(+), 3 deletions(-)
[PATCH v1] ACPI: scan: Clean up after recent changes
Posted by Rafael J. Wysocki 4 weeks ago
From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

Use LIST_HEAD() for initializing an on-stack list head in two places and
remove an empty code line added by mistake.

No intentional functional impact.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---

This is based on ACPI changes in linux-next.

---
 drivers/acpi/acpi_platform.c |    2 +-
 drivers/acpi/scan.c          |    3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

--- a/drivers/acpi/acpi_platform.c
+++ b/drivers/acpi/acpi_platform.c
@@ -136,7 +136,7 @@ struct platform_device *acpi_create_plat
 	}
 
 	if (adev->device_type == ACPI_BUS_TYPE_DEVICE && !adev->pnp.type.backlight) {
-		struct list_head resource_list = LIST_HEAD_INIT(resource_list);
+		LIST_HEAD(resource_list);
 
 		count = acpi_dev_get_resources(adev, &resource_list, NULL, NULL);
 		if (count < 0)
--- a/drivers/acpi/scan.c
+++ b/drivers/acpi/scan.c
@@ -2602,8 +2602,8 @@ static void acpi_scan_postponed(void)
 
 static void acpi_scan_claim_resources(struct acpi_device *adev)
 {
-	struct list_head resource_list = LIST_HEAD_INIT(resource_list);
 	struct resource_entry *rentry;
+	LIST_HEAD(resource_list);
 	unsigned int count = 0;
 	const char *regionid;
 
@@ -2660,7 +2660,6 @@ exit:
 	acpi_dev_free_resource_list(&resource_list);
 }
 
-
 static int __init acpi_reserve_motherboard_resources(void)
 {
 	struct acpi_scan_system_dev *sd, *tmp;
Re: [PATCH v1] ACPI: scan: Clean up after recent changes
Posted by Mario Limonciello (AMD) (kernel.org) 4 weeks ago

On 1/10/2026 5:50 AM, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> Use LIST_HEAD() for initializing an on-stack list head in two places and
> remove an empty code line added by mistake.
> 
> No intentional functional impact.
> 
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Mario Limonciello (AMD) <superm1@kernel.org>
> ---
> 
> This is based on ACPI changes in linux-next.
> 
> ---
>   drivers/acpi/acpi_platform.c |    2 +-
>   drivers/acpi/scan.c          |    3 +--
>   2 files changed, 2 insertions(+), 3 deletions(-)
> 
> --- a/drivers/acpi/acpi_platform.c
> +++ b/drivers/acpi/acpi_platform.c
> @@ -136,7 +136,7 @@ struct platform_device *acpi_create_plat
>   	}
>   
>   	if (adev->device_type == ACPI_BUS_TYPE_DEVICE && !adev->pnp.type.backlight) {
> -		struct list_head resource_list = LIST_HEAD_INIT(resource_list);
> +		LIST_HEAD(resource_list);
>   
>   		count = acpi_dev_get_resources(adev, &resource_list, NULL, NULL);
>   		if (count < 0)
> --- a/drivers/acpi/scan.c
> +++ b/drivers/acpi/scan.c
> @@ -2602,8 +2602,8 @@ static void acpi_scan_postponed(void)
>   
>   static void acpi_scan_claim_resources(struct acpi_device *adev)
>   {
> -	struct list_head resource_list = LIST_HEAD_INIT(resource_list);
>   	struct resource_entry *rentry;
> +	LIST_HEAD(resource_list);
>   	unsigned int count = 0;
>   	const char *regionid;
>   
> @@ -2660,7 +2660,6 @@ exit:
>   	acpi_dev_free_resource_list(&resource_list);
>   }
>   
> -
>   static int __init acpi_reserve_motherboard_resources(void)
>   {
>   	struct acpi_scan_system_dev *sd, *tmp;
> 
> 
>