linux-kernel - Re: [PATCH v1] ACPI: scan: Clean up after recent changes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <1aadd874-6400-4652-9a42-d1113db19184@kernel.org>
Date: Sat, 10 Jan 2026 07:25:55 -0600
From: "Mario Limonciello (AMD) (kernel.org)" <superm1@...nel.org>
To: "Rafael J. Wysocki" <rafael@...nel.org>,
 Linux ACPI <linux-acpi@...r.kernel.org>
Cc: LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v1] ACPI: scan: Clean up after recent changes



On 1/10/2026 5:50 AM, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@...el.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@...el.com>
Reviewed-by: Mario Limonciello (AMD) <superm1@...nel.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;
> 
> 
>