Re: [PATCH] HID: cp2112: fix to force single data-report reply

2015-07-11 Thread Antonio Borneo
On Wed, Jul 8, 2015 at 6:41 PM, Jiri Kosina wrote: > On Wed, 8 Jul 2015, Ellen Wang wrote: > >> > > Works as described. Thank you! >> > > >> > > By the way, I tested the code with and without your fix on my rev 2 chip, >> > > and >> > > it behaved the same way as you describe on your rev 1 chip.

Re: [PATCH] HID: cp2112: fix to force single data-report reply

2015-07-08 Thread Jiri Kosina
On Wed, 8 Jul 2015, Ellen Wang wrote: > > > Works as described. Thank you! > > > > > > By the way, I tested the code with and without your fix on my rev 2 chip, > > > and > > > it behaved the same way as you describe on your rev 1 chip. > > > > Ellen, are you okay with me applying the patch wit

Re: [PATCH] HID: cp2112: fix to force single data-report reply

2015-07-08 Thread Ellen Wang
On 07/08/2015 02:15 AM, Jiri Kosina wrote: On Wed, 1 Jul 2015, Ellen Wang wrote: Works as described. Thank you! By the way, I tested the code with and without your fix on my rev 2 chip, and it behaved the same way as you describe on your rev 1 chip. Ellen, are you okay with me applying the

Re: [PATCH] HID: cp2112: fix to force single data-report reply

2015-07-08 Thread Jiri Kosina
On Wed, 1 Jul 2015, Ellen Wang wrote: > Works as described. Thank you! > > By the way, I tested the code with and without your fix on my rev 2 chip, and > it behaved the same way as you describe on your rev 1 chip. Ellen, are you okay with me applying the patch with Tested-by: Ellen Wa

Re: [PATCH] HID: cp2112: fix to force single data-report reply

2015-07-01 Thread Ellen Wang
Works as described. Thank you! By the way, I tested the code with and without your fix on my rev 2 chip, and it behaved the same way as you describe on your rev 1 chip. On 06/20/2015 11:20 PM, Antonio Borneo wrote: Current implementation of cp2112_raw_event() only accepts one data report at

[PATCH] HID: cp2112: fix to force single data-report reply

2015-06-20 Thread Antonio Borneo
Current implementation of cp2112_raw_event() only accepts one data report at a time. If last received data report is not fully handled yet, a new incoming data report will overwrite it. In such case we don't guaranteed to propagate the correct incoming data. The trivial fix implemented here forces