linux-kernel - Re: [PATCH] HID: use multi input quirk for TouchPack touchscreen

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: <20100204175345.GL1445@holoscopio.com>
Date:	Thu, 4 Feb 2010 15:53:45 -0200
From:	Thadeu Lima de Souza Cascardo <cascardo@...oscopio.com>
To:	Jiri Kosina <jkosina@...e.cz>
Cc:	don@...t.com.br, linux-input@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-usb@...r.kernel.org
Subject: Re: [PATCH] HID: use multi input quirk for TouchPack touchscreen

On Thu, Feb 04, 2010 at 06:46:26PM +0100, Jiri Kosina wrote:
> On Thu, 4 Feb 2010, Thadeu Lima de Souza Cascardo wrote:
> 
> > This device generates ABS_Z and ABS_RX events, while it should be
> > generating ABS_X and ABS_Y instead. Using the MULTI_INPUT quirk solves
> > this issue.
> > 
> > Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@...oscopio.com>
> > Signed-off-by: Daniel Oliveira Nascimento <don@...t.com.br>
> > ---
> >  drivers/hid/hid-ids.h           |    3 +++
> >  drivers/hid/usbhid/hid-quirks.c |    1 +
> >  2 files changed, 4 insertions(+), 0 deletions(-)
> > 
> > diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h
> > index e99485f..ba39103 100644
> > --- a/drivers/hid/hid-ids.h
> > +++ b/drivers/hid/hid-ids.h
> > @@ -398,6 +398,9 @@
> >  
> >  #define USB_VENDOR_ID_THRUSTMASTER	0x044f
> >  
> > +#define USB_VENDOR_ID_TOUCHPACK		0x1bfd
> > +#define USB_DEVIDE_ID_TOUCHPAK_RTS	0x1688
> > +
> 
> Umm .. there are two typos in this single line. Please compile-test your 
> patches before submitting them.
> 
> I have fixed it and applied, thanks.
> 

Heh! Daniel has pointed this out to me too. I'm pretty sure I've
finished this build, but something must have happened. Perhaps, this was
when my computer has turned off because of bad wiring and I forgot
completely that this was building.

I am sorry for this. And thanks for the fix.

> >  #define USB_VENDOR_ID_TOPMAX		0x0663
> >  #define USB_DEVICE_ID_TOPMAX_COBRAPAD	0x0103
> >  
> > diff --git a/drivers/hid/usbhid/hid-quirks.c b/drivers/hid/usbhid/hid-quirks.c
> > index edc598d..b89717d 100644
> > --- a/drivers/hid/usbhid/hid-quirks.c
> > +++ b/drivers/hid/usbhid/hid-quirks.c
> > @@ -45,6 +45,7 @@ static const struct hid_blacklist {
> >  	{ USB_VENDOR_ID_ETURBOTOUCH, USB_DEVICE_ID_ETURBOTOUCH, HID_QUIRK_MULTI_INPUT },
> >  	{ USB_VENDOR_ID_PANTHERLORD, USB_DEVICE_ID_PANTHERLORD_TWIN_USB_JOYSTICK, HID_QUIRK_MULTI_INPUT | HID_QUIRK_SKIP_OUTPUT_REPORTS },
> >  	{ USB_VENDOR_ID_PLAYDOTCOM, USB_DEVICE_ID_PLAYDOTCOM_EMS_USBII, HID_QUIRK_MULTI_INPUT },
> > +	{ USB_VENDOR_ID_TOUCHPACK, USB_DEVICE_ID_TOUCHPACK_RTS, HID_QUIRK_MULTI_INPUT },
> >  
> >  	{ USB_VENDOR_ID_ATEN, USB_DEVICE_ID_ATEN_UC100KM, HID_QUIRK_NOGET },
> >  	{ USB_VENDOR_ID_ATEN, USB_DEVICE_ID_ATEN_CS124U, HID_QUIRK_NOGET },
> > -- 
> > 1.6.6.1
> > 
> 
> -- 
> Jiri Kosina
> SUSE Labs, Novell Inc.

Download attachment "signature.asc" of type "application/pgp-signature" (199 bytes)