All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Frank Schäfer" <fschaefer.oss@googlemail.com>
To: Jean-Francois Thibert <jfthibert@google.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH] Add support for KWorld UB435-Q V2
Date: Thu, 10 Oct 2013 15:38:18 +0200	[thread overview]
Message-ID: <5256ADCA.2020005@googlemail.com> (raw)
In-Reply-To: <CACxGHmN6Vj9sN1XB7W6-24MLCarSvhpxyMYNZHin5+uB9Rr6rQ@mail.gmail.com>

Am 09.10.2013 15:48, schrieb Jean-Francois Thibert:
> On Tue, Oct 8, 2013 at 12:24 PM, Frank Schäfer
> <fschaefer.oss@googlemail.com> wrote:
>>> This adds support for the UB435-Q V2. It seems that you might need to
>>> use the device once with the official driver to reprogram the device
>>> descriptors. Thanks to Jarod Wilson for the initial attempt at adding
>>> support for this device.
>> Could you please elaborate on this ?
>> What's the "official" driver and what changes after using it ?
>> Are these changes permanent ?
> From what I understand the Windows driver provided by KWorld will reprogram
> the eeprom so that the device descriptors are properly describing an Isochronous
> endpoint instead of Bulk. This only needs to be done once since it is permanent.
> I don't know if this is required since I don't have a device in the other state.
What do you mean with "properly describing an Isoc endpoint" ?
Did the Windows driver reprogram the device to provide an isoc instead
of a bulk video endpoint ?
Or did the device have no video endpoint and the bulk endpoint was used
for something different (likely device programming) ?
Any chance to get lsusb outputs ?

If you don't have a device in the other state and didn't test the bulk
endpoint's, how did you notice this issue ?

Regards,
Frank

>
>> The commit message should be included in the patch and not be sent as a
>> separate message.
>> Can you fix the patch and resend it ?
> Sure, I will resend the patch with the message included.
>
> Regards
> Jean-Francois


      reply	other threads:[~2013-10-10 13:38 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-30  0:45 [PATCH] Add support for KWorld UB435-Q V2 Jean-Francois Thibert
2013-09-30  0:45 ` Jean-Francois Thibert
2013-10-08 16:24 ` Frank Schäfer
2013-10-09 13:48   ` Jean-Francois Thibert
2013-10-10 13:38     ` Frank Schäfer [this message]

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=5256ADCA.2020005@googlemail.com \
    --to=fschaefer.oss@googlemail.com \
    --cc=jfthibert@google.com \
    --cc=linux-media@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.