On May 28, 2011, at 4:41 PM, Andy Walls <awalls@xxxxxxxxxxxxxxxx> wrote:
> On Fri, 2011-05-27 at 16:01 -0400, Jarod Wilson wrote:
>> Store the cdev pointer in struct irctl, allocated dynamically as needed,
>> rather than having a static array. At the same time, recycle some of the
>> saved memory to nudge the maximum number of lirc devices supported up a
>> ways -- its not that uncommon these days, now that we have the rc-core
>> lirc bridge driver, to see a system with at least 4 raw IR receivers.
>> (consider a mythtv backend with several video capture devices and the
>> possible need for IR transmit hardware).
>>
>> Signed-off-by: Jarod Wilson <jarod@xxxxxxxxxx>
>> ---
>> drivers/media/rc/lirc_dev.c | 33 ++++++++++++++++++++++++---------
>> include/media/lirc_dev.h | 2 +-
>> 2 files changed, 25 insertions(+), 10 deletions(-)
>>
>> diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c
>> index fd237ab..9e79692 100644
>> --- a/drivers/media/rc/lirc_dev.c
>> +++ b/drivers/media/rc/lirc_dev.c
>>
...
>>
>> + cdev = kzalloc(sizeof(*cdev), GFP_KERNEL);
>
> if (cdev == NULL)
>
>
>> if (d->fops) {
>> cdev_init(cdev, d->fops);
>
> generate_oops();
>
> ;)
D'oh, rookie mistake! I'll fix that right up... Thanks for lookin'.
--
Jarod Wilson
jarod@xxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
