Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Linda Knippers
On 8/18/2016 3:48 PM, Dan Williams wrote: > On Thu, Aug 18, 2016 at 11:48 AM, Vishal Verma > wrote: >> The nfit driver had an acpi event notification handler, but it never >> would've worked because we weren't setting the >> ACPI_DRIVER_ALL_NOTIFY_EVENTS flag in

Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Linda Knippers
On 8/18/2016 3:48 PM, Dan Williams wrote: > On Thu, Aug 18, 2016 at 11:48 AM, Vishal Verma > wrote: >> The nfit driver had an acpi event notification handler, but it never >> would've worked because we weren't setting the >> ACPI_DRIVER_ALL_NOTIFY_EVENTS flag in acpi_driver. > > Let's update

Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Dan Williams
On Thu, Aug 18, 2016 at 1:22 PM, Verma, Vishal L wrote: > On Thu, 2016-08-18 at 12:54 -0700, Dan Williams wrote: >> On Thu, Aug 18, 2016 at 12:52 PM, Linda Knippers > om> wrote: >> > >> > >> > >> > On 8/18/2016 3:48 PM, Dan Williams wrote: >> > >

Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Dan Williams
On Thu, Aug 18, 2016 at 1:22 PM, Verma, Vishal L wrote: > On Thu, 2016-08-18 at 12:54 -0700, Dan Williams wrote: >> On Thu, Aug 18, 2016 at 12:52 PM, Linda Knippers > om> wrote: >> > >> > >> > >> > On 8/18/2016 3:48 PM, Dan Williams wrote: >> > > >> > > On Thu, Aug 18, 2016 at 11:48 AM, Vishal

Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Verma, Vishal L
On Thu, 2016-08-18 at 12:54 -0700, Dan Williams wrote: > On Thu, Aug 18, 2016 at 12:52 PM, Linda Knippers om> wrote: > > > > > > > > On 8/18/2016 3:48 PM, Dan Williams wrote: > > > > > > On Thu, Aug 18, 2016 at 11:48 AM, Vishal Verma > > el.com>

Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Verma, Vishal L
On Thu, 2016-08-18 at 12:54 -0700, Dan Williams wrote: > On Thu, Aug 18, 2016 at 12:52 PM, Linda Knippers om> wrote: > > > > > > > > On 8/18/2016 3:48 PM, Dan Williams wrote: > > > > > > On Thu, Aug 18, 2016 at 11:48 AM, Vishal Verma > > el.com> wrote: > > > > > > > > The nfit driver had an

Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Dan Williams
On Thu, Aug 18, 2016 at 12:52 PM, Linda Knippers wrote: > > > On 8/18/2016 3:48 PM, Dan Williams wrote: >> On Thu, Aug 18, 2016 at 11:48 AM, Vishal Verma >> wrote: >>> The nfit driver had an acpi event notification handler, but it never >>>

Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Dan Williams
On Thu, Aug 18, 2016 at 12:52 PM, Linda Knippers wrote: > > > On 8/18/2016 3:48 PM, Dan Williams wrote: >> On Thu, Aug 18, 2016 at 11:48 AM, Vishal Verma >> wrote: >>> The nfit driver had an acpi event notification handler, but it never >>> would've worked because we weren't setting the >>>

[PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Vishal Verma
The nfit driver had an acpi event notification handler, but it never would've worked because we weren't setting the ACPI_DRIVER_ALL_NOTIFY_EVENTS flag in acpi_driver. Cc: Cc: Cc: Dan Williams Cc: Xiao Guangrong

[PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Vishal Verma
The nfit driver had an acpi event notification handler, but it never would've worked because we weren't setting the ACPI_DRIVER_ALL_NOTIFY_EVENTS flag in acpi_driver. Cc: Cc: Cc: Dan Williams Cc: Xiao Guangrong Signed-off-by: Vishal Verma --- drivers/acpi/nfit/core.c | 1 + 1 file changed,

Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Dan Williams
On Thu, Aug 18, 2016 at 11:48 AM, Vishal Verma wrote: > The nfit driver had an acpi event notification handler, but it never > would've worked because we weren't setting the > ACPI_DRIVER_ALL_NOTIFY_EVENTS flag in acpi_driver. Let's update the changelog to be helpful

Re: [PATCH] acpi, nfit: fix acpi event notifications for nfit

2016-08-18 Thread Dan Williams
On Thu, Aug 18, 2016 at 11:48 AM, Vishal Verma wrote: > The nfit driver had an acpi event notification handler, but it never > would've worked because we weren't setting the > ACPI_DRIVER_ALL_NOTIFY_EVENTS flag in acpi_driver. Let's update the changelog to be helpful for someone implementing a