Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Hanjun Guo
On 2015/3/17 22:33, Rafael J. Wysocki wrote: > On Tuesday, March 17, 2015 12:10:02 PM Hanjun Guo wrote: >> On 2015/3/17 11:23, Rafael J. Wysocki wrote: >>> On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: On 2015/3/17 10:28, Rafael J. Wysocki wrote: > On Tuesday, March 17, 2015

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Rafael J. Wysocki
On Tuesday, March 17, 2015 12:10:02 PM Hanjun Guo wrote: > On 2015/3/17 11:23, Rafael J. Wysocki wrote: > > On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: > >> On 2015/3/17 10:28, Rafael J. Wysocki wrote: > >>> On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: > On 2015/3/17

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Rafael J. Wysocki
On Tuesday, March 17, 2015 12:35:36 PM Lorenzo Pieralisi wrote: > On Tue, Mar 17, 2015 at 03:23:11AM +, Rafael J. Wysocki wrote: > > [...] > > > > Do you mean remove CONFIG_ACPI_GENERIC_SLEEP and > > > > > > +acpi-$(CONFIG_ACPI_SLEEP) += sleep.o > > > > > > as well (also need to remove

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Lorenzo Pieralisi
On Tue, Mar 17, 2015 at 03:23:11AM +, Rafael J. Wysocki wrote: [...] > > Do you mean remove CONFIG_ACPI_GENERIC_SLEEP and > > > > +acpi-$(CONFIG_ACPI_SLEEP) += sleep.o > > > > as well (also need to remove duplicate #ifdef CONFIG_ACPI_SLEEP in sleep.c > > if > > we doing so)? > > Well,

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Hanjun Guo
On 2015/3/17 13:59, Jon Masters wrote: > On 03/17/2015 12:10 AM, Hanjun Guo wrote: >> On 2015/3/17 11:23, Rafael J. Wysocki wrote: >>> On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: >>> Well, almost. There is one problem with that, becuase sleep.c contains code >>> outside of the

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Jon Masters
On 03/17/2015 12:10 AM, Hanjun Guo wrote: > On 2015/3/17 11:23, Rafael J. Wysocki wrote: >> On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: >> Well, almost. There is one problem with that, becuase sleep.c contains code >> outside of the ACPI_SLEEP-dependent blocks. That code is used

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Hanjun Guo
On 2015/3/17 13:59, Jon Masters wrote: On 03/17/2015 12:10 AM, Hanjun Guo wrote: On 2015/3/17 11:23, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: Well, almost. There is one problem with that, becuase sleep.c contains code outside of the

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Jon Masters
On 03/17/2015 12:10 AM, Hanjun Guo wrote: On 2015/3/17 11:23, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: Well, almost. There is one problem with that, becuase sleep.c contains code outside of the ACPI_SLEEP-dependent blocks. That code is used for

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Hanjun Guo
On 2015/3/17 22:33, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 12:10:02 PM Hanjun Guo wrote: On 2015/3/17 11:23, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: On 2015/3/17 10:28, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 09:08:45 AM

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Lorenzo Pieralisi
On Tue, Mar 17, 2015 at 03:23:11AM +, Rafael J. Wysocki wrote: [...] Do you mean remove CONFIG_ACPI_GENERIC_SLEEP and +acpi-$(CONFIG_ACPI_SLEEP) += sleep.o as well (also need to remove duplicate #ifdef CONFIG_ACPI_SLEEP in sleep.c if we doing so)? Well, almost. There is

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Rafael J. Wysocki
On Tuesday, March 17, 2015 12:10:02 PM Hanjun Guo wrote: On 2015/3/17 11:23, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: On 2015/3/17 10:28, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: On 2015/3/17 7:15, Rafael J.

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-17 Thread Rafael J. Wysocki
On Tuesday, March 17, 2015 12:35:36 PM Lorenzo Pieralisi wrote: On Tue, Mar 17, 2015 at 03:23:11AM +, Rafael J. Wysocki wrote: [...] Do you mean remove CONFIG_ACPI_GENERIC_SLEEP and +acpi-$(CONFIG_ACPI_SLEEP) += sleep.o as well (also need to remove duplicate #ifdef

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Hanjun Guo
On 2015/3/17 11:23, Rafael J. Wysocki wrote: > On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: >> On 2015/3/17 10:28, Rafael J. Wysocki wrote: >>> On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: On 2015/3/17 7:15, Rafael J. Wysocki wrote: > On Monday, March 16, 2015

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Rafael J. Wysocki
On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: > On 2015/3/17 10:28, Rafael J. Wysocki wrote: > > On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: > >> On 2015/3/17 7:15, Rafael J. Wysocki wrote: > >>> On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: > On 2015年03月14日

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Hanjun Guo
On 2015/3/17 10:28, Rafael J. Wysocki wrote: > On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: >> On 2015/3/17 7:15, Rafael J. Wysocki wrote: >>> On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: On 2015年03月14日 05:49, Rafael J. Wysocki wrote: > On Friday, March 13, 2015

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Rafael J. Wysocki
On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: > On 2015/3/17 7:15, Rafael J. Wysocki wrote: > > On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: > >> On 2015年03月14日 05:49, Rafael J. Wysocki wrote: > >>> On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: > [...] > >

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Hanjun Guo
On 2015/3/17 7:15, Rafael J. Wysocki wrote: > On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: >> On 2015年03月14日 05:49, Rafael J. Wysocki wrote: >>> On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: [...] diff --git a/arch/ia64/Kconfig b/arch/ia64/Kconfig index

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Rafael J. Wysocki
On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: > On 2015年03月14日 05:49, Rafael J. Wysocki wrote: > > On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: > >> From: Graeme Gregory > >> > >> ACPI 5.1 does not currently support S states for ARM64 hardware but > >> ACPI code will call

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Hanjun Guo
On 2015年03月14日 05:49, Rafael J. Wysocki wrote: On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: From: Graeme Gregory ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for device power management, so

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Rafael J. Wysocki
On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: On 2015年03月14日 05:49, Rafael J. Wysocki wrote: On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: From: Graeme Gregory graeme.greg...@linaro.org ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Hanjun Guo
On 2015/3/17 10:28, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: On 2015/3/17 7:15, Rafael J. Wysocki wrote: On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: On 2015年03月14日 05:49, Rafael J. Wysocki wrote: On Friday, March 13, 2015 04:14:29 PM Hanjun

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Rafael J. Wysocki
On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: On 2015/3/17 10:28, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: On 2015/3/17 7:15, Rafael J. Wysocki wrote: On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: On 2015年03月14日 05:49, Rafael

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Hanjun Guo
On 2015/3/17 7:15, Rafael J. Wysocki wrote: On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: On 2015年03月14日 05:49, Rafael J. Wysocki wrote: On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: [...] diff --git a/arch/ia64/Kconfig b/arch/ia64/Kconfig index 074e52b..e8728d7 100644

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Rafael J. Wysocki
On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: On 2015/3/17 7:15, Rafael J. Wysocki wrote: On Monday, March 16, 2015 08:14:52 PM Hanjun Guo wrote: On 2015年03月14日 05:49, Rafael J. Wysocki wrote: On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: [...] diff --git

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Hanjun Guo
On 2015年03月14日 05:49, Rafael J. Wysocki wrote: On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: From: Graeme Gregory graeme.greg...@linaro.org ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-16 Thread Hanjun Guo
On 2015/3/17 11:23, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 10:36:47 AM Hanjun Guo wrote: On 2015/3/17 10:28, Rafael J. Wysocki wrote: On Tuesday, March 17, 2015 09:08:45 AM Hanjun Guo wrote: On 2015/3/17 7:15, Rafael J. Wysocki wrote: On Monday, March 16, 2015 08:14:52 PM Hanjun

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-13 Thread Rafael J. Wysocki
On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: > From: Graeme Gregory > > ACPI 5.1 does not currently support S states for ARM64 hardware but > ACPI code will call acpi_target_system_state() and acpi_sleep_init() > for device power management, so introduce CONFIG_ACPI_GENERIC_SLEEP >

[update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-13 Thread Hanjun Guo
From: Graeme Gregory ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for device power management, so introduce CONFIG_ACPI_GENERIC_SLEEP and select it for x86 and ia64 only to make sleep functions available,

Re: [update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-13 Thread Rafael J. Wysocki
On Friday, March 13, 2015 04:14:29 PM Hanjun Guo wrote: From: Graeme Gregory graeme.greg...@linaro.org ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for device power management, so introduce

[update][PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-13 Thread Hanjun Guo
From: Graeme Gregory graeme.greg...@linaro.org ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for device power management, so introduce CONFIG_ACPI_GENERIC_SLEEP and select it for x86 and ia64 only to make

Re: [PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-12 Thread Hanjun Guo
On 2015/3/13 6:57, Rafael J. Wysocki wrote: > On Wednesday, March 11, 2015 08:39:32 PM Hanjun Guo wrote: >> From: Graeme Gregory >> >> ACPI 5.1 does not currently support S states for ARM64 hardware but >> ACPI code will call acpi_target_system_state() and acpi_sleep_init() >> for device power

Re: [PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-12 Thread Rafael J. Wysocki
On Wednesday, March 11, 2015 08:39:32 PM Hanjun Guo wrote: > From: Graeme Gregory > > ACPI 5.1 does not currently support S states for ARM64 hardware but > ACPI code will call acpi_target_system_state() and acpi_sleep_init() > for device power management, so introduce CONFIG_ACPI_GENERIC_SLEEP >

Re: [PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-12 Thread Lorenzo Pieralisi
On Wed, Mar 11, 2015 at 12:39:32PM +, Hanjun Guo wrote: > From: Graeme Gregory > > ACPI 5.1 does not currently support S states for ARM64 hardware but > ACPI code will call acpi_target_system_state() and acpi_sleep_init() > for device power management, so introduce CONFIG_ACPI_GENERIC_SLEEP

Re: [PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-12 Thread Lorenzo Pieralisi
On Wed, Mar 11, 2015 at 12:39:32PM +, Hanjun Guo wrote: From: Graeme Gregory graeme.greg...@linaro.org ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for device power management, so introduce

Re: [PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-12 Thread Hanjun Guo
On 2015/3/13 6:57, Rafael J. Wysocki wrote: On Wednesday, March 11, 2015 08:39:32 PM Hanjun Guo wrote: From: Graeme Gregory graeme.greg...@linaro.org ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for

Re: [PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-12 Thread Rafael J. Wysocki
On Wednesday, March 11, 2015 08:39:32 PM Hanjun Guo wrote: From: Graeme Gregory graeme.greg...@linaro.org ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for device power management, so introduce

[PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-11 Thread Hanjun Guo
From: Graeme Gregory ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for device power management, so introduce CONFIG_ACPI_GENERIC_SLEEP and select it for x86 and ia64 only to make sleep functions available,

[PATCH v10 06/21] ACPI / sleep: Introduce CONFIG_ACPI_GENERIC_SLEEP

2015-03-11 Thread Hanjun Guo
From: Graeme Gregory graeme.greg...@linaro.org ACPI 5.1 does not currently support S states for ARM64 hardware but ACPI code will call acpi_target_system_state() and acpi_sleep_init() for device power management, so introduce CONFIG_ACPI_GENERIC_SLEEP and select it for x86 and ia64 only to make