Re: [PATCH 19/21 v5] Input: atmel_mxt_ts - use T9 reportid range to init number of mt slots

["Henrik Rydberg" <rydberg@euromail.se>]

Hi Daniel,

> Atmel mxt devices can report one finger for each T9 reportid.
> Therefore, this range can be used to report the max number of MT-B slots
> to userspace instead of assuming a fixed 10.
> 
> Note that mxt_initialized() must complete early, since the input_dev
> properties now depend on values in the object table.
> 
> Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
> ---
>  drivers/input/touchscreen/atmel_mxt_ts.c |   16 ++++++++--------
>  1 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c
> index 4da12a9..b218b5f 100644
> --- a/drivers/input/touchscreen/atmel_mxt_ts.c
> +++ b/drivers/input/touchscreen/atmel_mxt_ts.c
> @@ -212,8 +212,6 @@
>  /* Touchscreen absolute values */
>  #define MXT_MAX_AREA		0xff
>  
> -#define MXT_MAX_FINGER		10
> -
>  struct mxt_info {
>  	u8 family_id;
>  	u8 variant_id;
> @@ -1074,6 +1072,7 @@ static int __devinit mxt_probe(struct i2c_client *client,
>  	struct mxt_data *data;
>  	struct input_dev *input_dev;
>  	int error;
> +	unsigned int num_mt_slots;
>  
>  	if (!pdata)
>  		return -EINVAL;
> @@ -1103,6 +1102,10 @@ static int __devinit mxt_probe(struct i2c_client *client,
>  
>  	mxt_calc_resolution(data);
>  
> +	error = mxt_initialize(data);
> +	if (error)
> +		goto err_free_object;
> +

The comment on the previous patch actually stems from problems in
mxt_initialize(); if a function fails, there should be no need to tear
down stuff created inside the failing function. IOW, it should be goto
err_free_mem here, and mxt_initialize() should be fixed accordingly.

>  	__set_bit(EV_ABS, input_dev->evbit);
>  	__set_bit(EV_KEY, input_dev->evbit);
>  	__set_bit(BTN_TOUCH, input_dev->keybit);
> @@ -1116,9 +1119,10 @@ static int __devinit mxt_probe(struct i2c_client *client,
>  			     0, 255, 0, 0);
>  
>  	/* For multi touch */
> -	error = input_mt_init_slots(input_dev, MXT_MAX_FINGER);
> +	num_mt_slots = data->T9_reportid_max - data->T9_reportid_min + 1;
> +	error = input_mt_init_slots(input_dev, num_mt_slots);
>  	if (error)
> -		goto err_free_mem;
> +		goto err_free_object;
>  	input_set_abs_params(input_dev, ABS_MT_TOUCH_MAJOR,
>  			     0, MXT_MAX_AREA, 0, 0);
>  	input_set_abs_params(input_dev, ABS_MT_POSITION_X,
> @@ -1131,10 +1135,6 @@ static int __devinit mxt_probe(struct i2c_client *client,
>  	input_set_drvdata(input_dev, data);
>  	i2c_set_clientdata(client, data);
>  
> -	error = mxt_initialize(data);
> -	if (error)
> -		goto err_free_object;
> -
>  	error = request_threaded_irq(client->irq, NULL, mxt_interrupt,
>  			pdata->irqflags, client->name, data);
>  	if (error) {
> -- 
> 1.7.7.3
> 

Thanks,
Henrik