linux-kernel - Re: [PATCH 6/6] Input: elan_i2c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <B2EA2005-9997-4780-9ABA-5083EB8E84CA@emc.com.tw>
Date:	Tue, 21 Apr 2015 08:45:07 +0800
From:	duson <dusonlin@....com.tw>
To:	linux-input@...r.kernel.org, linux-kernel@...r.kernel.org,
	Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc:	黃世鵬 <phoenix@....com.tw>
Subject: Re: [PATCH 6/6] Input: elan_i2c - Add hover detection flag

Hi Dmitry,

Is this patch looks good for you?
If you have any advice, please let me know.

----------------------------------------------
Thank you
ELAN Duson
✉ Email: dusonlin@....com.tw
----------------------------------------------





> duson <dusonlin@....com.tw> 於 2015年4月17日 上午9:56 寫道：
> 
> When hover event coming, set ABS_MT_DISTANCE as 1, otherwise clear to 0.
> 
> Signed-off-by: Duson Lin <dusonlin@....com.tw>
> ---
> drivers/input/mouse/elan_i2c_core.c |   15 +++++++++++----
> 1 file changed, 11 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/input/mouse/elan_i2c_core.c b/drivers/input/mouse/elan_i2c_core.c
> index 6333ba6..da7893f 100644
> --- a/drivers/input/mouse/elan_i2c_core.c
> +++ b/drivers/input/mouse/elan_i2c_core.c
> @@ -52,6 +52,7 @@
> #define ETP_REPORT_ID_OFFSET	2
> #define ETP_TOUCH_INFO_OFFSET	3
> #define ETP_FINGER_DATA_OFFSET	4
> +#define ETP_HOVER_INFO_OFFSET	30
> #define ETP_MAX_REPORT_LEN	34
> 
> /* The main device structure */
> @@ -725,7 +726,7 @@ static const struct attribute_group *elan_sysfs_groups[] = {
>  */
> static void elan_report_contact(struct elan_tp_data *data,
> 				int contact_num, bool contact_valid,
> -				u8 *finger_data)
> +				bool hover_event, u8 *finger_data)
> {
> 	struct input_dev *input = data->input;
> 	unsigned int pos_x, pos_y;
> @@ -769,7 +770,9 @@ static void elan_report_contact(struct elan_tp_data *data,
> 		input_mt_report_slot_state(input, MT_TOOL_FINGER, true);
> 		input_report_abs(input, ABS_MT_POSITION_X, pos_x);
> 		input_report_abs(input, ABS_MT_POSITION_Y, data->max_y - pos_y);
> -		input_report_abs(input, ABS_MT_PRESSURE, scaled_pressure);
> +		input_report_abs(input, ABS_MT_DISTANCE, hover_event);
> +		input_report_abs(input, ABS_MT_PRESSURE,
> +				 hover_event ? 0 : scaled_pressure);
> 		input_report_abs(input, ABS_TOOL_WIDTH, mk_x);
> 		input_report_abs(input, ABS_MT_TOUCH_MAJOR, major);
> 		input_report_abs(input, ABS_MT_TOUCH_MINOR, minor);
> @@ -785,11 +788,14 @@ static void elan_report_absolute(struct elan_tp_data *data, u8 *packet)
> 	u8 *finger_data = &packet[ETP_FINGER_DATA_OFFSET];
> 	int i;
> 	u8 tp_info = packet[ETP_TOUCH_INFO_OFFSET];
> -	bool contact_valid;
> +	u8 hover_info = packet[ETP_HOVER_INFO_OFFSET];
> +	bool contact_valid, hover_event;
> 
> +	hover_event = hover_info & 0x40;
> 	for (i = 0; i < ETP_MAX_FINGERS; i++) {
> 		contact_valid = tp_info & (1U << (3 + i));
> -		elan_report_contact(data, i, contact_valid, finger_data);
> +		elan_report_contact(data, i, contact_valid, hover_event,
> +				    finger_data);
> 
> 		if (contact_valid)
> 			finger_data += ETP_FINGER_DATA_LEN;
> @@ -883,6 +889,7 @@ static int elan_setup_input_device(struct elan_tp_data *data)
> 			     ETP_FINGER_WIDTH * max_width, 0, 0);
> 	input_set_abs_params(input, ABS_MT_TOUCH_MINOR, 0,
> 			     ETP_FINGER_WIDTH * min_width, 0, 0);
> +	input_set_abs_params(input, ABS_MT_DISTANCE, 0, 1, 0, 0);
> 
> 	data->input = input;
> 
> 
> 
> 
> 
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/