drivers/acpi/acpi_watchdog.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
Replace the separate declaration of 'resource_list' and subsequent
INIT_LIST_HEAD() call with LIST_HEAD(), which declares and initializes
the list head in one idiomatic step. This reduces code verbosity and
aligns with common kernel coding patterns, without functional change.
Signed-off-by: Can Peng <pengcan@kylinos.cn>
---
drivers/acpi/acpi_watchdog.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/acpi/acpi_watchdog.c b/drivers/acpi/acpi_watchdog.c
index 14b24157799c..709993c535d1 100644
--- a/drivers/acpi/acpi_watchdog.c
+++ b/drivers/acpi/acpi_watchdog.c
@@ -103,7 +103,7 @@ void __init acpi_watchdog_init(void)
{
const struct acpi_wdat_entry *entries;
const struct acpi_table_wdat *wdat;
- struct list_head resource_list;
+ LIST_HEAD(resource_list);
struct resource_entry *rentry;
struct platform_device *pdev;
struct resource *resources;
@@ -125,8 +125,6 @@ void __init acpi_watchdog_init(void)
wdat->pci_device != 0xff || wdat->pci_function != 0xff)
goto fail_put_wdat;
- INIT_LIST_HEAD(&resource_list);
-
entries = (struct acpi_wdat_entry *)(wdat + 1);
for (i = 0; i < wdat->entries; i++) {
const struct acpi_generic_address *gas;
--
2.25.1
On Tue, Dec 9, 2025 at 8:29 AM Can Peng <pengcan@kylinos.cn> wrote:
>
> Replace the separate declaration of 'resource_list' and subsequent
> INIT_LIST_HEAD() call with LIST_HEAD(), which declares and initializes
> the list head in one idiomatic step. This reduces code verbosity and
> aligns with common kernel coding patterns, without functional change.
>
> Signed-off-by: Can Peng <pengcan@kylinos.cn>
> ---
> drivers/acpi/acpi_watchdog.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/acpi/acpi_watchdog.c b/drivers/acpi/acpi_watchdog.c
> index 14b24157799c..709993c535d1 100644
> --- a/drivers/acpi/acpi_watchdog.c
> +++ b/drivers/acpi/acpi_watchdog.c
> @@ -103,7 +103,7 @@ void __init acpi_watchdog_init(void)
> {
> const struct acpi_wdat_entry *entries;
> const struct acpi_table_wdat *wdat;
> - struct list_head resource_list;
> + LIST_HEAD(resource_list);
> struct resource_entry *rentry;
> struct platform_device *pdev;
> struct resource *resources;
> @@ -125,8 +125,6 @@ void __init acpi_watchdog_init(void)
> wdat->pci_device != 0xff || wdat->pci_function != 0xff)
> goto fail_put_wdat;
>
> - INIT_LIST_HEAD(&resource_list);
> -
> entries = (struct acpi_wdat_entry *)(wdat + 1);
> for (i = 0; i < wdat->entries; i++) {
> const struct acpi_generic_address *gas;
> --
Applied as 6.20 material, thanks!
© 2016 - 2026 Red Hat, Inc.