From: "Rafael J. Wysocki" <rjw@rjwysocki.net> To: Hanjun Guo <hanjun.guo@linaro.org> Cc: Catalin Marinas <catalin.marinas@arm.com>, Graeme Gregory <graeme.gregory@linaro.org>, Tony Luck <tony.luck@intel.com>, Thomas Gleixner <tglx@linutronix.de>, linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linaro-acpi@lists.linaro.org Subject: Re: [PATCH 1/4] ACPI: ARM64 does not have a BIOS add config for BIOS table scan. Date: Mon, 07 Jul 2014 23:06:05 +0200 [thread overview] Message-ID: <1430951.SWzYBgFe0M@vostro.rjw.lan> (raw) In-Reply-To: <1404290847-7671-2-git-send-email-hanjun.guo@linaro.org> On Wednesday, July 02, 2014 04:47:23 PM Hanjun Guo wrote: > From: Graeme Gregory <graeme.gregory@linaro.org> > > With the addition of ARM64 that does not have a traditional BIOS to > scan, add a config option which is selected on x86 and ia64 to do > the traditional BIOS scanning for tables. > > Signed-off-by: Graeme Gregory <graeme.gregory@linaro.org> > Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> > --- > arch/ia64/Kconfig | 1 + > arch/x86/Kconfig | 1 + > drivers/acpi/Kconfig | 3 +++ > drivers/acpi/osl.c | 4 +++- > 4 files changed, 8 insertions(+), 1 deletion(-) > > diff --git a/arch/ia64/Kconfig b/arch/ia64/Kconfig > index 2f3abcf..f82b352 100644 > --- a/arch/ia64/Kconfig > +++ b/arch/ia64/Kconfig > @@ -47,6 +47,7 @@ config IA64 > select MODULES_USE_ELF_RELA > select ARCH_USE_CMPXCHG_LOCKREF > select HAVE_ARCH_AUDITSYSCALL > + select ACPI_SCAN_BIOS_NOT_EFI I would call this ACPI_LEGACY_TABLES_LOOKUP or something like that. Also please do + select ACPI_LEGACY_TABLES_LOOKUP if ACPI in analogy with patch [3/4] in this series. > default y > help > The Itanium Processor Family is Intel's 64-bit successor to > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index a8f749e..4865d44 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -131,6 +131,7 @@ config X86 > select HAVE_CC_STACKPROTECTOR > select GENERIC_CPU_AUTOPROBE > select HAVE_ARCH_AUDITSYSCALL > + select ACPI_SCAN_BIOS_NOT_EFI > > config INSTRUCTION_DECODER > def_bool y > diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig > index a34a228..70eaf7a 100644 > --- a/drivers/acpi/Kconfig > +++ b/drivers/acpi/Kconfig > @@ -2,6 +2,9 @@ > # ACPI Configuration > # > > +config ACPI_SCAN_BIOS_NOT_EFI > + bool > + > menuconfig ACPI > bool "ACPI (Advanced Configuration and Power Interface) Support" > depends on !IA64_HP_SIM > diff --git a/drivers/acpi/osl.c b/drivers/acpi/osl.c > index bad25b0..5ba0b74 100644 > --- a/drivers/acpi/osl.c > +++ b/drivers/acpi/osl.c > @@ -259,12 +259,14 @@ acpi_physical_address __init acpi_os_get_root_pointer(void) > "System description tables not found\n"); > return 0; > } > - } else { > + } else if (IS_ENABLED(CONFIG_ACPI_SCAN_BIOS_NOT_EFI)) { > acpi_physical_address pa = 0; > > acpi_find_root_pointer(&pa); > return pa; > } > + > + return 0; > } > > /* Must be called with 'acpi_ioremap_lock' or RCU read lock held. */ > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.
WARNING: multiple messages have this Message-ID (diff)
From: rjw@rjwysocki.net (Rafael J. Wysocki) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/4] ACPI: ARM64 does not have a BIOS add config for BIOS table scan. Date: Mon, 07 Jul 2014 23:06:05 +0200 [thread overview] Message-ID: <1430951.SWzYBgFe0M@vostro.rjw.lan> (raw) In-Reply-To: <1404290847-7671-2-git-send-email-hanjun.guo@linaro.org> On Wednesday, July 02, 2014 04:47:23 PM Hanjun Guo wrote: > From: Graeme Gregory <graeme.gregory@linaro.org> > > With the addition of ARM64 that does not have a traditional BIOS to > scan, add a config option which is selected on x86 and ia64 to do > the traditional BIOS scanning for tables. > > Signed-off-by: Graeme Gregory <graeme.gregory@linaro.org> > Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> > --- > arch/ia64/Kconfig | 1 + > arch/x86/Kconfig | 1 + > drivers/acpi/Kconfig | 3 +++ > drivers/acpi/osl.c | 4 +++- > 4 files changed, 8 insertions(+), 1 deletion(-) > > diff --git a/arch/ia64/Kconfig b/arch/ia64/Kconfig > index 2f3abcf..f82b352 100644 > --- a/arch/ia64/Kconfig > +++ b/arch/ia64/Kconfig > @@ -47,6 +47,7 @@ config IA64 > select MODULES_USE_ELF_RELA > select ARCH_USE_CMPXCHG_LOCKREF > select HAVE_ARCH_AUDITSYSCALL > + select ACPI_SCAN_BIOS_NOT_EFI I would call this ACPI_LEGACY_TABLES_LOOKUP or something like that. Also please do + select ACPI_LEGACY_TABLES_LOOKUP if ACPI in analogy with patch [3/4] in this series. > default y > help > The Itanium Processor Family is Intel's 64-bit successor to > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index a8f749e..4865d44 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -131,6 +131,7 @@ config X86 > select HAVE_CC_STACKPROTECTOR > select GENERIC_CPU_AUTOPROBE > select HAVE_ARCH_AUDITSYSCALL > + select ACPI_SCAN_BIOS_NOT_EFI > > config INSTRUCTION_DECODER > def_bool y > diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig > index a34a228..70eaf7a 100644 > --- a/drivers/acpi/Kconfig > +++ b/drivers/acpi/Kconfig > @@ -2,6 +2,9 @@ > # ACPI Configuration > # > > +config ACPI_SCAN_BIOS_NOT_EFI > + bool > + > menuconfig ACPI > bool "ACPI (Advanced Configuration and Power Interface) Support" > depends on !IA64_HP_SIM > diff --git a/drivers/acpi/osl.c b/drivers/acpi/osl.c > index bad25b0..5ba0b74 100644 > --- a/drivers/acpi/osl.c > +++ b/drivers/acpi/osl.c > @@ -259,12 +259,14 @@ acpi_physical_address __init acpi_os_get_root_pointer(void) > "System description tables not found\n"); > return 0; > } > - } else { > + } else if (IS_ENABLED(CONFIG_ACPI_SCAN_BIOS_NOT_EFI)) { > acpi_physical_address pa = 0; > > acpi_find_root_pointer(&pa); > return pa; > } > + > + return 0; > } > > /* Must be called with 'acpi_ioremap_lock' or RCU read lock held. */ > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.
next prev parent reply other threads:[~2014-07-07 21:06 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-07-02 8:47 [PATCH 0/4] Clean up ACPI core to prepare for running ACPI on ARM64 Hanjun Guo 2014-07-02 8:47 ` Hanjun Guo 2014-07-02 8:47 ` [PATCH 1/4] ACPI: ARM64 does not have a BIOS add config for BIOS table scan Hanjun Guo 2014-07-02 8:47 ` Hanjun Guo 2014-07-07 21:06 ` Rafael J. Wysocki [this message] 2014-07-07 21:06 ` Rafael J. Wysocki 2014-07-08 9:55 ` Hanjun Guo 2014-07-08 9:55 ` Hanjun Guo 2014-07-08 9:55 ` Hanjun Guo 2014-07-02 8:47 ` [PATCH 2/4] ACPI: Don't use acpi_lapic in ACPI core code Hanjun Guo 2014-07-02 8:47 ` Hanjun Guo 2014-07-07 21:08 ` Rafael J. Wysocki 2014-07-07 21:08 ` Rafael J. Wysocki 2014-07-08 10:07 ` Hanjun Guo 2014-07-08 10:07 ` Hanjun Guo 2014-07-08 10:07 ` Hanjun Guo 2014-07-02 8:47 ` [PATCH 3/4] ACPI / processor: Introduce ARCH_HAS_ACPI_PDC Hanjun Guo 2014-07-02 8:47 ` Hanjun Guo 2014-07-07 21:13 ` Rafael J. Wysocki 2014-07-07 21:13 ` Rafael J. Wysocki 2014-07-08 9:58 ` Hanjun Guo 2014-07-08 9:58 ` Hanjun Guo 2014-07-08 9:58 ` Hanjun Guo 2014-07-02 8:47 ` [PATCH 4/4] ACPI: Make EC debugfs depend on X86 || IA64 in Kconfig Hanjun Guo 2014-07-02 8:47 ` Hanjun Guo 2014-07-07 21:16 ` Rafael J. Wysocki 2014-07-07 21:16 ` Rafael J. Wysocki 2014-07-08 9:23 ` Hanjun Guo 2014-07-08 9:23 ` Hanjun Guo 2014-07-08 9:23 ` Hanjun Guo
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1430951.SWzYBgFe0M@vostro.rjw.lan \ --to=rjw@rjwysocki.net \ --cc=catalin.marinas@arm.com \ --cc=graeme.gregory@linaro.org \ --cc=hanjun.guo@linaro.org \ --cc=linaro-acpi@lists.linaro.org \ --cc=linux-acpi@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=tglx@linutronix.de \ --cc=tony.luck@intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.