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

[Huacai Chen]

Applied, thanks.

Huacai

On Tue, Feb 25, 2025 at 2:48 AM WangYuli <wangyuli@xxxxxxxxxxxxx> 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@xxxxxxxxxx/
> Signed-off-by: Yuli Wang <wangyuli@xxxxxxxxxxxxx>
> ---
> 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
>