Re: [PATCH v2 2/3] toshiba_acpi: Change notify funtion to handle more events

From: Darren Hart
Date: Thu Dec 04 2014 - 22:12:42 EST


On Thu, Dec 04, 2014 at 08:10:49PM -0700, Azael Avalos wrote:
> Hi Darren,
>
> 2014-12-03 6:12 GMT-07:00 Darren Hart <dvhart@xxxxxxxxxxxxx>:
> > On Wed, Dec 03, 2014 at 11:42:04PM -0700, Azael Avalos wrote:
> >> @@ -1971,41 +2008,22 @@ error:
> >> static void toshiba_acpi_notify(struct acpi_device *acpi_dev, u32 event)
> >> {
> >> struct toshiba_acpi_dev *dev = acpi_driver_data(acpi_dev);
> >> - u32 hci_result, value;
> >> - int retries = 3;
> >> - int scancode;
> >> -
> >> - if (event != 0x80)
> >> - return;
> >> + int ret;
> >
> > CC drivers/platform/x86/toshiba_acpi.o
> > drivers/platform/x86/toshiba_acpi.c: In function âtoshiba_acpi_notifyâ:
> > drivers/platform/x86/toshiba_acpi.c:2012:6: warning: unused variable âretâ [-Wunused-variable]
> > int ret;
> > ^
> >
> > Please compile check each patch.
>
> Sorry about that, was a left over from the split.
> Want me to send a V3? I can send them in few minutes.

Yes please.

--
Darren Hart
Intel Open Source Technology Center
--
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/