[PATCH] input: bcm5974: Adjust major / minor to scale

Chase Douglas chase.douglas at canonical.com
Wed Sep 1 12:42:50 UTC 2010


On Wed, 2010-09-01 at 13:46 +0200, Henrik Rydberg wrote:
> By visual inspection, the reported touch_major and touch_minor axes
> are a factor of two too small. Presumably the device actually reports
> the semi-major and semi-minor axes. Corrected with this patch.
> 
> Proposed for inclusion in maverick.
> 
> Signed-off-by: Henrik Rydberg <rydberg at euromail.se>

This is a really minor change that hopefully just makes things more
consistent across drivers.

Acked-by: Chase Douglas <chase.douglas at canonical.com>

> ---
>  drivers/input/mouse/bcm5974.c |   12 ++++++++----
>  1 files changed, 8 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/input/mouse/bcm5974.c b/drivers/input/mouse/bcm5974.c
> index ea67c49..b952317 100644
> --- a/drivers/input/mouse/bcm5974.c
> +++ b/drivers/input/mouse/bcm5974.c
> @@ -337,10 +337,14 @@ static void report_finger_data(struct input_dev *input,
>  			       const struct bcm5974_config *cfg,
>  			       const struct tp_finger *f)
>  {
> -	input_report_abs(input, ABS_MT_TOUCH_MAJOR, raw2int(f->force_major));
> -	input_report_abs(input, ABS_MT_TOUCH_MINOR, raw2int(f->force_minor));
> -	input_report_abs(input, ABS_MT_WIDTH_MAJOR, raw2int(f->size_major));
> -	input_report_abs(input, ABS_MT_WIDTH_MINOR, raw2int(f->size_minor));
> +	input_report_abs(input, ABS_MT_TOUCH_MAJOR,
> +			 raw2int(f->force_major) << 1);
> +	input_report_abs(input, ABS_MT_TOUCH_MINOR,
> +			 raw2int(f->force_minor) << 1);
> +	input_report_abs(input, ABS_MT_WIDTH_MAJOR,
> +			 raw2int(f->size_major) << 1);
> +	input_report_abs(input, ABS_MT_WIDTH_MINOR,
> +			 raw2int(f->size_minor) << 1);
>  	input_report_abs(input, ABS_MT_ORIENTATION,
>  			 MAX_FINGER_ORIENTATION - raw2int(f->orientation));
>  	input_report_abs(input, ABS_MT_POSITION_X, raw2int(f->abs_x));
> -- 
> 1.7.1





More information about the kernel-team mailing list