From: Jemma Denson <jdenson@gmail.com>
To: Patrick Boettcher <patrick.boettcher@posteo.de>
Cc: linux-media@vger.kernel.org,
"mchehab@osg.samsung.com >> Mauro Carvalho Chehab"
<mchehab@osg.samsung.com>
Subject: Re: [PATCH 3/4] cx24120: Take control of b2c2 receive stream
Date: Tue, 26 May 2015 10:21:11 +0100 [thread overview]
Message-ID: <55643B07.9010807@gmail.com> (raw)
In-Reply-To: <20150526110545.32c71335@dibcom294.coe.adi.dibcom.com>
Hi Patrick,
On 26/05/15 10:05, Patrick Boettcher wrote:
>> Now that b2c2 has an option to allow us to do so, turn off the
>> flexcop receive stream when we turn off mpeg output whilst tuning.
> Does this not fix (and your '[PATCH 2/4]') the problem of receiving
> PAT from the previously tuned transport-stream?
>
> Then patch 1 and 4 should not be necessary, should they?!
Only patch 1 fixes that problem, so out of the 4 here that one is the
most necessary. Controlling the flexcop receive stream and/or stopping
the cx24120 from sending doesn't actually appear to do much of anything
- it doesn't seem any better or worse doing one or the other, both or
even neither! (Apart from Patch 4 breaking things, as mentioned).
I'm including it though because I presume the reference driver advised
it was done, and it does tidy up the cx24120 codebase considerably by
being able to disable the whole turn off sending the stream whilst
tuning feature - I'm envisioning that in the future someone might want
to take on the task of merging cx24117 & cx24120 as they're quite
similar, and allowing what seems to just be a flexcop oddity to be
turned off would make this possible.
Jemma.
next prev parent reply other threads:[~2015-05-26 13:05 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-22 20:28 [PATCH 0/4] SkystarS2 pid filtering fix and stream control Jemma Denson
2015-05-22 20:28 ` [PATCH 1/4] b2c2: Add option to skip the first 6 pid filters Jemma Denson
2015-05-22 20:28 ` [PATCH 2/4] b2c2: Allow external stream control Jemma Denson
2015-05-22 20:28 ` [PATCH 3/4] cx24120: Take control of b2c2 receive stream Jemma Denson
2015-05-26 9:05 ` Patrick Boettcher
2015-05-26 9:21 ` Jemma Denson [this message]
2015-06-09 23:56 ` Mauro Carvalho Chehab
2015-06-10 7:32 ` Jemma Denson
2015-05-22 20:28 ` [PATCH 4/4] b2c2: Always turn off " Jemma Denson
2015-05-24 12:35 ` Jemma Denson
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=55643B07.9010807@gmail.com \
--to=jdenson@gmail.com \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@osg.samsung.com \
--cc=patrick.boettcher@posteo.de \
/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;
as well as URLs for NNTP newsgroup(s).