diff mbox series

[v1,2/2] x86: edison: Switch to ACPI mode

Message ID 20200109211235.79590-2-andriy.shevchenko@linux.intel.com
State Accepted
Commit b58d85363ff5bcd4293387e9e18e12e1add69e4a
Headers show
Series [v1,1/2] x86: edison: Enable command line editing | expand

Commit Message

Andy Shevchenko Jan. 9, 2020, 9:12 p.m. UTC
SFI is quite poor and useless resource provider. Moreover it makes hard
to develop and extend functionality in the Linux kernel.

Enable a necessary minimum to use ACPI on Intel Edison.

Linux kernel have been prepared for this change since v5.4, where the last
crucial driver, i.e. for Basin Cove PMIC, has been submitted.

Note, that stock image won't suffer by this change since it doesn't have
ACPI enabled on the kernel level.

Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
---
 configs/edison_defconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Andy Shevchenko Jan. 29, 2020, 4:16 p.m. UTC | #1
On Thu, Jan 09, 2020 at 11:12:35PM +0200, Andy Shevchenko wrote:
> SFI is quite poor and useless resource provider. Moreover it makes hard
> to develop and extend functionality in the Linux kernel.
> 
> Enable a necessary minimum to use ACPI on Intel Edison.
> 
> Linux kernel have been prepared for this change since v5.4, where the last
> crucial driver, i.e. for Basin Cove PMIC, has been submitted.
> 
> Note, that stock image won't suffer by this change since it doesn't have
> ACPI enabled on the kernel level.

Bin, Simon, can we get these into 2020.04?

> Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> ---
>  configs/edison_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/configs/edison_defconfig b/configs/edison_defconfig
> index 124f0f628f..58707d619a 100644
> --- a/configs/edison_defconfig
> +++ b/configs/edison_defconfig
> @@ -6,6 +6,7 @@ CONFIG_NR_DRAM_BANKS=3
>  CONFIG_VENDOR_INTEL=y
>  CONFIG_TARGET_EDISON=y
>  CONFIG_SMP=y
> +CONFIG_GENERATE_ACPI_TABLE=y
>  CONFIG_BOARD_EARLY_INIT_R=y
>  CONFIG_LAST_STAGE_INIT=y
>  CONFIG_HUSH_PARSER=y
> -- 
> 2.24.1
>
Simon Glass Jan. 30, 2020, 2:17 a.m. UTC | #2
On Wed, 29 Jan 2020 at 09:16, Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
>
> On Thu, Jan 09, 2020 at 11:12:35PM +0200, Andy Shevchenko wrote:
> > SFI is quite poor and useless resource provider. Moreover it makes hard
> > to develop and extend functionality in the Linux kernel.
> >
> > Enable a necessary minimum to use ACPI on Intel Edison.
> >
> > Linux kernel have been prepared for this change since v5.4, where the last
> > crucial driver, i.e. for Basin Cove PMIC, has been submitted.
> >
> > Note, that stock image won't suffer by this change since it doesn't have
> > ACPI enabled on the kernel level.
>
> Bin, Simon, can we get these into 2020.04?

+Tom Rini FYI

Reviewed-by: Simon Glass <sjg at chromium.org>

I hope so. There's a small mountain of pending x86 patches. I think
Bin is probably away until next week?


- Simon
diff mbox series

Patch

diff --git a/configs/edison_defconfig b/configs/edison_defconfig
index 124f0f628f..58707d619a 100644
--- a/configs/edison_defconfig
+++ b/configs/edison_defconfig
@@ -6,6 +6,7 @@  CONFIG_NR_DRAM_BANKS=3
 CONFIG_VENDOR_INTEL=y
 CONFIG_TARGET_EDISON=y
 CONFIG_SMP=y
+CONFIG_GENERATE_ACPI_TABLE=y
 CONFIG_BOARD_EARLY_INIT_R=y
 CONFIG_LAST_STAGE_INIT=y
 CONFIG_HUSH_PARSER=y