Re: bisected: IR press/release behavior changed in 3.17, repeat events
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
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
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
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
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 --