Messages in this thread | | | Date | Fri, 25 Jul 2014 13:06:56 -0700 | From | Dmitry Torokhov <> | Subject | Re: [PATCH v2] Input: add i2c/smbus driver for elan touchpad |
| |
On Fri, Jul 25, 2014 at 11:20:53AM -0700, Dmitry Torokhov wrote: > Hi Duson, > > On Sat, Jul 26, 2014 at 02:05:02AM +0800, cheng wei Lin wrote: > > Hi Dmitry, > > > > Sorry for late to reply you and thanks for your command > > > > Yes, I also agree, it looks great than my original code. > > Thank you for confirming. > > > So, should I modify this and upstream v3 patch for you? > > No, that is fine, I'll fix it up on my end.
Hmm, I am also looking at elan_get_x_resolution() and in case of SMB we'll never get negative result since we are doing "elan_convert_res(val[1] & 0x0F)". Was it intended as such, or do we need to extend sign here via
(((char)(val[1] & 0x0F)) << 4) >> 4;
Thanks.
-- Dmitry
| |