All of lore.kernel.org
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: LMML <linux-media@vger.kernel.org>
Subject: Re: em28xx: Too many ISO frames scheduled when starting stream
Date: Sun, 28 Sep 2014 21:13:59 +0300	[thread overview]
Message-ID: <54284FE7.5090805@iki.fi> (raw)
In-Reply-To: <54284488.60404@iki.fi>

On 09/28/2014 08:25 PM, Antti Palosaari wrote:
> I want raise that bug:
> Too many ISO frames scheduled when starting stream
> https://bugzilla.kernel.org/show_bug.cgi?id=72891
>
> Is there anyone who cares to study it? It looks like em28xx driver bug
> or USB host controller driver or both.
>
> According to comments bug appeared on kernel 3.13.

em28xx didn't even get any notable changes at that time... I looked all 
the 3.13 em28xx patches and there is no patch that could cause issues 
listed. So root of cause is somewhere else or there is reports which has 
kernel with media_build installed.
Also, em28xx uses ISOC to data transferred, whilst most devices are 
using BULK. No other reports from other ISOC DVB devices so far though. 
I suspect it may be some compatibility issue with em28xx chip / em28xx 
driver / USB stack / USB host controller.

There were em28xx patches went to 3.13 (stable patches not included):
bdee6bd [media] em28xx-video: Swap release order to avoid lock nesting
6dbea9f [media] Add support for KWorld UB435-Q V2
be353fa [media] V4L2: em28xx: tell the ov2640 driver to balance clock 
enabling internally
fc5d0f8 [media] V4L2: em28xx: register a V4L2 clock source
032f1dd [media] em28xx: fix error path in em28xx_start_analog_streaming()
b68cafc [media] em28xx: fix and unify the coding style of the GPIO 
register write sequences
de0fc46 [media] em28xx: MaxMedia UB425-TC change demod settings
b6c7abb [media] em28xx: MaxMedia UB425-TC switch RF tuner driver to another
8d100b2 [media] em28xx: MaxMedia UB425-TC offer firmware for demodulator


> Is there anyone knowing em28xx internals who wants (non surely really
> want, but hates less to start examine) to look it?
>
> It is very near I will fork em28xx to DVB only driver and move devices
> there. Current em28xx is too complex for my taste and has had more bugs
> than any other DVB driver I have ever seen.


regards
Antti

-- 
http://palosaari.fi/

  reply	other threads:[~2014-09-28 18:14 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-28 17:25 em28xx: Too many ISO frames scheduled when starting stream Antti Palosaari
2014-09-28 18:13 ` Antti Palosaari [this message]
2014-09-28 19:57   ` Mauro Carvalho Chehab
2014-09-28 20:43     ` Antti Palosaari
2014-09-28 21:05       ` Mauro Carvalho Chehab

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=54284FE7.5090805@iki.fi \
    --to=crope@iki.fi \
    --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.