Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1311680

Re: [PATCH] driver-core: platform: automatically mark wakeup devices

Show all headers | View raw

On Mon, Jan 18, 2016 at 4:58 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
>
>
> On 18/01/16 15:41, Rafael J. Wysocki wrote:
>>
>> On Monday, January 18, 2016 03:23:18 PM Sudeep Holla wrote:
>>>
>>> On Mon, Jan 18, 2016 at 2:47 PM, Rafael J. Wysocki <rjw@rjwysocki.net>
>>> wrote:
>>>>
>>>> On Sunday, January 17, 2016 06:11:38 PM Dmitry Torokhov wrote:
>>>>>
>>>>> When probing platform drivers let's check if corresponding devices have
>>>>> "wakeup-source" property defined (either in device tree, ACPI, or
>>>>> static
>>>>> platform properties) and automatically enable such devices as wakeup
>>>>> sources for the system. This will help us standardize on the name for
>>>>> this
>>>>> property and reduce amount of boilerplate code in the drivers.
>>>>
>>>>
>>>> ACPI has other ways of telling the OS that the device is wakeup-capable,
>>>> but I guess the property in question can be used too (as long as it is
>>>> consistent with the other methods).
>>>>
>>>
>>> Just curious to know what you mean when you say this property can also
>>> be used with ACPI. Do you mean we could use "wakeup-source" DSD ?
>>
>>
>> Yes.
>>
>>> If so, won't that go against rule for DSD (i.e we *should not* bypass the
>>> existing mechanisms defined by the ACPI, e.g. _SxW in this case)
>>
>>
>> Not necessarily.
>>
>> What if the device doesn't use ACPI PM and still can wake up the system?
>>
>
> May be I don't understand the exact configuration you are referring, but
> if you don't use ACPI PM, how will that system enter sleep states ?
> IIUC you are referring suspend-to-idle case only here which doesn't
> require any ACPI _Sx methods, which make sense.

That plus PM on HW-reduced ACPI platforms where GPIOs can be used for
system wakeup in principle (although admittedly I have no experience
with such platforms, so this is just a theory).

> But still I don't see a strong reason to provide an alternate method to
> specify the same information. Firmware responsible for ACPI tables have
> to specify this DSD in question, instead it can use the existing
> mechanism in ACPI.

We actually don't use _SxW to determine whether or not devices are
wakeup-capable.  We look at _PRW and we set "wakeup-capable" if it is
present and the contents is valid.

> Let me know if I am missing some information about the systems you are
> referring ?

That said I'm actually quite unsure about the property suggested by
Dmitry, but let me reply to him on that. :-)

Thanks,
Rafael

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] driver-core: platform: automatically mark wakeup devices Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-01-18 03:20 +0100
  Re: [PATCH] driver-core: platform: automatically mark wakeup devices Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-01-18 06:20 +0100
    Re: [PATCH] driver-core: platform: automatically mark wakeup devices Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-01-18 07:20 +0100
      Re: [PATCH] driver-core: platform: automatically mark wakeup devices Sudeep Holla <sudeep.holla@arm.com> - 2016-01-18 16:40 +0100
  Re: [PATCH] driver-core: platform: automatically mark wakeup devices "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-01-18 15:50 +0100
    Re: [PATCH] driver-core: platform: automatically mark wakeup devices Sudeep Holla <sudeep.holla@arm.com> - 2016-01-18 16:30 +0100
      Re: [PATCH] driver-core: platform: automatically mark wakeup devices "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-01-18 16:50 +0100
        Re: [PATCH] driver-core: platform: automatically mark wakeup devices Sudeep Holla <sudeep.holla@arm.com> - 2016-01-18 17:00 +0100
          Re: [PATCH] driver-core: platform: automatically mark wakeup devices "Rafael J. Wysocki" <rafael@kernel.org> - 2016-01-18 18:10 +0100
    Re: [PATCH] driver-core: platform: automatically mark wakeup devices Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-01-18 17:30 +0100
      Re: [PATCH] driver-core: platform: automatically mark wakeup devices "Rafael J. Wysocki" <rafael@kernel.org> - 2016-01-18 18:20 +0100
        Re: [PATCH] driver-core: platform: automatically mark wakeup devices Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-01-18 19:00 +0100
          Re: [PATCH] driver-core: platform: automatically mark wakeup devices "Rafael J. Wysocki" <rafael@kernel.org> - 2016-01-18 23:20 +0100
            Re: [PATCH] driver-core: platform: automatically mark wakeup devices Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-01-20 02:00 +0100
              Re: [PATCH] driver-core: platform: automatically mark wakeup devices "Rafael J. Wysocki" <rafael@kernel.org> - 2016-01-20 03:50 +0100
                Re: [PATCH] driver-core: platform: automatically mark wakeup devices "Rafael J. Wysocki" <rafael@kernel.org> - 2016-01-20 15:00 +0100
                Re: [PATCH] driver-core: platform: automatically mark wakeup devices "Rafael J. Wysocki" <rafael@kernel.org> - 2016-01-21 00:10 +0100
                Re: [PATCH] driver-core: platform: automatically mark wakeup devices Dmitry Torokhov <dmitry.torokhov@gmail.com> - 2016-01-21 01:30 +0100
                Re: [PATCH] driver-core: platform: automatically mark wakeup devices "Rafael J. Wysocki" <rafael@kernel.org> - 2016-01-26 17:50 +0100

csiph-web