Hi Jiri,
This patchset has gone through two rounds of testing/review. It is
also a necessary set to support future userland LED configuration
features.
Do you see any issues with the patches?
Cheers,
Ping
On Wed, Jul 13, 2016 at 2:36 PM, Aaron Armstrong Skomra
wrote:
> On
Hi Jiri,
This patchset has gone through two rounds of testing/review. It is
also a necessary set to support future userland LED configuration
features.
Do you see any issues with the patches?
Cheers,
Ping
On Wed, Jul 13, 2016 at 2:36 PM, Aaron Armstrong Skomra
wrote:
> On Wed, Jul 13, 2016
tricky part.
> Untested.
Aaron tested it. So, your code is safe ;).
> Signed-off-by: Benjamin Tissoires <benjamin.tissoi...@redhat.com>
The whole set is
Acked-by: Ping Cheng <pi...@wacom.com>
Thanks for your effort!
Ping
> ---
>
> No changes in v2
> ---
&g
on tested it. So, your code is safe ;).
> Signed-off-by: Benjamin Tissoires
The whole set is
Acked-by: Ping Cheng
Thanks for your effort!
Ping
> ---
>
> No changes in v2
> ---
> drivers/hid/wacom_sys.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> d
On Mon, May 23, 2016 at 9:52 AM, Dmitry Torokhov
<dmitry.torok...@gmail.com> wrote:
> On Sun, May 22, 2016 at 10:21:45PM -0700, Ping Cheng wrote:
>> Hi Chris,
>>
>> On Sun, May 22, 2016 at 6:42 PM, Chris J Arges
>> <christopherar...@gmail.com> wrote:
>&
On Mon, May 23, 2016 at 9:52 AM, Dmitry Torokhov
wrote:
> On Sun, May 22, 2016 at 10:21:45PM -0700, Ping Cheng wrote:
>> Hi Chris,
>>
>> On Sun, May 22, 2016 at 6:42 PM, Chris J Arges
>> wrote:
>> > I've noticed crashes when using my x60t using a coreboot
Hi Chris,
On Sun, May 22, 2016 at 6:42 PM, Chris J Arges
wrote:
> I've noticed crashes when using my x60t using a coreboot bios. When using
> the pen I can produce a crash simply by tapping a few times. This
> generates an event which has an idx of 0xc. This in turn
Hi Chris,
On Sun, May 22, 2016 at 6:42 PM, Chris J Arges
wrote:
> I've noticed crashes when using my x60t using a coreboot bios. When using
> the pen I can produce a crash simply by tapping a few times. This
> generates an event which has an idx of 0xc. This in turn crashes the
> machine because
ical pad connected to it.
> The weird part is that the pad is still sending events and given that
> there is no input node connected to it, we get anull pointer exception.
>
> Link: https://bugzilla.redhat.com/show_bug.cgi?id=1317116
>
> Signed-off-by: Benjamin Tissoires <benjami
The weird part is that the pad is still sending events and given that
> there is no input node connected to it, we get anull pointer exception.
>
> Link: https://bugzilla.redhat.com/show_bug.cgi?id=1317116
>
> Signed-off-by: Benjamin Tissoires
Acked-by: Ping Cheng
Thank you Benjamin
Hi Jiri,
On Tue, Dec 1, 2015 at 8:36 AM, Jiri Kosina wrote:
> On Fri, 20 Nov 2015, Ioan-Adrian Ratiu wrote:
>
>> The critical section protected by usbhid->lock in hid_ctrl() is too
>> big and because of this it causes a recursive deadlock. "Too big" means
>> the case statement and the call to
Hi Jiri,
On Tue, Dec 1, 2015 at 8:36 AM, Jiri Kosina wrote:
> On Fri, 20 Nov 2015, Ioan-Adrian Ratiu wrote:
>
>> The critical section protected by usbhid->lock in hid_ctrl() is too
>> big and because of this it causes a recursive deadlock. "Too big" means
>> the case statement
n't
> any of these types (e.g. the "Cintiq 24HDT Monitor Control"). This patch
> updates the logic so that no suffix will be added to the device name if
> the device type is unknown.
>
> Signed-off-by: Jason Gerecke
Reviewed-by: Ping Cheng for the whole set.
Cheers,
Ping
>
aren't
any of these types (e.g. the Cintiq 24HDT Monitor Control). This patch
updates the logic so that no suffix will be added to the device name if
the device type is unknown.
Signed-off-by: Jason Gerecke jason.gere...@wacom.com
Reviewed-by: Ping Cheng pi...@wacom.com for the whole set.
Cheers
On Thu, Mar 5, 2015 at 2:36 PM, Benjamin Tissoires
wrote:
> 486b908 (HID: wacom: do not send pen events before touch is up/forced out)
> introduces a kernel oops when plugging a tablet without touch.
Thank you for the catch. You must have tried an Intuos 4 or earlier
model.
On Thu, Mar 5, 2015 at 2:36 PM, Benjamin Tissoires
benjamin.tissoi...@redhat.com wrote:
486b908 (HID: wacom: do not send pen events before touch is up/forced out)
introduces a kernel oops when plugging a tablet without touch.
Thank you for the catch. You must have tried an Intuos 4 or earlier
Tested-by: Josep Sanchez Ferreres
> Signed-off-by: Benjamin Tissoires
Acked-by: Ping Cheng for the series.
Cheers,
Ping
> ---
> changes in v3:
> - store touch_down information to not send pen events before the touch has
> been
> released
>
> changes
...@est.fib.upc.edu
Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com
Acked-by: Ping Cheng pi...@wacom.com for the series.
Cheers,
Ping
---
changes in v3:
- store touch_down information to not send pen events before the touch has
been
released
changes in v2:
- re-route the pen
On Wed, Feb 25, 2015 at 8:43 AM, Benjamin Tissoires
wrote:
> The stylus of this device works just fine out of the box.
> The touch is seen by default as a mouse with relative events and some
> gestures.
> The wireless and the wired version have slightly different firmwares, but
> the debug mode 2
On Wed, Feb 25, 2015 at 8:43 AM, Benjamin Tissoires
benjamin.tissoi...@redhat.com wrote:
The stylus of this device works just fine out of the box.
The touch is seen by default as a mouse with relative events and some
gestures.
The wireless and the wired version have slightly different
in this driver, it is better to
> use input_mt_sync_frame().
It is sensible to me, except I think you meant input_mt_slot() instead
of input_mt_assign_slot().
> Signed-off-by: Benjamin Tissoires
Reviewed-by: Ping Cheng
Ping
> ---
> drivers/hid/wacom_wac.c | 10 +-
> 1 fi
input_mt_assign_slot() in this driver, it is better to
use input_mt_sync_frame().
It is sensible to me, except I think you meant input_mt_slot() instead
of input_mt_assign_slot().
Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com
Reviewed-by: Ping Cheng pi...@wacom.com
Ping
---
drivers/hid
Hi Benjamin,
The relevant code was introduced by
d70420b914c98a3758674c6e9858810e0ab4ea30. Can you take a look and let
us know if Aniroop's patch fits your original thought or not?
Thanks,
Ping
On Tue, Jan 6, 2015 at 6:32 AM, Aniroop Mathur wrote:
> Dear Mr. Jason and Mr. Ping,
>
> Please
Hi Benjamin,
The relevant code was introduced by
d70420b914c98a3758674c6e9858810e0ab4ea30. Can you take a look and let
us know if Aniroop's patch fits your original thought or not?
Thanks,
Ping
On Tue, Jan 6, 2015 at 6:32 AM, Aniroop Mathur aniroop.mat...@gmail.com wrote:
Dear Mr. Jason and
and was rejected, we may have to keep clients
confused. With that said, the patch is
>
> This way, tablets connected through WL can be used from the user point of
> view in the same way they are used while connected through wire.
>
> Signed-off-by: Benjamin Tissoires
Reviewed-by:
-by: Ping Cheng pi...@wacom.com
Cheers,
Ping
---
Hi guys,
to continue on Ping's work regarding the Wireless reciever, here is a patch
which allows to have the same user experience regardless the transport layer.
Unfortunately, a libwacom patch is also required, but once both are applied
.
> For OLEDs, Bluetooth devices handle only 1-bit images instead of 4 for USB.
> The documentation has been updated to match this.
>
> Signed-off-by: Benjamin Tissoires
Reviewed-by: Ping Cheng
Ping
> Documentation/ABI/testing/sysfs-driver-wacom | 70
> +++---
devices handle only 1-bit images instead of 4 for USB.
The documentation has been updated to match this.
Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com
Reviewed-by: Ping Cheng pi...@wacom.com
Ping
Documentation/ABI/testing/sysfs-driver-wacom | 70
On Tue, Aug 5, 2014 at 2:34 PM, Benjamin Tissoires
wrote:
> Hi Ping,
>
> On Aug 05 2014 or thereabouts, Ping Cheng wrote:
>> Thank you Benjamin for updating the ABI docs. My comments are inline.
>>
>> With those lines updated, this patch is:
>>
>&
Thank you Benjamin for updating the ABI docs. My comments are inline.
With those lines updated, this patch is:
Reviewed-by: Ping Cheng
Cheers,
Ping
On Tue, Aug 5, 2014 at 8:34 AM, Benjamin Tissoires
wrote:
> Now the devices show up under hid no matter the connection. So update the
>
Thank you Benjamin for updating the ABI docs. My comments are inline.
With those lines updated, this patch is:
Reviewed-by: Ping Cheng pi...@wacom.com
Cheers,
Ping
On Tue, Aug 5, 2014 at 8:34 AM, Benjamin Tissoires
benjamin.tissoi...@redhat.com wrote:
Now the devices show up under hid
On Tue, Aug 5, 2014 at 2:34 PM, Benjamin Tissoires
benjamin.tissoi...@redhat.com wrote:
Hi Ping,
On Aug 05 2014 or thereabouts, Ping Cheng wrote:
Thank you Benjamin for updating the ABI docs. My comments are inline.
With those lines updated, this patch is:
Reviewed-by: Ping Cheng pi
t;bus == BUS_BLUETOOTH) {
>> + len = 256;
>> + xfer_id = WAC_CMD_ICON_BT_XFER;
>> + } else {
>> + len = 1024;
>> + xfer_id = WAC_CMD_ICON_XFER;
>> + }
>> +
>> + if (count != len)
>>
;
mutex_lock(wacom-lock);
- err = wacom_led_putimage(wacom, button_id, buf);
+ err = wacom_led_putimage(wacom, button_id, xfer_id, len, buf);
mutex_unlock(wacom-lock);
Signed-off-by: Przemo Firszt prz...@firszt.eu
Reviewed-by: Ping Cheng pi...@wacom.com
I'll test the whole series
entally - the patch is still sound as far as I
> am concerned. I will wait a bit for other to chime in with comments or
> acks before applying this series.
Thank you Benjamin and Dmitry for your support. The whole series is:
Acked-by: Ping Cheng
Cheers,
Ping
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
about doing it incrementally - the patch is still sound as far as I
am concerned. I will wait a bit for other to chime in with comments or
acks before applying this series.
Thank you Benjamin and Dmitry for your support. The whole series is:
Acked-by: Ping Cheng pi...@wacom.com
Cheers,
Ping
ency, the function wacom_register_input() is renamed into
> wacom_register_input().
wacom_register_inputs().
>
> Signed-off-by: Benjamin Tissoires
Reviewed-by: Ping Cheng .
Ping
> ---
>
> Hi,
>
> I noticed this afternoon that the "pad-in-a-separate-device" was missing the
>
, the function wacom_register_input() is renamed into
wacom_register_input().
wacom_register_inputs().
Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com
Reviewed-by: Ping Cheng pi...@wacom.com.
Ping
---
Hi,
I noticed this afternoon that the pad-in-a-separate-device was missing
Unless you plan to use other means to tell userland those events are
from PAD tool, ABS_MISC is necessary and is a reasonable way to group
PAD events. So, I do not think it is for backward compatibility. It is
there to stay. With that said, the whole patchset is
> Signed-off-by: Benj
, ABS_MISC is necessary and is a reasonable way to group
PAD events. So, I do not think it is for backward compatibility. It is
there to stay. With that said, the whole patchset is
Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com
Reviewed-by: Ping Cheng pi...@wacom.com
Thank you
Hi Benjamin,
On Monday, June 16, 2014, Benjamin Tissoires
wrote:
>
> Hi Ping,
>
> On Jun 13 2014 or thereabouts, Ping Cheng wrote:
> > Hi Benjamin,
> >
> > On Fri, Jun 13, 2014 at 1:29 PM, Benjamin Tissoires
> > wrote:
> > > This solves a bug with t
Hi Benjamin,
On Monday, June 16, 2014, Benjamin Tissoires
benjamin.tissoi...@redhat.com wrote:
Hi Ping,
On Jun 13 2014 or thereabouts, Ping Cheng wrote:
Hi Benjamin,
On Fri, Jun 13, 2014 at 1:29 PM, Benjamin Tissoires
benjamin.tissoi...@redhat.com wrote:
This solves a bug
ected.
> Signed-off-by: Benjamin Tissoires
Thank you for your support. But, sorry
NAKed-by: Ping Cheng
Ping
> ---
> drivers/input/tablet/wacom.h | 1 +
> drivers/input/tablet/wacom_sys.c | 6 ++
> 2 files changed, 7 insertions(+)
>
> diff --git a/drivers/input/tablet/
-by: Benjamin Tissoires benjamin.tissoi...@redhat.com
Thank you for your support. But, sorry
NAKed-by: Ping Cheng pi...@wacom.com
Ping
---
drivers/input/tablet/wacom.h | 1 +
drivers/input/tablet/wacom_sys.c | 6 ++
2 files changed, 7 insertions(+)
diff --git a/drivers/input/tablet
Hi Carl,
Thank you for the heads up. I believe Jason's patchset 4 of 4
(http://www.spinics.net/lists/linux-input/msg29435.html) fixed the
issue for your device and for other's. The patch was submitted last
month. If you can test the set on your device and give us a Tested-by
here, it will help
Hi Carl,
Thank you for the heads up. I believe Jason's patchset 4 of 4
(http://www.spinics.net/lists/linux-input/msg29435.html) fixed the
issue for your device and for other's. The patch was submitted last
month. If you can test the set on your device and give us a Tested-by
here, it will help
One of Wacom's pen and touch capable models added a switch for
users to turn on/off touch events. We need to report the state of
this switch to userland. But, there is no existing switch event
defined for this purpose. Luckily enough, there is a room for a
new switch event.
Signed-off-by: Ping
One of Wacom's pen and touch capable models added a switch for
users to turn on/off touch events. We need to report the state of
this switch to userland. But, there is no existing switch event
defined for this purpose. Luckily enough, there is a room for a
new switch event.
Signed-off-by: Ping
a555fa58a79b6ecbb6c531b8bab0650778d:
>
> Linux 3.6-rc4 (2012-09-01 10:39:58 -0700)
>
> are available in the git repository at:
>
> git://github.com/rydberg/linux.git maybe
This branch is tested with a Wacom Bamboo3 and an Intuos 5 M.
Tested-by: Ping Cheng
Ping
> fo
3.6-rc4 (2012-09-01 10:39:58 -0700)
are available in the git repository at:
git://github.com/rydberg/linux.git maybe
This branch is tested with a Wacom Bamboo3 and an Intuos 5 M.
Tested-by: Ping Cheng pi...@wacom.com
Ping
for you to fetch changes up
On Thu, Aug 16, 2012 at 11:07 AM, Henrik Rydberg wrote:
> On Wed, Aug 15, 2012 at 04:28:17PM -0700, Ping Cheng wrote:
>> On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg wrote:
>> > Collect common frame synchronization tasks in a new function,
>> > input_mt_sync_fram
On Thu, Aug 16, 2012 at 11:07 AM, Henrik Rydberg wrote:
> On Wed, Aug 15, 2012 at 04:28:17PM -0700, Ping Cheng wrote:
>> On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg wrote:
>> > Collect common frame synchronization tasks in a new function,
>> > input_mt_sync_fram
On Thu, Aug 16, 2012 at 11:07 AM, Henrik Rydberg rydb...@euromail.se wrote:
On Wed, Aug 15, 2012 at 04:28:17PM -0700, Ping Cheng wrote:
On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg rydb...@euromail.se wrote:
Collect common frame synchronization tasks in a new function,
input_mt_sync_frame
On Thu, Aug 16, 2012 at 11:07 AM, Henrik Rydberg rydb...@euromail.se wrote:
On Wed, Aug 15, 2012 at 04:28:17PM -0700, Ping Cheng wrote:
On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg rydb...@euromail.se wrote:
Collect common frame synchronization tasks in a new function,
input_mt_sync_frame
On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg wrote:
> Collect common frame synchronization tasks in a new function,
> input_mt_sync_frame(). Depending on the flags set, it drops
> unseen contacts and performs pointer emulation.
>
> Signed-off-by: Henrik Rydberg
I went through the patchset
On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg rydb...@euromail.se wrote:
Collect common frame synchronization tasks in a new function,
input_mt_sync_frame(). Depending on the flags set, it drops
unseen contacts and performs pointer emulation.
Signed-off-by: Henrik Rydberg
On Tue, Aug 14, 2012 at 2:12 PM, Dmitry Torokhov
wrote:
> >> On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg
> >> wrote:
> >> > Many MT devices send a number of keys along with the mt information.
> >> > This patch makes sure that there is room for them in the packet
> >> > buffer.
>>
>> So,
On Tue, Aug 14, 2012 at 1:01 PM, Henrik Rydberg wrote:
> Hi Ping,
>
> Long time no see. :-)
>
>> > + /* Make room for KEY and MSC events */
>> > + events += 7;
>>
>> It is nice to get rid of the redundant pieces and to incorporate
>> common functions. Thank you.
>>
>> I have a
On Tue, Aug 14, 2012 at 12:53 PM, Dmitry Torokhov
wrote:
> On Tuesday, August 14, 2012 12:32:21 PM Ping Cheng wrote:
>> On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg wrote:
>> > Many MT devices send a number of keys along with the mt information.
>> > This patch ma
On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg wrote:
> Many MT devices send a number of keys along with the mt information.
> This patch makes sure that there is room for them in the packet
> buffer.
>
> Signed-off-by: Henrik Rydberg
> ---
> drivers/input/input.c | 10 +++---
> 1 file
On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg rydb...@euromail.se wrote:
Many MT devices send a number of keys along with the mt information.
This patch makes sure that there is room for them in the packet
buffer.
Signed-off-by: Henrik Rydberg rydb...@euromail.se
---
On Tue, Aug 14, 2012 at 12:53 PM, Dmitry Torokhov
dmitry.torok...@gmail.com wrote:
On Tuesday, August 14, 2012 12:32:21 PM Ping Cheng wrote:
On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg rydb...@euromail.se wrote:
Many MT devices send a number of keys along with the mt information
On Tue, Aug 14, 2012 at 1:01 PM, Henrik Rydberg rydb...@euromail.se wrote:
Hi Ping,
Long time no see. :-)
+ /* Make room for KEY and MSC events */
+ events += 7;
It is nice to get rid of the redundant pieces and to incorporate
common functions. Thank you.
I have a question
On Tue, Aug 14, 2012 at 2:12 PM, Dmitry Torokhov
dmitry.torok...@gmail.com wrote:
On Sun, Aug 12, 2012 at 2:42 PM, Henrik Rydberg rydb...@euromail.se
wrote:
Many MT devices send a number of keys along with the mt information.
This patch makes sure that there is room for them in the
64 matches
Mail list logo