All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Istvan Varga <istvan_v@mailbox.hu>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [GIT PULL for v3.0] media updates for v3.1
Date: Sun, 31 Jul 2011 01:05:21 -0300	[thread overview]
Message-ID: <4E34D481.1040302@redhat.com> (raw)
In-Reply-To: <4E3411D3.90703@mailbox.hu>

Em 30-07-2011 11:14, Istvan Varga escreveu:
> On 07/30/2011 04:03 PM, Mauro Carvalho Chehab wrote:
> 
>> Btw, It may actually make sense to not allow using the PAL filter with a
>> NTSC source and vice-versa, e. g. reducing the notch filter to only 3
>> possible values:
>>
>>     0 - 4xFSC            (00)
>>     1 - square pixel        (01)
>>     2 - std-optimized filter    (10 or 11)
>>
>> Where 2 would man 10 for NTSC standard or 11 for PAL standard. I suspect,
>> however, that the std-optimized filter only works if the sampling frequency
>> is set to 27 MHz. However, at cx88 code, we set the sampling frequency to
>> be 8xFSC, instead of fixing it to 27MHz. Due to that, I doubt that the
>> PAL or NTSC optimized filters will give a good result. So, maybe we can change
>> it to just:
>>
>>     0 - 4xFSC
>>     1 - square pixel
>>
>> In other words, except if you found that the std-optimized filters are giving
>> better results, I would change the control to only select between 00 and 01,
>> and initialize it at device init, with 00.
> 
> OK, I have no problem with removing the standard optimized filters and
> restricting the control to 2 settings; I only used the square pixel one
> anyway, as I found it often looks better than the default filter.

Ok then. Please prepare the patches for that.

Thanks!
Mauro

  reply	other threads:[~2011-07-31  4:05 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-29 17:31 [GIT PULL for v3.0] media updates for v3.1 Mauro Carvalho Chehab
2011-07-29 20:01 ` Alina Friedrichsen
2011-07-29 20:11   ` Mauro Carvalho Chehab
2011-07-30  8:43 ` Istvan Varga
2011-07-30 14:03   ` Mauro Carvalho Chehab
2011-07-30 14:14     ` Istvan Varga
2011-07-31  4:05       ` Mauro Carvalho Chehab [this message]
     [not found]   ` <CAGoCfiw0f1puvj33eOHsh8bqVP-EgOUwCPQigF3u5gTEry839Q@mail.gmail.com>
2011-07-31 14:29     ` Istvan Varga
2011-07-31  7:57 ` Hans de Goede
2011-07-31 10:38   ` Mauro Carvalho Chehab
2011-08-01  7:11     ` Hans de Goede
2011-08-01  7:45       ` Hans Verkuil
2011-08-01 14:02         ` Hans de Goede

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=4E34D481.1040302@redhat.com \
    --to=mchehab@redhat.com \
    --cc=istvan_v@mailbox.hu \
    --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.