From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH] Input: elan_i2c - Expand maxmium product_id form 0xFF to 0xFFFF Date: Mon, 21 Sep 2015 09:31:42 -0700 Message-ID: <20150921163142.GF17389@dtor-ws> References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: duson Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Arker , =?utf-8?B?5L2V5Ye955KL?= , kyle_lin List-Id: linux-input@vger.kernel.org On Thu, Sep 10, 2015 at 09:55:47PM +0800, duson wrote: > In order to support more projects in the feture, we expand the > maxmium product_id value form 0xFF to 0xFFFF. >=20 > Signed-off by: Duson Lin Applied, thank you. > --- > drivers/input/mouse/elan_i2c.h | 2 +- > drivers/input/mouse/elan_i2c_core.c | 6 +++--- > drivers/input/mouse/elan_i2c_i2c.c | 4 ++-- > drivers/input/mouse/elan_i2c_smbus.c | 4 ++-- > 4 files changed, 8 insertions(+), 8 deletions(-) >=20 > diff --git a/drivers/input/mouse/elan_i2c.h b/drivers/input/mouse/ela= n_i2c.h > index 73670f2..c0ec261 100644 > --- a/drivers/input/mouse/elan_i2c.h > +++ b/drivers/input/mouse/elan_i2c.h > @@ -60,7 +60,7 @@ struct elan_transport_ops { > int (*get_sm_version)(struct i2c_client *client, > u8* ic_type, u8 *version); > int (*get_checksum)(struct i2c_client *client, bool iap, u16 *csum)= ; > - int (*get_product_id)(struct i2c_client *client, u8 *id); > + int (*get_product_id)(struct i2c_client *client, u16 *id); > =20 > int (*get_max)(struct i2c_client *client, > unsigned int *max_x, unsigned int *max_y); > diff --git a/drivers/input/mouse/elan_i2c_core.c b/drivers/input/mous= e/elan_i2c_core.c > index e2b7420..4a5450d 100644 > --- a/drivers/input/mouse/elan_i2c_core.c > +++ b/drivers/input/mouse/elan_i2c_core.c > @@ -4,7 +4,7 @@ > * Copyright (c) 2013 ELAN Microelectronics Corp. > * > * Author: =E6=9E=97=E6=94=BF=E7=B6=AD (Duson Lin) > - * Version: 1.6.0 > + * Version: 1.6.1 > * > * Based on cyapa driver: > * copyright (c) 2011-2012 Cypress Semiconductor, Inc. > @@ -40,7 +40,7 @@ > #include "elan_i2c.h" > =20 > #define DRIVER_NAME "elan_i2c" > -#define ELAN_DRIVER_VERSION "1.6.0" > +#define ELAN_DRIVER_VERSION "1.6.1" > #define ETP_MAX_PRESSURE 255 > #define ETP_FWIDTH_REDUCE 90 > #define ETP_FINGER_WIDTH 15 > @@ -76,7 +76,7 @@ struct elan_tp_data { > unsigned int x_res; > unsigned int y_res; > =20 > - u8 product_id; > + u16 product_id; > u8 fw_version; > u8 sm_version; > u8 iap_version; > diff --git a/drivers/input/mouse/elan_i2c_i2c.c b/drivers/input/mouse= /elan_i2c_i2c.c > index 683c840..a679e56 100644 > --- a/drivers/input/mouse/elan_i2c_i2c.c > +++ b/drivers/input/mouse/elan_i2c_i2c.c > @@ -276,7 +276,7 @@ static int elan_i2c_get_sm_version(struct i2c_cli= ent *client, > return 0; > } > =20 > -static int elan_i2c_get_product_id(struct i2c_client *client, u8 *id= ) > +static int elan_i2c_get_product_id(struct i2c_client *client, u16 *i= d) > { > int error; > u8 val[3]; > @@ -287,7 +287,7 @@ static int elan_i2c_get_product_id(struct i2c_cli= ent *client, u8 *id) > return error; > } > =20 > - *id =3D val[0]; > + *id =3D le16_to_cpup((__le16 *)val); > return 0; > } > =20 > diff --git a/drivers/input/mouse/elan_i2c_smbus.c b/drivers/input/mou= se/elan_i2c_smbus.c > index ff36a36..cb6aecb 100644 > --- a/drivers/input/mouse/elan_i2c_smbus.c > +++ b/drivers/input/mouse/elan_i2c_smbus.c > @@ -183,7 +183,7 @@ static int elan_smbus_get_sm_version(struct i2c_c= lient *client, > return 0; > } > =20 > -static int elan_smbus_get_product_id(struct i2c_client *client, u8 *= id) > +static int elan_smbus_get_product_id(struct i2c_client *client, u16 = *id) > { > int error; > u8 val[3]; > @@ -195,7 +195,7 @@ static int elan_smbus_get_product_id(struct i2c_c= lient *client, u8 *id) > return error; > } > =20 > - *id =3D val[1]; > + *id =3D be16_to_cpup((__be16 *)val); > return 0; > } >=20 --=20 Dmitry