All of lore.kernel.org
 help / color / mirror / Atom feed
From: Felipe Balbi <balbi@ti.com>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: Felipe Balbi <balbi@ti.com>, Greg KH <greg@kroah.com>,
	Jack Pham <jackp@codeaurora.org>,
	linux-usb@vger.kernel.org, Tim Sander <tim@krieglstein.org>,
	Manu Gautam <mgautam@codeaurora.org>,
	linux-arm-msm@vger.kernel.org
Subject: Re: [RFC/PATCH 2/2] usb: ehci: Add support for SINGLE_STEP_SET_FEATURE test of EHSET
Date: Fri, 9 Aug 2013 17:44:52 +0300	[thread overview]
Message-ID: <20130809144452.GR12041@radagast> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1308091035100.1405-100000@iolanthe.rowland.org>

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

Hi,

On Fri, Aug 09, 2013 at 10:37:50AM -0400, Alan Stern wrote:
> > > > Wait a minute, didn't we discuss a while back that these test features
> > > > should be built into usbcore so that we could have a usbcv clone for
> > > > linux ?
> > > 
> > > There's no way this can be built into the core.  This test requires the
> > > behavior of the host controller to be modified at a low level
> > > (introducing a 15-second delay between the Setup and Data stages of a
> > > control transfer).  Only the HCD can do that.
> > 
> > heh, it doesn't need to be entirely in the core. Core could have the
> > generic calls and HCDs could implement some callbacks, but I think quite
> > a bit of the code will be similar if we implement the same thing on all
> > HCDs.
> 
> What generic calls and callbacks would you suggest?  I assume you want 
> enough to cover not just this one test but the entire USB-CV suite.

maybe a single callback for supporting 'testmodes' ? which receives the
test mode as argument ?

> > > Besides, doesn't USB-CV install its own version of Windows' EHCI driver 
> > > while it runs its tests?
> > 
> > IIRC only USB20CV, the new USB30CV (which also tests USB2) doesn't do
> > that.
> 
> I don't have more than the foggiest notion of how these things really
> work under Windows.

and that's hardly necessary to understand...

-- 
balbi

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

  reply	other threads:[~2013-08-09 14:45 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1591113.JYQdTBbFW7@dabox>
2013-07-03  3:13 ` [PATCH 0/2] usb: Add support for EHSET Jack Pham
2013-07-03  3:13   ` [PATCH 1/2] usb: misc: EHSET Test Fixture device driver for host compliance Jack Pham
2013-07-03  3:13   ` [RFC/PATCH 2/2] usb: ehci: Add support for SINGLE_STEP_SET_FEATURE test of EHSET Jack Pham
2013-07-25 18:46     ` Greg KH
2013-07-25 19:44       ` Alan Stern
2013-07-25 20:54         ` Felipe Balbi
2013-07-25 21:33           ` Alan Stern
2013-08-09 13:41             ` Felipe Balbi
2013-08-09 14:37               ` Alan Stern
2013-08-09 14:44                 ` Felipe Balbi [this message]
2013-08-09 15:04                   ` Alan Stern
     [not found]                     ` <Pine.LNX.4.44L0.1308091053450.1405-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2013-08-12 17:53                       ` Felipe Balbi
2013-08-12 18:52                         ` Alan Stern
2013-08-13 15:32                           ` Felipe Balbi
2013-08-13 16:47                             ` Alan Stern
2013-08-13 23:05                             ` Jack Pham
2013-08-14 14:10                               ` Alan Stern
2013-08-14 17:05                                 ` Jack Pham
2013-07-25 22:09     ` Jack Pham
     [not found]       ` <20130725220925.GA28634-NjF/qFWh7jSrUKQWM4GlyCPyLMyjRtWwAL8bYrjMMd8@public.gmane.org>
2013-08-08 23:49         ` [PATCH " Jack Pham
2013-08-09  0:05           ` Greg KH
2013-08-09  2:12             ` Jack Pham
2013-08-09  2:32               ` Greg KH

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=20130809144452.GR12041@radagast \
    --to=balbi@ti.com \
    --cc=greg@kroah.com \
    --cc=jackp@codeaurora.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=mgautam@codeaurora.org \
    --cc=stern@rowland.harvard.edu \
    --cc=tim@krieglstein.org \
    /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.