[PATCH] cache: hisi_soc_hha: use named initializers for acpi_device_id

Pawel Zalewski via B4 Relay posted 1 patch 1 month, 4 weeks ago
drivers/cache/hisi_soc_hha.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] cache: hisi_soc_hha: use named initializers for acpi_device_id
Posted by Pawel Zalewski via B4 Relay 1 month, 4 weeks ago
From: "Pawel Zalewski (The Capable Hub)" <pzalewski@thegoodpenguin.co.uk>

Use a named initializer for the acpi_device_id fields which
makes the code more readable and consistent with how lists
are initialized in the rest of the kernel code base.

Signed-off-by: Pawel Zalewski (The Capable Hub) <pzalewski@thegoodpenguin.co.uk>
---
 drivers/cache/hisi_soc_hha.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/cache/hisi_soc_hha.c b/drivers/cache/hisi_soc_hha.c
index 756c43398515..f0f6c0354108 100644
--- a/drivers/cache/hisi_soc_hha.c
+++ b/drivers/cache/hisi_soc_hha.c
@@ -170,7 +170,7 @@ static void hisi_soc_hha_remove(struct platform_device *pdev)
 }
 
 static const struct acpi_device_id hisi_soc_hha_ids[] = {
-	{ "HISI0511", },
+	{ .id = "HISI0511" },
 	{ }
 };
 MODULE_DEVICE_TABLE(acpi, hisi_soc_hha_ids);

---
base-commit: 075b74841bd0065a3bda3440873c747938e69b68
change-id: 20260803-cache-55f8e58f7d94

Best regards,
--  
Pawel Zalewski (The Capable Hub) <pzalewski@thegoodpenguin.co.uk>
Re: [PATCH] cache: hisi_soc_hha: use named initializers for acpi_device_id
Posted by Conor Dooley 1 month, 3 weeks ago
From: Conor Dooley <conor.dooley@microchip.com>

On Tue, 04 Aug 2026 12:17:45 +0100, Pawel Zalewski (The Capable Hub) wrote:
> Use a named initializer for the acpi_device_id fields which
> makes the code more readable and consistent with how lists
> are initialized in the rest of the kernel code base.

Applied to cache-for-next, thanks!

[1/1] cache: hisi_soc_hha: use named initializers for acpi_device_id
      https://git.kernel.org/conor/c/8a5d6c13cc7c

Thanks,
Conor.
Re: [PATCH] cache: hisi_soc_hha: use named initializers for acpi_device_id
Posted by Yushan Wang 1 month, 4 weeks ago

On 8/4/2026 7:17 PM, Pawel Zalewski via B4 Relay wrote:
> From: "Pawel Zalewski (The Capable Hub)" <pzalewski@thegoodpenguin.co.uk>
>
> Use a named initializer for the acpi_device_id fields which
> makes the code more readable and consistent with how lists
> are initialized in the rest of the kernel code base.
>
> Signed-off-by: Pawel Zalewski (The Capable Hub) <pzalewski@thegoodpenguin.co.uk>

Reviewed-by: Yushan Wang <wangyushan12@huawei.com>

> ---
>  drivers/cache/hisi_soc_hha.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/cache/hisi_soc_hha.c b/drivers/cache/hisi_soc_hha.c
> index 756c43398515..f0f6c0354108 100644
> --- a/drivers/cache/hisi_soc_hha.c
> +++ b/drivers/cache/hisi_soc_hha.c
> @@ -170,7 +170,7 @@ static void hisi_soc_hha_remove(struct platform_device *pdev)
>  }
>  
>  static const struct acpi_device_id hisi_soc_hha_ids[] = {
> -	{ "HISI0511", },
> +	{ .id = "HISI0511" },
>  	{ }
>  };
>  MODULE_DEVICE_TABLE(acpi, hisi_soc_hha_ids);
>
> ---
> base-commit: 075b74841bd0065a3bda3440873c747938e69b68
> change-id: 20260803-cache-55f8e58f7d94
>
> Best regards,
> --  
> Pawel Zalewski (The Capable Hub) <pzalewski@thegoodpenguin.co.uk>
>
>
Re: [PATCH] cache: hisi_soc_hha: use named initializers for acpi_device_id
Posted by Jonathan Cameron 1 month, 4 weeks ago
On Tue, 04 Aug 2026 12:17:45 +0100
Pawel Zalewski via B4 Relay <devnull+pzalewski.thegoodpenguin.co.uk@kernel.org> wrote:

> From: "Pawel Zalewski (The Capable Hub)" <pzalewski@thegoodpenguin.co.uk>
> 
> Use a named initializer for the acpi_device_id fields which
> makes the code more readable and consistent with how lists
> are initialized in the rest of the kernel code base.
> 
> Signed-off-by: Pawel Zalewski (The Capable Hub) <pzalewski@thegoodpenguin.co.uk>

Acked-by: Jonathan Cameron <jonathan.cameron@oss.qualcomm.com>

> ---
>  drivers/cache/hisi_soc_hha.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/cache/hisi_soc_hha.c b/drivers/cache/hisi_soc_hha.c
> index 756c43398515..f0f6c0354108 100644
> --- a/drivers/cache/hisi_soc_hha.c
> +++ b/drivers/cache/hisi_soc_hha.c
> @@ -170,7 +170,7 @@ static void hisi_soc_hha_remove(struct platform_device *pdev)
>  }
>  
>  static const struct acpi_device_id hisi_soc_hha_ids[] = {
> -	{ "HISI0511", },
> +	{ .id = "HISI0511" },
>  	{ }
>  };
>  MODULE_DEVICE_TABLE(acpi, hisi_soc_hha_ids);
> 
> ---
> base-commit: 075b74841bd0065a3bda3440873c747938e69b68
> change-id: 20260803-cache-55f8e58f7d94
> 
> Best regards,
> --  
> Pawel Zalewski (The Capable Hub) <pzalewski@thegoodpenguin.co.uk>
> 
>