Re: [PATCH 2/5] HID: multitouch: fix handling of buggy reportsdescriptors for Dell ST2220T

From: Jiri Kosina
Date: Mon Mar 12 2012 - 06:25:40 EST


On Tue, 6 Mar 2012, benjamin.tissoires wrote:

> From: Benjamin Tissoires <benjamin.tissoires@xxxxxxx>
>
> LG multitouch panels, such as the one found in Dell ST2220T, has buggy
> reports descriptors. With the previous implementation, it was impossible
> to rely on the reports descriptors to determine how the different
> touches are emitted from the device.
>
> This patch changes the splitting of the different touches in the report
> in a more robust way.
>
> Signed-off-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxx>
> ---
>
> This one gives me warning under Ubuntu (11.10 and 12.04 at least):
> drivers/hid/hid-multitouch.c: In function 'mt_input_mapping':
> arch/x86/include/asm/bitops.h:312:8: warning: array subscript is above array bounds [-Warray-bounds]
> arch/x86/include/asm/bitops.h:312:8: warning: array subscript is above array bounds [-Warray-bounds]
>
> I don't get any warnings under Fedora. Any help would be great.

I am not getting any with my 4.3.4 and I am not able to spot any problem
in that function. Seems like a gcc bug to me.

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