Re: [PATCH v4 05/18] ARM64 / ACPI: Introduce sleep-arm.c

[Graeme Gregory]

On Fri, Sep 12, 2014 at 04:49:03PM +0100, Catalin Marinas wrote:
> On Fri, Sep 12, 2014 at 04:18:51PM +0100, Graeme Gregory wrote:
> > On Fri, Sep 12, 2014 at 03:51:02PM +0100, Catalin Marinas wrote:
> > > On Fri, Sep 12, 2014 at 03:00:03PM +0100, Hanjun Guo wrote:
> > > > --- a/drivers/acpi/Makefile
> > > > +++ b/drivers/acpi/Makefile
> > > > @@ -23,7 +23,11 @@ acpi-y				+= nvs.o
> > > >  
> > > >  # Power management related files
> > > >  acpi-y				+= wakeup.o
> > > > +ifeq ($(ARCH), arm64)
> > > > +acpi-y				+= sleep-arm.o
> > > > +else # X86, IA64
> > > >  acpi-y				+= sleep.o
> > > > +endif
> > > >  acpi-y				+= device_pm.o
> > > >  acpi-$(CONFIG_ACPI_SLEEP)	+= proc.o
> > > 
> > > What about:
> > > 
> > > -acpi-y				+= sleep.o
> > > +acpi-$(CONFIG_ACPI_SLEEP)	+= sleep.o
> > > 
> > > and make CONFIG_ACPI_SLEEP depend on !ARM64? Does the kernel compile if
> > > you disable CONFIG_ACPI_SLEEP?
> > 
> > sleep.c is still needed in the case CONFIG_ACPI_SLEEP=n
> > 
> > basically the two functions I stubbed are always called by bus.c
> 
> So the two functions are:
> 
> acpi_target_system_state() - already defined in acpi_bus.h when
> CONFIG_ACPI_SLEEP is off.
> 
> acpi_sleep_init() - this one doesn't have a dummy implementation for
> !ACPI_SLEEP.
> 
> Now, for x86, do we need acpi_sleep_init() when CONFIG_ACPI_SLEEP is
> off? If not, we could simply add a dummy acpi_sleep_init() function when
> !ACPI_SLEEP and don't enable the config option for arm64.
> 
It looks to me that it still does processing for the S5 state which is
soft off.

It sets up the operations to be called when entering this state and also
sets up a notifier.

Graeme

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/