All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Larsson <benjamin@southpole.se>
To: Antti Palosaari <crope@iki.fi>, linux-media@vger.kernel.org
Subject: Re: [PATCH 3/3] rtl28xxu: change Astrometa DVB-T2 to always use hardware pid filters
Date: Sun, 20 Dec 2015 10:47:39 +0100	[thread overview]
Message-ID: <5676793B.1070209@southpole.se> (raw)
In-Reply-To: <5676224F.3030803@iki.fi>

On 12/20/2015 04:36 AM, Antti Palosaari wrote:
> Moikka!
> I did some testing and I cannot see reason to force hw pid filter for
> that device. I assume you somehow think it does not work without
> filtering, but I think it does.
>

[... proof that I was wrong ...]

>
> So point me the reason hw PID filters need to be forced.

The only reason would be that I suspected that high rates would not be 
possible over the bridge and that this was the default mode for the 
binary driver but as you proved that the bridge is able to support the 
rate this patch can be nacked.

>
> regards
> Antti

MvH
Benjamin Larsson

  reply	other threads:[~2015-12-20  9:47 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-29  2:10 [PATCH 1/3] rtl2832: add support for slave ts pid filter Benjamin Larsson
2015-11-29  2:10 ` [PATCH 2/3] mn88472: add work around for failing firmware loading Benjamin Larsson
2015-12-21  2:52   ` Antti Palosaari
2015-12-21 10:07     ` Benjamin Larsson
2015-11-29  2:10 ` [PATCH 3/3] rtl28xxu: change Astrometa DVB-T2 to always use hardware pid filters Benjamin Larsson
2015-12-20  3:36   ` Antti Palosaari
2015-12-20  9:47     ` Benjamin Larsson [this message]
2015-12-21  2:42 ` [PATCH 1/3] rtl2832: add support for slave ts pid filter Antti Palosaari
2015-12-21  9:51   ` Benjamin Larsson

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=5676793B.1070209@southpole.se \
    --to=benjamin@southpole.se \
    --cc=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.