lkml.org 
[lkml]   [2015]   [Mar]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 3/3] toshiba_acpi: Fix the enabling of the Special Functions
    On Thu, Feb 26, 2015 at 10:57:14AM -0700, Azael Avalos wrote:
    > Some Toshiba laptops with the "Special Functions" feature enabled
    > fail to properly enable such feature unless a specific value is
    > used to enable the hotkey events.

    The specific value being... HCI_HOTKEY_ENABLE_SPECIAL? And we do this for ALL
    systems that support special keys right (and have them enabled)? Not just the
    ones that fail to properly enable them?

    The above makes it sound like a work around, but nothing in the implementation
    suggests that to me. Is this a workaround?

    > This patch adds a new function called "*_enable_special_functions",
    > that simply makes a call to the HCI_HOTKEY_EVENT call, but this time
    > we are using a different parameter to make the "Special Functions"
    > mode work as expected.
    >
    > Signed-off-by: Azael Avalos <coproscefalo@gmail.com>
    > ---
    > drivers/platform/x86/toshiba_acpi.c | 20 ++++++++++++++++++++
    > 1 file changed, 20 insertions(+)
    >
    > diff --git a/drivers/platform/x86/toshiba_acpi.c b/drivers/platform/x86/toshiba_acpi.c
    > index 10e0773..09c6a2f 100644
    > --- a/drivers/platform/x86/toshiba_acpi.c
    > +++ b/drivers/platform/x86/toshiba_acpi.c
    > @@ -2336,6 +2336,20 @@ static int toshiba_acpi_enable_hotkeys(struct toshiba_acpi_dev *dev)
    > return 0;
    > }
    >
    > +static void toshiba_acpi_enable_special_functions(struct toshiba_acpi_dev *dev)
    > +{
    > + u32 result;
    > +
    > + /*
    > + * Re-activate the hotkeys, but this time, we are using the
    > + * "Special Functions" mode.
    > + */
    > + result = hci_write1(dev, HCI_HOTKEY_EVENT,
    > + HCI_HOTKEY_ENABLE_SPECIAL);

    One line.


    --
    Darren Hart
    Intel Open Source Technology Center


    \
     
     \ /
      Last update: 2015-03-06 19:41    [W:3.169 / U:0.176 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site