linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH RESEND] toshiba_acpi: Initialize hotkey_event_type variable
@ 2015-11-04 16:28 Azael Avalos
  2015-11-04 16:47 ` Darren Hart
  2015-11-06  0:11 ` Darren Hart
  0 siblings, 2 replies; 3+ messages in thread
From: Azael Avalos @ 2015-11-04 16:28 UTC (permalink / raw)
  To: Darren Hart, platform-driver-x86, linux-kernel; +Cc: Azael Avalos, stable

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@vger.kernel.org> # 4.1+
Signed-off-by: Azael Avalos <coproscefalo@gmail.com>
---
 drivers/platform/x86/toshiba_acpi.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/platform/x86/toshiba_acpi.c b/drivers/platform/x86/toshiba_acpi.c
index beb709f..c013029 100644
--- a/drivers/platform/x86/toshiba_acpi.c
+++ b/drivers/platform/x86/toshiba_acpi.c
@@ -2665,6 +2665,7 @@ static int toshiba_acpi_add(struct acpi_device *acpi_dev)
 	ret = toshiba_function_keys_get(dev, &dev->special_functions);
 	dev->kbd_function_keys_supported = !ret;
 
+	dev->hotkey_event_type = 0;
 	if (toshiba_acpi_setup_keyboard(dev))
 		pr_info("Unable to activate hotkeys\n");
 
-- 
2.6.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH RESEND] toshiba_acpi: Initialize hotkey_event_type variable
  2015-11-04 16:28 [PATCH RESEND] toshiba_acpi: Initialize hotkey_event_type variable Azael Avalos
@ 2015-11-04 16:47 ` Darren Hart
  2015-11-06  0:11 ` Darren Hart
  1 sibling, 0 replies; 3+ messages in thread
From: Darren Hart @ 2015-11-04 16:47 UTC (permalink / raw)
  To: Azael Avalos; +Cc: platform-driver-x86, linux-kernel, stable

On Wed, Nov 04, 2015 at 09:28:26AM -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@vger.kernel.org> # 4.1+
> Signed-off-by: Azael Avalos <coproscefalo@gmail.com>

Applied, thanks Azael.
-- 
Darren Hart
Intel Open Source Technology Center

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH RESEND] toshiba_acpi: Initialize hotkey_event_type variable
  2015-11-04 16:28 [PATCH RESEND] toshiba_acpi: Initialize hotkey_event_type variable Azael Avalos
  2015-11-04 16:47 ` Darren Hart
@ 2015-11-06  0:11 ` Darren Hart
  1 sibling, 0 replies; 3+ messages in thread
From: Darren Hart @ 2015-11-06  0:11 UTC (permalink / raw)
  To: Azael Avalos; +Cc: platform-driver-x86, linux-kernel, stable

On Wed, Nov 04, 2015 at 09:28:26AM -0700, Azael Avalos wrote:
> Commit 23f8f4326a15 ("toshiba_acpi: Fix hotkeys registration on some

Note that the above commit ID does not exist in mainline. The corresponding ID
is 53147b6cabee5e8d1997b5682fcc0c3b72ddf9c2. Please always use the mainling
commit ID (as opposed to one from your development tree) when referring to a
commit. I've corrected this it the version I'll submit to Linus.

-- 
Darren Hart
Intel Open Source Technology Center

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-11-06  0:11 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-04 16:28 [PATCH RESEND] toshiba_acpi: Initialize hotkey_event_type variable Azael Avalos
2015-11-04 16:47 ` Darren Hart
2015-11-06  0:11 ` Darren Hart

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).