Re: [PATCH] lirc: remove backwards compatibility macro obfuscation

From: Christoph Bartelmus
Date: Sat Feb 03 2007 - 22:12:58 EST


Hi Vincent,

on 02 Feb 07 at 12:04, you wrote:
> On 2/2/07, Pekka J Enberg <penberg@xxxxxxxxxxxxxx> wrote:
>> From: Pekka Enberg <penberg@xxxxxxxxxxxxxx>
>> [...]
>> drivers/lirc_atiusb/lirc_atiusb.c | 102 -----------------
> ^^^^^^^^^^^^^^
>
> I may be mistaken, but the lirc_atiusb module looks redondant with
> the driver already in drivers/usb/input/ati_remote.c.
> Moreover, I was under the impression that the input layer was
> currently considered the "right way" to implement the kernel side lirc
> needs

No. Using the input layer makes no sense for most lirc drivers. The
hardare used does not decode the IR signals itself but only provides raw
signal timings. Those signals are decoded in userspace by lircd.
It would be a really bad idea trying to do this decoding in kernel
space.

Christoph
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/