linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Antti Palosaari <crope@iki.fi>
Cc: linux-media <linux-media@vger.kernel.org>
Subject: Re: [GIT PULL FOR v3.7] all the rest patches!
Date: Mon, 1 Oct 2012 16:32:25 -0300	[thread overview]
Message-ID: <20121001163225.67ff5319@redhat.com> (raw)
In-Reply-To: <506740E5.1030708@iki.fi>

Em Sat, 29 Sep 2012 21:41:41 +0300
Antti Palosaari <crope@iki.fi> escreveu:

> Updated, one new USB ID for RTL2832U.
> 
> The following changes since commit 8928b6d1568eb9104cc9e2e6627d7086437b2fb3:
> 
>    [media] media: mx2_camera: use managed functions to clean up code 
> (2012-09-27 15:56:47 -0300)
> 
> are available in the git repository at:
> 
>    git://linuxtv.org/anttip/media_tree.git for_v3.7_mauro-3
> 
> for you to fetch changes up to bf342b50ac6c5801a95d6a089086587446c8d6cf:
> 
>    rtl28xxu: [0ccd:00d3] TerraTec Cinergy T Stick RC (Rev. 3) 
> (2012-09-29 21:39:26 +0300)
> 
> ----------------------------------------------------------------
> Antti Palosaari (5):
>        em28xx: implement FE set_lna() callback
>        cxd2820r: use static GPIO config when GPIOLIB is undefined
>        em28xx: do not set PCTV 290e LNA handler if fe attach fail
>        em28xx: PCTV 520e workaround for DRX-K fw loading

All applied except for the above.

As I said before: sleeping for 2 seconds doesn't give any warranty that
the firmware got loaded (and I know one system where firmware load generally
takes more than 2 seconds to start - probably because the root fs is using
nfs, and the machine uses an Atom single core processor).

As I've explained, if the driver needs to wait for a firmware load, it
should use something that will actually wait for firmware load to complete,
instead of just sleeping in the hope that the amount of sleeping time would
be enough.

Regards,
Mauro

  reply	other threads:[~2012-10-01 19:32 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-27 22:15 [GIT PULL FOR v3.7] all the rest patches! Antti Palosaari
     [not found] ` <50657B0C.70706@iki.fi>
2012-09-29 18:41   ` Antti Palosaari
2012-10-01 19:32     ` Mauro Carvalho Chehab [this message]
2012-10-01 19:38       ` Antti Palosaari
2012-10-01 19:53         ` Mauro Carvalho Chehab

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=20121001163225.67ff5319@redhat.com \
    --to=mchehab@redhat.com \
    --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 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).