Hi, Rafael,


> -----原始邮件-----
> 发件人: "Rafael J. Wysocki" [off-list ref]
> 发送时间: 2021-07-14 20:30:33 (星期三)
> 收件人: "Huacai Chen" [off-list ref]
> 抄送: "Rafael J . Wysocki" [off-list ref], "Len Brown" [off-list ref], "Robert Moore" [off-list ref], "Erik Kaneda" [off-list ref], "ACPI Devel Maling List" [off-list ref], "open list:ACPI COMPONENT ARCHITECTURE (ACPICA)" [off-list ref], "Xuefeng Li" [off-list ref], "Jiaxun Yang" [off-list ref]
> 主题: Re: [PATCH 1/3] ACPI: Add LoongArch support for ACPI_PROCESSOR/ACPI_NUMA
> 
> On Mon, Jul 5, 2021 at 2:41 PM Huacai Chen [off-list ref] wrote:
> >
> > We are preparing to add new Loongson (based on LoongArch, not MIPS)
> > support. LoongArch use ACPI other than DT as its boot protocol, so
> > add its support for ACPI_PROCESSOR/ACPI_NUMA.
> >
> > Signed-off-by: Huacai Chen [off-list ref]
> > ---
> >  drivers/acpi/Kconfig      | 4 ++--
> >  drivers/acpi/numa/Kconfig | 2 +-
> >  drivers/acpi/numa/srat.c  | 2 +-
> >  include/linux/acpi.h      | 2 +-
> >  4 files changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig
> > index fe0bb6277e4d..90afa44efcba 100644
> > --- a/drivers/acpi/Kconfig
> > +++ b/drivers/acpi/Kconfig
> > @@ -280,9 +280,9 @@ config ACPI_CPPC_LIB
> >
> >  config ACPI_PROCESSOR
> >         tristate "Processor"
> > -       depends on X86 || IA64 || ARM64
> > +       depends on X86 || IA64 || ARM64 || LOONGARCH
> >         select ACPI_PROCESSOR_IDLE
> > -       select ACPI_CPU_FREQ_PSS if X86 || IA64
> > +       select ACPI_CPU_FREQ_PSS if X86 || IA64 || LOONGARCH
> >         default y
> >         help
> >           This driver adds support for the ACPI Processor package. It is required
> > diff --git a/drivers/acpi/numa/Kconfig b/drivers/acpi/numa/Kconfig
> > index fcf2e556d69d..39b1f34c21df 100644
> > --- a/drivers/acpi/numa/Kconfig
> > +++ b/drivers/acpi/numa/Kconfig
> > @@ -2,7 +2,7 @@
> >  config ACPI_NUMA
> >         bool "NUMA support"
> >         depends on NUMA
> > -       depends on (X86 || IA64 || ARM64)
> > +       depends on (X86 || IA64 || ARM64 || LOONGARCH)
> >         default y if IA64 || ARM64
> >
> >  config ACPI_HMAT
> > diff --git a/drivers/acpi/numa/srat.c b/drivers/acpi/numa/srat.c
> > index 6021a1013442..b8795fc49097 100644
> > --- a/drivers/acpi/numa/srat.c
> > +++ b/drivers/acpi/numa/srat.c
> > @@ -206,7 +206,7 @@ int __init srat_disabled(void)
> >         return acpi_numa < 0;
> >  }
> >
> > -#if defined(CONFIG_X86) || defined(CONFIG_ARM64)
> > +#if defined(CONFIG_X86) || defined(CONFIG_ARM64) || defined(CONFIG_LOONGARCH)
> >  /*
> >   * 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
> > diff --git a/include/linux/acpi.h b/include/linux/acpi.h
> > index 6bb36fd6ba31..3ba8511cbede 100644
> > --- a/include/linux/acpi.h
> > +++ b/include/linux/acpi.h
> > @@ -249,7 +249,7 @@ void acpi_table_print_madt_entry (struct acpi_subtable_header *madt);
> >  /* the following numa functions are architecture-dependent */
> >  void acpi_numa_slit_init (struct acpi_table_slit *slit);
> >
> > -#if defined(CONFIG_X86) || defined(CONFIG_IA64)
> > +#if defined(CONFIG_X86) || defined(CONFIG_IA64) || defined(CONFIG_LOONGARCH)
> >  void acpi_numa_processor_affinity_init (struct acpi_srat_cpu_affinity *pa);
> >  #else
> >  static inline void
> > --
> 
> Does this patch alone make sense without the other two in the series?
> If so, I can queue it up for 5.15, so please let me know.
Yes, this patch has no dependency with other two.

Huacai
> 
> Thanks!
[off-list ref][off-list ref][off-list ref][off-list ref][off-list ref][off-list ref][off-list ref][off-list ref][off-list ref][off-list ref][off-list ref][off-list ref]