Re: [PATCHv2 1/2] media: rc: Introduce RX51 IR transmitter driver

2012-08-13 Thread Sean Young
On Fri, Aug 10, 2012 at 01:16:36PM +0300, Timo Kokkonen wrote: This is the driver for the IR transmitter diode found on the Nokia N900 (also known as RX51) device. The driver is mostly the same as found in the original 2.6.28 based kernel that comes with the device. The following

Re: [PATCHv2 1/2] media: rc: Introduce RX51 IR transmitter driver

2012-08-13 Thread Timo Kokkonen
On 08/13/12 21:36, Sean Young wrote: On Fri, Aug 10, 2012 at 01:16:36PM +0300, Timo Kokkonen wrote: +static ssize_t lirc_rx51_write(struct file *file, const char *buf, + size_t n, loff_t *ppos) +{ +int count, i; +struct lirc_rx51 *lirc_rx51 = file-private_data;

[PATCHv2 1/2] media: rc: Introduce RX51 IR transmitter driver

2012-08-10 Thread Timo Kokkonen
This is the driver for the IR transmitter diode found on the Nokia N900 (also known as RX51) device. The driver is mostly the same as found in the original 2.6.28 based kernel that comes with the device. The following modifications have been made compared to the original driver version: - Adopt

Re: [PATCHv2 1/2] media: rc: Introduce RX51 IR transmitter driver

2012-08-10 Thread Timo Kokkonen
I should have probably tried this before, but when I enabled lock debugging with this driver, I got this: [ 663.848083] BUG: sleeping function called from invalid context at kernel/mutex.c:269 [ 663.856262] in_atomic(): 1, irqs_disabled(): 128, pid: 889, name: lircd [ 663.863220] 1 lock

Re: [PATCHv2 1/2] media: rc: Introduce RX51 IR transmitter driver

2012-08-10 Thread Timo Kokkonen
On 08.10 2012 14:06:38, Timo Kokkonen wrote: I should have probably tried this before, but when I enabled lock debugging with this driver, I got this: [ 663.848083] BUG: sleeping function called from invalid context at kernel/mutex.c:269 [ 663.856262] in_atomic(): 1, irqs_disabled():