Em 08-09-2010 19:49, David Härdeman escreveu:
> On Wed, Sep 08, 2010 at 07:42:04PM -0300, Mauro Carvalho Chehab wrote:
>> Em 06-09-2010 18:26, Maxim Levitsky escreveu:
>>> diff --git a/drivers/media/IR/ir-rc6-decoder.c b/drivers/media/IR/ir-rc6-decoder.c
>>> index f1624b8..d25da91 100644
>>> --- a/drivers/media/IR/ir-rc6-decoder.c
>>> +++ b/drivers/media/IR/ir-rc6-decoder.c
>>> @@ -85,8 +85,9 @@ static int ir_rc6_decode(struct input_dev *input_dev, struct ir_raw_event ev)
>>> if (!(ir_dev->raw->enabled_protocols & IR_TYPE_RC6))
>>> return 0;
>>>
>>> - if (IS_RESET(ev)) {
>>> - data->state = STATE_INACTIVE;
>>> + if (!is_timing_event(ev)) {
>>> + if (ev.reset)
>>
>> Again, why do you need to test first for !is_timing_event()?
>
> Because the decoder should return early if the event is not a timing
> event (the return 0 two lines below)...think carrier report event...
Yeah, I saw that. I was supposed to remove all the comments about that, but I
forgot to remove the last one ;)
Cheers,
Mauro.
--
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
