All of lore.kernel.org
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: linux-media@vger.kernel.org, Michael Krufky <mkrufky@kernellabs.com>
Subject: Re: [GIT PULL FOR 3.2] misc small changes, mostly get/set IF related
Date: Fri, 25 Nov 2011 01:11:51 +0200	[thread overview]
Message-ID: <4ECECF37.2010202@iki.fi> (raw)
In-Reply-To: <4ECEA1E9.2000107@redhat.com>

On 11/24/2011 09:58 PM, Mauro Carvalho Chehab wrote:
> Em 13-11-2011 17:20, Antti Palosaari escreveu:
>> Mauro,
>> and these too for 3.3. Sorry about mistakes.
>
> In fact, those 3 patches seemed to be good for 3.2:
>
> 576b849 [media] mxl5007t: fix reg read
> d7d89dc [media] tda18218: fix 6 MHz default IF frequency
> ff83bd8 [media] af9015: limit I2C access to keep FW happy
>
> So, I've backported d7d89dc to 3.2, and applied there. the others,
> I've added for 3.3.

You could put mxl5007t and tda18218. But as I changed tda18218 to 
support .get_if_frequency() just before that fix and it touches same 
code lines, it will not apply unless you do some hand editing or apply 
tda18218 get_if_frequency() patch too.

DO *NOT* push af9015 I2C patch to the 3.2. I see there could be 
potential problems if there is heavy demodulator I2C traffic since it 
forces to wait some I2C related tasks. I suspect that could lead bad 
situation when there is more traffic we can handle due to I2C access 
waiting coming from patch.

I have optimized af9013 I2C load, reduced it rather much switching 
multibyte I2C where possible and running statistic polls using timers in 
backround. Now it returns all statistics from cache and not make any 
statistics queries directly when userspace app is asking.

After my af9013 optimizations are ready those could be put to master, 
which will happen likely 3.3 schedule. It is absolutely too risky put 
that single patch to 3.2.


regards
Antti

-- 
http://palosaari.fi/

      reply	other threads:[~2011-11-24 23:11 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-13 19:12 [GIT PULL FOR 3.2] misc small changes, mostly get/set IF related Antti Palosaari
2011-11-13 19:20 ` Antti Palosaari
2011-11-24 19:58   ` Mauro Carvalho Chehab
2011-11-24 23:11     ` Antti Palosaari [this message]

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=4ECECF37.2010202@iki.fi \
    --to=crope@iki.fi \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.com \
    --cc=mkrufky@kernellabs.com \
    /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.