public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: David Brownell <david-b@pacbell.net>
To: Greg KH <greg@kroah.com>, Marcel Janssen <marcel.janssen@admesy.nl>
Cc: Marcel Janssen <korgull@home.nl>,
	Stefan Kopp <stefan_kopp@agilent.com>,
	Felipe Balbi <me@felipebalbi.com>,
	linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: Fwd: Re: [PATCH] USB: add USB test and measurement class driver
Date: Thu, 28 Aug 2008 15:37:50 -0700	[thread overview]
Message-ID: <200808281537.50952.david-b@pacbell.net> (raw)
In-Reply-To: <20080828203557.GB20537@kroah.com>


> > I either have not seen that happening before or I missed something when 
> > writing my firmware. I'll keep an eye on my USB analyzer though :-)
> > Although it's a good thing to keep to the spec, I also believe that in the 
> > case of a test and measurement device suspending the device may be an 
> > unwanted behaviour but perhaps I do misinterpret this part of the spec here.

For the record, the "gadgetfs" interface was originally developed with
userspace USB TMC implementations in mind.  I don't recall whether that
effort got out of the lab though.

The issue of suspend/resume explicitly came up.  The scenario which
stuck in my mind was that when the TMC device was in USB-level suspend
state, the code on the peripheral would need to know about that so
it could re-enable manual controls.  The host could no longer be
issuing commands.  (Some of these folk were on the TMC design team.)

One of the customers didn't seem to like the notion of not being able
to turn off stress tests on a $US 20,000 engine ... and watching the
device(s) under test blow up.  Accidental destruction == bad, it seems!

So I think you probably do misinterpret the spec.  That issue was not
only discussed, but proper handling for it was identified.

- Dave

  reply	other threads:[~2008-08-28 22:38 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200808272342.38208.korgull@home.nl>
     [not found] ` <48B671D4.7090509@admesy.nl>
     [not found]   ` <20080828162109.GC18132@kroah.com>
2008-08-28 18:22     ` Fwd: Re: [PATCH] USB: add USB test and measurement class driver Marcel Janssen
2008-08-28 20:35       ` Greg KH
2008-08-28 22:37         ` David Brownell [this message]
     [not found]           ` <48B793E1.4000504@admesy.nl>
2008-08-30  6:09             ` 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=200808281537.50952.david-b@pacbell.net \
    --to=david-b@pacbell.net \
    --cc=greg@kroah.com \
    --cc=korgull@home.nl \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=marcel.janssen@admesy.nl \
    --cc=me@felipebalbi.com \
    --cc=stefan_kopp@agilent.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox