linux-kernel - Re: [PATCH] [media] rc: do not enable remote controller adapters by default.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Wed, 16 Feb 2011 15:43:57 -0200
From:	Mauro Carvalho Chehab <mchehab@...radead.org>
To:	VDR User <user.vdr@...il.com>
CC:	Jarod Wilson <jarod@...hat.com>, Stephen Wilson <wilsons@...rt.ca>,
	Andy Walls <awalls@...metrocast.net>,
	David Härdeman <david@...deman.nu>,
	linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] [media] rc: do not enable remote controller adapters
 by default.

Em 16-02-2011 15:25, VDR User escreveu:
> On Wed, Feb 16, 2011 at 7:20 AM, Jarod Wilson <jarod@...hat.com> wrote:
>>> It is not a need.  I simply observed that after the IR_ to RC_ rename
>>> there was another set of drivers being built which I did not ask for.
>>
>> So disable them. I think most people would rather have this support
>> enabled so that remotes Just Work if a DTV card or stand-alone IR receiver
>> is plugged in without having to hunt back through Kconfig options to
>> figure out why it doesn't...
> 
> Unfortunately _ALL_ the usb DVB devices are unavailable if you do not
> enable IR_CORE "Infrared remote controller adapters" in v4l.  This is
> a little annoying as the usb device I use doesn't even have IR
> capabilities.  It doesn't seem like something that should be forced on
> the user -- enable IR or you can't even compile the driver you need,
> which doesn't even use IR.
> 
> It's not the end of the world but I don't particularly appreciate the
> enable-everything approach.  It would be nice to at least have the
> option to trim the fat if you want.. Isn't that partially what Linux
> is supposed to be about in the first place?

Unfortunately, the way almost all drivers/media/[video|dvb] was made,
the IR (and input support on webcams) are tighted to the driver.

It would be possible to change it (and it is not that hard), but it requires 
people with enough time and skills to go into each driver that can support IR, 
split the IR code into a separate module and be sure that the driver will 
compile and work with or without IR support.

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