Re: [PATCH 3.0-rc1] Input: omap-keypad: add missing input_sync()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Janusz,

> Otherwise the updated evdev driver (commit 
> cdda911c34006f1089f3c87b1a1f31ab3a4722f2, "Input: evdev - only signal 
> polls on full packets") no longer works on top of omap-keypad.
> 
> Tested on Amstrad Delta.
> 
> Signed-off-by: Janusz Krzysztofik <jkrzyszt@xxxxxxxxxxxx>
> ---
>  drivers/input/keyboard/omap-keypad.c |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> --- git/drivers/input/keyboard/omap-keypad.c.orig	2011-05-30 22:16:50.000000000 +0200
> +++ git/drivers/input/keyboard/omap-keypad.c	2011-06-03 22:22:18.000000000 +0200
> @@ -169,6 +169,7 @@ static void omap_kp_tasklet(unsigned lon
>  	unsigned char new_state[8], changed, key_down = 0;
>  	int col, row;
>  	int spurious = 0;
> +	bool need_sync = false;

No need for a special variable here - the input core already does the same thing.

>  
>  	/* check for any changes */
>  	omap_kp_scan_keypad(omap_kp_data, new_state);
> @@ -206,9 +207,12 @@ static void omap_kp_tasklet(unsigned lon
>  			kp_cur_group = key & GROUP_MASK;
>  			input_report_key(omap_kp_data->input, key & ~GROUP_MASK,
>  					 new_state[col] & (1 << row));
> +			need_sync = true;
>  #endif
>  		}
>  	}
> +	if (need_sync)
> +		input_sync(omap_kp_data->input);
>  	memcpy(keypad_state, new_state, sizeof(keypad_state));
>  
>  	if (key_down) {
> --
> To unsubscribe from this list: send the line "unsubscribe linux-input" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Thanks,
Henrik
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux