From: "Tanya Brokhman" <tlinder@codeaurora.org>
To: 'Sebastian Andrzej Siewior' <bigeasy@linutronix.de>
Cc: balbi@ti.com, linux-usb@vger.kernel.org,
linux-arm-msm@vger.kernel.org, ablay@codeaurora.org,
'open list' <linux-kernel@vger.kernel.org>
Subject: RE: [RFC/PATCH v2 1/2] usb:dummy_hcd: connect/disconnect test support
Date: Wed, 22 Jun 2011 13:21:43 +0300 [thread overview]
Message-ID: <021101cc30c6$30dd7760$92986620$@org> (raw)
In-Reply-To: <20110622095205.GA24674@linutronix.de>
Hi Sebastian,
> * Tatyana Brokhman | 2011-06-22 11:39:00 [+0300]:
>
> >This implementation adds a new proprietary device control requests (to
> be
> >handled by the dummy_hcd) that initiates a connect/disconnect
> sequence.
> >The bRequest value of the new control request is 0x52.
> >It is used by the user-space Unit testing application.
> >
> >Signed-off-by: Tatyana Linder <tlinder@codeaurora.org>
>
> As far as I understand this should be used to test real Hardware. Mind
> moving this to composite_setup() so it can be re-used by other ucd as
> well? usb_gadget_*nect() should do the pullup() trick.
>
Felipe also suggested that on the last version of this patch. Here is a
quote of his email on the subject:
"actually, this is quite useful. Specially for the controllers which _can_
do soft-connect by toggling data pullups. I would rather have these sort of
thing maybe in composite.c, so that we can build tests with all gadget
drivers/controllers, not only dummy_hcd.
That said, I think now it's not the time to fiddle too much with these
details right now. So best to keep this in dummy_hcd until we know the test
tools are actually good."
So, right now I'll leave it in dummy_hcd if it's ok with you.
Thanks,
Tanya Brokhman
---
Sent by an consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
WARNING: multiple messages have this Message-ID (diff)
From: "Tanya Brokhman" <tlinder@codeaurora.org>
To: "'Sebastian Andrzej Siewior'" <bigeasy@linutronix.de>
Cc: <balbi@ti.com>, <linux-usb@vger.kernel.org>,
<linux-arm-msm@vger.kernel.org>, <ablay@codeaurora.org>,
"'open list'" <linux-kernel@vger.kernel.org>
Subject: RE: [RFC/PATCH v2 1/2] usb:dummy_hcd: connect/disconnect test support
Date: Wed, 22 Jun 2011 13:21:43 +0300 [thread overview]
Message-ID: <021101cc30c6$30dd7760$92986620$@org> (raw)
In-Reply-To: <20110622095205.GA24674@linutronix.de>
Hi Sebastian,
> * Tatyana Brokhman | 2011-06-22 11:39:00 [+0300]:
>
> >This implementation adds a new proprietary device control requests (to
> be
> >handled by the dummy_hcd) that initiates a connect/disconnect
> sequence.
> >The bRequest value of the new control request is 0x52.
> >It is used by the user-space Unit testing application.
> >
> >Signed-off-by: Tatyana Linder <tlinder@codeaurora.org>
>
> As far as I understand this should be used to test real Hardware. Mind
> moving this to composite_setup() so it can be re-used by other ucd as
> well? usb_gadget_*nect() should do the pullup() trick.
>
Felipe also suggested that on the last version of this patch. Here is a
quote of his email on the subject:
"actually, this is quite useful. Specially for the controllers which _can_
do soft-connect by toggling data pullups. I would rather have these sort of
thing maybe in composite.c, so that we can build tests with all gadget
drivers/controllers, not only dummy_hcd.
That said, I think now it's not the time to fiddle too much with these
details right now. So best to keep this in dummy_hcd until we know the test
tools are actually good."
So, right now I'll leave it in dummy_hcd if it's ok with you.
Thanks,
Tanya Brokhman
---
Sent by an consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
next prev parent reply other threads:[~2011-06-22 10:21 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-22 8:38 [RFC/PATCH v2 0/2] usb:tests: kernel side patches Tatyana Brokhman
2011-06-22 8:39 ` [RFC/PATCH v2 1/2] usb:dummy_hcd: connect/disconnect test support Tatyana Brokhman
2011-06-22 8:39 ` Tatyana Brokhman
[not found] ` <1308731945-7601-2-git-send-email-tlinder-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2011-06-22 9:52 ` Sebastian Andrzej Siewior
2011-06-22 9:52 ` Sebastian Andrzej Siewior
2011-06-22 10:21 ` Tanya Brokhman [this message]
2011-06-22 10:21 ` Tanya Brokhman
2011-06-22 8:39 ` [RFC/PATCH v2 2/2] usb:g_zero: bulk in/out unittest support Tatyana Brokhman
2011-06-22 8:39 ` Tatyana Brokhman
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='021101cc30c6$30dd7760$92986620$@org' \
--to=tlinder@codeaurora.org \
--cc=ablay@codeaurora.org \
--cc=balbi@ti.com \
--cc=bigeasy@linutronix.de \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.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.