All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: "Ezequiel García" <elezegarcia@gmail.com>
Cc: Rui Salvaterra <rsalvaterra@gmail.com>, linux-media@vger.kernel.org
Subject: Re: eMPIA EM2710 Webcam (em28xx) and LIRC
Date: Thu, 15 Mar 2012 14:37:24 -0300	[thread overview]
Message-ID: <4F6228D4.6090706@redhat.com> (raw)
In-Reply-To: <CALF0-+WJ9c579+=2QMamxpAngHJKWfZaWqOp_z=GvZKGy97VnA@mail.gmail.com>

Em 15-03-2012 14:15, Ezequiel García escreveu:
> Hi Mauro,
> 
> On 3/15/12, Mauro Carvalho Chehab <mchehab@redhat.com> wrote:
>> If you won't take it, it is likely that some day someone will do it,
>> but, as this is just a cleanup, the main developers won't likely
>> have time for doing it, as they're generally busy adding support for
>> new hardware.
>>
> 
> I have no problem submitting the two patches in discussion.
> (Actually, I've already wroten the first one).

Good!

> If Rui wants to help me, it would be nice to get this tested,
> since I don't have the necessary hardware.

I might test, but I'm currently very busy with some other stuff,
so I won't likely have time for it anytime soon.

> One thing I haven't got clear is this: I've seen that ir_raw_init()
> does not call request_module() directly but rather defers it
> through a work queue.

Deferring request_module to a work_queue is a common procedure we
do at v4l drivers. It helps to initialize faster, and prevents some
dependency issues, as the decoders depend on rc_core to be initialized,
in order to load.

> I don't understand fully the rc code, but still I don't get what
> happens if I need the raw decoders *now* (so to speak)
> but the work queue hasn't been run yet?

Well, if you press an RC key while the needed decoded is not loaded,
that key will be lost. No other problems should happen.

Regards,
Mauro.

  reply	other threads:[~2012-03-15 17:37 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-14 11:28 eMPIA EM2710 Webcam (em28xx) and LIRC Rui Salvaterra
2012-03-14 17:42 ` Ezequiel García
2012-03-14 18:47   ` Rui Salvaterra
2012-03-14 19:47     ` Ezequiel García
2012-03-15  0:28       ` Rui Salvaterra
2012-03-15  3:03         ` Ezequiel García
2012-03-15 10:42         ` Mauro Carvalho Chehab
2012-03-15 12:34           ` Frank Schäfer
2012-03-15 13:05             ` Mauro Carvalho Chehab
2012-03-16 22:18               ` Move em27xx/em28xx webcams to a gspca subdriver ? Frank Schäfer
2012-03-21 18:01                 ` Frank Schäfer
2012-03-21 20:39                   ` Mauro Carvalho Chehab
2012-03-15 13:10           ` eMPIA EM2710 Webcam (em28xx) and LIRC Rui Salvaterra
2012-03-15 14:39             ` Mauro Carvalho Chehab
2012-03-15 17:15               ` Ezequiel García
2012-03-15 17:37                 ` Mauro Carvalho Chehab [this message]
2012-03-16  9:08                   ` Rui Salvaterra

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=4F6228D4.6090706@redhat.com \
    --to=mchehab@redhat.com \
    --cc=elezegarcia@gmail.com \
    --cc=linux-media@vger.kernel.org \
    --cc=rsalvaterra@gmail.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.