Hi Fabio, On Fri, Mar 04, 2016 at 07:07:58PM -0300, Fabio Estevam wrote: > From: Fabio Estevam <fabio.estevam@xxxxxxx> > > REPORT_MODE_MOUSE and REPORT_MODE_VENDOR are never used in the driver, > so let's just remove them. > > Signed-off-by: Fabio Estevam <fabio.estevam@xxxxxxx> > --- > drivers/input/touchscreen/egalax_ts.c | 12 ------------ > 1 file changed, 12 deletions(-) > > diff --git a/drivers/input/touchscreen/egalax_ts.c b/drivers/input/touchscreen/egalax_ts.c > index 1afc08b..a260476 100644 > --- a/drivers/input/touchscreen/egalax_ts.c > +++ b/drivers/input/touchscreen/egalax_ts.c > @@ -29,18 +29,6 @@ > #include <linux/input/mt.h> > #include <linux/of_gpio.h> > > -/* > - * Mouse Mode: some panel may configure the controller to mouse mode, > - * which can only report one point at a given time. > - * This driver will ignore events in this mode. > - */ > -#define REPORT_MODE_MOUSE 0x1 > -/* > - * Vendor Mode: this mode is used to transfer some vendor specific > - * messages. > - * This driver will ignore events in this mode. > - */ > -#define REPORT_MODE_VENDOR 0x3 Keeping this does not add to runtime size and is not excessive, I do not see harm in not keeping it. > /* Multiple Touch Mode */ > #define REPORT_MODE_MTTOUCH 0x4 > > -- > 1.9.1 > Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
