Re: bisected: IR press/release behavior changed in 3.17, repeat events

2014-11-19 Thread Stephan Raue

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-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: bisected: IR press/release behavior changed in 3.17, repeat events

2014-11-19 Thread 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);


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


Re: bisected: IR press/release behavior changed in 3.17, repeat events

2014-11-19 Thread Stephan Raue

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.17] 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) prot

Re: bisected: IR press/release behavior changed in 3.17, repeat events

2014-11-19 Thread 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

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


Re: bisected: IR press/release behavior changed in 3.17, repeat events

2014-11-15 Thread Mauro Carvalho Chehab
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  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 --