linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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.

  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).