On Thu, Feb 09, 2023 at 09:53:59PM +0000, Conor Dooley wrote: > On Mon, Jan 30, 2023 at 11:52:22PM +0530, Sunil V L wrote: > > Initialize ACPI tables for RISC-V during boot. > > > > Signed-off-by: Sunil V L <sunilvl@xxxxxxxxxxxxxxxx> > > --- > > arch/riscv/kernel/setup.c | 19 ++++++++++++++----- > > 1 file changed, 14 insertions(+), 5 deletions(-) > > > > diff --git a/arch/riscv/kernel/setup.c b/arch/riscv/kernel/setup.c > > index 4335f08ffaf2..5b4ad1baf664 100644 > > --- a/arch/riscv/kernel/setup.c > > +++ b/arch/riscv/kernel/setup.c > > @@ -8,6 +8,7 @@ > > * Nick Kossifidis <mick@xxxxxxxxxxxx> > > */ > > > > +#include <linux/acpi.h> > > #include <linux/init.h> > > #include <linux/mm.h> > > #include <linux/memblock.h> > > @@ -276,14 +277,22 @@ void __init setup_arch(char **cmdline_p) > > > > efi_init(); > > paging_init(); > > + > > + /* Parse the ACPI tables for possible boot-time configuration */ > > + acpi_boot_table_init(); > > + if (acpi_disabled) { > > #if IS_ENABLED(CONFIG_BUILTIN_DTB) > > I only poked it with a stick, but I think this `#if IS_ENABLED()` can > be changed to a normal `if (IS_ENABLED())` while you're already > modifying this code. > Sure, Thanks Sunil