Re: [PATCH v2] media: st-rc: Add ST remote control driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 29/08/13 13:01, Sean Young wrote:
> On Thu, Aug 29, 2013 at 12:29:00PM +0100, Srinivas KANDAGATLA wrote:
>> On 29/08/13 10:11, Sean Young wrote:
>>> On Wed, Aug 28, 2013 at 04:33:50PM +0100, Srinivas KANDAGATLA wrote:
>>>> From: Srinivas Kandagatla <srinivas.kandagatla@xxxxxx>
>>>> +config RC_ST
>>>> +	tristate "ST remote control receiver"
>>>> +	depends on ARCH_STI && LIRC && OF
>>>
>>> Minor nitpick, this should not depend on LIRC, it depends on RC_CORE.
>> Yes, I will make it depend on RC_CORE, remove OF as suggested by Mauro
>> CC and select LIRC to something like.
>>
>> depends on ARCH_STI && RC_CORE
>> select LIRC
> 
> The driver is usable with the kernel ir decoders, there is no need to 
> select LIRC or use lirc at all.
> 
> You can either define a remote in drivers/media/rc/keymaps and set 
> the rcdev->map_name, or it can be loaded at runtime using ir-keytable(1);
> either way you don't need to use a lirc userspace daemon.
Yep.. got it.
I will remove LIRC selection.

Thanks,
srini
> 
> 
> Sean
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux