All of lore.kernel.org
 help / color / mirror / Atom feed
From: Forest Bond <forest@alittletooquiet.net>
To: Jiri Kosina <jkosina@suse.cz>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Daniel Ritz <daniel.ritz@gmx.ch>,
	Alan Stern <stern@rowland.harvard.edu>,
	Sergei Shtylyov <sshtylyov@mvista.com>,
	linux-input@vger.kernel.org, linux-usb@vger.kernel.org
Subject: Re: [PATCH resend2] Input: usbtouchscreen - initialize eGalax devices
Date: Fri, 2 Nov 2012 15:51:30 -0400	[thread overview]
Message-ID: <20121102195130.GJ12501@alittletooquiet.net> (raw)
In-Reply-To: <alpine.LNX.2.00.1211011137260.6606@pobox.suse.cz>

[-- Attachment #1: Type: text/plain, Size: 1571 bytes --]

Hi Jiri,

Thanks for your message.  Pretty good timing, actually, as I was just about to
pick this back up. ;)

On Thu, Nov 01, 2012 at 11:38:19AM +0100, Jiri Kosina wrote:
> On Mon, 10 Sep 2012, Dmitry Torokhov wrote:
> 
> > > The other piece to this puzzle is that usbhid should blacklist these devices
> > > to avoid binding if it happens to be loaded before usbtouchscreen.  To do
> > > this, usbhid needs to be able to blacklist devices based on interface
> > > protocol (right now it only supports blacklist on VID + PID).
> > > 
> > > Would you accept a patch set that implements this?
> > 
> > Juri, this question is really for you...
> 
> Generally, I am not objecting to this idea in general.
> 
> What is the particular use case here, so that it's needed?

We have some eGalax devices with class HID and protocol None that both usbhid
and usbtouchscreen will bind to, but we only want them bound to usbtouchscreen.
Some do in fact work with usbhid, but not all of them do.  OTOH they all work
with usbtouchscreen as of commit 037a833ed05a86d01ea27a2c32043b86c549be1b.

We want to blacklist these devices in usbhid to avoid binding to it if it is
loaded first.  But usbhid should continue to handle eGalax devices with class
HID and protocol other than None (e.g. Mouse).  They all have the same vendor
and product IDs, so we need to be able to blacklist on (VID, PID, protocol)
instead of just (VID, PID).

Does that make sense?

Thanks,
Forest
-- 
Forest Bond
http://www.alittletooquiet.net
http://www.rapidrollout.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

  reply	other threads:[~2012-11-02 20:04 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-31 13:56 [PATCH] Input: usbtouchscreen - initialize eGalax devices Forest Bond
2012-08-31 14:51 ` Dmitry Torokhov
2012-08-31 18:50 ` Sergei Shtylyov
2012-08-31 19:26   ` Dmitry Torokhov
     [not found]     ` <20120831192632.GA30202-WlK9ik9hQGAhIp7JRqBPierSzoNAToWh@public.gmane.org>
2012-08-31 20:04       ` Alan Stern
     [not found]         ` <Pine.LNX.4.44L0.1208311555140.1328-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2012-08-31 20:22           ` Forest Bond
2012-08-31 22:53           ` Forest Bond
     [not found]             ` <20120831225353.GE24820-B/PTSs0AgtP3p6jHtUh95NHuzzzSOjJt@public.gmane.org>
2012-08-31 23:10               ` Dmitry Torokhov
     [not found]                 ` <20120831231047.GA22142-WlK9ik9hQGAhIp7JRqBPierSzoNAToWh@public.gmane.org>
2012-08-31 23:23                   ` Forest Bond
2012-09-01  0:37                     ` [PATCH resend] " Forest Bond
2012-09-03 13:26                       ` Sergey Vlasov
     [not found]                         ` <20120903132648.GC11919-TEYkr/UGJhVKdHEj4xO92LjjLBE8jN/0@public.gmane.org>
2012-09-03 17:33                           ` [PATCH resend2] " Forest Bond
     [not found]                             ` <20120903173349.GA18666-B/PTSs0AgtP3p6jHtUh95NHuzzzSOjJt@public.gmane.org>
2012-09-05  6:07                               ` Dmitry Torokhov
     [not found]                                 ` <20120905060704.GC25962-WlK9ik9hQGAhIp7JRqBPierSzoNAToWh@public.gmane.org>
2012-09-07 20:42                                   ` Forest Bond
2012-09-10 21:11                                     ` Dmitry Torokhov
2012-11-01 10:38                                       ` Jiri Kosina
2012-11-02 19:51                                         ` Forest Bond [this message]
2012-11-05 14:19                                           ` Jiri Kosina
2012-11-05 18:34                                             ` Forest Bond

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20121102195130.GJ12501@alittletooquiet.net \
    --to=forest@alittletooquiet.net \
    --cc=daniel.ritz@gmx.ch \
    --cc=dmitry.torokhov@gmail.com \
    --cc=jkosina@suse.cz \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=sshtylyov@mvista.com \
    --cc=stern@rowland.harvard.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.