lkml.org 
[lkml]   [2008]   [Nov]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[patch 09/23] ACPI: Always report a sync event after a lid state change
    2.6.26-stable review patch.  If anyone has any objections, please let us know.

    ------------------

    From: Guillem Jover <guillem.jover@nokia.com>

    upstream commit df316e939100e789b3c5d4d102619ccf5834bd00

    Currently not always an EV_SYN event is reported to userland
    after the EV_SW SW_LID event has been sent. This is easy to verify
    by using “input-events” from input-utils and just closing and opening
    the lid.

    Signed-off-by: Guillem Jover <guillem.jover@nokia.com>
    Acked-by: Dmitry Torokhov <dtor@mail.ru>
    Signed-off-by: Len Brown <len.brown@intel.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    drivers/acpi/button.c | 3 ++-
    1 file changed, 2 insertions(+), 1 deletion(-)

    --- a/drivers/acpi/button.c
    +++ b/drivers/acpi/button.c
    @@ -262,6 +262,7 @@ static int acpi_lid_send_state(struct ac
    return -ENODEV;
    /* input layer checks if event is redundant */
    input_report_switch(button->input, SW_LID, !state);
    + input_sync(button->input);
    return 0;
    }

    @@ -285,8 +286,8 @@ static void acpi_button_notify(acpi_hand
    input_report_key(input, keycode, 1);
    input_sync(input);
    input_report_key(input, keycode, 0);
    + input_sync(input);
    }
    - input_sync(input);

    acpi_bus_generate_proc_event(button->device, event,
    ++button->pushed);
    --
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2008-11-08 00:29    [W:0.021 / U:30.100 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site