linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* bisected: IR press/release behavior changed in 3.17, repeat events
@ 2014-11-15 17:59 Stephan Raue
  2014-11-15 18:54 ` Mauro Carvalho Chehab
  2014-11-19 19:50 ` David Härdeman
  0 siblings, 2 replies; 6+ messages in thread
From: Stephan Raue @ 2014-11-15 17:59 UTC (permalink / raw)
  To: linux-input; +Cc: david, m.chehab

Hi

with kernel 3.17 using a RC6 remote with a buildin nuvoton IR receiver 
(not tested others, but i think its a common problem) when 
pressing/releasing the same button often within 1 second there will no 
release event sent. Instead we get repeat events. To get the release 
event i must press the same button with a delay of ~ 1sec.

the evtest output for kernel with the difference 3.16 and 3.17 looks like

kernel 3.16

Event: time 1415452412.497503, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
Event: time 1415452412.497503, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415452412.497503, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
Event: time 1415452412.497503, -------------- SYN_REPORT ------------
Event: time 1415452412.672387, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415452412.672387, -------------- SYN_REPORT ------------
Event: time 1415452412.919799, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
Event: time 1415452412.919799, -------------- SYN_REPORT ------------
Event: time 1415452414.363169, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415452414.363169, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
Event: time 1415452414.363169, -------------- SYN_REPORT ------------
Event: time 1415452414.538010, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415452414.538010, -------------- SYN_REPORT ------------
Event: time 1415452414.621916, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
Event: time 1415452414.621916, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415452414.621916, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
Event: time 1415452414.621916, -------------- SYN_REPORT ------------
Event: time 1415452414.818869, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
Event: time 1415452414.818869, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415452414.818869, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
Event: time 1415452414.818869, -------------- SYN_REPORT ------------
Event: time 1415452414.994902, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
Event: time 1415452414.994902, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415452414.994902, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
Event: time 1415452414.994902, -------------- SYN_REPORT ------------



kernel 3.17

Event: time 1415454057.620687, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454057.620687, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
Event: time 1415454057.620687, -------------- SYN_REPORT ------------
Event: time 1415454057.795567, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454057.795567, -------------- SYN_REPORT ------------
Event: time 1415454057.896636, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454057.896636, -------------- SYN_REPORT ------------
Event: time 1415454058.056369, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454058.056369, -------------- SYN_REPORT ------------
Event: time 1415454058.210349, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454058.210349, -------------- SYN_REPORT ------------
Event: time 1415454058.371157, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454058.371157, -------------- SYN_REPORT ------------
Event: time 1415454058.540551, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454058.540551, -------------- SYN_REPORT ------------
Event: time 1415454058.622935, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
Event: time 1415454058.622935, -------------- SYN_REPORT ------------
Event: time 1415454058.696211, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454058.696211, -------------- SYN_REPORT ------------
Event: time 1415454058.749595, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
Event: time 1415454058.749595, -------------- SYN_REPORT ------------
Event: time 1415454058.849992, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454058.849992, -------------- SYN_REPORT ------------
Event: time 1415454058.876332, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
Event: time 1415454058.876332, -------------- SYN_REPORT ------------
Event: time 1415454059.002998, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
Event: time 1415454059.002998, -------------- SYN_REPORT ------------
Event: time 1415454059.008823, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454059.008823, -------------- SYN_REPORT ------------
Event: time 1415454059.129614, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
Event: time 1415454059.129614, -------------- SYN_REPORT ------------
Event: time 1415454059.179093, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454059.179093, -------------- SYN_REPORT ------------
Event: time 1415454059.256285, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
Event: time 1415454059.256285, -------------- SYN_REPORT ------------
Event: time 1415454059.346881, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
800f041f
Event: time 1415454059.346881, -------------- SYN_REPORT ------------
Event: time 1415454059.382993, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
Event: time 1415454059.382993, -------------- SYN_REPORT ------------
Event: time 1415454059.509617, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
Event: time 1415454059.509617, -------------- SYN_REPORT ------------
Event: time 1415454059.596281, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0

with irw it looks like:

kernel 3.16
OpenELEC:~ # irw
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput

kernel 3.17 (the first 2 presses was pressed with a delay of more then 1 
sec:
OpenELEC:~ # irw
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput
6c 0 KEY_DOWN devinput
6c 1 KEY_DOWN devinput
6c 2 KEY_DOWN devinput
6c 3 KEY_DOWN devinput
6c 4 KEY_DOWN devinput
6c 5 KEY_DOWN devinput
6c 6 KEY_DOWN devinput
6c 7 KEY_DOWN devinput
6c 8 KEY_DOWN devinput
6c 9 KEY_DOWN devinput
6c a KEY_DOWN devinput
6c b KEY_DOWN devinput
6c c KEY_DOWN devinput
6c d KEY_DOWN devinput
6c e KEY_DOWN devinput
6c f KEY_DOWN devinput
6c 10 KEY_DOWN devinput
6c 11 KEY_DOWN devinput
6c 12 KEY_DOWN devinput
6c 13 KEY_DOWN devinput
6c 14 KEY_DOWN devinput
6c 15 KEY_DOWN devinput
6c 0 KEY_DOWN_UP devinput


i have bisected the issue:

[stephan@buildserver linux-3.17-bisect]$ git bisect good
120703f9eb32033f0e39bdc552c0273c8ab45f33 is the first bad commit
commit 120703f9eb32033f0e39bdc552c0273c8ab45f33
Author: David Härdeman <david@hardeman.nu>
Date:   Thu Apr 3 20:31:30 2014 -0300

     [media] rc-core: document the protocol type

     Right now the protocol information is not preserved, rc-core gets 
handed a
     scancode but has no idea which protocol it corresponds to.

     This patch (which required reading through the source/keymap for 
all drivers,
     not fun) makes the protocol information explicit which is important
     documentation and makes it easier to e.g. support multiple 
protocols with one
     decoder (think rc5 and rc-streamzap). The information isn't used 
yet so there
     should be no functional changes.

     [m.chehab@samsung.com: rebased, added cxusb and removed bad 
whitespacing]
     Signed-off-by: David Härdeman <david@hardeman.nu>
     Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

:040000 040000 3db25c8acb78f27a4c6613e9fddbf9af8d1ea65e 
bc5866551b8c1a7dc8d4eaf35def332f20321122 M    drivers
:040000 040000 e69773356627779a7cdf905e11619a310fbfaeee 
aef9c358ea71385d2b83b498ce1e2c5568f257a7 M    include

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: bisected: IR press/release behavior changed in 3.17, repeat events
  2014-11-15 17:59 bisected: IR press/release behavior changed in 3.17, repeat events Stephan Raue
@ 2014-11-15 18:54 ` Mauro Carvalho Chehab
  2014-11-19 19:50 ` David Härdeman
  1 sibling, 0 replies; 6+ messages in thread
From: Mauro Carvalho Chehab @ 2014-11-15 18:54 UTC (permalink / raw)
  To: Stephan Raue; +Cc: linux-input, david, linux-media

Hi Stephan,

C/C linux-media, as this is the right ML for IR discussions.

Em Sat, 15 Nov 2014 18:59:05 +0100
Stephan Raue <mailinglists@openelec.tv> escreveu:

> Hi
> 
> with kernel 3.17 using a RC6 remote with a buildin nuvoton IR receiver 
> (not tested others, but i think its a common problem) when 
> pressing/releasing the same button often within 1 second there will no 
> release event sent. Instead we get repeat events. To get the release 
> event i must press the same button with a delay of ~ 1sec.
> 
> the evtest output for kernel with the difference 3.16 and 3.17 looks like
> 
> kernel 3.16
> 
> Event: time 1415452412.497503, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
> Event: time 1415452412.497503, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415452412.497503, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
> Event: time 1415452412.497503, -------------- SYN_REPORT ------------
> Event: time 1415452412.672387, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415452412.672387, -------------- SYN_REPORT ------------
> Event: time 1415452412.919799, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
> Event: time 1415452412.919799, -------------- SYN_REPORT ------------
> Event: time 1415452414.363169, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415452414.363169, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
> Event: time 1415452414.363169, -------------- SYN_REPORT ------------
> Event: time 1415452414.538010, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415452414.538010, -------------- SYN_REPORT ------------
> Event: time 1415452414.621916, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
> Event: time 1415452414.621916, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415452414.621916, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
> Event: time 1415452414.621916, -------------- SYN_REPORT ------------
> Event: time 1415452414.818869, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
> Event: time 1415452414.818869, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415452414.818869, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
> Event: time 1415452414.818869, -------------- SYN_REPORT ------------
> Event: time 1415452414.994902, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
> Event: time 1415452414.994902, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415452414.994902, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
> Event: time 1415452414.994902, -------------- SYN_REPORT ------------
> 
> 
> 
> kernel 3.17
> 
> Event: time 1415454057.620687, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454057.620687, type 1 (EV_KEY), code 108 (KEY_DOWN), value 1
> Event: time 1415454057.620687, -------------- SYN_REPORT ------------
> Event: time 1415454057.795567, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454057.795567, -------------- SYN_REPORT ------------
> Event: time 1415454057.896636, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454057.896636, -------------- SYN_REPORT ------------
> Event: time 1415454058.056369, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454058.056369, -------------- SYN_REPORT ------------
> Event: time 1415454058.210349, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454058.210349, -------------- SYN_REPORT ------------
> Event: time 1415454058.371157, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454058.371157, -------------- SYN_REPORT ------------
> Event: time 1415454058.540551, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454058.540551, -------------- SYN_REPORT ------------
> Event: time 1415454058.622935, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
> Event: time 1415454058.622935, -------------- SYN_REPORT ------------
> Event: time 1415454058.696211, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454058.696211, -------------- SYN_REPORT ------------
> Event: time 1415454058.749595, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
> Event: time 1415454058.749595, -------------- SYN_REPORT ------------
> Event: time 1415454058.849992, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454058.849992, -------------- SYN_REPORT ------------
> Event: time 1415454058.876332, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
> Event: time 1415454058.876332, -------------- SYN_REPORT ------------
> Event: time 1415454059.002998, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
> Event: time 1415454059.002998, -------------- SYN_REPORT ------------
> Event: time 1415454059.008823, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454059.008823, -------------- SYN_REPORT ------------
> Event: time 1415454059.129614, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
> Event: time 1415454059.129614, -------------- SYN_REPORT ------------
> Event: time 1415454059.179093, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454059.179093, -------------- SYN_REPORT ------------
> Event: time 1415454059.256285, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
> Event: time 1415454059.256285, -------------- SYN_REPORT ------------
> Event: time 1415454059.346881, type 4 (EV_MSC), code 4 (MSC_SCAN), value 
> 800f041f
> Event: time 1415454059.346881, -------------- SYN_REPORT ------------
> Event: time 1415454059.382993, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
> Event: time 1415454059.382993, -------------- SYN_REPORT ------------
> Event: time 1415454059.509617, type 1 (EV_KEY), code 108 (KEY_DOWN), value 2
> Event: time 1415454059.509617, -------------- SYN_REPORT ------------
> Event: time 1415454059.596281, type 1 (EV_KEY), code 108 (KEY_DOWN), value 0
> 
> with irw it looks like:
> 
> kernel 3.16
> OpenELEC:~ # irw
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 
> kernel 3.17 (the first 2 presses was pressed with a delay of more then 1 
> sec:
> OpenELEC:~ # irw
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 6c 0 KEY_DOWN devinput
> 6c 1 KEY_DOWN devinput
> 6c 2 KEY_DOWN devinput
> 6c 3 KEY_DOWN devinput
> 6c 4 KEY_DOWN devinput
> 6c 5 KEY_DOWN devinput
> 6c 6 KEY_DOWN devinput
> 6c 7 KEY_DOWN devinput
> 6c 8 KEY_DOWN devinput
> 6c 9 KEY_DOWN devinput
> 6c a KEY_DOWN devinput
> 6c b KEY_DOWN devinput
> 6c c KEY_DOWN devinput
> 6c d KEY_DOWN devinput
> 6c e KEY_DOWN devinput
> 6c f KEY_DOWN devinput
> 6c 10 KEY_DOWN devinput
> 6c 11 KEY_DOWN devinput
> 6c 12 KEY_DOWN devinput
> 6c 13 KEY_DOWN devinput
> 6c 14 KEY_DOWN devinput
> 6c 15 KEY_DOWN devinput
> 6c 0 KEY_DOWN_UP devinput
> 
> 
> i have bisected the issue:
> 
> [stephan@buildserver linux-3.17-bisect]$ git bisect good
> 120703f9eb32033f0e39bdc552c0273c8ab45f33 is the first bad commit
> commit 120703f9eb32033f0e39bdc552c0273c8ab45f33
> Author: David Härdeman <david@hardeman.nu>
> Date:   Thu Apr 3 20:31:30 2014 -0300
> 
>      [media] rc-core: document the protocol type
> 
>      Right now the protocol information is not preserved, rc-core gets 
> handed a
>      scancode but has no idea which protocol it corresponds to.
> 
>      This patch (which required reading through the source/keymap for 
> all drivers,
>      not fun) makes the protocol information explicit which is important
>      documentation and makes it easier to e.g. support multiple 
> protocols with one
>      decoder (think rc5 and rc-streamzap). The information isn't used 
> yet so there
>      should be no functional changes.
> 
>      [m.chehab@samsung.com: rebased, added cxusb and removed bad 
> whitespacing]
>      Signed-off-by: David Härdeman <david@hardeman.nu>
>      Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
> 
> :040000 040000 3db25c8acb78f27a4c6613e9fddbf9af8d1ea65e 
> bc5866551b8c1a7dc8d4eaf35def332f20321122 M    drivers
> :040000 040000 e69773356627779a7cdf905e11619a310fbfaeee 
> aef9c358ea71385d2b83b498ce1e2c5568f257a7 M    include
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: bisected: IR press/release behavior changed in 3.17, repeat events
  2014-11-15 17:59 bisected: IR press/release behavior changed in 3.17, repeat events Stephan Raue
  2014-11-15 18:54 ` Mauro Carvalho Chehab
@ 2014-11-19 19:50 ` David Härdeman
  2014-11-19 23:20   ` Stephan Raue
  1 sibling, 1 reply; 6+ messages in thread
From: David Härdeman @ 2014-11-19 19:50 UTC (permalink / raw)
  To: Stephan Raue; +Cc: linux-input, m.chehab, linux-media

On Sat, Nov 15, 2014 at 06:59:05PM +0100, Stephan Raue wrote:
>Hi
>
>with kernel 3.17 using a RC6 remote with a buildin nuvoton IR receiver (not
>tested others, but i think its a common problem) when pressing/releasing the
>same button often within 1 second there will no release event sent. Instead
>we get repeat events. To get the release event i must press the same button
>with a delay of ~ 1sec.
>
>the evtest output for kernel with the difference 3.16 and 3.17 looks like

Hi,

could you try the working and non-working versions with debugging output
enabled from the in-kernel rc6 decoder (i.e. set debug for the rc-core
module) and post the two different outputs?

//David

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: bisected: IR press/release behavior changed in 3.17, repeat events
  2014-11-19 19:50 ` David Härdeman
@ 2014-11-19 23:20   ` Stephan Raue
  2014-11-19 23:45     ` David Härdeman
  0 siblings, 1 reply; 6+ messages in thread
From: Stephan Raue @ 2014-11-19 23:20 UTC (permalink / raw)
  To: David Härdeman; +Cc: linux-input, m.chehab, linux-media

Am 19.11.2014 um 20:50 schrieb David Härdeman:
> On Sat, Nov 15, 2014 at 06:59:05PM +0100, Stephan Raue wrote:
>> Hi
>>
>> with kernel 3.17 using a RC6 remote with a buildin nuvoton IR receiver (not
>> tested others, but i think its a common problem) when pressing/releasing the
>> same button often within 1 second there will no release event sent. Instead
>> we get repeat events. To get the release event i must press the same button
>> with a delay of ~ 1sec.
>>
>> the evtest output for kernel with the difference 3.16 and 3.17 looks like
> Hi,
>
> could you try the working and non-working versions with debugging output
> enabled from the in-kernel rc6 decoder (i.e. set debug for the rc-core
> module) and post the two different outputs?
>
> //David
>

Hi David

with kernel 3.17: (you dont see the messages with "toggle 1" here)
if i press once and wait:

[   72.175548] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[   72.175555] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[   72.175559] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, protocol 0x0011, scancode 0x800f041f
[   72.350377] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[   72.350385] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[   72.598265] keyup key 0x006c
[   81.456175] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[   81.456182] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[   81.456186] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, protocol 0x0011, scancode 0x800f041f
[   81.631033] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[   81.631045] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[   81.878230] keyup key 0x006c
[   98.976060] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[   98.976067] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[   98.976071] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, protocol 0x0011, scancode 0x800f041f
[   99.150910] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[   99.150918] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[   99.398575] keyup key 0x006c

with kernel 3.17 if i press the same key often without a longer break:

[  298.971043] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  298.971051] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  298.971055] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, protocol 0x0011, scancode 0x800f041f
[  299.162854] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  299.162863] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  299.273112] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  299.273119] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  299.396907] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  299.396913] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  299.484521] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  299.484533] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  299.649523] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  299.649533] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  299.822100] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  299.822107] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  299.970903] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  299.970910] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  300.133381] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  300.133392] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  300.310163] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  300.310168] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  300.496736] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  300.496743] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  300.660526] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  300.660535] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  300.829385] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  300.829390] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  301.005423] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  301.005430] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  301.167183] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  301.167195] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  301.330419] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  301.330426] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  301.505621] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  301.505628] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  301.686007] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  301.686013] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  301.846361] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  301.846370] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  302.016169] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  302.016180] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  302.186990] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  302.186997] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  302.364885] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  302.364893] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  302.507666] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  302.507673] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  302.634184] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  302.634191] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  302.762198] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  302.762206] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  302.942671] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  302.942678] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  303.085449] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  303.085456] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  303.186777] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  303.186784] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  303.271129] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  303.271143] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  303.348733] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  303.348743] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  303.513377] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  303.513382] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  303.616009] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  303.616019] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  303.688259] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  303.688265] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  303.752897] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  303.752905] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  303.940450] RC6(6A) proto 0x0011, scancode 0x800f041f (toggle: 0)
[  303.940458] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  304.187848] keyup key 0x006c
[  311.405294] RC6 decode failed at state 0 (250us pulse)
[  311.405302] RC6 decode failed at state 0 (6350us space)
[  331.899445] RC6 decode failed at state 0 (250us pulse)
[  331.899454] RC6 decode failed at state 0 (6350us space)

with kernel 3.16: (you see messages with "RC6(6A) scancode 0x800f041f 
(toggle: 1)"

pressing the buttons with some delay between:

[  112.360318] RC6(6A) scancode 0x800f041f (toggle: 1)
[  112.360326] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  112.360330] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  112.484703] RC6(6A) scancode 0x800f041f (toggle: 1)
[  112.484711] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  112.733389] keyup key 0x006c
[  114.605782] RC6(6A) scancode 0x800f041f (toggle: 0)
[  114.605795] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  114.605802] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  114.730174] RC6(6A) scancode 0x800f041f (toggle: 0)
[  114.730182] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  114.979244] keyup key 0x006c
[  117.093793] RC6(6A) scancode 0x800f041f (toggle: 1)
[  117.093800] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  117.093805] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  117.218137] RC6(6A) scancode 0x800f041f (toggle: 1)
[  117.218144] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  117.465012] keyup key 0x006c
[  125.029704] RC6 decode failed at state 0 (300us pulse)
[  125.029714] RC6 decode failed at state 0 (95250us space)
[  125.795347] RC6(6A) scancode 0x800f041f (toggle: 0)
[  125.795354] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  125.795358] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  125.919612] RC6(6A) scancode 0x800f041f (toggle: 0)
[  125.919623] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  126.168501] keyup key 0x006c
[  128.740748] RC6 decode failed at state 0 (250us pulse)
[  128.740756] RC6 decode failed at state 0 (95250us space)


pressing fast:

[  192.644815] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  192.753373] RC6(6A) scancode 0x800f041f (toggle: 0)
[  192.753384] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  192.854618] RC6(6A) scancode 0x800f041f (toggle: 1)
[  192.854625] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  192.854627] keyup key 0x006c
[  192.854631] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  192.960172] RC6(6A) scancode 0x800f041f (toggle: 1)
[  192.960178] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  193.063672] RC6(6A) scancode 0x800f041f (toggle: 0)
[  193.063686] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  193.063690] keyup key 0x006c
[  193.063698] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  193.161351] RC6(6A) scancode 0x800f041f (toggle: 0)
[  193.161359] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  193.263223] RC6(6A) scancode 0x800f041f (toggle: 1)
[  193.263235] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  193.263238] keyup key 0x006c
[  193.263245] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  193.366792] RC6(6A) scancode 0x800f041f (toggle: 1)
[  193.366800] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  193.487952] RC6(6A) scancode 0x800f041f (toggle: 0)
[  193.487959] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  193.487962] keyup key 0x006c
[  193.487966] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  193.655452] RC6(6A) scancode 0x800f041f (toggle: 1)
[  193.655459] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  193.655462] keyup key 0x006c
[  193.655466] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  193.757056] RC6(6A) scancode 0x800f041f (toggle: 1)
[  193.757064] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  193.858806] RC6(6A) scancode 0x800f041f (toggle: 0)
[  193.858811] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  193.858813] keyup key 0x006c
[  193.858816] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  193.975154] RC6(6A) scancode 0x800f041f (toggle: 0)
[  193.975165] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  194.078759] RC6(6A) scancode 0x800f041f (toggle: 1)
[  194.078769] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  194.078772] keyup key 0x006c
[  194.078776] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  194.204414] RC6(6A) scancode 0x800f041f (toggle: 1)
[  194.204420] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  194.376145] RC6(6A) scancode 0x800f041f (toggle: 0)
[  194.376153] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  194.376156] keyup key 0x006c
[  194.376160] Nuvoton w836x7hg Infrared Remote Transceiver: key down 
event, key 0x006c, scancode 0x800f041f
[  194.500541] RC6(6A) scancode 0x800f041f (toggle: 0)
[  194.500548] Nuvoton w836x7hg Infrared Remote Transceiver: scancode 
0x800f041f keycode 0x6c
[  194.750393] keyup key 0x006c

greetings and thanks for your help

Stephan

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: bisected: IR press/release behavior changed in 3.17, repeat events
  2014-11-19 23:20   ` Stephan Raue
@ 2014-11-19 23:45     ` David Härdeman
  2014-11-20  0:30       ` Stephan Raue
  0 siblings, 1 reply; 6+ messages in thread
From: David Härdeman @ 2014-11-19 23:45 UTC (permalink / raw)
  To: Stephan Raue; +Cc: linux-input, m.chehab, linux-media

On Thu, Nov 20, 2014 at 12:20:55AM +0100, Stephan Raue wrote:
>with kernel 3.17: (you dont see the messages with "toggle 1" here)
>if i press once and wait:

Ummm...kinda embarassing...try swapping the order of the scancode and
toggle lines in the rc6 decoder (drivers/media/rc/ir-rc6-decoder.c).

They're somewhere around line 259, right after the case 32 statement.

case 32:
	if ((scancode & RC6_6A_LCC_MASK) == RC6_6A_MCE_CC) {
		protocol = RC_TYPE_RC6_MCE;
                scancode &= ~RC6_6A_MCE_TOGGLE_MASK;
		toggle = !!(scancode & RC6_6A_MCE_TOGGLE_MASK);



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: bisected: IR press/release behavior changed in 3.17, repeat events
  2014-11-19 23:45     ` David Härdeman
@ 2014-11-20  0:30       ` Stephan Raue
  0 siblings, 0 replies; 6+ messages in thread
From: Stephan Raue @ 2014-11-20  0:30 UTC (permalink / raw)
  To: David Härdeman; +Cc: linux-input, m.chehab, linux-media

Am 20.11.2014 um 00:45 schrieb David Härdeman:
> On Thu, Nov 20, 2014 at 12:20:55AM +0100, Stephan Raue wrote:
>> with kernel 3.17: (you dont see the messages with "toggle 1" here)
>> if i press once and wait:
> Ummm...kinda embarassing...try swapping the order of the scancode and
> toggle lines in the rc6 decoder (drivers/media/rc/ir-rc6-decoder.c).
>
> They're somewhere around line 259, right after the case 32 statement.
>
> case 32:
> 	if ((scancode & RC6_6A_LCC_MASK) == RC6_6A_MCE_CC) {
> 		protocol = RC_TYPE_RC6_MCE;
>                  scancode &= ~RC6_6A_MCE_TOGGLE_MASK;
> 		toggle = !!(scancode & RC6_6A_MCE_TOGGLE_MASK);
>
>
many thanks!!! this works :-)
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-11-20  0:30 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-15 17:59 bisected: IR press/release behavior changed in 3.17, repeat events Stephan Raue
2014-11-15 18:54 ` Mauro Carvalho Chehab
2014-11-19 19:50 ` David Härdeman
2014-11-19 23:20   ` Stephan Raue
2014-11-19 23:45     ` David Härdeman
2014-11-20  0:30       ` Stephan Raue

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).