Re: [PATCH] toshiba_acpi: Initialize hotkey_event_type variable

From: Azael Avalos
Date: Wed Nov 04 2015 - 11:07:20 EST


Hi Darren,

2015-11-03 17:44 GMT-07:00 Darren Hart <dvhart@xxxxxxxxxxxxx>:
> On Sun, Nov 01, 2015 at 07:30:30PM -0700, Azael Avalos wrote:
>> Commit 23f8f4326a15 ("toshiba_acpi: Fix hotkeys registration on some
>> toshiba models") fixed an issue on some laptops regarding hotkeys
>> registration, however, if failed to address the initialization of the
>> hotkey_event_type variable, and thus, it can lead to potential
>> unwanted effects as the variable is being checked.
>>
>> This patch initializes such variable to avoid such unwanted effects.
>>
>> Cc: <stable@xxxxxxxxxxxxxxx> # 4.1+
>> Signed-off-by: Azael Avalos <coproscefalo@xxxxxxxxx>
>> ---
>> Darren,
>>
>> Sorry for sending this patch this late, I had a really busy week :-(
>
> It's a trivial patch, so I'm not concerned with the timing. Thanks for getting
> it out!
>
> Unfortunately, it doesn't apply and appears to be whitespace mangled. Checkpatch
> complains and tabs appear to have been replaced with spaces. Please check your
> send mail mechanism and resend - possibly using git-send-email or something like
> mutt -H PATCHNAME. Try sending to yourself first and verifying you can apply it.

Ughh, sorry about this, I sent this email directly instead of git-send-email,
so the copy-paste might have changed something.

The version of git I'm currently using parses the stable address as
stable@xxxxxxxxxxxxxxx#4.1+ and gsmtp complains about a malformed address...

I'll try to resend this patch ASAP.


>
> Thanks,
>

Cheers
Azael



--
-- El mundo apesta y vosotros apestais tambien --
--
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/