[PATCH v2] LoongArch: acpi: Eliminate superfluous get_numa_distances_cnt()

WangYuli posted 1 patch 9 months, 3 weeks ago
arch/loongarch/kernel/acpi.c | 12 ------------
1 file changed, 12 deletions(-)
[PATCH v2] LoongArch: acpi: Eliminate superfluous get_numa_distances_cnt()
Posted by WangYuli 9 months, 3 weeks ago
In LoongArch, get_numa_distances_cnt() was not in use, resulting in
a compiler warning.

Fix follow errors with clang-18 when W=1e:

arch/loongarch/kernel/acpi.c:259:28: error: unused function 'get_numa_distances_cnt' [-Werror,-Wunused-function]
  259 | static inline unsigned int get_numa_distances_cnt(struct acpi_table_slit *slit)
      |                            ^~~~~~~~~~~~~~~~~~~~~~
1 error generated.

Link: https://lore.kernel.org/all/Z7bHPVUH4lAezk0E@kernel.org/
Signed-off-by: Yuli Wang <wangyuli@uniontech.com>
---
Changelog:
 *v1->v2: Drop the relocation and one more useless variable.
---
 arch/loongarch/kernel/acpi.c | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/arch/loongarch/kernel/acpi.c b/arch/loongarch/kernel/acpi.c
index ee471a80763e..193f81920074 100644
--- a/arch/loongarch/kernel/acpi.c
+++ b/arch/loongarch/kernel/acpi.c
@@ -249,18 +249,6 @@ static __init int setup_node(int pxm)
 	return acpi_map_pxm_to_node(pxm);
 }
 
-/*
- * Callback for SLIT parsing.  pxm_to_node() returns NUMA_NO_NODE for
- * I/O localities since SRAT does not list them.  I/O localities are
- * not supported at this point.
- */
-unsigned int numa_distance_cnt;
-
-static inline unsigned int get_numa_distances_cnt(struct acpi_table_slit *slit)
-{
-	return slit->locality_count;
-}
-
 void __init numa_set_distance(int from, int to, int distance)
 {
 	if ((u8)distance != distance || (from == to && distance != LOCAL_DISTANCE)) {
-- 
2.47.2
Re: [PATCH v2] LoongArch: acpi: Eliminate superfluous get_numa_distances_cnt()
Posted by Huacai Chen 9 months, 3 weeks ago
Applied, thanks.

Huacai

On Tue, Feb 25, 2025 at 2:48 AM WangYuli <wangyuli@uniontech.com> wrote:
>
> In LoongArch, get_numa_distances_cnt() was not in use, resulting in
> a compiler warning.
>
> Fix follow errors with clang-18 when W=1e:
>
> arch/loongarch/kernel/acpi.c:259:28: error: unused function 'get_numa_distances_cnt' [-Werror,-Wunused-function]
>   259 | static inline unsigned int get_numa_distances_cnt(struct acpi_table_slit *slit)
>       |                            ^~~~~~~~~~~~~~~~~~~~~~
> 1 error generated.
>
> Link: https://lore.kernel.org/all/Z7bHPVUH4lAezk0E@kernel.org/
> Signed-off-by: Yuli Wang <wangyuli@uniontech.com>
> ---
> Changelog:
>  *v1->v2: Drop the relocation and one more useless variable.
> ---
>  arch/loongarch/kernel/acpi.c | 12 ------------
>  1 file changed, 12 deletions(-)
>
> diff --git a/arch/loongarch/kernel/acpi.c b/arch/loongarch/kernel/acpi.c
> index ee471a80763e..193f81920074 100644
> --- a/arch/loongarch/kernel/acpi.c
> +++ b/arch/loongarch/kernel/acpi.c
> @@ -249,18 +249,6 @@ static __init int setup_node(int pxm)
>         return acpi_map_pxm_to_node(pxm);
>  }
>
> -/*
> - * Callback for SLIT parsing.  pxm_to_node() returns NUMA_NO_NODE for
> - * I/O localities since SRAT does not list them.  I/O localities are
> - * not supported at this point.
> - */
> -unsigned int numa_distance_cnt;
> -
> -static inline unsigned int get_numa_distances_cnt(struct acpi_table_slit *slit)
> -{
> -       return slit->locality_count;
> -}
> -
>  void __init numa_set_distance(int from, int to, int distance)
>  {
>         if ((u8)distance != distance || (from == to && distance != LOCAL_DISTANCE)) {
> --
> 2.47.2
>