From mboxrd@z Thu Jan  1 00:00:00 1970
Return-path: <linux-media-owner@vger.kernel.org>
Received: from mx1.redhat.com ([209.132.183.28]:48197 "EHLO mx1.redhat.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1030464Ab2CORh2 (ORCPT <rfc822;linux-media@vger.kernel.org>);
	Thu, 15 Mar 2012 13:37:28 -0400
Message-ID: <4F6228D4.6090706@redhat.com>
Date: Thu, 15 Mar 2012 14:37:24 -0300
From: Mauro Carvalho Chehab <mchehab@redhat.com>
MIME-Version: 1.0
To: =?ISO-8859-1?Q?Ezequiel_Garc=EDa?= <elezegarcia@gmail.com>
CC: Rui Salvaterra <rsalvaterra@gmail.com>, linux-media@vger.kernel.org
Subject: Re: eMPIA EM2710 Webcam (em28xx) and LIRC
References: <CALjTZvZy4npSE0aELnmsZzzgsxUC1xjeNYVwQ_CvJG59PizfEQ@mail.gmail.com> <CALF0-+Wp03vsbiaJFUt=ymnEncEvDg_KmnV+2OWjtO-_0qqBVg@mail.gmail.com> <CALjTZvYVtuSm0v-_Q7od=iUDvHbkMe4c5ycAQZwoErCCe=N+Bg@mail.gmail.com> <CALF0-+W3HenNpUt_yGxqs+fohcZ22ozDw9MhTWua0B++ZFA2vA@mail.gmail.com> <CALjTZvYJZ32Red-UfZXubB-Lk503DWbHGTL_kEoV4DVDDYJ46w@mail.gmail.com> <4F61C79E.6090603@redhat.com> <CALjTZvZR=Mr-eSVwy=Wd8ToikAX9bG23NLARRw_K0scT-_YeCg@mail.gmail.com> <4F61FF2D.6010505@redhat.com> <CALF0-+WJ9c579+=2QMamxpAngHJKWfZaWqOp_z=GvZKGy97VnA@mail.gmail.com>
In-Reply-To: <CALF0-+WJ9c579+=2QMamxpAngHJKWfZaWqOp_z=GvZKGy97VnA@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
Sender: linux-media-owner@vger.kernel.org
List-ID: <linux-media.vger.kernel.org>

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.