All of lore.kernel.org
 help / color / mirror / Atom feed
* [Patch v2] input:rohm based bu21013 touch panel controller driver support
@ 2010-09-09 11:36 Naveen Kumar GADDIPATI
  2010-09-09 12:10 ` Trilok Soni
  2010-09-09 15:59 ` Dmitry Torokhov
  0 siblings, 2 replies; 13+ messages in thread
From: Naveen Kumar GADDIPATI @ 2010-09-09 11:36 UTC (permalink / raw)
  To: Dmitry Torokhov; +Cc: Trilok Soni, linux-input, STEricsson_nomadik_linux

Hi Dmitry,

From: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>

Added the ROHM based bu21013 capacitive touch panel controller
driver support with i2c interface.

Acked-by: Linus Walleij <linus.walleij@stericsson.com>
Signed-off-by: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
---
Modifications in v2:
        --Updated with the Dmitry comments on Patch v1
        --Updated with the Trilok comments on Patch v1


 drivers/input/touchscreen/Kconfig      |   12 +
 drivers/input/touchscreen/Makefile     |    1 +
 drivers/input/touchscreen/bu21013_ts.c |  770 ++++++++++++++++++++++++++++++++
 include/linux/input/bu21013.h          |   44 ++
 4 files changed, 827 insertions(+), 0 deletions(-)
 create mode 100644 drivers/input/touchscreen/bu21013_ts.c
 create mode 100644 include/linux/input/bu21013.h

diff --git a/drivers/input/touchscreen/Kconfig b/drivers/input/touchscreen/Kconfig
index 0069d97..2380533 100644
--- a/drivers/input/touchscreen/Kconfig
+++ b/drivers/input/touchscreen/Kconfig
@@ -98,6 +98,18 @@ config TOUCHSCREEN_BITSY
          To compile this driver as a module, choose M here: the
          module will be called h3600_ts_input.

+config TOUCHSCREEN_BU21013
+       tristate "BU21013 based touch panel controllers"
+       depends on I2C
+       help
+         Say Y here if you have a bu21013 touchscreen connected to
+         your system.
+
+         If unsure, say N.
+
+         To compile this driver as a module, choose M here: the
+         module will be called bu21013_ts.
+
 config TOUCHSCREEN_CY8CTMG110
        tristate "cy8ctmg110 touchscreen"
        depends on I2C
diff --git a/drivers/input/touchscreen/Makefile b/drivers/input/touchscreen/Makefile
index 28217e1..a0a2021 100644
--- a/drivers/input/touchscreen/Makefile
+++ b/drivers/input/touchscreen/Makefile
@@ -14,6 +14,7 @@ obj-$(CONFIG_TOUCHSCREEN_AD7879_SPI)  += ad7879-spi.o
 obj-$(CONFIG_TOUCHSCREEN_ADS7846)      += ads7846.o
 obj-$(CONFIG_TOUCHSCREEN_ATMEL_TSADCC) += atmel_tsadcc.o
 obj-$(CONFIG_TOUCHSCREEN_BITSY)                += h3600_ts_input.o
+obj-$(CONFIG_TOUCHSCREEN_BU21013)       += bu21013_ts.o
 obj-$(CONFIG_TOUCHSCREEN_CY8CTMG110)   += cy8ctmg110_ts.o
 obj-$(CONFIG_TOUCHSCREEN_DA9034)       += da9034-ts.o
 obj-$(CONFIG_TOUCHSCREEN_DYNAPRO)      += dynapro.o
diff --git a/drivers/input/touchscreen/bu21013_ts.c b/drivers/input/touchscreen/bu21013_ts.c
new file mode 100644
index 0000000..117d9c5
--- /dev/null
+++ b/drivers/input/touchscreen/bu21013_ts.c
@@ -0,0 +1,770 @@
+/*
+ * Copyright (C) ST-Ericsson SA 2010
+ * Author: Naveen Kumar G <naveen.gaddipati@stericsson.com> for ST-Ericsson
+ * License terms:GNU General Public License (GPL) version 2
+ */
+
+#include <linux/kernel.h>
+#include <linux/init.h>
+#include <linux/delay.h>
+#include <linux/interrupt.h>
+#include <linux/i2c.h>
+#include <linux/workqueue.h>
+#include <linux/input.h>
+#include <linux/types.h>
+#include <linux/timer.h>
+#include <linux/input/bu21013.h>
+#include <linux/slab.h>
+
+#define PEN_DOWN_INTR                  0
+#define PEN_UP_INTR                    1
+#define RESET_DELAY                    30
+#define POLLING_DELAY                  100
+#define MAX_TOOL_WIDTH                 15
+#define MAX_TOUCH_MAJOR                        255
+#define MAX_TOUCH_MINOR                        15
+#define MAX_PRESSURE                   1
+#define PENUP_TIMEOUT                  (2)
+#define DELTA_MIN                      16
+#define MASK_BITS                      0x03
+#define SHIFT_8                                8
+#define SHIFT_2                                2
+#define LENGTH_OF_BUFFER               11
+
+#define BU21013_SENSORS_BTN_0_7_REG    0x70
+#define BU21013_SENSORS_BTN_8_15_REG   0x71
+#define BU21013_SENSORS_BTN_16_23_REG  0x72
+#define BU21013_X1_POS_MSB_REG         0x73
+#define BU21013_X1_POS_LSB_REG         0x74
+#define BU21013_Y1_POS_MSB_REG         0x75
+#define BU21013_Y1_POS_LSB_REG         0x76
+#define BU21013_X2_POS_MSB_REG         0x77
+#define BU21013_X2_POS_LSB_REG         0x78
+#define BU21013_Y2_POS_MSB_REG         0x79
+#define BU21013_Y2_POS_LSB_REG         0x7A
+#define BU21013_INT_CLR_REG            0xE8
+#define BU21013_INT_MODE_REG           0xE9
+#define BU21013_GAIN_REG               0xEA
+#define BU21013_OFFSET_MODE_REG                0xEB
+#define BU21013_XY_EDGE_REG            0xEC
+#define BU21013_RESET_REG              0xED
+#define BU21013_CALIB_REG              0xEE
+#define BU21013_DONE_REG               0xEF
+#define BU21013_SENSOR_0_7_REG         0xF0
+#define BU21013_SENSOR_8_15_REG                0xF1
+#define BU21013_SENSOR_16_23_REG       0xF2
+#define BU21013_POS_MODE1_REG          0xF3
+#define BU21013_POS_MODE2_REG          0xF4
+#define BU21013_CLK_MODE_REG           0xF5
+#define BU21013_IDLE_REG               0xFA
+#define BU21013_FILTER_REG             0xFB
+#define BU21013_TH_ON_REG              0xFC
+#define BU21013_TH_OFF_REG             0xFD
+
+#define BU21013_RESET_ENABLE           0x01
+
+#define BU21013_SENSORS_EN_0_7         0x3F
+#define BU21013_SENSORS_EN_8_15                0xFC
+#define BU21013_SENSORS_EN_16_23       0x1F
+
+#define BU21013_POS_MODE1_0            0x02
+#define BU21013_POS_MODE1_1            0x04
+#define BU21013_POS_MODE1_2            0x08
+
+#define BU21013_POS_MODE2_ZERO         0x01
+#define BU21013_POS_MODE2_AVG1         0x02
+#define BU21013_POS_MODE2_AVG2         0x04
+#define BU21013_POS_MODE2_EN_XY                0x08
+#define BU21013_POS_MODE2_EN_RAW       0x10
+#define BU21013_POS_MODE2_MULTI                0x80
+
+#define BU21013_CLK_MODE_DIV           0x01
+#define BU21013_CLK_MODE_EXT           0x02
+#define BU21013_CLK_MODE_CALIB         0x80
+
+#define BU21013_IDLET_0                        0x01
+#define BU21013_IDLET_1                        0x02
+#define BU21013_IDLET_2                        0x04
+#define BU21013_IDLET_3                        0x08
+#define BU21013_IDLE_INTERMIT_EN       0x10
+
+#define BU21013_DELTA_0_6              0x7F
+#define BU21013_FILTER_EN              0x80
+
+#define BU21013_INT_MODE_LEVEL         0x00
+#define BU21013_INT_MODE_EDGE          0x01
+
+#define BU21013_GAIN_0                 0x01
+#define BU21013_GAIN_1                 0x02
+#define BU21013_GAIN_2                 0x04
+
+#define BU21013_OFFSET_MODE_DEFAULT    0x00
+#define BU21013_OFFSET_MODE_MOVE       0x01
+#define BU21013_OFFSET_MODE_DISABLE    0x02
+
+#define BU21013_TH_ON_0                        0x01
+#define BU21013_TH_ON_1                        0x02
+#define BU21013_TH_ON_2                        0x04
+#define BU21013_TH_ON_3                        0x08
+#define BU21013_TH_ON_4                        0x10
+#define BU21013_TH_ON_5                        0x20
+#define BU21013_TH_ON_6                        0x40
+#define BU21013_TH_ON_7                        0x80
+#define BU21013_TH_ON_MAX              0xFF
+
+#define BU21013_TH_OFF_0               0x01
+#define BU21013_TH_OFF_1               0x02
+#define BU21013_TH_OFF_2               0x04
+#define BU21013_TH_OFF_3               0x08
+#define BU21013_TH_OFF_4               0x10
+#define BU21013_TH_OFF_5               0x20
+#define BU21013_TH_OFF_6               0x40
+#define BU21013_TH_OFF_7               0x80
+#define BU21013_TH_OFF_MAX             0xFF
+
+#define BU21013_X_EDGE_0               0x01
+#define BU21013_X_EDGE_1               0x02
+#define BU21013_X_EDGE_2               0x04
+#define BU21013_X_EDGE_3               0x08
+#define BU21013_Y_EDGE_0               0x10
+#define BU21013_Y_EDGE_1               0x20
+#define BU21013_Y_EDGE_2               0x40
+#define BU21013_Y_EDGE_3               0x80
+
+#define BU21013_DONE                   0x01
+#define BU21013_NUMBER_OF_X_SENSORS    (6)
+#define BU21013_NUMBER_OF_Y_SENSORS    (11)
+
+#define DRIVER_TP      "bu21013_tp"
+
+/**
+ * struct bu21013_ts_data - touch panel data structure
+ * @client:    pointer to the i2c client
+ * @chip:      pointer to the touch panel controller
+ * @in_dev:    pointer to the input device structure
+ * @wait: variable to wait_queue_head_t structure
+ * @intr_pin: interrupt pin value
+ * @touch_stopped: touch stop flag
+ * @x_pos: x position for two co-ordinates
+ * @y_pos: y position for two co-ordinates
+ * @previous_press_reported: last reported flag
+ *
+ * Touch panel device data structure
+ */
+struct bu21013_ts_data {
+       struct i2c_client *client;
+       struct bu21013_platform_device *chip;
+       struct input_dev *in_dev;
+       wait_queue_head_t wait;
+       bool touch_stopped;
+       unsigned int intr_pin;
+       signed short    x_pos[2];
+       signed short    y_pos[2];
+       bool previous_press_reported;
+};
+
+/**
+ * bu21013_report_pen_down() - reports the pen down event
+ * @data:bu21013_ts_data structure pointer
+ * @count:touch count
+ *
+ * This function used to report the pen down interrupt to
+ * input subsystem and returns none
+ */
+static void bu21013_report_pen_down(struct bu21013_ts_data *data, int count)
+{
+       int i;
+
+       input_report_abs(data->in_dev, ABS_X, data->x_pos[0]);
+       input_report_abs(data->in_dev, ABS_Y, data->y_pos[0]);
+       input_report_key(data->in_dev, BTN_TOUCH, count);
+
+       if (data->chip->multi_touch) {
+               for (i = 0; i < count; i++) {
+                       input_report_abs(data->in_dev, ABS_MT_POSITION_X,
+                                                       data->x_pos[i]);
+                       input_report_abs(data->in_dev, ABS_MT_POSITION_Y,
+                                                       data->y_pos[i]);
+                       input_mt_sync(data->in_dev);
+               }
+       }
+       input_sync(data->in_dev);
+       data->previous_press_reported = count;
+}
+/**
+ * bu21013_report_pen_up() - reports the pen up event
+ * @data:bu21013_ts_data structure pointer
+ *
+ * This function used to report the pen up interrupt
+ * to input subsystem and returns none
+ */
+static void bu21013_report_pen_up(struct bu21013_ts_data *data)
+{
+       input_report_key(data->in_dev, BTN_TOUCH, 0);
+       if (data->chip->multi_touch) {
+               input_report_key(data->in_dev, BTN_2, 0);
+               input_mt_sync(data->in_dev);
+       }
+       input_sync(data->in_dev);
+       data->previous_press_reported = 0;
+}
+/**
+ * bu21013_touch_calc() - calculates the co-ordinates delta
+ * @data: bu21013_ts_data structure pointer
+ * @x: x position
+ * @y: y position
+ * @count: touch count
+ *
+ * This function calculates the exact co-ordinates with respect to
+ * display resolution and returns none
+ */
+static void bu21013_touch_calc
+       (struct bu21013_ts_data *data, int x, int y, int count)
+{
+       data->x_pos[count] = x;
+       data->y_pos[count] = y;
+       if (data->chip->x_flip)
+               data->x_pos[count] =
+                       data->chip->touch_x_max - data->x_pos[count];
+       if (data->chip->y_flip)
+               data->y_pos[count] =
+                       data->chip->touch_y_max - data->y_pos[count];
+}
+/**
+ * bu21013_verify_delta() - verify the co-ordinates delta
+ * @x1: x1 position
+ * @y1: y1 position
+ * @x2: x2 position
+ * @y2: y2 position
+ *
+ * This function verifies the delta of the
+ * co-ordinates and returns boolean.
+ */
+static bool bu21013_verify_delta(int x1, int y1, int x2, int y2)
+{
+       int delta_x, delta_y;
+
+       if ((x1 != 0) && (y1 != 0)) {
+               delta_x = x2 - x1;
+               if (x1 > x2)
+                       delta_x = x1 - x2;
+               delta_y = y2 - y1;
+               if (y1 > y2)
+                       delta_y = y1 - y2;
+               if ((delta_x < DELTA_MIN) || (delta_y < DELTA_MIN))
+                       return false;
+       }
+       return true;
+}
+
+/**
+ * bu21013_do_touch_report(): Get the touch co-ordinates
+ * @data:bu21013_ts_data structure pointer
+ *
+ * Get the touch co-ordinates from touch sensor registers and writes
+ * into device structure and returns integer.
+ */
+static int bu21013_do_touch_report(struct bu21013_ts_data *data)
+{
+       u8      buf[LENGTH_OF_BUFFER];
+       int     finger1_valid = 0;
+       int     finger2_valid = 0;
+       unsigned int    finger1_pos_x;
+       unsigned int    finger1_pos_y;
+       unsigned int    finger2_pos_x = 0;
+       unsigned int    finger2_pos_y = 0;
+       int     number_of_active_x_sensors;
+       int     number_of_active_y_sensors;
+       int     total_number_of_active_sensors;
+       int     finger_down_count = 0;
+       int     ret = 0;
+
+       if (data == NULL)
+               return -EINVAL;
+
+       ret = i2c_smbus_read_i2c_block_data(data->client,
+                                               BU21013_SENSORS_BTN_0_7_REG,
+                                                       LENGTH_OF_BUFFER, buf);
+       if (ret < LENGTH_OF_BUFFER)
+               return -EINVAL;
+
+       number_of_active_x_sensors = hweight32(buf[0] &
+                                       BU21013_SENSORS_EN_0_7);
+       number_of_active_y_sensors = hweight32(
+                       ((buf[1] & BU21013_SENSORS_EN_8_15) |
+               ((buf[2] & BU21013_SENSORS_EN_16_23) << SHIFT_8)) >> SHIFT_2);
+       if (((number_of_active_x_sensors != 0) &&
+               (number_of_active_y_sensors == 0)) ||
+               ((number_of_active_x_sensors == 0) &&
+               (number_of_active_y_sensors != 0)))
+               return 0;
+
+       total_number_of_active_sensors =
+               number_of_active_x_sensors + number_of_active_y_sensors;
+
+       finger1_pos_x = buf[3] << SHIFT_2 | (buf[4] & MASK_BITS);
+       finger1_pos_y = buf[5] << SHIFT_2 | (buf[6] & MASK_BITS);
+
+       if (data->chip->multi_touch) {
+               finger2_pos_x = buf[7] << SHIFT_2 | (buf[8] & MASK_BITS);
+               finger2_pos_y = buf[9] << SHIFT_2 | (buf[10] & MASK_BITS);
+       }
+
+       if (total_number_of_active_sensors) {
+               if ((finger2_pos_x != 0) && (finger2_pos_y != 0)) {
+                       if ((finger1_pos_x == 0) || (finger1_pos_y == 0))
+                               return 0;
+                       ret = bu21013_verify_delta(finger1_pos_x,
+                               finger1_pos_y, finger2_pos_x, finger2_pos_y);
+                       if (!ret)
+                               total_number_of_active_sensors = 0;
+               }
+       }
+
+       if (total_number_of_active_sensors) {
+               finger1_valid = 1;
+               if ((finger1_pos_x == 0) || (finger1_pos_y == 0) ||
+                       (finger1_pos_x >= data->chip->touch_x_max) ||
+                       (finger1_pos_y >= data->chip->touch_y_max))
+                       finger1_valid = 0;
+               finger2_valid = 1;
+               if ((finger2_pos_x == 0) || (finger2_pos_y == 0) ||
+                       (finger2_pos_x >= data->chip->touch_x_max) ||
+                       (finger2_pos_y >= data->chip->touch_y_max))
+                       finger2_valid = 0;
+               if ((finger1_valid == 0) && (finger2_valid == 0))
+                       return 0;
+               if (finger1_valid) {
+                       bu21013_touch_calc(data, finger1_pos_x,
+                               finger1_pos_y, finger_down_count);
+                       finger_down_count++;
+               }
+               if (finger2_valid) {
+                       bu21013_touch_calc(data, finger2_pos_x,
+                               finger2_pos_y, finger_down_count);
+                       finger_down_count++;
+               }
+       }
+
+       if (finger_down_count > 0)
+               bu21013_report_pen_down(data, finger_down_count);
+       else {
+               if (data->previous_press_reported)
+                       bu21013_report_pen_up(data);
+       }
+
+       return ret;
+}
+
+/**
+ * bu21013_gpio_irq() - gpio thread function for touch interrupt
+ * @irq: irq value
+ * @device_data:void pointer
+ *
+ * This gpio thread function for touch interrupt
+ * and returns irqreturn_t.
+ */
+static irqreturn_t bu21013_gpio_irq(int irq, void *device_data)
+{
+       struct bu21013_ts_data *data = device_data;
+       struct i2c_client *i2c = data->client;
+       int retval;
+
+       do {
+               retval = bu21013_do_touch_report(data);
+               if (retval < 0) {
+                       dev_err(&i2c->dev, "bu21013_do_touch_report failed\n");
+                       return IRQ_NONE;
+               }
+
+               data->intr_pin = data->chip->irq_read_val();
+               if (data->intr_pin == PEN_DOWN_INTR)
+                       wait_event_timeout(data->wait, data->touch_stopped,
+                                                       msecs_to_jiffies(10));
+       } while (!data->intr_pin && !data->touch_stopped);
+
+       return IRQ_HANDLED;
+}
+
+/**
+ * bu21013_init_chip() - power on sequence for the bu21013 controller
+ * @data: device structure pointer
+ *
+ * This function is used to power on
+ * the bu21013 controller and returns integer.
+ */
+static int bu21013_init_chip(struct bu21013_ts_data *data)
+{
+       int retval;
+       struct i2c_client *i2c = data->client;
+
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_RESET_REG,
+                                       BU21013_RESET_ENABLE);
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_RESET reg write failed\n");
+               goto err;
+       }
+       msleep(RESET_DELAY);
+
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_0_7_REG,
+                                       BU21013_SENSORS_EN_0_7);
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_SENSOR_0_7 reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_8_15_REG,
+                                       BU21013_SENSORS_EN_8_15);
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_SENSOR_8_15 reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_16_23_REG,
+                                       BU21013_SENSORS_EN_16_23);
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_SENSOR_16_23 reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE1_REG,
+                               (BU21013_POS_MODE1_0 | BU21013_POS_MODE1_1));
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_POS_MODE1 reg write failed\n");
+               goto err;
+       }
+       if (data->chip->multi_touch)
+               retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE2_REG,
+                       (BU21013_POS_MODE2_ZERO | BU21013_POS_MODE2_AVG1 |
+                       BU21013_POS_MODE2_AVG2 | BU21013_POS_MODE2_EN_RAW |
+                       BU21013_POS_MODE2_MULTI));
+       else
+               retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE2_REG,
+                       (BU21013_POS_MODE2_ZERO | BU21013_POS_MODE2_AVG1 |
+                       BU21013_POS_MODE2_AVG2 | BU21013_POS_MODE2_EN_RAW));
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_POS_MODE2 reg write failed\n");
+               goto err;
+       }
+       if (data->chip->ext_clk)
+               retval = i2c_smbus_write_byte_data(i2c, BU21013_CLK_MODE_REG,
+                       (BU21013_CLK_MODE_EXT | BU21013_CLK_MODE_CALIB));
+       else
+               retval = i2c_smbus_write_byte_data(i2c, BU21013_CLK_MODE_REG,
+                       (BU21013_CLK_MODE_DIV | BU21013_CLK_MODE_CALIB));
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_CLK_MODE reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_IDLE_REG,
+                               (BU21013_IDLET_0 | BU21013_IDLE_INTERMIT_EN));
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_IDLE reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_INT_MODE_REG,
+                                               BU21013_INT_MODE_LEVEL);
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_INT_MODE reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_FILTER_REG,
+                                               (BU21013_DELTA_0_6 |
+                                                       BU21013_FILTER_EN));
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_FILTER reg write failed\n");
+               goto err;
+       }
+
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_TH_ON_REG,
+                                       (BU21013_TH_ON_3 | BU21013_TH_ON_4));
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_TH_ON reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_TH_OFF_REG,
+                                                       BU21013_TH_OFF_4);
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_TH_OFF reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_GAIN_REG,
+                                       (BU21013_GAIN_0 | BU21013_GAIN_1));
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_GAIN reg write failed\n");
+               goto err;
+       }
+
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_OFFSET_MODE_REG,
+                                       BU21013_OFFSET_MODE_DEFAULT);
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_OFFSET_MODE reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_XY_EDGE_REG,
+                       (BU21013_X_EDGE_0 | BU21013_X_EDGE_2 |
+                               BU21013_Y_EDGE_1 | BU21013_Y_EDGE_3));
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_XY_EDGE reg write failed\n");
+               goto err;
+       }
+       retval = i2c_smbus_write_byte_data(i2c, BU21013_DONE_REG,
+                                                       BU21013_DONE);
+       if (retval < 0) {
+               dev_err(&i2c->dev, "BU21013_REG_DONE reg write failed\n");
+               goto err;
+       }
+
+       data->x_pos[0] = 0;
+       data->x_pos[1] = 0;
+       data->y_pos[0] = 0;
+       data->y_pos[1] = 0;
+       data->previous_press_reported = 0;
+
+err:
+       return retval;
+}
+
+#ifdef CONFIG_PM
+/**
+ * bu21013_suspend() - suspend the touch screen controller
+ * @client: pointer to i2c client structure
+ * @mesg: message from power manager
+ *
+ * This funtion is used to suspend the
+ * touch panel controller and returns integer
+ */
+static int bu21013_suspend(struct i2c_client *client, pm_message_t mesg)
+{
+       struct bu21013_ts_data *bu21013_data = i2c_get_clientdata(client);
+
+       bu21013_data->touch_stopped = true;
+       wake_up(&bu21013_data->wait);
+       if (device_may_wakeup(&client->dev))
+               enable_irq_wake(bu21013_data->chip->irq);
+       else
+               disable_irq(bu21013_data->chip->irq);
+
+       return 0;
+}
+
+/**
+ * bu21013_resume() - resume the touch screen controller
+ * @client: pointer to i2c client structure
+ *
+ * This funtion is used to resume the touch panel
+ * controller and returns integer.
+ */
+static int bu21013_resume(struct i2c_client *client)
+{
+       int retval;
+       struct bu21013_ts_data *bu21013_data = i2c_get_clientdata(client);
+
+       retval = bu21013_init_chip(bu21013_data);
+       if (retval < 0) {
+               dev_err(&client->dev, "tsc config failed\n");
+               return retval;
+       }
+
+       bu21013_data->touch_stopped = false;
+       if (device_may_wakeup(&client->dev))
+               disable_irq_wake(bu21013_data->chip->irq);
+       else
+               enable_irq(bu21013_data->chip->irq);
+
+       return 0;
+}
+#endif
+
+/**
+ * bu21013_probe() - initialzes the i2c-client touchscreen driver
+ * @i2c: i2c client structure pointer
+ * @id:i2c device id pointer
+ *
+ * This function used to initializes the i2c-client touchscreen
+ * driver and returns integer.
+ */
+static int __devinit bu21013_probe(struct i2c_client *i2c,
+                                       const struct i2c_device_id *id)
+{
+       int retval;
+       struct bu21013_ts_data *bu21013_data;
+       struct input_dev *in_dev;
+       short x_max;
+       short y_max;
+       struct bu21013_platform_device *pdata = i2c->dev.platform_data;
+
+       if (!i2c) {
+               dev_err(&i2c->dev, "i2c client not defined\n");
+               retval = -EINVAL;
+               return retval;
+       }
+
+       if (!pdata) {
+               dev_err(&i2c->dev, "platform data not defined\n");
+               retval = -EINVAL;
+               return retval;
+       }
+
+       bu21013_data = kzalloc(sizeof(struct bu21013_ts_data), GFP_KERNEL);
+       if (!bu21013_data) {
+               dev_err(&i2c->dev, "device memory alloc failed\n");
+               retval = -ENOMEM;
+               return retval;
+       }
+
+       /* allocate input device */
+       in_dev = input_allocate_device();
+       if (!in_dev) {
+               dev_err(&i2c->dev, "input device memory alloc failed\n");
+               retval = -ENOMEM;
+               goto err_alloc;
+       }
+       bu21013_data->in_dev = in_dev;
+
+       bu21013_data->chip = pdata;
+       bu21013_data->client = i2c;
+       bu21013_data->touch_stopped = false;
+
+       x_max = pdata->x_max_res;
+       y_max = pdata->y_max_res;
+
+       init_waitqueue_head(&bu21013_data->wait);
+
+       i2c_set_clientdata(i2c, bu21013_data);
+
+       /* configure the gpio pins */
+       if (pdata->cs_en) {
+               retval = pdata->cs_en(pdata->cs_pin);
+               if (retval < 0) {
+                       dev_err(&i2c->dev, "chip init failed\n");
+                       goto err_init_cs;
+               }
+       }
+
+       /* configure the touch panel controller */
+       retval = bu21013_init_chip(bu21013_data);
+       if (retval < 0) {
+               dev_err(&i2c->dev, "error in bu21013 config\n");
+               goto err_init_config;
+       }
+
+       /* register the device to input subsystem */
+       in_dev->name = DRIVER_TP;
+       in_dev->id.bustype = BUS_I2C;
+       in_dev->dev.parent = &i2c->dev;
+       __set_bit(EV_SYN, in_dev->evbit);
+       __set_bit(EV_KEY, in_dev->evbit);
+       __set_bit(EV_ABS, in_dev->evbit);
+       __set_bit(BTN_TOUCH, in_dev->keybit);
+
+       input_set_abs_params(in_dev, ABS_X, 0, x_max, 0, 0);
+       input_set_abs_params(in_dev, ABS_Y, 0, y_max, 0, 0);
+       input_set_abs_params(in_dev, ABS_PRESSURE, 0, MAX_PRESSURE, 0, 0);
+       input_set_abs_params(in_dev, ABS_TOOL_WIDTH, 0, MAX_TOOL_WIDTH, 0, 0);
+
+       if (pdata->multi_touch) {
+               input_set_abs_params(in_dev, ABS_MT_POSITION_X, 0, x_max,
+                                                                       0, 0);
+               input_set_abs_params(in_dev, ABS_MT_POSITION_Y, 0, y_max,
+                                                                       0, 0);
+               input_set_abs_params(in_dev, ABS_MT_TOUCH_MAJOR, 0,
+                                               MAX_TOUCH_MAJOR, 0, 0);
+               input_set_abs_params(in_dev, ABS_MT_TOUCH_MINOR, 0,
+                                               MAX_TOUCH_MINOR, 0, 0);
+               input_set_abs_params(in_dev, ABS_MT_ORIENTATION, 0, 1, 0, 0);
+               input_set_abs_params(in_dev, ABS_MT_WIDTH_MAJOR, 0,
+                                               MAX_TOOL_WIDTH, 0, 0);
+       }
+       retval = input_register_device(in_dev);
+       if (retval)
+               goto err_input_register;
+
+       retval = request_threaded_irq(pdata->irq, NULL, bu21013_gpio_irq,
+                                       (IRQF_TRIGGER_FALLING | IRQF_SHARED),
+                                       DRIVER_TP, bu21013_data);
+       if (retval) {
+               dev_err(&i2c->dev, "request irq %d failed\n", pdata->irq);
+               goto err_init_irq;
+       }
+
+       return retval;
+
+err_init_irq:
+       input_unregister_device(bu21013_data->in_dev);
+       bu21013_data->in_dev = NULL;
+err_input_register:
+err_init_config:
+       pdata->cs_dis(pdata->cs_pin);
+err_init_cs:
+       input_free_device(bu21013_data->in_dev);
+err_alloc:
+       kfree(bu21013_data);
+
+       return retval;
+}
+/**
+ * bu21013_remove() - removes the i2c-client touchscreen driver
+ * @client: i2c client structure pointer
+ *
+ * This function uses to remove the i2c-client
+ * touchscreen driver and returns integer.
+ */
+static int __devexit bu21013_remove(struct i2c_client *client)
+{
+       struct bu21013_ts_data *data = i2c_get_clientdata(client);
+
+       data->touch_stopped = true;
+       free_irq(data->chip->irq, data);
+       data->chip->cs_dis(data->chip->cs_pin);
+       input_unregister_device(data->in_dev);
+       kfree(data);
+
+       return 0;
+}
+
+static const struct i2c_device_id bu21013_id[] = {
+       { DRIVER_TP, 0 },
+       { }
+};
+MODULE_DEVICE_TABLE(i2c, bu21013_id);
+
+static struct i2c_driver bu21013_driver = {
+       .driver = {
+               .name   =       DRIVER_TP,
+               .owner  =       THIS_MODULE,
+       },
+       .probe          =       bu21013_probe,
+#ifdef CONFIG_PM
+       .suspend        =       bu21013_suspend,
+       .resume         =       bu21013_resume,
+#endif
+       .remove         =       __devexit_p(bu21013_remove),
+       .id_table       =       bu21013_id,
+};
+
+/**
+ * bu21013_init() - initializes the bu21013 touchscreen driver
+ *
+ * This function used to initializes the bu21013
+ * touchscreen driver and returns integer.
+ */
+static int __init bu21013_init(void)
+{
+       return i2c_add_driver(&bu21013_driver);
+}
+
+/**
+ * bu21013_exit() - de-initializes the bu21013 touchscreen driver
+ *
+ * This function uses to de-initializes the bu21013
+ * touchscreen driver and returns none.
+ */
+static void __exit bu21013_exit(void)
+{
+       i2c_del_driver(&bu21013_driver);
+}
+
+module_init(bu21013_init);
+module_exit(bu21013_exit);
+
+MODULE_LICENSE("GPL v2");
+MODULE_AUTHOR("NAVEEN KUMAR G<naveen.gaddipati@stericsson.com");
+MODULE_DESCRIPTION("bu21013 touch screen controller driver");
diff --git a/include/linux/input/bu21013.h b/include/linux/input/bu21013.h
new file mode 100644
index 0000000..6c64bda
--- /dev/null
+++ b/include/linux/input/bu21013.h
@@ -0,0 +1,44 @@
+/*
+ * Copyright (C) ST-Ericsson SA 2010
+ * Author: Naveen Kumar G <naveen.gaddipati@stericsson.com> for ST-Ericsson
+ * License terms:GNU General Public License (GPL) version 2
+ */
+
+#ifndef _BU21013_H
+#define _BU21013_H
+
+/**
+ * struct bu21013_platform_device - Handle the platform data
+ * @cs_en:     pointer to the cs enable function
+ * @cs_dis:    pointer to the cs disable function
+ * @irq_read_val:    pointer to read the pen irq value function
+ * @x_max_res: xmax resolution
+ * @y_max_res: ymax resolution
+ * @touch_x_max: touch x max
+ * @touch_y_max: touch y max
+ * @cs_pin: chip select pin
+ * @irq: irq pin
+ * @ext_clk: external clock flag
+ * @x_flip: x flip flag
+ * @y_flip: y flip flag
+ * @multi_touch: multi_touch flag
+ *
+ * This is used to handle the platform data
+ */
+struct bu21013_platform_device {
+       int (*cs_en)(int reset_pin);
+       int (*cs_dis)(int reset_pin);
+       int (*irq_read_val)(void);
+       int x_max_res;
+       int y_max_res;
+       int touch_x_max;
+       int touch_y_max;
+       unsigned int cs_pin;
+       unsigned int irq;
+       bool ext_clk;
+       bool x_flip;
+       bool y_flip;
+       bool multi_touch;
+};
+
+#endif
--
1.7.2.dirty

Thanks & Regards,
Naveen

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 11:36 [Patch v2] input:rohm based bu21013 touch panel controller driver support Naveen Kumar GADDIPATI
@ 2010-09-09 12:10 ` Trilok Soni
  2010-09-09 13:20   ` Naveen Kumar GADDIPATI
  2010-09-09 15:59 ` Dmitry Torokhov
  1 sibling, 1 reply; 13+ messages in thread
From: Trilok Soni @ 2010-09-09 12:10 UTC (permalink / raw)
  To: Naveen Kumar GADDIPATI
  Cc: Dmitry Torokhov, linux-input, STEricsson_nomadik_linux

Hi Naveen,

On 9/9/2010 5:06 PM, Naveen Kumar GADDIPATI wrote:
> Hi Dmitry,
> 
> From: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
> 
> Added the ROHM based bu21013 capacitive touch panel controller
> driver support with i2c interface.
> 
> Acked-by: Linus Walleij <linus.walleij@stericsson.com>
> Signed-off-by: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
> ---
> Modifications in v2:
>         --Updated with the Dmitry comments on Patch v1
>         --Updated with the Trilok comments on Patch v1

Thanks for the updates. Few comments.

> +
> +/**
> + * bu21013_report_pen_down() - reports the pen down event
> + * @data:bu21013_ts_data structure pointer

One space after ":" right? Applies to whole patch.

> + * @count:touch count
> + *
> + * This function used to report the pen down interrupt to
> + * input subsystem and returns none
> + */
> +static void bu21013_report_pen_down(struct bu21013_ts_data *data, int count)
> +{
> +       int i;
> +
> +       input_report_abs(data->in_dev, ABS_X, data->x_pos[0]);
> +       input_report_abs(data->in_dev, ABS_Y, data->y_pos[0]);
> +       input_report_key(data->in_dev, BTN_TOUCH, count);
> +
> +       if (data->chip->multi_touch) {
> +               for (i = 0; i < count; i++) {
> +                       input_report_abs(data->in_dev, ABS_MT_POSITION_X,
> +                                                       data->x_pos[i]);
> +                       input_report_abs(data->in_dev, ABS_MT_POSITION_Y,
> +                                                       data->y_pos[i]);

Wondering why you don't need to report TOUCH_MAJOR and WIDTH_MAJOR?

> +static int bu21013_suspend(struct i2c_client *client, pm_message_t mesg)
> +{
> +       struct bu21013_ts_data *bu21013_data = i2c_get_clientdata(client);
> +
> +       bu21013_data->touch_stopped = true;
> +       wake_up(&bu21013_data->wait);
> +       if (device_may_wakeup(&client->dev))

I don't find the device_init_wakeup call in the probe function.

> +static int __devinit bu21013_probe(struct i2c_client *i2c,
> +                                       const struct i2c_device_id *id)
> +{
> +       int retval;
> +       struct bu21013_ts_data *bu21013_data;
> +       struct input_dev *in_dev;
> +       short x_max;
> +       short y_max;
> +       struct bu21013_platform_device *pdata = i2c->dev.platform_data;
> +
> +       if (!i2c) {
> +               dev_err(&i2c->dev, "i2c client not defined\n");
> +               retval = -EINVAL;
> +               return retval;
> +       }

Do we think this will ever happen?

Please also add i2c_check_functionality call.

> +
> +static struct i2c_driver bu21013_driver = {
> +       .driver = {
> +               .name   =       DRIVER_TP,
> +               .owner  =       THIS_MODULE,
> +       },
> +       .probe          =       bu21013_probe,
> +#ifdef CONFIG_PM
> +       .suspend        =       bu21013_suspend,
> +       .resume         =       bu21013_resume,
> +#endif

Better to move these suspend and resume to dev_pm_ops.

> +       .remove         =       __devexit_p(bu21013_remove),
> +       .id_table       =       bu21013_id,
> +};
> +

---Trilok Soni

-- 
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* RE: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 12:10 ` Trilok Soni
@ 2010-09-09 13:20   ` Naveen Kumar GADDIPATI
  2010-09-09 13:46     ` Trilok Soni
  0 siblings, 1 reply; 13+ messages in thread
From: Naveen Kumar GADDIPATI @ 2010-09-09 13:20 UTC (permalink / raw)
  To: Trilok Soni; +Cc: Dmitry Torokhov, linux-input, STEricsson_nomadik_linux

Hi Trilok,

> -----Original Message-----
> From: Trilok Soni [mailto:tsoni@codeaurora.org]
> Sent: Thursday, September 09, 2010 5:41 PM
> To: Naveen Kumar GADDIPATI
> Cc: Dmitry Torokhov; linux-input@vger.kernel.org;
> STEricsson_nomadik_linux


> > +
> > +static struct i2c_driver bu21013_driver = {
> > +       .driver = {
> > +               .name   =       DRIVER_TP,
> > +               .owner  =       THIS_MODULE,
> > +       },
> > +       .probe          =       bu21013_probe,
> > +#ifdef CONFIG_PM
> > +       .suspend        =       bu21013_suspend,
> > +       .resume         =       bu21013_resume,
> > +#endif
> 
> Better to move these suspend and resume to dev_pm_ops.
> 
	How we could dev_pm_ops in i2c driver structure?

Thanks & Regards,
Naveen


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 13:20   ` Naveen Kumar GADDIPATI
@ 2010-09-09 13:46     ` Trilok Soni
  2010-09-09 15:39       ` Dmitry Torokhov
  0 siblings, 1 reply; 13+ messages in thread
From: Trilok Soni @ 2010-09-09 13:46 UTC (permalink / raw)
  To: Naveen Kumar GADDIPATI
  Cc: Dmitry Torokhov, linux-input, STEricsson_nomadik_linux

On 9/9/2010 6:50 PM, Naveen Kumar GADDIPATI wrote:
> Hi Trilok,
> 
>> -----Original Message-----
>> From: Trilok Soni [mailto:tsoni@codeaurora.org]
>> Sent: Thursday, September 09, 2010 5:41 PM
>> To: Naveen Kumar GADDIPATI
>> Cc: Dmitry Torokhov; linux-input@vger.kernel.org;
>> STEricsson_nomadik_linux
> 
> 
>>> +
>>> +static struct i2c_driver bu21013_driver = {
>>> +       .driver = {
>>> +               .name   =       DRIVER_TP,
>>> +               .owner  =       THIS_MODULE,
>>> +       },
>>> +       .probe          =       bu21013_probe,
>>> +#ifdef CONFIG_PM
>>> +       .suspend        =       bu21013_suspend,
>>> +       .resume         =       bu21013_resume,
>>> +#endif
>>
>> Better to move these suspend and resume to dev_pm_ops.
>>
> 	How we could dev_pm_ops in i2c driver structure?
> 

Refer: drivers/input/keyboard/adp5588-keys.c as example.

---Trilok Soni
-- 
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 13:46     ` Trilok Soni
@ 2010-09-09 15:39       ` Dmitry Torokhov
  2010-09-09 18:44         ` Trilok Soni
  0 siblings, 1 reply; 13+ messages in thread
From: Dmitry Torokhov @ 2010-09-09 15:39 UTC (permalink / raw)
  To: Trilok Soni; +Cc: Naveen Kumar GADDIPATI, linux-input, STEricsson_nomadik_linux

On Thu, Sep 09, 2010 at 07:16:19PM +0530, Trilok Soni wrote:
> On 9/9/2010 6:50 PM, Naveen Kumar GADDIPATI wrote:
> > Hi Trilok,
> > 
> >> -----Original Message-----
> >> From: Trilok Soni [mailto:tsoni@codeaurora.org]
> >> Sent: Thursday, September 09, 2010 5:41 PM
> >> To: Naveen Kumar GADDIPATI
> >> Cc: Dmitry Torokhov; linux-input@vger.kernel.org;
> >> STEricsson_nomadik_linux
> > 
> > 
> >>> +
> >>> +static struct i2c_driver bu21013_driver = {
> >>> +       .driver = {
> >>> +               .name   =       DRIVER_TP,
> >>> +               .owner  =       THIS_MODULE,
> >>> +       },
> >>> +       .probe          =       bu21013_probe,
> >>> +#ifdef CONFIG_PM
> >>> +       .suspend        =       bu21013_suspend,
> >>> +       .resume         =       bu21013_resume,
> >>> +#endif
> >>
> >> Better to move these suspend and resume to dev_pm_ops.
> >>
> > 	How we could dev_pm_ops in i2c driver structure?
> > 
> 
> Refer: drivers/input/keyboard/adp5588-keys.c as example.
> 

Hm, indeed I forgot that i2c has been fully wired for dev_pm_ops. I
should enforce its use then. Thank you for reminding Trilok.

I wonder if someone could wire SPI as well...

-- 
Dmitry

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 11:36 [Patch v2] input:rohm based bu21013 touch panel controller driver support Naveen Kumar GADDIPATI
  2010-09-09 12:10 ` Trilok Soni
@ 2010-09-09 15:59 ` Dmitry Torokhov
  2010-09-09 16:01   ` Dmitry Torokhov
  1 sibling, 1 reply; 13+ messages in thread
From: Dmitry Torokhov @ 2010-09-09 15:59 UTC (permalink / raw)
  To: Naveen Kumar GADDIPATI; +Cc: Trilok Soni, linux-input, STEricsson_nomadik_linux

Hi Naveen,

On Thu, Sep 09, 2010 at 01:36:12PM +0200, Naveen Kumar GADDIPATI wrote:
> Hi Dmitry,
> 
> From: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
> 
> Added the ROHM based bu21013 capacitive touch panel controller
> driver support with i2c interface.
> 
> Acked-by: Linus Walleij <linus.walleij@stericsson.com>
> Signed-off-by: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>

Thank you for making the changes.

Henrik, could you glance over MT handling please?

> ---
> Modifications in v2:
>         --Updated with the Dmitry comments on Patch v1
>         --Updated with the Trilok comments on Patch v1
> 
> 
>  drivers/input/touchscreen/Kconfig      |   12 +
>  drivers/input/touchscreen/Makefile     |    1 +
>  drivers/input/touchscreen/bu21013_ts.c |  770 ++++++++++++++++++++++++++++++++
>  include/linux/input/bu21013.h          |   44 ++
>  4 files changed, 827 insertions(+), 0 deletions(-)
>  create mode 100644 drivers/input/touchscreen/bu21013_ts.c
>  create mode 100644 include/linux/input/bu21013.h
> 
> diff --git a/drivers/input/touchscreen/Kconfig b/drivers/input/touchscreen/Kconfig
> index 0069d97..2380533 100644
> --- a/drivers/input/touchscreen/Kconfig
> +++ b/drivers/input/touchscreen/Kconfig
> @@ -98,6 +98,18 @@ config TOUCHSCREEN_BITSY
>           To compile this driver as a module, choose M here: the
>           module will be called h3600_ts_input.
> 
> +config TOUCHSCREEN_BU21013
> +       tristate "BU21013 based touch panel controllers"
> +       depends on I2C
> +       help
> +         Say Y here if you have a bu21013 touchscreen connected to
> +         your system.
> +
> +         If unsure, say N.
> +
> +         To compile this driver as a module, choose M here: the
> +         module will be called bu21013_ts.
> +
>  config TOUCHSCREEN_CY8CTMG110
>         tristate "cy8ctmg110 touchscreen"
>         depends on I2C
> diff --git a/drivers/input/touchscreen/Makefile b/drivers/input/touchscreen/Makefile
> index 28217e1..a0a2021 100644
> --- a/drivers/input/touchscreen/Makefile
> +++ b/drivers/input/touchscreen/Makefile
> @@ -14,6 +14,7 @@ obj-$(CONFIG_TOUCHSCREEN_AD7879_SPI)  += ad7879-spi.o
>  obj-$(CONFIG_TOUCHSCREEN_ADS7846)      += ads7846.o
>  obj-$(CONFIG_TOUCHSCREEN_ATMEL_TSADCC) += atmel_tsadcc.o
>  obj-$(CONFIG_TOUCHSCREEN_BITSY)                += h3600_ts_input.o
> +obj-$(CONFIG_TOUCHSCREEN_BU21013)       += bu21013_ts.o
>  obj-$(CONFIG_TOUCHSCREEN_CY8CTMG110)   += cy8ctmg110_ts.o
>  obj-$(CONFIG_TOUCHSCREEN_DA9034)       += da9034-ts.o
>  obj-$(CONFIG_TOUCHSCREEN_DYNAPRO)      += dynapro.o
> diff --git a/drivers/input/touchscreen/bu21013_ts.c b/drivers/input/touchscreen/bu21013_ts.c
> new file mode 100644
> index 0000000..117d9c5
> --- /dev/null
> +++ b/drivers/input/touchscreen/bu21013_ts.c
> @@ -0,0 +1,770 @@
> +/*
> + * Copyright (C) ST-Ericsson SA 2010
> + * Author: Naveen Kumar G <naveen.gaddipati@stericsson.com> for ST-Ericsson
> + * License terms:GNU General Public License (GPL) version 2
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/delay.h>
> +#include <linux/interrupt.h>
> +#include <linux/i2c.h>
> +#include <linux/workqueue.h>
> +#include <linux/input.h>
> +#include <linux/types.h>
> +#include <linux/timer.h>
> +#include <linux/input/bu21013.h>
> +#include <linux/slab.h>
> +
> +#define PEN_DOWN_INTR                  0
> +#define PEN_UP_INTR                    1
> +#define RESET_DELAY                    30
> +#define POLLING_DELAY                  100
> +#define MAX_TOOL_WIDTH                 15
> +#define MAX_TOUCH_MAJOR                        255
> +#define MAX_TOUCH_MINOR                        15
> +#define MAX_PRESSURE                   1
> +#define PENUP_TIMEOUT                  (2)
> +#define DELTA_MIN                      16
> +#define MASK_BITS                      0x03
> +#define SHIFT_8                                8
> +#define SHIFT_2                                2
> +#define LENGTH_OF_BUFFER               11
> +
> +#define BU21013_SENSORS_BTN_0_7_REG    0x70
> +#define BU21013_SENSORS_BTN_8_15_REG   0x71
> +#define BU21013_SENSORS_BTN_16_23_REG  0x72
> +#define BU21013_X1_POS_MSB_REG         0x73
> +#define BU21013_X1_POS_LSB_REG         0x74
> +#define BU21013_Y1_POS_MSB_REG         0x75
> +#define BU21013_Y1_POS_LSB_REG         0x76
> +#define BU21013_X2_POS_MSB_REG         0x77
> +#define BU21013_X2_POS_LSB_REG         0x78
> +#define BU21013_Y2_POS_MSB_REG         0x79
> +#define BU21013_Y2_POS_LSB_REG         0x7A
> +#define BU21013_INT_CLR_REG            0xE8
> +#define BU21013_INT_MODE_REG           0xE9
> +#define BU21013_GAIN_REG               0xEA
> +#define BU21013_OFFSET_MODE_REG                0xEB
> +#define BU21013_XY_EDGE_REG            0xEC
> +#define BU21013_RESET_REG              0xED
> +#define BU21013_CALIB_REG              0xEE
> +#define BU21013_DONE_REG               0xEF
> +#define BU21013_SENSOR_0_7_REG         0xF0
> +#define BU21013_SENSOR_8_15_REG                0xF1
> +#define BU21013_SENSOR_16_23_REG       0xF2
> +#define BU21013_POS_MODE1_REG          0xF3
> +#define BU21013_POS_MODE2_REG          0xF4
> +#define BU21013_CLK_MODE_REG           0xF5
> +#define BU21013_IDLE_REG               0xFA
> +#define BU21013_FILTER_REG             0xFB
> +#define BU21013_TH_ON_REG              0xFC
> +#define BU21013_TH_OFF_REG             0xFD
> +
> +#define BU21013_RESET_ENABLE           0x01
> +
> +#define BU21013_SENSORS_EN_0_7         0x3F
> +#define BU21013_SENSORS_EN_8_15                0xFC
> +#define BU21013_SENSORS_EN_16_23       0x1F
> +
> +#define BU21013_POS_MODE1_0            0x02
> +#define BU21013_POS_MODE1_1            0x04
> +#define BU21013_POS_MODE1_2            0x08
> +
> +#define BU21013_POS_MODE2_ZERO         0x01
> +#define BU21013_POS_MODE2_AVG1         0x02
> +#define BU21013_POS_MODE2_AVG2         0x04
> +#define BU21013_POS_MODE2_EN_XY                0x08
> +#define BU21013_POS_MODE2_EN_RAW       0x10
> +#define BU21013_POS_MODE2_MULTI                0x80
> +
> +#define BU21013_CLK_MODE_DIV           0x01
> +#define BU21013_CLK_MODE_EXT           0x02
> +#define BU21013_CLK_MODE_CALIB         0x80
> +
> +#define BU21013_IDLET_0                        0x01
> +#define BU21013_IDLET_1                        0x02
> +#define BU21013_IDLET_2                        0x04
> +#define BU21013_IDLET_3                        0x08
> +#define BU21013_IDLE_INTERMIT_EN       0x10
> +
> +#define BU21013_DELTA_0_6              0x7F
> +#define BU21013_FILTER_EN              0x80
> +
> +#define BU21013_INT_MODE_LEVEL         0x00
> +#define BU21013_INT_MODE_EDGE          0x01
> +
> +#define BU21013_GAIN_0                 0x01
> +#define BU21013_GAIN_1                 0x02
> +#define BU21013_GAIN_2                 0x04
> +
> +#define BU21013_OFFSET_MODE_DEFAULT    0x00
> +#define BU21013_OFFSET_MODE_MOVE       0x01
> +#define BU21013_OFFSET_MODE_DISABLE    0x02
> +
> +#define BU21013_TH_ON_0                        0x01
> +#define BU21013_TH_ON_1                        0x02
> +#define BU21013_TH_ON_2                        0x04
> +#define BU21013_TH_ON_3                        0x08
> +#define BU21013_TH_ON_4                        0x10
> +#define BU21013_TH_ON_5                        0x20
> +#define BU21013_TH_ON_6                        0x40
> +#define BU21013_TH_ON_7                        0x80
> +#define BU21013_TH_ON_MAX              0xFF
> +
> +#define BU21013_TH_OFF_0               0x01
> +#define BU21013_TH_OFF_1               0x02
> +#define BU21013_TH_OFF_2               0x04
> +#define BU21013_TH_OFF_3               0x08
> +#define BU21013_TH_OFF_4               0x10
> +#define BU21013_TH_OFF_5               0x20
> +#define BU21013_TH_OFF_6               0x40
> +#define BU21013_TH_OFF_7               0x80
> +#define BU21013_TH_OFF_MAX             0xFF
> +
> +#define BU21013_X_EDGE_0               0x01
> +#define BU21013_X_EDGE_1               0x02
> +#define BU21013_X_EDGE_2               0x04
> +#define BU21013_X_EDGE_3               0x08
> +#define BU21013_Y_EDGE_0               0x10
> +#define BU21013_Y_EDGE_1               0x20
> +#define BU21013_Y_EDGE_2               0x40
> +#define BU21013_Y_EDGE_3               0x80
> +
> +#define BU21013_DONE                   0x01
> +#define BU21013_NUMBER_OF_X_SENSORS    (6)
> +#define BU21013_NUMBER_OF_Y_SENSORS    (11)
> +
> +#define DRIVER_TP      "bu21013_tp"
> +
> +/**
> + * struct bu21013_ts_data - touch panel data structure
> + * @client:    pointer to the i2c client
> + * @chip:      pointer to the touch panel controller
> + * @in_dev:    pointer to the input device structure
> + * @wait: variable to wait_queue_head_t structure
> + * @intr_pin: interrupt pin value
> + * @touch_stopped: touch stop flag
> + * @x_pos: x position for two co-ordinates
> + * @y_pos: y position for two co-ordinates
> + * @previous_press_reported: last reported flag
> + *
> + * Touch panel device data structure
> + */
> +struct bu21013_ts_data {
> +       struct i2c_client *client;
> +       struct bu21013_platform_device *chip;

Please make int "const".

> +       struct input_dev *in_dev;
> +       wait_queue_head_t wait;
> +       bool touch_stopped;
> +       unsigned int intr_pin;
> +       signed short    x_pos[2];
> +       signed short    y_pos[2];
> +       bool previous_press_reported;
> +};
> +
> +/**
> + * bu21013_report_pen_down() - reports the pen down event
> + * @data:bu21013_ts_data structure pointer
> + * @count:touch count
> + *
> + * This function used to report the pen down interrupt to
> + * input subsystem and returns none
> + */
> +static void bu21013_report_pen_down(struct bu21013_ts_data *data, int count)
> +{
> +       int i;
> +
> +       input_report_abs(data->in_dev, ABS_X, data->x_pos[0]);
> +       input_report_abs(data->in_dev, ABS_Y, data->y_pos[0]);
> +       input_report_key(data->in_dev, BTN_TOUCH, count);
> +
> +       if (data->chip->multi_touch) {

Why would we want to condition this? If hardware is capable of MT we
should always support it.

> +               for (i = 0; i < count; i++) {
> +                       input_report_abs(data->in_dev, ABS_MT_POSITION_X,
> +                                                       data->x_pos[i]);
> +                       input_report_abs(data->in_dev, ABS_MT_POSITION_Y,
> +                                                       data->y_pos[i]);
> +                       input_mt_sync(data->in_dev);
> +               }
> +       }
> +       input_sync(data->in_dev);
> +       data->previous_press_reported = count;
> +}
> +/**
> + * bu21013_report_pen_up() - reports the pen up event
> + * @data:bu21013_ts_data structure pointer
> + *
> + * This function used to report the pen up interrupt
> + * to input subsystem and returns none
> + */
> +static void bu21013_report_pen_up(struct bu21013_ts_data *data)
> +{
> +       input_report_key(data->in_dev, BTN_TOUCH, 0);
> +       if (data->chip->multi_touch) {
> +               input_report_key(data->in_dev, BTN_2, 0);

Remnants of old patch... please remove.

> +               input_mt_sync(data->in_dev);
> +       }
> +       input_sync(data->in_dev);
> +       data->previous_press_reported = 0;

 = false;

> +}
> +/**
> + * bu21013_touch_calc() - calculates the co-ordinates delta
> + * @data: bu21013_ts_data structure pointer
> + * @x: x position
> + * @y: y position
> + * @count: touch count
> + *
> + * This function calculates the exact co-ordinates with respect to
> + * display resolution and returns none
> + */
> +static void bu21013_touch_calc
> +       (struct bu21013_ts_data *data, int x, int y, int count)
> +{
> +       data->x_pos[count] = x;
> +       data->y_pos[count] = y;
> +       if (data->chip->x_flip)
> +               data->x_pos[count] =
> +                       data->chip->touch_x_max - data->x_pos[count];
> +       if (data->chip->y_flip)
> +               data->y_pos[count] =
> +                       data->chip->touch_y_max - data->y_pos[count];
> +}
> +/**
> + * bu21013_verify_delta() - verify the co-ordinates delta
> + * @x1: x1 position
> + * @y1: y1 position
> + * @x2: x2 position
> + * @y2: y2 position
> + *
> + * This function verifies the delta of the
> + * co-ordinates and returns boolean.
> + */
> +static bool bu21013_verify_delta(int x1, int y1, int x2, int y2)
> +{
> +       int delta_x, delta_y;
> +
> +       if ((x1 != 0) && (y1 != 0)) {
> +               delta_x = x2 - x1;
> +               if (x1 > x2)
> +                       delta_x = x1 - x2;
> +               delta_y = y2 - y1;
> +               if (y1 > y2)
> +                       delta_y = y1 - y2;
> +               if ((delta_x < DELTA_MIN) || (delta_y < DELTA_MIN))
> +                       return false;
> +       }
> +       return true;

Hmm, I remember - the fuzz handling indeed will not work for MT A protocol.
Still, I do not think it is correct. Don't you need to do
"abs(delta_x) < DELTA_MIN"?

> +}
> +
> +/**
> + * bu21013_do_touch_report(): Get the touch co-ordinates
> + * @data:bu21013_ts_data structure pointer
> + *
> + * Get the touch co-ordinates from touch sensor registers and writes
> + * into device structure and returns integer.
> + */
> +static int bu21013_do_touch_report(struct bu21013_ts_data *data)
> +{
> +       u8      buf[LENGTH_OF_BUFFER];
> +       int     finger1_valid = 0;
> +       int     finger2_valid = 0;
> +       unsigned int    finger1_pos_x;
> +       unsigned int    finger1_pos_y;
> +       unsigned int    finger2_pos_x = 0;
> +       unsigned int    finger2_pos_y = 0;
> +       int     number_of_active_x_sensors;
> +       int     number_of_active_y_sensors;
> +       int     total_number_of_active_sensors;
> +       int     finger_down_count = 0;
> +       int     ret = 0;
> +
> +       if (data == NULL)
> +               return -EINVAL;
> +
> +       ret = i2c_smbus_read_i2c_block_data(data->client,
> +                                               BU21013_SENSORS_BTN_0_7_REG,
> +                                                       LENGTH_OF_BUFFER, buf);
> +       if (ret < LENGTH_OF_BUFFER)
> +               return -EINVAL;
> +
> +       number_of_active_x_sensors = hweight32(buf[0] &
> +                                       BU21013_SENSORS_EN_0_7);
> +       number_of_active_y_sensors = hweight32(
> +                       ((buf[1] & BU21013_SENSORS_EN_8_15) |
> +               ((buf[2] & BU21013_SENSORS_EN_16_23) << SHIFT_8)) >> SHIFT_2);
> +       if (((number_of_active_x_sensors != 0) &&
> +               (number_of_active_y_sensors == 0)) ||
> +               ((number_of_active_x_sensors == 0) &&
> +               (number_of_active_y_sensors != 0)))
> +               return 0;
> +
> +       total_number_of_active_sensors =
> +               number_of_active_x_sensors + number_of_active_y_sensors;
> +
> +       finger1_pos_x = buf[3] << SHIFT_2 | (buf[4] & MASK_BITS);
> +       finger1_pos_y = buf[5] << SHIFT_2 | (buf[6] & MASK_BITS);
> +
> +       if (data->chip->multi_touch) {
> +               finger2_pos_x = buf[7] << SHIFT_2 | (buf[8] & MASK_BITS);
> +               finger2_pos_y = buf[9] << SHIFT_2 | (buf[10] & MASK_BITS);
> +       }
> +
> +       if (total_number_of_active_sensors) {
> +               if ((finger2_pos_x != 0) && (finger2_pos_y != 0)) {
> +                       if ((finger1_pos_x == 0) || (finger1_pos_y == 0))
> +                               return 0;
> +                       ret = bu21013_verify_delta(finger1_pos_x,
> +                               finger1_pos_y, finger2_pos_x, finger2_pos_y);
> +                       if (!ret)
> +                               total_number_of_active_sensors = 0;
> +               }
> +       }
> +
> +       if (total_number_of_active_sensors) {
> +               finger1_valid = 1;
> +               if ((finger1_pos_x == 0) || (finger1_pos_y == 0) ||
> +                       (finger1_pos_x >= data->chip->touch_x_max) ||
> +                       (finger1_pos_y >= data->chip->touch_y_max))
> +                       finger1_valid = 0;
> +               finger2_valid = 1;
> +               if ((finger2_pos_x == 0) || (finger2_pos_y == 0) ||
> +                       (finger2_pos_x >= data->chip->touch_x_max) ||
> +                       (finger2_pos_y >= data->chip->touch_y_max))
> +                       finger2_valid = 0;
> +               if ((finger1_valid == 0) && (finger2_valid == 0))
> +                       return 0;
> +               if (finger1_valid) {
> +                       bu21013_touch_calc(data, finger1_pos_x,
> +                               finger1_pos_y, finger_down_count);
> +                       finger_down_count++;
> +               }
> +               if (finger2_valid) {
> +                       bu21013_touch_calc(data, finger2_pos_x,
> +                               finger2_pos_y, finger_down_count);
> +                       finger_down_count++;
> +               }
> +       }
> +
> +       if (finger_down_count > 0)
> +               bu21013_report_pen_down(data, finger_down_count);
> +       else {
> +               if (data->previous_press_reported)
> +                       bu21013_report_pen_up(data);
> +       }
> +
> +       return ret;
> +}
> +
> +/**
> + * bu21013_gpio_irq() - gpio thread function for touch interrupt
> + * @irq: irq value
> + * @device_data:void pointer
> + *
> + * This gpio thread function for touch interrupt
> + * and returns irqreturn_t.
> + */
> +static irqreturn_t bu21013_gpio_irq(int irq, void *device_data)
> +{
> +       struct bu21013_ts_data *data = device_data;
> +       struct i2c_client *i2c = data->client;
> +       int retval;
> +
> +       do {
> +               retval = bu21013_do_touch_report(data);
> +               if (retval < 0) {
> +                       dev_err(&i2c->dev, "bu21013_do_touch_report failed\n");
> +                       return IRQ_NONE;
> +               }
> +
> +               data->intr_pin = data->chip->irq_read_val();
> +               if (data->intr_pin == PEN_DOWN_INTR)
> +                       wait_event_timeout(data->wait, data->touch_stopped,
> +                                                       msecs_to_jiffies(10));
> +       } while (!data->intr_pin && !data->touch_stopped);
> +
> +       return IRQ_HANDLED;
> +}
> +
> +/**
> + * bu21013_init_chip() - power on sequence for the bu21013 controller
> + * @data: device structure pointer
> + *
> + * This function is used to power on
> + * the bu21013 controller and returns integer.
> + */
> +static int bu21013_init_chip(struct bu21013_ts_data *data)
> +{
> +       int retval;
> +       struct i2c_client *i2c = data->client;
> +
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_RESET_REG,
> +                                       BU21013_RESET_ENABLE);
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_RESET reg write failed\n");
> +               goto err;
> +       }
> +       msleep(RESET_DELAY);
> +
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_0_7_REG,
> +                                       BU21013_SENSORS_EN_0_7);
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_SENSOR_0_7 reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_8_15_REG,
> +                                       BU21013_SENSORS_EN_8_15);
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_SENSOR_8_15 reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_16_23_REG,
> +                                       BU21013_SENSORS_EN_16_23);
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_SENSOR_16_23 reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE1_REG,
> +                               (BU21013_POS_MODE1_0 | BU21013_POS_MODE1_1));
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_POS_MODE1 reg write failed\n");
> +               goto err;
> +       }
> +       if (data->chip->multi_touch)
> +               retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE2_REG,
> +                       (BU21013_POS_MODE2_ZERO | BU21013_POS_MODE2_AVG1 |
> +                       BU21013_POS_MODE2_AVG2 | BU21013_POS_MODE2_EN_RAW |
> +                       BU21013_POS_MODE2_MULTI));
> +       else
> +               retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE2_REG,
> +                       (BU21013_POS_MODE2_ZERO | BU21013_POS_MODE2_AVG1 |
> +                       BU21013_POS_MODE2_AVG2 | BU21013_POS_MODE2_EN_RAW));
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_POS_MODE2 reg write failed\n");
> +               goto err;
> +       }
> +       if (data->chip->ext_clk)
> +               retval = i2c_smbus_write_byte_data(i2c, BU21013_CLK_MODE_REG,
> +                       (BU21013_CLK_MODE_EXT | BU21013_CLK_MODE_CALIB));
> +       else
> +               retval = i2c_smbus_write_byte_data(i2c, BU21013_CLK_MODE_REG,
> +                       (BU21013_CLK_MODE_DIV | BU21013_CLK_MODE_CALIB));
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_CLK_MODE reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_IDLE_REG,
> +                               (BU21013_IDLET_0 | BU21013_IDLE_INTERMIT_EN));
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_IDLE reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_INT_MODE_REG,
> +                                               BU21013_INT_MODE_LEVEL);
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_INT_MODE reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_FILTER_REG,
> +                                               (BU21013_DELTA_0_6 |
> +                                                       BU21013_FILTER_EN));
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_FILTER reg write failed\n");
> +               goto err;
> +       }
> +
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_TH_ON_REG,
> +                                       (BU21013_TH_ON_3 | BU21013_TH_ON_4));
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_TH_ON reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_TH_OFF_REG,
> +                                                       BU21013_TH_OFF_4);
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_TH_OFF reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_GAIN_REG,
> +                                       (BU21013_GAIN_0 | BU21013_GAIN_1));
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_GAIN reg write failed\n");
> +               goto err;
> +       }
> +
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_OFFSET_MODE_REG,
> +                                       BU21013_OFFSET_MODE_DEFAULT);
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_OFFSET_MODE reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_XY_EDGE_REG,
> +                       (BU21013_X_EDGE_0 | BU21013_X_EDGE_2 |
> +                               BU21013_Y_EDGE_1 | BU21013_Y_EDGE_3));
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_XY_EDGE reg write failed\n");
> +               goto err;
> +       }
> +       retval = i2c_smbus_write_byte_data(i2c, BU21013_DONE_REG,
> +                                                       BU21013_DONE);
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "BU21013_REG_DONE reg write failed\n");
> +               goto err;
> +       }
> +
> +       data->x_pos[0] = 0;
> +       data->x_pos[1] = 0;
> +       data->y_pos[0] = 0;
> +       data->y_pos[1] = 0;
> +       data->previous_press_reported = 0;
> +
> +err:
> +       return retval;
> +}
> +
> +#ifdef CONFIG_PM
> +/**
> + * bu21013_suspend() - suspend the touch screen controller
> + * @client: pointer to i2c client structure
> + * @mesg: message from power manager
> + *
> + * This funtion is used to suspend the
> + * touch panel controller and returns integer
> + */
> +static int bu21013_suspend(struct i2c_client *client, pm_message_t mesg)
> +{
> +       struct bu21013_ts_data *bu21013_data = i2c_get_clientdata(client);
> +
> +       bu21013_data->touch_stopped = true;
> +       wake_up(&bu21013_data->wait);
> +       if (device_may_wakeup(&client->dev))
> +               enable_irq_wake(bu21013_data->chip->irq);
> +       else
> +               disable_irq(bu21013_data->chip->irq);
> +
> +       return 0;
> +}
> +
> +/**
> + * bu21013_resume() - resume the touch screen controller
> + * @client: pointer to i2c client structure
> + *
> + * This funtion is used to resume the touch panel
> + * controller and returns integer.
> + */
> +static int bu21013_resume(struct i2c_client *client)
> +{
> +       int retval;
> +       struct bu21013_ts_data *bu21013_data = i2c_get_clientdata(client);
> +
> +       retval = bu21013_init_chip(bu21013_data);
> +       if (retval < 0) {
> +               dev_err(&client->dev, "tsc config failed\n");
> +               return retval;
> +       }
> +
> +       bu21013_data->touch_stopped = false;
> +       if (device_may_wakeup(&client->dev))
> +               disable_irq_wake(bu21013_data->chip->irq);
> +       else
> +               enable_irq(bu21013_data->chip->irq);
> +
> +       return 0;
> +}
> +#endif
> +
> +/**
> + * bu21013_probe() - initialzes the i2c-client touchscreen driver
> + * @i2c: i2c client structure pointer
> + * @id:i2c device id pointer
> + *
> + * This function used to initializes the i2c-client touchscreen
> + * driver and returns integer.
> + */
> +static int __devinit bu21013_probe(struct i2c_client *i2c,
> +                                       const struct i2c_device_id *id)
> +{
> +       int retval;
> +       struct bu21013_ts_data *bu21013_data;
> +       struct input_dev *in_dev;
> +       short x_max;
> +       short y_max;
> +       struct bu21013_platform_device *pdata = i2c->dev.platform_data;
> +
> +       if (!i2c) {
> +               dev_err(&i2c->dev, "i2c client not defined\n");
> +               retval = -EINVAL;
> +               return retval;
> +       }
> +
> +       if (!pdata) {
> +               dev_err(&i2c->dev, "platform data not defined\n");
> +               retval = -EINVAL;
> +               return retval;
> +       }
> +
> +       bu21013_data = kzalloc(sizeof(struct bu21013_ts_data), GFP_KERNEL);
> +       if (!bu21013_data) {
> +               dev_err(&i2c->dev, "device memory alloc failed\n");
> +               retval = -ENOMEM;
> +               return retval;
> +       }
> +
> +       /* allocate input device */
> +       in_dev = input_allocate_device();
> +       if (!in_dev) {
> +               dev_err(&i2c->dev, "input device memory alloc failed\n");
> +               retval = -ENOMEM;
> +               goto err_alloc;
> +       }
> +       bu21013_data->in_dev = in_dev;
> +
> +       bu21013_data->chip = pdata;
> +       bu21013_data->client = i2c;
> +       bu21013_data->touch_stopped = false;
> +
> +       x_max = pdata->x_max_res;
> +       y_max = pdata->y_max_res;
> +
> +       init_waitqueue_head(&bu21013_data->wait);
> +
> +       i2c_set_clientdata(i2c, bu21013_data);
> +
> +       /* configure the gpio pins */
> +       if (pdata->cs_en) {
> +               retval = pdata->cs_en(pdata->cs_pin);
> +               if (retval < 0) {
> +                       dev_err(&i2c->dev, "chip init failed\n");
> +                       goto err_init_cs;
> +               }
> +       }
> +
> +       /* configure the touch panel controller */
> +       retval = bu21013_init_chip(bu21013_data);
> +       if (retval < 0) {
> +               dev_err(&i2c->dev, "error in bu21013 config\n");
> +               goto err_init_config;
> +       }
> +
> +       /* register the device to input subsystem */
> +       in_dev->name = DRIVER_TP;
> +       in_dev->id.bustype = BUS_I2C;
> +       in_dev->dev.parent = &i2c->dev;
> +       __set_bit(EV_SYN, in_dev->evbit);
> +       __set_bit(EV_KEY, in_dev->evbit);
> +       __set_bit(EV_ABS, in_dev->evbit);
> +       __set_bit(BTN_TOUCH, in_dev->keybit);
> +
> +       input_set_abs_params(in_dev, ABS_X, 0, x_max, 0, 0);
> +       input_set_abs_params(in_dev, ABS_Y, 0, y_max, 0, 0);
> +       input_set_abs_params(in_dev, ABS_PRESSURE, 0, MAX_PRESSURE, 0, 0);
> +       input_set_abs_params(in_dev, ABS_TOOL_WIDTH, 0, MAX_TOOL_WIDTH, 0, 0);
> +
> +       if (pdata->multi_touch) {
> +               input_set_abs_params(in_dev, ABS_MT_POSITION_X, 0, x_max,
> +                                                                       0, 0);
> +               input_set_abs_params(in_dev, ABS_MT_POSITION_Y, 0, y_max,
> +                                                                       0, 0);
> +               input_set_abs_params(in_dev, ABS_MT_TOUCH_MAJOR, 0,
> +                                               MAX_TOUCH_MAJOR, 0, 0);
> +               input_set_abs_params(in_dev, ABS_MT_TOUCH_MINOR, 0,
> +                                               MAX_TOUCH_MINOR, 0, 0);
> +               input_set_abs_params(in_dev, ABS_MT_ORIENTATION, 0, 1, 0, 0);
> +               input_set_abs_params(in_dev, ABS_MT_WIDTH_MAJOR, 0,
> +                                               MAX_TOOL_WIDTH, 0, 0);

If we are not sending events we should not declare capabilities either.

> +       }
> +       retval = input_register_device(in_dev);
> +       if (retval)
> +               goto err_input_register;
> +
> +       retval = request_threaded_irq(pdata->irq, NULL, bu21013_gpio_irq,
> +                                       (IRQF_TRIGGER_FALLING | IRQF_SHARED),
> +                                       DRIVER_TP, bu21013_data);
> +       if (retval) {
> +               dev_err(&i2c->dev, "request irq %d failed\n", pdata->irq);
> +               goto err_init_irq;
> +       }
> +
> +       return retval;
> +
> +err_init_irq:
> +       input_unregister_device(bu21013_data->in_dev);
> +       bu21013_data->in_dev = NULL;
> +err_input_register:
> +err_init_config:
> +       pdata->cs_dis(pdata->cs_pin);
> +err_init_cs:
> +       input_free_device(bu21013_data->in_dev);
> +err_alloc:
> +       kfree(bu21013_data);
> +
> +       return retval;
> +}
> +/**
> + * bu21013_remove() - removes the i2c-client touchscreen driver
> + * @client: i2c client structure pointer
> + *
> + * This function uses to remove the i2c-client
> + * touchscreen driver and returns integer.
> + */
> +static int __devexit bu21013_remove(struct i2c_client *client)
> +{
> +       struct bu21013_ts_data *data = i2c_get_clientdata(client);
> +
> +       data->touch_stopped = true;
> +       free_irq(data->chip->irq, data);
> +       data->chip->cs_dis(data->chip->cs_pin);
> +       input_unregister_device(data->in_dev);
> +       kfree(data);
> +
> +       return 0;
> +}
> +
> +static const struct i2c_device_id bu21013_id[] = {
> +       { DRIVER_TP, 0 },
> +       { }
> +};
> +MODULE_DEVICE_TABLE(i2c, bu21013_id);
> +
> +static struct i2c_driver bu21013_driver = {
> +       .driver = {
> +               .name   =       DRIVER_TP,
> +               .owner  =       THIS_MODULE,
> +       },
> +       .probe          =       bu21013_probe,
> +#ifdef CONFIG_PM
> +       .suspend        =       bu21013_suspend,
> +       .resume         =       bu21013_resume,
> +#endif
> +       .remove         =       __devexit_p(bu21013_remove),
> +       .id_table       =       bu21013_id,
> +};
> +
> +/**
> + * bu21013_init() - initializes the bu21013 touchscreen driver
> + *
> + * This function used to initializes the bu21013
> + * touchscreen driver and returns integer.
> + */
> +static int __init bu21013_init(void)
> +{
> +       return i2c_add_driver(&bu21013_driver);
> +}
> +
> +/**
> + * bu21013_exit() - de-initializes the bu21013 touchscreen driver
> + *
> + * This function uses to de-initializes the bu21013
> + * touchscreen driver and returns none.
> + */
> +static void __exit bu21013_exit(void)
> +{
> +       i2c_del_driver(&bu21013_driver);
> +}
> +
> +module_init(bu21013_init);
> +module_exit(bu21013_exit);
> +
> +MODULE_LICENSE("GPL v2");
> +MODULE_AUTHOR("NAVEEN KUMAR G<naveen.gaddipati@stericsson.com");
> +MODULE_DESCRIPTION("bu21013 touch screen controller driver");
> diff --git a/include/linux/input/bu21013.h b/include/linux/input/bu21013.h
> new file mode 100644
> index 0000000..6c64bda
> --- /dev/null
> +++ b/include/linux/input/bu21013.h
> @@ -0,0 +1,44 @@
> +/*
> + * Copyright (C) ST-Ericsson SA 2010
> + * Author: Naveen Kumar G <naveen.gaddipati@stericsson.com> for ST-Ericsson
> + * License terms:GNU General Public License (GPL) version 2
> + */
> +
> +#ifndef _BU21013_H
> +#define _BU21013_H
> +
> +/**
> + * struct bu21013_platform_device - Handle the platform data
> + * @cs_en:     pointer to the cs enable function
> + * @cs_dis:    pointer to the cs disable function
> + * @irq_read_val:    pointer to read the pen irq value function
> + * @x_max_res: xmax resolution
> + * @y_max_res: ymax resolution
> + * @touch_x_max: touch x max
> + * @touch_y_max: touch y max
> + * @cs_pin: chip select pin
> + * @irq: irq pin
> + * @ext_clk: external clock flag
> + * @x_flip: x flip flag
> + * @y_flip: y flip flag
> + * @multi_touch: multi_touch flag
> + *
> + * This is used to handle the platform data
> + */
> +struct bu21013_platform_device {
> +       int (*cs_en)(int reset_pin);
> +       int (*cs_dis)(int reset_pin);
> +       int (*irq_read_val)(void);
> +       int x_max_res;
> +       int y_max_res;
> +       int touch_x_max;
> +       int touch_y_max;
> +       unsigned int cs_pin;
> +       unsigned int irq;
> +       bool ext_clk;
> +       bool x_flip;
> +       bool y_flip;
> +       bool multi_touch;
> +};
> +
> +#endif
> --
> 1.7.2.dirty
> 
> Thanks & Regards,
> Naveen

-- 
Dmitry

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 15:59 ` Dmitry Torokhov
@ 2010-09-09 16:01   ` Dmitry Torokhov
  2010-09-09 18:26     ` Henrik Rydberg
  0 siblings, 1 reply; 13+ messages in thread
From: Dmitry Torokhov @ 2010-09-09 16:01 UTC (permalink / raw)
  To: Naveen Kumar GADDIPATI
  Cc: Trilok Soni, linux-input, STEricsson_nomadik_linux, Henrik Rydberg

On Thu, Sep 09, 2010 at 08:59:19AM -0700, Dmitry Torokhov wrote:
> Hi Naveen,
> 
> On Thu, Sep 09, 2010 at 01:36:12PM +0200, Naveen Kumar GADDIPATI wrote:
> > Hi Dmitry,
> > 
> > From: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
> > 
> > Added the ROHM based bu21013 capacitive touch panel controller
> > driver support with i2c interface.
> > 
> > Acked-by: Linus Walleij <linus.walleij@stericsson.com>
> > Signed-off-by: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
> 
> Thank you for making the changes.
> 
> Henrik, could you glance over MT handling please?

Argh, now really adding Henrik...

> 
> > ---
> > Modifications in v2:
> >         --Updated with the Dmitry comments on Patch v1
> >         --Updated with the Trilok comments on Patch v1
> > 
> > 
> >  drivers/input/touchscreen/Kconfig      |   12 +
> >  drivers/input/touchscreen/Makefile     |    1 +
> >  drivers/input/touchscreen/bu21013_ts.c |  770 ++++++++++++++++++++++++++++++++
> >  include/linux/input/bu21013.h          |   44 ++
> >  4 files changed, 827 insertions(+), 0 deletions(-)
> >  create mode 100644 drivers/input/touchscreen/bu21013_ts.c
> >  create mode 100644 include/linux/input/bu21013.h
> > 
> > diff --git a/drivers/input/touchscreen/Kconfig b/drivers/input/touchscreen/Kconfig
> > index 0069d97..2380533 100644
> > --- a/drivers/input/touchscreen/Kconfig
> > +++ b/drivers/input/touchscreen/Kconfig
> > @@ -98,6 +98,18 @@ config TOUCHSCREEN_BITSY
> >           To compile this driver as a module, choose M here: the
> >           module will be called h3600_ts_input.
> > 
> > +config TOUCHSCREEN_BU21013
> > +       tristate "BU21013 based touch panel controllers"
> > +       depends on I2C
> > +       help
> > +         Say Y here if you have a bu21013 touchscreen connected to
> > +         your system.
> > +
> > +         If unsure, say N.
> > +
> > +         To compile this driver as a module, choose M here: the
> > +         module will be called bu21013_ts.
> > +
> >  config TOUCHSCREEN_CY8CTMG110
> >         tristate "cy8ctmg110 touchscreen"
> >         depends on I2C
> > diff --git a/drivers/input/touchscreen/Makefile b/drivers/input/touchscreen/Makefile
> > index 28217e1..a0a2021 100644
> > --- a/drivers/input/touchscreen/Makefile
> > +++ b/drivers/input/touchscreen/Makefile
> > @@ -14,6 +14,7 @@ obj-$(CONFIG_TOUCHSCREEN_AD7879_SPI)  += ad7879-spi.o
> >  obj-$(CONFIG_TOUCHSCREEN_ADS7846)      += ads7846.o
> >  obj-$(CONFIG_TOUCHSCREEN_ATMEL_TSADCC) += atmel_tsadcc.o
> >  obj-$(CONFIG_TOUCHSCREEN_BITSY)                += h3600_ts_input.o
> > +obj-$(CONFIG_TOUCHSCREEN_BU21013)       += bu21013_ts.o
> >  obj-$(CONFIG_TOUCHSCREEN_CY8CTMG110)   += cy8ctmg110_ts.o
> >  obj-$(CONFIG_TOUCHSCREEN_DA9034)       += da9034-ts.o
> >  obj-$(CONFIG_TOUCHSCREEN_DYNAPRO)      += dynapro.o
> > diff --git a/drivers/input/touchscreen/bu21013_ts.c b/drivers/input/touchscreen/bu21013_ts.c
> > new file mode 100644
> > index 0000000..117d9c5
> > --- /dev/null
> > +++ b/drivers/input/touchscreen/bu21013_ts.c
> > @@ -0,0 +1,770 @@
> > +/*
> > + * Copyright (C) ST-Ericsson SA 2010
> > + * Author: Naveen Kumar G <naveen.gaddipati@stericsson.com> for ST-Ericsson
> > + * License terms:GNU General Public License (GPL) version 2
> > + */
> > +
> > +#include <linux/kernel.h>
> > +#include <linux/init.h>
> > +#include <linux/delay.h>
> > +#include <linux/interrupt.h>
> > +#include <linux/i2c.h>
> > +#include <linux/workqueue.h>
> > +#include <linux/input.h>
> > +#include <linux/types.h>
> > +#include <linux/timer.h>
> > +#include <linux/input/bu21013.h>
> > +#include <linux/slab.h>
> > +
> > +#define PEN_DOWN_INTR                  0
> > +#define PEN_UP_INTR                    1
> > +#define RESET_DELAY                    30
> > +#define POLLING_DELAY                  100
> > +#define MAX_TOOL_WIDTH                 15
> > +#define MAX_TOUCH_MAJOR                        255
> > +#define MAX_TOUCH_MINOR                        15
> > +#define MAX_PRESSURE                   1
> > +#define PENUP_TIMEOUT                  (2)
> > +#define DELTA_MIN                      16
> > +#define MASK_BITS                      0x03
> > +#define SHIFT_8                                8
> > +#define SHIFT_2                                2
> > +#define LENGTH_OF_BUFFER               11
> > +
> > +#define BU21013_SENSORS_BTN_0_7_REG    0x70
> > +#define BU21013_SENSORS_BTN_8_15_REG   0x71
> > +#define BU21013_SENSORS_BTN_16_23_REG  0x72
> > +#define BU21013_X1_POS_MSB_REG         0x73
> > +#define BU21013_X1_POS_LSB_REG         0x74
> > +#define BU21013_Y1_POS_MSB_REG         0x75
> > +#define BU21013_Y1_POS_LSB_REG         0x76
> > +#define BU21013_X2_POS_MSB_REG         0x77
> > +#define BU21013_X2_POS_LSB_REG         0x78
> > +#define BU21013_Y2_POS_MSB_REG         0x79
> > +#define BU21013_Y2_POS_LSB_REG         0x7A
> > +#define BU21013_INT_CLR_REG            0xE8
> > +#define BU21013_INT_MODE_REG           0xE9
> > +#define BU21013_GAIN_REG               0xEA
> > +#define BU21013_OFFSET_MODE_REG                0xEB
> > +#define BU21013_XY_EDGE_REG            0xEC
> > +#define BU21013_RESET_REG              0xED
> > +#define BU21013_CALIB_REG              0xEE
> > +#define BU21013_DONE_REG               0xEF
> > +#define BU21013_SENSOR_0_7_REG         0xF0
> > +#define BU21013_SENSOR_8_15_REG                0xF1
> > +#define BU21013_SENSOR_16_23_REG       0xF2
> > +#define BU21013_POS_MODE1_REG          0xF3
> > +#define BU21013_POS_MODE2_REG          0xF4
> > +#define BU21013_CLK_MODE_REG           0xF5
> > +#define BU21013_IDLE_REG               0xFA
> > +#define BU21013_FILTER_REG             0xFB
> > +#define BU21013_TH_ON_REG              0xFC
> > +#define BU21013_TH_OFF_REG             0xFD
> > +
> > +#define BU21013_RESET_ENABLE           0x01
> > +
> > +#define BU21013_SENSORS_EN_0_7         0x3F
> > +#define BU21013_SENSORS_EN_8_15                0xFC
> > +#define BU21013_SENSORS_EN_16_23       0x1F
> > +
> > +#define BU21013_POS_MODE1_0            0x02
> > +#define BU21013_POS_MODE1_1            0x04
> > +#define BU21013_POS_MODE1_2            0x08
> > +
> > +#define BU21013_POS_MODE2_ZERO         0x01
> > +#define BU21013_POS_MODE2_AVG1         0x02
> > +#define BU21013_POS_MODE2_AVG2         0x04
> > +#define BU21013_POS_MODE2_EN_XY                0x08
> > +#define BU21013_POS_MODE2_EN_RAW       0x10
> > +#define BU21013_POS_MODE2_MULTI                0x80
> > +
> > +#define BU21013_CLK_MODE_DIV           0x01
> > +#define BU21013_CLK_MODE_EXT           0x02
> > +#define BU21013_CLK_MODE_CALIB         0x80
> > +
> > +#define BU21013_IDLET_0                        0x01
> > +#define BU21013_IDLET_1                        0x02
> > +#define BU21013_IDLET_2                        0x04
> > +#define BU21013_IDLET_3                        0x08
> > +#define BU21013_IDLE_INTERMIT_EN       0x10
> > +
> > +#define BU21013_DELTA_0_6              0x7F
> > +#define BU21013_FILTER_EN              0x80
> > +
> > +#define BU21013_INT_MODE_LEVEL         0x00
> > +#define BU21013_INT_MODE_EDGE          0x01
> > +
> > +#define BU21013_GAIN_0                 0x01
> > +#define BU21013_GAIN_1                 0x02
> > +#define BU21013_GAIN_2                 0x04
> > +
> > +#define BU21013_OFFSET_MODE_DEFAULT    0x00
> > +#define BU21013_OFFSET_MODE_MOVE       0x01
> > +#define BU21013_OFFSET_MODE_DISABLE    0x02
> > +
> > +#define BU21013_TH_ON_0                        0x01
> > +#define BU21013_TH_ON_1                        0x02
> > +#define BU21013_TH_ON_2                        0x04
> > +#define BU21013_TH_ON_3                        0x08
> > +#define BU21013_TH_ON_4                        0x10
> > +#define BU21013_TH_ON_5                        0x20
> > +#define BU21013_TH_ON_6                        0x40
> > +#define BU21013_TH_ON_7                        0x80
> > +#define BU21013_TH_ON_MAX              0xFF
> > +
> > +#define BU21013_TH_OFF_0               0x01
> > +#define BU21013_TH_OFF_1               0x02
> > +#define BU21013_TH_OFF_2               0x04
> > +#define BU21013_TH_OFF_3               0x08
> > +#define BU21013_TH_OFF_4               0x10
> > +#define BU21013_TH_OFF_5               0x20
> > +#define BU21013_TH_OFF_6               0x40
> > +#define BU21013_TH_OFF_7               0x80
> > +#define BU21013_TH_OFF_MAX             0xFF
> > +
> > +#define BU21013_X_EDGE_0               0x01
> > +#define BU21013_X_EDGE_1               0x02
> > +#define BU21013_X_EDGE_2               0x04
> > +#define BU21013_X_EDGE_3               0x08
> > +#define BU21013_Y_EDGE_0               0x10
> > +#define BU21013_Y_EDGE_1               0x20
> > +#define BU21013_Y_EDGE_2               0x40
> > +#define BU21013_Y_EDGE_3               0x80
> > +
> > +#define BU21013_DONE                   0x01
> > +#define BU21013_NUMBER_OF_X_SENSORS    (6)
> > +#define BU21013_NUMBER_OF_Y_SENSORS    (11)
> > +
> > +#define DRIVER_TP      "bu21013_tp"
> > +
> > +/**
> > + * struct bu21013_ts_data - touch panel data structure
> > + * @client:    pointer to the i2c client
> > + * @chip:      pointer to the touch panel controller
> > + * @in_dev:    pointer to the input device structure
> > + * @wait: variable to wait_queue_head_t structure
> > + * @intr_pin: interrupt pin value
> > + * @touch_stopped: touch stop flag
> > + * @x_pos: x position for two co-ordinates
> > + * @y_pos: y position for two co-ordinates
> > + * @previous_press_reported: last reported flag
> > + *
> > + * Touch panel device data structure
> > + */
> > +struct bu21013_ts_data {
> > +       struct i2c_client *client;
> > +       struct bu21013_platform_device *chip;
> 
> Please make int "const".
> 
> > +       struct input_dev *in_dev;
> > +       wait_queue_head_t wait;
> > +       bool touch_stopped;
> > +       unsigned int intr_pin;
> > +       signed short    x_pos[2];
> > +       signed short    y_pos[2];
> > +       bool previous_press_reported;
> > +};
> > +
> > +/**
> > + * bu21013_report_pen_down() - reports the pen down event
> > + * @data:bu21013_ts_data structure pointer
> > + * @count:touch count
> > + *
> > + * This function used to report the pen down interrupt to
> > + * input subsystem and returns none
> > + */
> > +static void bu21013_report_pen_down(struct bu21013_ts_data *data, int count)
> > +{
> > +       int i;
> > +
> > +       input_report_abs(data->in_dev, ABS_X, data->x_pos[0]);
> > +       input_report_abs(data->in_dev, ABS_Y, data->y_pos[0]);
> > +       input_report_key(data->in_dev, BTN_TOUCH, count);
> > +
> > +       if (data->chip->multi_touch) {
> 
> Why would we want to condition this? If hardware is capable of MT we
> should always support it.
> 
> > +               for (i = 0; i < count; i++) {
> > +                       input_report_abs(data->in_dev, ABS_MT_POSITION_X,
> > +                                                       data->x_pos[i]);
> > +                       input_report_abs(data->in_dev, ABS_MT_POSITION_Y,
> > +                                                       data->y_pos[i]);
> > +                       input_mt_sync(data->in_dev);
> > +               }
> > +       }
> > +       input_sync(data->in_dev);
> > +       data->previous_press_reported = count;
> > +}
> > +/**
> > + * bu21013_report_pen_up() - reports the pen up event
> > + * @data:bu21013_ts_data structure pointer
> > + *
> > + * This function used to report the pen up interrupt
> > + * to input subsystem and returns none
> > + */
> > +static void bu21013_report_pen_up(struct bu21013_ts_data *data)
> > +{
> > +       input_report_key(data->in_dev, BTN_TOUCH, 0);
> > +       if (data->chip->multi_touch) {
> > +               input_report_key(data->in_dev, BTN_2, 0);
> 
> Remnants of old patch... please remove.
> 
> > +               input_mt_sync(data->in_dev);
> > +       }
> > +       input_sync(data->in_dev);
> > +       data->previous_press_reported = 0;
> 
>  = false;
> 
> > +}
> > +/**
> > + * bu21013_touch_calc() - calculates the co-ordinates delta
> > + * @data: bu21013_ts_data structure pointer
> > + * @x: x position
> > + * @y: y position
> > + * @count: touch count
> > + *
> > + * This function calculates the exact co-ordinates with respect to
> > + * display resolution and returns none
> > + */
> > +static void bu21013_touch_calc
> > +       (struct bu21013_ts_data *data, int x, int y, int count)
> > +{
> > +       data->x_pos[count] = x;
> > +       data->y_pos[count] = y;
> > +       if (data->chip->x_flip)
> > +               data->x_pos[count] =
> > +                       data->chip->touch_x_max - data->x_pos[count];
> > +       if (data->chip->y_flip)
> > +               data->y_pos[count] =
> > +                       data->chip->touch_y_max - data->y_pos[count];
> > +}
> > +/**
> > + * bu21013_verify_delta() - verify the co-ordinates delta
> > + * @x1: x1 position
> > + * @y1: y1 position
> > + * @x2: x2 position
> > + * @y2: y2 position
> > + *
> > + * This function verifies the delta of the
> > + * co-ordinates and returns boolean.
> > + */
> > +static bool bu21013_verify_delta(int x1, int y1, int x2, int y2)
> > +{
> > +       int delta_x, delta_y;
> > +
> > +       if ((x1 != 0) && (y1 != 0)) {
> > +               delta_x = x2 - x1;
> > +               if (x1 > x2)
> > +                       delta_x = x1 - x2;
> > +               delta_y = y2 - y1;
> > +               if (y1 > y2)
> > +                       delta_y = y1 - y2;
> > +               if ((delta_x < DELTA_MIN) || (delta_y < DELTA_MIN))
> > +                       return false;
> > +       }
> > +       return true;
> 
> Hmm, I remember - the fuzz handling indeed will not work for MT A protocol.
> Still, I do not think it is correct. Don't you need to do
> "abs(delta_x) < DELTA_MIN"?
> 
> > +}
> > +
> > +/**
> > + * bu21013_do_touch_report(): Get the touch co-ordinates
> > + * @data:bu21013_ts_data structure pointer
> > + *
> > + * Get the touch co-ordinates from touch sensor registers and writes
> > + * into device structure and returns integer.
> > + */
> > +static int bu21013_do_touch_report(struct bu21013_ts_data *data)
> > +{
> > +       u8      buf[LENGTH_OF_BUFFER];
> > +       int     finger1_valid = 0;
> > +       int     finger2_valid = 0;
> > +       unsigned int    finger1_pos_x;
> > +       unsigned int    finger1_pos_y;
> > +       unsigned int    finger2_pos_x = 0;
> > +       unsigned int    finger2_pos_y = 0;
> > +       int     number_of_active_x_sensors;
> > +       int     number_of_active_y_sensors;
> > +       int     total_number_of_active_sensors;
> > +       int     finger_down_count = 0;
> > +       int     ret = 0;
> > +
> > +       if (data == NULL)
> > +               return -EINVAL;
> > +
> > +       ret = i2c_smbus_read_i2c_block_data(data->client,
> > +                                               BU21013_SENSORS_BTN_0_7_REG,
> > +                                                       LENGTH_OF_BUFFER, buf);
> > +       if (ret < LENGTH_OF_BUFFER)
> > +               return -EINVAL;
> > +
> > +       number_of_active_x_sensors = hweight32(buf[0] &
> > +                                       BU21013_SENSORS_EN_0_7);
> > +       number_of_active_y_sensors = hweight32(
> > +                       ((buf[1] & BU21013_SENSORS_EN_8_15) |
> > +               ((buf[2] & BU21013_SENSORS_EN_16_23) << SHIFT_8)) >> SHIFT_2);
> > +       if (((number_of_active_x_sensors != 0) &&
> > +               (number_of_active_y_sensors == 0)) ||
> > +               ((number_of_active_x_sensors == 0) &&
> > +               (number_of_active_y_sensors != 0)))
> > +               return 0;
> > +
> > +       total_number_of_active_sensors =
> > +               number_of_active_x_sensors + number_of_active_y_sensors;
> > +
> > +       finger1_pos_x = buf[3] << SHIFT_2 | (buf[4] & MASK_BITS);
> > +       finger1_pos_y = buf[5] << SHIFT_2 | (buf[6] & MASK_BITS);
> > +
> > +       if (data->chip->multi_touch) {
> > +               finger2_pos_x = buf[7] << SHIFT_2 | (buf[8] & MASK_BITS);
> > +               finger2_pos_y = buf[9] << SHIFT_2 | (buf[10] & MASK_BITS);
> > +       }
> > +
> > +       if (total_number_of_active_sensors) {
> > +               if ((finger2_pos_x != 0) && (finger2_pos_y != 0)) {
> > +                       if ((finger1_pos_x == 0) || (finger1_pos_y == 0))
> > +                               return 0;
> > +                       ret = bu21013_verify_delta(finger1_pos_x,
> > +                               finger1_pos_y, finger2_pos_x, finger2_pos_y);
> > +                       if (!ret)
> > +                               total_number_of_active_sensors = 0;
> > +               }
> > +       }
> > +
> > +       if (total_number_of_active_sensors) {
> > +               finger1_valid = 1;
> > +               if ((finger1_pos_x == 0) || (finger1_pos_y == 0) ||
> > +                       (finger1_pos_x >= data->chip->touch_x_max) ||
> > +                       (finger1_pos_y >= data->chip->touch_y_max))
> > +                       finger1_valid = 0;
> > +               finger2_valid = 1;
> > +               if ((finger2_pos_x == 0) || (finger2_pos_y == 0) ||
> > +                       (finger2_pos_x >= data->chip->touch_x_max) ||
> > +                       (finger2_pos_y >= data->chip->touch_y_max))
> > +                       finger2_valid = 0;
> > +               if ((finger1_valid == 0) && (finger2_valid == 0))
> > +                       return 0;
> > +               if (finger1_valid) {
> > +                       bu21013_touch_calc(data, finger1_pos_x,
> > +                               finger1_pos_y, finger_down_count);
> > +                       finger_down_count++;
> > +               }
> > +               if (finger2_valid) {
> > +                       bu21013_touch_calc(data, finger2_pos_x,
> > +                               finger2_pos_y, finger_down_count);
> > +                       finger_down_count++;
> > +               }
> > +       }
> > +
> > +       if (finger_down_count > 0)
> > +               bu21013_report_pen_down(data, finger_down_count);
> > +       else {
> > +               if (data->previous_press_reported)
> > +                       bu21013_report_pen_up(data);
> > +       }
> > +
> > +       return ret;
> > +}
> > +
> > +/**
> > + * bu21013_gpio_irq() - gpio thread function for touch interrupt
> > + * @irq: irq value
> > + * @device_data:void pointer
> > + *
> > + * This gpio thread function for touch interrupt
> > + * and returns irqreturn_t.
> > + */
> > +static irqreturn_t bu21013_gpio_irq(int irq, void *device_data)
> > +{
> > +       struct bu21013_ts_data *data = device_data;
> > +       struct i2c_client *i2c = data->client;
> > +       int retval;
> > +
> > +       do {
> > +               retval = bu21013_do_touch_report(data);
> > +               if (retval < 0) {
> > +                       dev_err(&i2c->dev, "bu21013_do_touch_report failed\n");
> > +                       return IRQ_NONE;
> > +               }
> > +
> > +               data->intr_pin = data->chip->irq_read_val();
> > +               if (data->intr_pin == PEN_DOWN_INTR)
> > +                       wait_event_timeout(data->wait, data->touch_stopped,
> > +                                                       msecs_to_jiffies(10));
> > +       } while (!data->intr_pin && !data->touch_stopped);
> > +
> > +       return IRQ_HANDLED;
> > +}
> > +
> > +/**
> > + * bu21013_init_chip() - power on sequence for the bu21013 controller
> > + * @data: device structure pointer
> > + *
> > + * This function is used to power on
> > + * the bu21013 controller and returns integer.
> > + */
> > +static int bu21013_init_chip(struct bu21013_ts_data *data)
> > +{
> > +       int retval;
> > +       struct i2c_client *i2c = data->client;
> > +
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_RESET_REG,
> > +                                       BU21013_RESET_ENABLE);
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_RESET reg write failed\n");
> > +               goto err;
> > +       }
> > +       msleep(RESET_DELAY);
> > +
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_0_7_REG,
> > +                                       BU21013_SENSORS_EN_0_7);
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_SENSOR_0_7 reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_8_15_REG,
> > +                                       BU21013_SENSORS_EN_8_15);
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_SENSOR_8_15 reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_16_23_REG,
> > +                                       BU21013_SENSORS_EN_16_23);
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_SENSOR_16_23 reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE1_REG,
> > +                               (BU21013_POS_MODE1_0 | BU21013_POS_MODE1_1));
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_POS_MODE1 reg write failed\n");
> > +               goto err;
> > +       }
> > +       if (data->chip->multi_touch)
> > +               retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE2_REG,
> > +                       (BU21013_POS_MODE2_ZERO | BU21013_POS_MODE2_AVG1 |
> > +                       BU21013_POS_MODE2_AVG2 | BU21013_POS_MODE2_EN_RAW |
> > +                       BU21013_POS_MODE2_MULTI));
> > +       else
> > +               retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE2_REG,
> > +                       (BU21013_POS_MODE2_ZERO | BU21013_POS_MODE2_AVG1 |
> > +                       BU21013_POS_MODE2_AVG2 | BU21013_POS_MODE2_EN_RAW));
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_POS_MODE2 reg write failed\n");
> > +               goto err;
> > +       }
> > +       if (data->chip->ext_clk)
> > +               retval = i2c_smbus_write_byte_data(i2c, BU21013_CLK_MODE_REG,
> > +                       (BU21013_CLK_MODE_EXT | BU21013_CLK_MODE_CALIB));
> > +       else
> > +               retval = i2c_smbus_write_byte_data(i2c, BU21013_CLK_MODE_REG,
> > +                       (BU21013_CLK_MODE_DIV | BU21013_CLK_MODE_CALIB));
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_CLK_MODE reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_IDLE_REG,
> > +                               (BU21013_IDLET_0 | BU21013_IDLE_INTERMIT_EN));
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_IDLE reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_INT_MODE_REG,
> > +                                               BU21013_INT_MODE_LEVEL);
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_INT_MODE reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_FILTER_REG,
> > +                                               (BU21013_DELTA_0_6 |
> > +                                                       BU21013_FILTER_EN));
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_FILTER reg write failed\n");
> > +               goto err;
> > +       }
> > +
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_TH_ON_REG,
> > +                                       (BU21013_TH_ON_3 | BU21013_TH_ON_4));
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_TH_ON reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_TH_OFF_REG,
> > +                                                       BU21013_TH_OFF_4);
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_TH_OFF reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_GAIN_REG,
> > +                                       (BU21013_GAIN_0 | BU21013_GAIN_1));
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_GAIN reg write failed\n");
> > +               goto err;
> > +       }
> > +
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_OFFSET_MODE_REG,
> > +                                       BU21013_OFFSET_MODE_DEFAULT);
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_OFFSET_MODE reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_XY_EDGE_REG,
> > +                       (BU21013_X_EDGE_0 | BU21013_X_EDGE_2 |
> > +                               BU21013_Y_EDGE_1 | BU21013_Y_EDGE_3));
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_XY_EDGE reg write failed\n");
> > +               goto err;
> > +       }
> > +       retval = i2c_smbus_write_byte_data(i2c, BU21013_DONE_REG,
> > +                                                       BU21013_DONE);
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "BU21013_REG_DONE reg write failed\n");
> > +               goto err;
> > +       }
> > +
> > +       data->x_pos[0] = 0;
> > +       data->x_pos[1] = 0;
> > +       data->y_pos[0] = 0;
> > +       data->y_pos[1] = 0;
> > +       data->previous_press_reported = 0;
> > +
> > +err:
> > +       return retval;
> > +}
> > +
> > +#ifdef CONFIG_PM
> > +/**
> > + * bu21013_suspend() - suspend the touch screen controller
> > + * @client: pointer to i2c client structure
> > + * @mesg: message from power manager
> > + *
> > + * This funtion is used to suspend the
> > + * touch panel controller and returns integer
> > + */
> > +static int bu21013_suspend(struct i2c_client *client, pm_message_t mesg)
> > +{
> > +       struct bu21013_ts_data *bu21013_data = i2c_get_clientdata(client);
> > +
> > +       bu21013_data->touch_stopped = true;
> > +       wake_up(&bu21013_data->wait);
> > +       if (device_may_wakeup(&client->dev))
> > +               enable_irq_wake(bu21013_data->chip->irq);
> > +       else
> > +               disable_irq(bu21013_data->chip->irq);
> > +
> > +       return 0;
> > +}
> > +
> > +/**
> > + * bu21013_resume() - resume the touch screen controller
> > + * @client: pointer to i2c client structure
> > + *
> > + * This funtion is used to resume the touch panel
> > + * controller and returns integer.
> > + */
> > +static int bu21013_resume(struct i2c_client *client)
> > +{
> > +       int retval;
> > +       struct bu21013_ts_data *bu21013_data = i2c_get_clientdata(client);
> > +
> > +       retval = bu21013_init_chip(bu21013_data);
> > +       if (retval < 0) {
> > +               dev_err(&client->dev, "tsc config failed\n");
> > +               return retval;
> > +       }
> > +
> > +       bu21013_data->touch_stopped = false;
> > +       if (device_may_wakeup(&client->dev))
> > +               disable_irq_wake(bu21013_data->chip->irq);
> > +       else
> > +               enable_irq(bu21013_data->chip->irq);
> > +
> > +       return 0;
> > +}
> > +#endif
> > +
> > +/**
> > + * bu21013_probe() - initialzes the i2c-client touchscreen driver
> > + * @i2c: i2c client structure pointer
> > + * @id:i2c device id pointer
> > + *
> > + * This function used to initializes the i2c-client touchscreen
> > + * driver and returns integer.
> > + */
> > +static int __devinit bu21013_probe(struct i2c_client *i2c,
> > +                                       const struct i2c_device_id *id)
> > +{
> > +       int retval;
> > +       struct bu21013_ts_data *bu21013_data;
> > +       struct input_dev *in_dev;
> > +       short x_max;
> > +       short y_max;
> > +       struct bu21013_platform_device *pdata = i2c->dev.platform_data;
> > +
> > +       if (!i2c) {
> > +               dev_err(&i2c->dev, "i2c client not defined\n");
> > +               retval = -EINVAL;
> > +               return retval;
> > +       }
> > +
> > +       if (!pdata) {
> > +               dev_err(&i2c->dev, "platform data not defined\n");
> > +               retval = -EINVAL;
> > +               return retval;
> > +       }
> > +
> > +       bu21013_data = kzalloc(sizeof(struct bu21013_ts_data), GFP_KERNEL);
> > +       if (!bu21013_data) {
> > +               dev_err(&i2c->dev, "device memory alloc failed\n");
> > +               retval = -ENOMEM;
> > +               return retval;
> > +       }
> > +
> > +       /* allocate input device */
> > +       in_dev = input_allocate_device();
> > +       if (!in_dev) {
> > +               dev_err(&i2c->dev, "input device memory alloc failed\n");
> > +               retval = -ENOMEM;
> > +               goto err_alloc;
> > +       }
> > +       bu21013_data->in_dev = in_dev;
> > +
> > +       bu21013_data->chip = pdata;
> > +       bu21013_data->client = i2c;
> > +       bu21013_data->touch_stopped = false;
> > +
> > +       x_max = pdata->x_max_res;
> > +       y_max = pdata->y_max_res;
> > +
> > +       init_waitqueue_head(&bu21013_data->wait);
> > +
> > +       i2c_set_clientdata(i2c, bu21013_data);
> > +
> > +       /* configure the gpio pins */
> > +       if (pdata->cs_en) {
> > +               retval = pdata->cs_en(pdata->cs_pin);
> > +               if (retval < 0) {
> > +                       dev_err(&i2c->dev, "chip init failed\n");
> > +                       goto err_init_cs;
> > +               }
> > +       }
> > +
> > +       /* configure the touch panel controller */
> > +       retval = bu21013_init_chip(bu21013_data);
> > +       if (retval < 0) {
> > +               dev_err(&i2c->dev, "error in bu21013 config\n");
> > +               goto err_init_config;
> > +       }
> > +
> > +       /* register the device to input subsystem */
> > +       in_dev->name = DRIVER_TP;
> > +       in_dev->id.bustype = BUS_I2C;
> > +       in_dev->dev.parent = &i2c->dev;
> > +       __set_bit(EV_SYN, in_dev->evbit);
> > +       __set_bit(EV_KEY, in_dev->evbit);
> > +       __set_bit(EV_ABS, in_dev->evbit);
> > +       __set_bit(BTN_TOUCH, in_dev->keybit);
> > +
> > +       input_set_abs_params(in_dev, ABS_X, 0, x_max, 0, 0);
> > +       input_set_abs_params(in_dev, ABS_Y, 0, y_max, 0, 0);
> > +       input_set_abs_params(in_dev, ABS_PRESSURE, 0, MAX_PRESSURE, 0, 0);
> > +       input_set_abs_params(in_dev, ABS_TOOL_WIDTH, 0, MAX_TOOL_WIDTH, 0, 0);
> > +
> > +       if (pdata->multi_touch) {
> > +               input_set_abs_params(in_dev, ABS_MT_POSITION_X, 0, x_max,
> > +                                                                       0, 0);
> > +               input_set_abs_params(in_dev, ABS_MT_POSITION_Y, 0, y_max,
> > +                                                                       0, 0);
> > +               input_set_abs_params(in_dev, ABS_MT_TOUCH_MAJOR, 0,
> > +                                               MAX_TOUCH_MAJOR, 0, 0);
> > +               input_set_abs_params(in_dev, ABS_MT_TOUCH_MINOR, 0,
> > +                                               MAX_TOUCH_MINOR, 0, 0);
> > +               input_set_abs_params(in_dev, ABS_MT_ORIENTATION, 0, 1, 0, 0);
> > +               input_set_abs_params(in_dev, ABS_MT_WIDTH_MAJOR, 0,
> > +                                               MAX_TOOL_WIDTH, 0, 0);
> 
> If we are not sending events we should not declare capabilities either.
> 
> > +       }
> > +       retval = input_register_device(in_dev);
> > +       if (retval)
> > +               goto err_input_register;
> > +
> > +       retval = request_threaded_irq(pdata->irq, NULL, bu21013_gpio_irq,
> > +                                       (IRQF_TRIGGER_FALLING | IRQF_SHARED),
> > +                                       DRIVER_TP, bu21013_data);
> > +       if (retval) {
> > +               dev_err(&i2c->dev, "request irq %d failed\n", pdata->irq);
> > +               goto err_init_irq;
> > +       }
> > +
> > +       return retval;
> > +
> > +err_init_irq:
> > +       input_unregister_device(bu21013_data->in_dev);
> > +       bu21013_data->in_dev = NULL;
> > +err_input_register:
> > +err_init_config:
> > +       pdata->cs_dis(pdata->cs_pin);
> > +err_init_cs:
> > +       input_free_device(bu21013_data->in_dev);
> > +err_alloc:
> > +       kfree(bu21013_data);
> > +
> > +       return retval;
> > +}
> > +/**
> > + * bu21013_remove() - removes the i2c-client touchscreen driver
> > + * @client: i2c client structure pointer
> > + *
> > + * This function uses to remove the i2c-client
> > + * touchscreen driver and returns integer.
> > + */
> > +static int __devexit bu21013_remove(struct i2c_client *client)
> > +{
> > +       struct bu21013_ts_data *data = i2c_get_clientdata(client);
> > +
> > +       data->touch_stopped = true;
> > +       free_irq(data->chip->irq, data);
> > +       data->chip->cs_dis(data->chip->cs_pin);
> > +       input_unregister_device(data->in_dev);
> > +       kfree(data);
> > +
> > +       return 0;
> > +}
> > +
> > +static const struct i2c_device_id bu21013_id[] = {
> > +       { DRIVER_TP, 0 },
> > +       { }
> > +};
> > +MODULE_DEVICE_TABLE(i2c, bu21013_id);
> > +
> > +static struct i2c_driver bu21013_driver = {
> > +       .driver = {
> > +               .name   =       DRIVER_TP,
> > +               .owner  =       THIS_MODULE,
> > +       },
> > +       .probe          =       bu21013_probe,
> > +#ifdef CONFIG_PM
> > +       .suspend        =       bu21013_suspend,
> > +       .resume         =       bu21013_resume,
> > +#endif
> > +       .remove         =       __devexit_p(bu21013_remove),
> > +       .id_table       =       bu21013_id,
> > +};
> > +
> > +/**
> > + * bu21013_init() - initializes the bu21013 touchscreen driver
> > + *
> > + * This function used to initializes the bu21013
> > + * touchscreen driver and returns integer.
> > + */
> > +static int __init bu21013_init(void)
> > +{
> > +       return i2c_add_driver(&bu21013_driver);
> > +}
> > +
> > +/**
> > + * bu21013_exit() - de-initializes the bu21013 touchscreen driver
> > + *
> > + * This function uses to de-initializes the bu21013
> > + * touchscreen driver and returns none.
> > + */
> > +static void __exit bu21013_exit(void)
> > +{
> > +       i2c_del_driver(&bu21013_driver);
> > +}
> > +
> > +module_init(bu21013_init);
> > +module_exit(bu21013_exit);
> > +
> > +MODULE_LICENSE("GPL v2");
> > +MODULE_AUTHOR("NAVEEN KUMAR G<naveen.gaddipati@stericsson.com");
> > +MODULE_DESCRIPTION("bu21013 touch screen controller driver");
> > diff --git a/include/linux/input/bu21013.h b/include/linux/input/bu21013.h
> > new file mode 100644
> > index 0000000..6c64bda
> > --- /dev/null
> > +++ b/include/linux/input/bu21013.h
> > @@ -0,0 +1,44 @@
> > +/*
> > + * Copyright (C) ST-Ericsson SA 2010
> > + * Author: Naveen Kumar G <naveen.gaddipati@stericsson.com> for ST-Ericsson
> > + * License terms:GNU General Public License (GPL) version 2
> > + */
> > +
> > +#ifndef _BU21013_H
> > +#define _BU21013_H
> > +
> > +/**
> > + * struct bu21013_platform_device - Handle the platform data
> > + * @cs_en:     pointer to the cs enable function
> > + * @cs_dis:    pointer to the cs disable function
> > + * @irq_read_val:    pointer to read the pen irq value function
> > + * @x_max_res: xmax resolution
> > + * @y_max_res: ymax resolution
> > + * @touch_x_max: touch x max
> > + * @touch_y_max: touch y max
> > + * @cs_pin: chip select pin
> > + * @irq: irq pin
> > + * @ext_clk: external clock flag
> > + * @x_flip: x flip flag
> > + * @y_flip: y flip flag
> > + * @multi_touch: multi_touch flag
> > + *
> > + * This is used to handle the platform data
> > + */
> > +struct bu21013_platform_device {
> > +       int (*cs_en)(int reset_pin);
> > +       int (*cs_dis)(int reset_pin);
> > +       int (*irq_read_val)(void);
> > +       int x_max_res;
> > +       int y_max_res;
> > +       int touch_x_max;
> > +       int touch_y_max;
> > +       unsigned int cs_pin;
> > +       unsigned int irq;
> > +       bool ext_clk;
> > +       bool x_flip;
> > +       bool y_flip;
> > +       bool multi_touch;
> > +};
> > +
> > +#endif
> > --
> > 1.7.2.dirty
> > 
> > Thanks & Regards,
> > Naveen
> 
> -- 
> Dmitry

-- 
Dmitry

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 16:01   ` Dmitry Torokhov
@ 2010-09-09 18:26     ` Henrik Rydberg
  2010-09-13 10:12       ` Naveen Kumar GADDIPATI
  0 siblings, 1 reply; 13+ messages in thread
From: Henrik Rydberg @ 2010-09-09 18:26 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: Naveen Kumar GADDIPATI, Trilok Soni, linux-input,
	STEricsson_nomadik_linux

Hi Naveen, Dmitry,

comments below.

>>> +/**
>>> + * bu21013_verify_delta() - verify the co-ordinates delta
>>> + * @x1: x1 position
>>> + * @y1: y1 position
>>> + * @x2: x2 position
>>> + * @y2: y2 position
>>> + *
>>> + * This function verifies the delta of the
>>> + * co-ordinates and returns boolean.
>>> + */
>>> +static bool bu21013_verify_delta(int x1, int y1, int x2, int y2)
>>> +{
>>> +       int delta_x, delta_y;
>>> +
>>> +       if ((x1 != 0) && (y1 != 0)) {
>>> +               delta_x = x2 - x1;
>>> +               if (x1 > x2)
>>> +                       delta_x = x1 - x2;
>>> +               delta_y = y2 - y1;
>>> +               if (y1 > y2)
>>> +                       delta_y = y1 - y2;
>>> +               if ((delta_x < DELTA_MIN) || (delta_y < DELTA_MIN))
>>> +                       return false;
>>> +       }
>>> +       return true;
>>
>> Hmm, I remember - the fuzz handling indeed will not work for MT A protocol.
>> Still, I do not think it is correct. Don't you need to do
>> "abs(delta_x) < DELTA_MIN"?


True, fuzz wont be processed in the kernel, but the information can be used
outside the kernel, like in mtdev.

I do not understand why deltas are verified in the first place. Either one can
rely on the identity of the fingers, in which case MT slots is simpler to
handle, or one cannot rely on it, in which case the above code is wrong, since
it assumes the values compared have the same identity.

>>> +static int bu21013_do_touch_report(struct bu21013_ts_data *data)

[...]
>>> +       finger1_pos_x = buf[3] << SHIFT_2 | (buf[4] & MASK_BITS);

>>> +       finger1_pos_y = buf[5] << SHIFT_2 | (buf[6] & MASK_BITS);
>>> +
>>> +       if (data->chip->multi_touch) {
>>> +               finger2_pos_x = buf[7] << SHIFT_2 | (buf[8] & MASK_BITS);
>>> +               finger2_pos_y = buf[9] << SHIFT_2 | (buf[10] & MASK_BITS);
>>> +       }


Yes, why branching on multi_touch here?

>>> +       if (total_number_of_active_sensors) {


Perhaps the logic could be inverted so that the case when the total number is
zero is treated first, to avoid repeated tests below.

>>> +               if ((finger2_pos_x != 0) && (finger2_pos_y != 0)) {
>>> +                       if ((finger1_pos_x == 0) || (finger1_pos_y == 0))
>>> +                               return 0;


So zero is some kind of null value? No continuation here?

>>> +                       ret = bu21013_verify_delta(finger1_pos_x,
>>> +                               finger1_pos_y, finger2_pos_x, finger2_pos_y);
>>> +                       if (!ret)
>>> +                               total_number_of_active_sensors = 0;
>>> +               }


So this is a tracking-plus-filtering solution? Unless the form factor forbids
it, having driver-specific tracking solutions in the kernel is not encouraged.
Two fingers might be borderline.

>>> +       }
>>> +
>>> +       if (total_number_of_active_sensors) {
>>> +               finger1_valid = 1;
>>> +               if ((finger1_pos_x == 0) || (finger1_pos_y == 0) ||
>>> +                       (finger1_pos_x >= data->chip->touch_x_max) ||
>>> +                       (finger1_pos_y >= data->chip->touch_y_max))
>>> +                       finger1_valid = 0;
>>> +               finger2_valid = 1;
>>> +               if ((finger2_pos_x == 0) || (finger2_pos_y == 0) ||
>>> +                       (finger2_pos_x >= data->chip->touch_x_max) ||
>>> +                       (finger2_pos_y >= data->chip->touch_y_max))
>>> +                       finger2_valid = 0;
>>> +               if ((finger1_valid == 0) && (finger2_valid == 0))
>>> +                       return 0;
>>> +               if (finger1_valid) {
>>> +                       bu21013_touch_calc(data, finger1_pos_x,
>>> +                               finger1_pos_y, finger_down_count);
>>> +                       finger_down_count++;
>>> +               }
>>> +               if (finger2_valid) {
>>> +                       bu21013_touch_calc(data, finger2_pos_x,
>>> +                               finger2_pos_y, finger_down_count);
>>> +                       finger_down_count++;
>>> +               }
>>> +       }


Surely this code could be made more comprehensible. Is it even necessary?
It is not an error from the perspective of the input core to send values outside
limits. If we are treating null values, perhaps defining them at the beginning
could help seeing the logic more clearly.

[...]

>>> +static int __devinit bu21013_probe(struct i2c_client *i2c,
[...]
>>> +       /* register the device to input subsystem */
>>> +       in_dev->name = DRIVER_TP;
>>> +       in_dev->id.bustype = BUS_I2C;
>>> +       in_dev->dev.parent = &i2c->dev;
>>> +       __set_bit(EV_SYN, in_dev->evbit);
>>> +       __set_bit(EV_KEY, in_dev->evbit);
>>> +       __set_bit(EV_ABS, in_dev->evbit);
>>> +       __set_bit(BTN_TOUCH, in_dev->keybit);
>>> +
>>> +       input_set_abs_params(in_dev, ABS_X, 0, x_max, 0, 0);
>>> +       input_set_abs_params(in_dev, ABS_Y, 0, y_max, 0, 0);
>>> +       input_set_abs_params(in_dev, ABS_PRESSURE, 0, MAX_PRESSURE, 0, 0);
>>> +       input_set_abs_params(in_dev, ABS_TOOL_WIDTH, 0, MAX_TOOL_WIDTH, 0, 0);
>>> +
>>> +       if (pdata->multi_touch) {
>>> +               input_set_abs_params(in_dev, ABS_MT_POSITION_X, 0, x_max,
>>> +                                                                       0, 0);
>>> +               input_set_abs_params(in_dev, ABS_MT_POSITION_Y, 0, y_max,
>>> +                                                                       0, 0);
>>> +               input_set_abs_params(in_dev, ABS_MT_TOUCH_MAJOR, 0,
>>> +                                               MAX_TOUCH_MAJOR, 0, 0);
>>> +               input_set_abs_params(in_dev, ABS_MT_TOUCH_MINOR, 0,
>>> +                                               MAX_TOUCH_MINOR, 0, 0);
>>> +               input_set_abs_params(in_dev, ABS_MT_ORIENTATION, 0, 1, 0, 0);
>>> +               input_set_abs_params(in_dev, ABS_MT_WIDTH_MAJOR, 0,
>>> +                                               MAX_TOOL_WIDTH, 0, 0);


TOUCH_MAJOR/MINOR and ORIENTATION are not used anywhere.

Thanks,
Henrik

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 15:39       ` Dmitry Torokhov
@ 2010-09-09 18:44         ` Trilok Soni
  0 siblings, 0 replies; 13+ messages in thread
From: Trilok Soni @ 2010-09-09 18:44 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: Naveen Kumar GADDIPATI, linux-input, STEricsson_nomadik_linux

Hi Dmitry,
>>
>> Refer: drivers/input/keyboard/adp5588-keys.c as example.
>>
> 
> Hm, indeed I forgot that i2c has been fully wired for dev_pm_ops. I
> should enforce its use then. Thank you for reminding Trilok.
> 
> I wonder if someone could wire SPI as well...
> 

Let me see if I could do that or atleast inspire someone to do it :)

---Trilok Soni

-- 
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* RE: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 18:26     ` Henrik Rydberg
@ 2010-09-13 10:12       ` Naveen Kumar GADDIPATI
  2010-09-13 14:36         ` Henrik Rydberg
  0 siblings, 1 reply; 13+ messages in thread
From: Naveen Kumar GADDIPATI @ 2010-09-13 10:12 UTC (permalink / raw)
  To: Henrik Rydberg, Dmitry Torokhov
  Cc: Trilok Soni, linux-input, STEricsson_nomadik_linux

Hi Henrik,

Thanks for your comments.

> >>> +static bool bu21013_verify_delta(int x1, int y1, int x2, int y2)
> >>> +{
> >>> +       int delta_x, delta_y;
> >>> +
> >>> +       if ((x1 != 0) && (y1 != 0)) {
> >>> +               delta_x = x2 - x1;
> >>> +               if (x1 > x2)
> >>> +                       delta_x = x1 - x2;
> >>> +               delta_y = y2 - y1;
> >>> +               if (y1 > y2)
> >>> +                       delta_y = y1 - y2;
> >>> +               if ((delta_x < DELTA_MIN) || (delta_y < DELTA_MIN))
> >>> +                       return false;
> >>> +       }
> >>> +       return true;
> >>
> >> Hmm, I remember - the fuzz handling indeed will not work for MT A
> protocol.
> >> Still, I do not think it is correct. Don't you need to do
> >> "abs(delta_x) < DELTA_MIN"?
> 
> 
> True, fuzz wont be processed in the kernel, but the information can be
> used
> outside the kernel, like in mtdev.
> 
> I do not understand why deltas are verified in the first place. Either
> one can
> rely on the identity of the fingers, in which case MT slots is simpler
> to
> handle, or one cannot rely on it, in which case the above code is
> wrong, since
> it assumes the values compared have the same identity.
> 

	This case according to controller specification, when the two fingers delta is of
	16 either in x or y, we should ignore those co-ordinates and report as pen-up.


> Perhaps the logic could be inverted so that the case when the total
> number is
> zero is treated first, to avoid repeated tests below.
> 
> >>> +               if ((finger2_pos_x != 0) && (finger2_pos_y != 0)) {
> >>> +                       if ((finger1_pos_x == 0) || (finger1_pos_y
> == 0))
> >>> +                               return 0;
> 
> 
> So zero is some kind of null value? No continuation here?
> 
> >>> +                       ret = bu21013_verify_delta(finger1_pos_x,
> >>> +                               finger1_pos_y, finger2_pos_x,
> finger2_pos_y);
> >>> +                       if (!ret)
> >>> +                               total_number_of_active_sensors = 0;
> >>> +               }
> 
> 
> So this is a tracking-plus-filtering solution? Unless the form factor
> forbids
> it, having driver-specific tracking solutions in the kernel is not
> encouraged.
> Two fingers might be borderline.
	
	We need to keep this filtering solution according to the controller specification.

Thanks & Regards,
Naveen

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-13 10:12       ` Naveen Kumar GADDIPATI
@ 2010-09-13 14:36         ` Henrik Rydberg
  0 siblings, 0 replies; 13+ messages in thread
From: Henrik Rydberg @ 2010-09-13 14:36 UTC (permalink / raw)
  To: Naveen Kumar GADDIPATI
  Cc: Dmitry Torokhov, Trilok Soni, linux-input, STEricsson_nomadik_linux

On 09/13/2010 12:12 PM, Naveen Kumar GADDIPATI wrote:
[...]
>>>>> +static bool bu21013_verify_delta(int x1, int y1, int x2, int y2)

>>>>> +{
>>>>> +       int delta_x, delta_y;
>>>>> +
>>>>> +       if ((x1 != 0) && (y1 != 0)) {
>>>>> +               delta_x = x2 - x1;
>>>>> +               if (x1 > x2)
>>>>> +                       delta_x = x1 - x2;
>>>>> +               delta_y = y2 - y1;
>>>>> +               if (y1 > y2)
>>>>> +                       delta_y = y1 - y2;
>>>>> +               if ((delta_x < DELTA_MIN) || (delta_y < DELTA_MIN))
>>>>> +                       return false;
>>>>> +       }
>>>>> +       return true;
[...]

>> I do not understand why deltas are verified in the first place.
>> Either one can rely on the identity of the fingers, in which
>> case MT slots is simpler to handle, or one cannot rely on it,
>> in which case the above code is wrong, since it assumes the
>> values compared have the same identity.

> 
> 	This case according to controller specification, when the two fingers delta is of
> 	16 either in x or y, we should ignore those co-ordinates and report as pen-up.


And not as a single finger with larger perimeter? It would be interesting to
know how this driver has been tested. There are some applications that can help
examining the quality of MT output, one of them can be found at
http://bitmath.org/code/mtview/.

Henrik

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
  2010-09-09 12:12 Naveen Kumar GADDIPATI
@ 2010-09-09 12:13 ` Trilok Soni
  0 siblings, 0 replies; 13+ messages in thread
From: Trilok Soni @ 2010-09-09 12:13 UTC (permalink / raw)
  To: Naveen Kumar GADDIPATI
  Cc: Dmitry Torokhov, linux-input, STEricsson_nomadik_linux

On 9/9/2010 5:42 PM, Naveen Kumar GADDIPATI wrote:
> Hi Dmitry,
> 
> Please ignore this patch. Just now I sent a new PATCHv2.
> 
> Thanks & Regards,
> Naveen
> 

Looks like I am very quick at review. I had reviewed your earlier patchv2. I hope the
comments still apply.

---Trilok Soni

-- 
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* RE:  [Patch v2] input:rohm based bu21013 touch panel controller driver support
@ 2010-09-09 12:12 Naveen Kumar GADDIPATI
  2010-09-09 12:13 ` Trilok Soni
  0 siblings, 1 reply; 13+ messages in thread
From: Naveen Kumar GADDIPATI @ 2010-09-09 12:12 UTC (permalink / raw)
  To: Dmitry Torokhov; +Cc: Trilok Soni, linux-input, STEricsson_nomadik_linux

Hi Dmitry,

Please ignore this patch. Just now I sent a new PATCHv2.

Thanks & Regards,
Naveen

> -----Original Message-----
> From: Naveen Kumar GADDIPATI
> Sent: Thursday, September 09, 2010 5:06 PM
> To: Dmitry Torokhov
> Cc: 'Trilok Soni'; linux-input@vger.kernel.org;
> STEricsson_nomadik_linux
> Subject: [Patch v2] input:rohm based bu21013 touch panel controller
> driver support
>
> Hi Dmitry,
>
> From: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
>
> Added the ROHM based bu21013 capacitive touch panel controller
> driver support with i2c interface.
>
> Acked-by: Linus Walleij <linus.walleij@stericsson.com>
> Signed-off-by: Naveen Kumar Gaddipati <naveen.gaddipati@stericsson.com>
> ---
> Modifications in v2:
>       --Updated with the Dmitry comments on Patch v1
>       --Updated with the Trilok comments on Patch v1
>
>
>  drivers/input/touchscreen/Kconfig      |   12 +
>  drivers/input/touchscreen/Makefile     |    1 +
>  drivers/input/touchscreen/bu21013_ts.c |  770
> ++++++++++++++++++++++++++++++++
>  include/linux/input/bu21013.h          |   44 ++
>  4 files changed, 827 insertions(+), 0 deletions(-)
>  create mode 100644 drivers/input/touchscreen/bu21013_ts.c
>  create mode 100644 include/linux/input/bu21013.h
>
> diff --git a/drivers/input/touchscreen/Kconfig
> b/drivers/input/touchscreen/Kconfig
> index 0069d97..2380533 100644
> --- a/drivers/input/touchscreen/Kconfig
> +++ b/drivers/input/touchscreen/Kconfig
> @@ -98,6 +98,18 @@ config TOUCHSCREEN_BITSY
>         To compile this driver as a module, choose M here: the
>         module will be called h3600_ts_input.
>
> +config TOUCHSCREEN_BU21013
> +     tristate "BU21013 based touch panel controllers"
> +     depends on I2C
> +     help
> +       Say Y here if you have a bu21013 touchscreen connected to
> +       your system.
> +
> +       If unsure, say N.
> +
> +       To compile this driver as a module, choose M here: the
> +       module will be called bu21013_ts.
> +
>  config TOUCHSCREEN_CY8CTMG110
>       tristate "cy8ctmg110 touchscreen"
>       depends on I2C
> diff --git a/drivers/input/touchscreen/Makefile
> b/drivers/input/touchscreen/Makefile
> index 28217e1..a0a2021 100644
> --- a/drivers/input/touchscreen/Makefile
> +++ b/drivers/input/touchscreen/Makefile
> @@ -14,6 +14,7 @@ obj-$(CONFIG_TOUCHSCREEN_AD7879_SPI)        += ad7879-
> spi.o
>  obj-$(CONFIG_TOUCHSCREEN_ADS7846)    += ads7846.o
>  obj-$(CONFIG_TOUCHSCREEN_ATMEL_TSADCC)       += atmel_tsadcc.o
>  obj-$(CONFIG_TOUCHSCREEN_BITSY)              += h3600_ts_input.o
> +obj-$(CONFIG_TOUCHSCREEN_BU21013)       += bu21013_ts.o
>  obj-$(CONFIG_TOUCHSCREEN_CY8CTMG110) += cy8ctmg110_ts.o
>  obj-$(CONFIG_TOUCHSCREEN_DA9034)     += da9034-ts.o
>  obj-$(CONFIG_TOUCHSCREEN_DYNAPRO)    += dynapro.o
> diff --git a/drivers/input/touchscreen/bu21013_ts.c
> b/drivers/input/touchscreen/bu21013_ts.c
> new file mode 100644
> index 0000000..117d9c5
> --- /dev/null
> +++ b/drivers/input/touchscreen/bu21013_ts.c
> @@ -0,0 +1,770 @@
> +/*
> + * Copyright (C) ST-Ericsson SA 2010
> + * Author: Naveen Kumar G <naveen.gaddipati@stericsson.com> for ST-
> Ericsson
> + * License terms:GNU General Public License (GPL) version 2
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/delay.h>
> +#include <linux/interrupt.h>
> +#include <linux/i2c.h>
> +#include <linux/workqueue.h>
> +#include <linux/input.h>
> +#include <linux/types.h>
> +#include <linux/timer.h>
> +#include <linux/input/bu21013.h>
> +#include <linux/slab.h>
> +
> +#define PEN_DOWN_INTR                        0
> +#define PEN_UP_INTR                  1
> +#define RESET_DELAY                  30
> +#define POLLING_DELAY                        100
> +#define MAX_TOOL_WIDTH                       15
> +#define MAX_TOUCH_MAJOR                      255
> +#define MAX_TOUCH_MINOR                      15
> +#define MAX_PRESSURE                 1
> +#define PENUP_TIMEOUT                        (2)
> +#define DELTA_MIN                    16
> +#define MASK_BITS                    0x03
> +#define SHIFT_8                              8
> +#define SHIFT_2                              2
> +#define LENGTH_OF_BUFFER             11
> +
> +#define BU21013_SENSORS_BTN_0_7_REG  0x70
> +#define BU21013_SENSORS_BTN_8_15_REG 0x71
> +#define BU21013_SENSORS_BTN_16_23_REG        0x72
> +#define BU21013_X1_POS_MSB_REG               0x73
> +#define BU21013_X1_POS_LSB_REG               0x74
> +#define BU21013_Y1_POS_MSB_REG               0x75
> +#define BU21013_Y1_POS_LSB_REG               0x76
> +#define BU21013_X2_POS_MSB_REG               0x77
> +#define BU21013_X2_POS_LSB_REG               0x78
> +#define BU21013_Y2_POS_MSB_REG               0x79
> +#define BU21013_Y2_POS_LSB_REG               0x7A
> +#define BU21013_INT_CLR_REG          0xE8
> +#define BU21013_INT_MODE_REG         0xE9
> +#define BU21013_GAIN_REG             0xEA
> +#define BU21013_OFFSET_MODE_REG              0xEB
> +#define BU21013_XY_EDGE_REG          0xEC
> +#define BU21013_RESET_REG            0xED
> +#define BU21013_CALIB_REG            0xEE
> +#define BU21013_DONE_REG             0xEF
> +#define BU21013_SENSOR_0_7_REG               0xF0
> +#define BU21013_SENSOR_8_15_REG              0xF1
> +#define BU21013_SENSOR_16_23_REG     0xF2
> +#define BU21013_POS_MODE1_REG                0xF3
> +#define BU21013_POS_MODE2_REG                0xF4
> +#define BU21013_CLK_MODE_REG         0xF5
> +#define BU21013_IDLE_REG             0xFA
> +#define BU21013_FILTER_REG           0xFB
> +#define BU21013_TH_ON_REG            0xFC
> +#define BU21013_TH_OFF_REG           0xFD
> +
> +#define BU21013_RESET_ENABLE         0x01
> +
> +#define BU21013_SENSORS_EN_0_7               0x3F
> +#define BU21013_SENSORS_EN_8_15              0xFC
> +#define BU21013_SENSORS_EN_16_23     0x1F
> +
> +#define BU21013_POS_MODE1_0          0x02
> +#define BU21013_POS_MODE1_1          0x04
> +#define BU21013_POS_MODE1_2          0x08
> +
> +#define BU21013_POS_MODE2_ZERO               0x01
> +#define BU21013_POS_MODE2_AVG1               0x02
> +#define BU21013_POS_MODE2_AVG2               0x04
> +#define BU21013_POS_MODE2_EN_XY              0x08
> +#define BU21013_POS_MODE2_EN_RAW     0x10
> +#define BU21013_POS_MODE2_MULTI              0x80
> +
> +#define BU21013_CLK_MODE_DIV         0x01
> +#define BU21013_CLK_MODE_EXT         0x02
> +#define BU21013_CLK_MODE_CALIB               0x80
> +
> +#define BU21013_IDLET_0                      0x01
> +#define BU21013_IDLET_1                      0x02
> +#define BU21013_IDLET_2                      0x04
> +#define BU21013_IDLET_3                      0x08
> +#define BU21013_IDLE_INTERMIT_EN     0x10
> +
> +#define BU21013_DELTA_0_6            0x7F
> +#define BU21013_FILTER_EN            0x80
> +
> +#define BU21013_INT_MODE_LEVEL               0x00
> +#define BU21013_INT_MODE_EDGE                0x01
> +
> +#define BU21013_GAIN_0                       0x01
> +#define BU21013_GAIN_1                       0x02
> +#define BU21013_GAIN_2                       0x04
> +
> +#define BU21013_OFFSET_MODE_DEFAULT  0x00
> +#define BU21013_OFFSET_MODE_MOVE     0x01
> +#define BU21013_OFFSET_MODE_DISABLE  0x02
> +
> +#define BU21013_TH_ON_0                      0x01
> +#define BU21013_TH_ON_1                      0x02
> +#define BU21013_TH_ON_2                      0x04
> +#define BU21013_TH_ON_3                      0x08
> +#define BU21013_TH_ON_4                      0x10
> +#define BU21013_TH_ON_5                      0x20
> +#define BU21013_TH_ON_6                      0x40
> +#define BU21013_TH_ON_7                      0x80
> +#define BU21013_TH_ON_MAX            0xFF
> +
> +#define BU21013_TH_OFF_0             0x01
> +#define BU21013_TH_OFF_1             0x02
> +#define BU21013_TH_OFF_2             0x04
> +#define BU21013_TH_OFF_3             0x08
> +#define BU21013_TH_OFF_4             0x10
> +#define BU21013_TH_OFF_5             0x20
> +#define BU21013_TH_OFF_6             0x40
> +#define BU21013_TH_OFF_7             0x80
> +#define BU21013_TH_OFF_MAX           0xFF
> +
> +#define BU21013_X_EDGE_0             0x01
> +#define BU21013_X_EDGE_1             0x02
> +#define BU21013_X_EDGE_2             0x04
> +#define BU21013_X_EDGE_3             0x08
> +#define BU21013_Y_EDGE_0             0x10
> +#define BU21013_Y_EDGE_1             0x20
> +#define BU21013_Y_EDGE_2             0x40
> +#define BU21013_Y_EDGE_3             0x80
> +
> +#define BU21013_DONE                 0x01
> +#define BU21013_NUMBER_OF_X_SENSORS  (6)
> +#define BU21013_NUMBER_OF_Y_SENSORS  (11)
> +
> +#define DRIVER_TP    "bu21013_tp"
> +
> +/**
> + * struct bu21013_ts_data - touch panel data structure
> + * @client:  pointer to the i2c client
> + * @chip:    pointer to the touch panel controller
> + * @in_dev:    pointer to the input device structure
> + * @wait: variable to wait_queue_head_t structure
> + * @intr_pin: interrupt pin value
> + * @touch_stopped: touch stop flag
> + * @x_pos: x position for two co-ordinates
> + * @y_pos: y position for two co-ordinates
> + * @previous_press_reported: last reported flag
> + *
> + * Touch panel device data structure
> + */
> +struct bu21013_ts_data {
> +     struct i2c_client *client;
> +     struct bu21013_platform_device *chip;
> +     struct input_dev *in_dev;
> +     wait_queue_head_t wait;
> +     bool touch_stopped;
> +     unsigned int intr_pin;
> +     signed short    x_pos[2];
> +     signed short    y_pos[2];
> +     bool previous_press_reported;
> +};
> +
> +/**
> + * bu21013_report_pen_down() - reports the pen down event
> + * @data:bu21013_ts_data structure pointer
> + * @count:touch count
> + *
> + * This function used to report the pen down interrupt to
> + * input subsystem and returns none
> + */
> +static void bu21013_report_pen_down(struct bu21013_ts_data *data, int
> count)
> +{
> +     int i;
> +
> +     input_report_abs(data->in_dev, ABS_X, data->x_pos[0]);
> +     input_report_abs(data->in_dev, ABS_Y, data->y_pos[0]);
> +     input_report_key(data->in_dev, BTN_TOUCH, count);
> +
> +     if (data->chip->multi_touch) {
> +             for (i = 0; i < count; i++) {
> +                     input_report_abs(data->in_dev, ABS_MT_POSITION_X,
> +                                                     data->x_pos[i]);
> +                     input_report_abs(data->in_dev, ABS_MT_POSITION_Y,
> +                                                     data->y_pos[i]);
> +                     input_mt_sync(data->in_dev);
> +             }
> +     }
> +     input_sync(data->in_dev);
> +     data->previous_press_reported = count;
> +}
> +/**
> + * bu21013_report_pen_up() - reports the pen up event
> + * @data:bu21013_ts_data structure pointer
> + *
> + * This function used to report the pen up interrupt
> + * to input subsystem and returns none
> + */
> +static void bu21013_report_pen_up(struct bu21013_ts_data *data)
> +{
> +     input_report_key(data->in_dev, BTN_TOUCH, 0);
> +     if (data->chip->multi_touch) {
> +             input_report_key(data->in_dev, BTN_2, 0);
> +             input_mt_sync(data->in_dev);
> +     }
> +     input_sync(data->in_dev);
> +     data->previous_press_reported = 0;
> +}
> +/**
> + * bu21013_touch_calc() - calculates the co-ordinates delta
> + * @data: bu21013_ts_data structure pointer
> + * @x: x position
> + * @y: y position
> + * @count: touch count
> + *
> + * This function calculates the exact co-ordinates with respect to
> + * display resolution and returns none
> + */
> +static void bu21013_touch_calc
> +     (struct bu21013_ts_data *data, int x, int y, int count)
> +{
> +     data->x_pos[count] = x;
> +     data->y_pos[count] = y;
> +     if (data->chip->x_flip)
> +             data->x_pos[count] =
> +                     data->chip->touch_x_max - data->x_pos[count];
> +     if (data->chip->y_flip)
> +             data->y_pos[count] =
> +                     data->chip->touch_y_max - data->y_pos[count];
> +}
> +/**
> + * bu21013_verify_delta() - verify the co-ordinates delta
> + * @x1: x1 position
> + * @y1: y1 position
> + * @x2: x2 position
> + * @y2: y2 position
> + *
> + * This function verifies the delta of the
> + * co-ordinates and returns boolean.
> + */
> +static bool bu21013_verify_delta(int x1, int y1, int x2, int y2)
> +{
> +     int delta_x, delta_y;
> +
> +     if ((x1 != 0) && (y1 != 0)) {
> +             delta_x = x2 - x1;
> +             if (x1 > x2)
> +                     delta_x = x1 - x2;
> +             delta_y = y2 - y1;
> +             if (y1 > y2)
> +                     delta_y = y1 - y2;
> +             if ((delta_x < DELTA_MIN) || (delta_y < DELTA_MIN))
> +                     return false;
> +     }
> +     return true;
> +}
> +
> +/**
> + * bu21013_do_touch_report(): Get the touch co-ordinates
> + * @data:bu21013_ts_data structure pointer
> + *
> + * Get the touch co-ordinates from touch sensor registers and writes
> + * into device structure and returns integer.
> + */
> +static int bu21013_do_touch_report(struct bu21013_ts_data *data)
> +{
> +     u8      buf[LENGTH_OF_BUFFER];
> +     int     finger1_valid = 0;
> +     int     finger2_valid = 0;
> +     unsigned int    finger1_pos_x;
> +     unsigned int    finger1_pos_y;
> +     unsigned int    finger2_pos_x = 0;
> +     unsigned int    finger2_pos_y = 0;
> +     int     number_of_active_x_sensors;
> +     int     number_of_active_y_sensors;
> +     int     total_number_of_active_sensors;
> +     int     finger_down_count = 0;
> +     int     ret = 0;
> +
> +     if (data == NULL)
> +             return -EINVAL;
> +
> +     ret = i2c_smbus_read_i2c_block_data(data->client,
> +                                             BU21013_SENSORS_BTN_0_7_REG,
> +                                                     LENGTH_OF_BUFFER, buf);
> +     if (ret < LENGTH_OF_BUFFER)
> +             return -EINVAL;
> +
> +     number_of_active_x_sensors = hweight32(buf[0] &
> +                                     BU21013_SENSORS_EN_0_7);
> +     number_of_active_y_sensors = hweight32(
> +                     ((buf[1] & BU21013_SENSORS_EN_8_15) |
> +             ((buf[2] & BU21013_SENSORS_EN_16_23) << SHIFT_8)) >>
> SHIFT_2);
> +     if (((number_of_active_x_sensors != 0) &&
> +             (number_of_active_y_sensors == 0)) ||
> +             ((number_of_active_x_sensors == 0) &&
> +             (number_of_active_y_sensors != 0)))
> +             return 0;
> +
> +     total_number_of_active_sensors =
> +             number_of_active_x_sensors + number_of_active_y_sensors;
> +
> +     finger1_pos_x = buf[3] << SHIFT_2 | (buf[4] & MASK_BITS);
> +     finger1_pos_y = buf[5] << SHIFT_2 | (buf[6] & MASK_BITS);
> +
> +     if (data->chip->multi_touch) {
> +             finger2_pos_x = buf[7] << SHIFT_2 | (buf[8] & MASK_BITS);
> +             finger2_pos_y = buf[9] << SHIFT_2 | (buf[10] & MASK_BITS);
> +     }
> +
> +     if (total_number_of_active_sensors) {
> +             if ((finger2_pos_x != 0) && (finger2_pos_y != 0)) {
> +                     if ((finger1_pos_x == 0) || (finger1_pos_y == 0))
> +                             return 0;
> +                     ret = bu21013_verify_delta(finger1_pos_x,
> +                             finger1_pos_y, finger2_pos_x, finger2_pos_y);
> +                     if (!ret)
> +                             total_number_of_active_sensors = 0;
> +             }
> +     }
> +
> +     if (total_number_of_active_sensors) {
> +             finger1_valid = 1;
> +             if ((finger1_pos_x == 0) || (finger1_pos_y == 0) ||
> +                     (finger1_pos_x >= data->chip->touch_x_max) ||
> +                     (finger1_pos_y >= data->chip->touch_y_max))
> +                     finger1_valid = 0;
> +             finger2_valid = 1;
> +             if ((finger2_pos_x == 0) || (finger2_pos_y == 0) ||
> +                     (finger2_pos_x >= data->chip->touch_x_max) ||
> +                     (finger2_pos_y >= data->chip->touch_y_max))
> +                     finger2_valid = 0;
> +             if ((finger1_valid == 0) && (finger2_valid == 0))
> +                     return 0;
> +             if (finger1_valid) {
> +                     bu21013_touch_calc(data, finger1_pos_x,
> +                             finger1_pos_y, finger_down_count);
> +                     finger_down_count++;
> +             }
> +             if (finger2_valid) {
> +                     bu21013_touch_calc(data, finger2_pos_x,
> +                             finger2_pos_y, finger_down_count);
> +                     finger_down_count++;
> +             }
> +     }
> +
> +     if (finger_down_count > 0)
> +             bu21013_report_pen_down(data, finger_down_count);
> +     else {
> +             if (data->previous_press_reported)
> +                     bu21013_report_pen_up(data);
> +     }
> +
> +     return ret;
> +}
> +
> +/**
> + * bu21013_gpio_irq() - gpio thread function for touch interrupt
> + * @irq: irq value
> + * @device_data:void pointer
> + *
> + * This gpio thread function for touch interrupt
> + * and returns irqreturn_t.
> + */
> +static irqreturn_t bu21013_gpio_irq(int irq, void *device_data)
> +{
> +     struct bu21013_ts_data *data = device_data;
> +     struct i2c_client *i2c = data->client;
> +     int retval;
> +
> +     do {
> +             retval = bu21013_do_touch_report(data);
> +             if (retval < 0) {
> +                     dev_err(&i2c->dev, "bu21013_do_touch_report
> failed\n");
> +                     return IRQ_NONE;
> +             }
> +
> +             data->intr_pin = data->chip->irq_read_val();
> +             if (data->intr_pin == PEN_DOWN_INTR)
> +                     wait_event_timeout(data->wait, data->touch_stopped,
> +                                                     msecs_to_jiffies(10));
> +     } while (!data->intr_pin && !data->touch_stopped);
> +
> +     return IRQ_HANDLED;
> +}
> +
> +/**
> + * bu21013_init_chip() - power on sequence for the bu21013 controller
> + * @data: device structure pointer
> + *
> + * This function is used to power on
> + * the bu21013 controller and returns integer.
> + */
> +static int bu21013_init_chip(struct bu21013_ts_data *data)
> +{
> +     int retval;
> +     struct i2c_client *i2c = data->client;
> +
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_RESET_REG,
> +                                     BU21013_RESET_ENABLE);
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_RESET reg write failed\n");
> +             goto err;
> +     }
> +     msleep(RESET_DELAY);
> +
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_0_7_REG,
> +                                     BU21013_SENSORS_EN_0_7);
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_SENSOR_0_7 reg write
> failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_8_15_REG,
> +                                     BU21013_SENSORS_EN_8_15);
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_SENSOR_8_15 reg write
> failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_SENSOR_16_23_REG,
> +                                     BU21013_SENSORS_EN_16_23);
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_SENSOR_16_23 reg write
> failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_POS_MODE1_REG,
> +                             (BU21013_POS_MODE1_0 | BU21013_POS_MODE1_1));
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_POS_MODE1 reg write failed\n");
> +             goto err;
> +     }
> +     if (data->chip->multi_touch)
> +             retval = i2c_smbus_write_byte_data(i2c,
> BU21013_POS_MODE2_REG,
> +                     (BU21013_POS_MODE2_ZERO | BU21013_POS_MODE2_AVG1 |
> +                     BU21013_POS_MODE2_AVG2 | BU21013_POS_MODE2_EN_RAW |
> +                     BU21013_POS_MODE2_MULTI));
> +     else
> +             retval = i2c_smbus_write_byte_data(i2c,
> BU21013_POS_MODE2_REG,
> +                     (BU21013_POS_MODE2_ZERO | BU21013_POS_MODE2_AVG1 |
> +                     BU21013_POS_MODE2_AVG2 | BU21013_POS_MODE2_EN_RAW));
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_POS_MODE2 reg write failed\n");
> +             goto err;
> +     }
> +     if (data->chip->ext_clk)
> +             retval = i2c_smbus_write_byte_data(i2c,
> BU21013_CLK_MODE_REG,
> +                     (BU21013_CLK_MODE_EXT | BU21013_CLK_MODE_CALIB));
> +     else
> +             retval = i2c_smbus_write_byte_data(i2c,
> BU21013_CLK_MODE_REG,
> +                     (BU21013_CLK_MODE_DIV | BU21013_CLK_MODE_CALIB));
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_CLK_MODE reg write failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_IDLE_REG,
> +                             (BU21013_IDLET_0 | BU21013_IDLE_INTERMIT_EN));
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_IDLE reg write failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_INT_MODE_REG,
> +                                             BU21013_INT_MODE_LEVEL);
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_INT_MODE reg write failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_FILTER_REG,
> +                                             (BU21013_DELTA_0_6 |
> +                                                     BU21013_FILTER_EN));
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_FILTER reg write failed\n");
> +             goto err;
> +     }
> +
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_TH_ON_REG,
> +                                     (BU21013_TH_ON_3 | BU21013_TH_ON_4));
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_TH_ON reg write failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_TH_OFF_REG,
> +                                                     BU21013_TH_OFF_4);
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_TH_OFF reg write failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_GAIN_REG,
> +                                     (BU21013_GAIN_0 | BU21013_GAIN_1));
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_GAIN reg write failed\n");
> +             goto err;
> +     }
> +
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_OFFSET_MODE_REG,
> +                                     BU21013_OFFSET_MODE_DEFAULT);
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_OFFSET_MODE reg write
> failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_XY_EDGE_REG,
> +                     (BU21013_X_EDGE_0 | BU21013_X_EDGE_2 |
> +                             BU21013_Y_EDGE_1 | BU21013_Y_EDGE_3));
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_XY_EDGE reg write failed\n");
> +             goto err;
> +     }
> +     retval = i2c_smbus_write_byte_data(i2c, BU21013_DONE_REG,
> +                                                     BU21013_DONE);
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "BU21013_REG_DONE reg write failed\n");
> +             goto err;
> +     }
> +
> +     data->x_pos[0] = 0;
> +     data->x_pos[1] = 0;
> +     data->y_pos[0] = 0;
> +     data->y_pos[1] = 0;
> +     data->previous_press_reported = 0;
> +
> +err:
> +     return retval;
> +}
> +
> +#ifdef CONFIG_PM
> +/**
> + * bu21013_suspend() - suspend the touch screen controller
> + * @client: pointer to i2c client structure
> + * @mesg: message from power manager
> + *
> + * This funtion is used to suspend the
> + * touch panel controller and returns integer
> + */
> +static int bu21013_suspend(struct i2c_client *client, pm_message_t
> mesg)
> +{
> +     struct bu21013_ts_data *bu21013_data =
> i2c_get_clientdata(client);
> +
> +     bu21013_data->touch_stopped = true;
> +     wake_up(&bu21013_data->wait);
> +     if (device_may_wakeup(&client->dev))
> +             enable_irq_wake(bu21013_data->chip->irq);
> +     else
> +             disable_irq(bu21013_data->chip->irq);
> +
> +     return 0;
> +}
> +
> +/**
> + * bu21013_resume() - resume the touch screen controller
> + * @client: pointer to i2c client structure
> + *
> + * This funtion is used to resume the touch panel
> + * controller and returns integer.
> + */
> +static int bu21013_resume(struct i2c_client *client)
> +{
> +     int retval;
> +     struct bu21013_ts_data *bu21013_data =
> i2c_get_clientdata(client);
> +
> +     retval = bu21013_init_chip(bu21013_data);
> +     if (retval < 0) {
> +             dev_err(&client->dev, "tsc config failed\n");
> +             return retval;
> +     }
> +
> +     bu21013_data->touch_stopped = false;
> +     if (device_may_wakeup(&client->dev))
> +             disable_irq_wake(bu21013_data->chip->irq);
> +     else
> +             enable_irq(bu21013_data->chip->irq);
> +
> +     return 0;
> +}
> +#endif
> +
> +/**
> + * bu21013_probe() - initialzes the i2c-client touchscreen driver
> + * @i2c: i2c client structure pointer
> + * @id:i2c device id pointer
> + *
> + * This function used to initializes the i2c-client touchscreen
> + * driver and returns integer.
> + */
> +static int __devinit bu21013_probe(struct i2c_client *i2c,
> +                                     const struct i2c_device_id *id)
> +{
> +     int retval;
> +     struct bu21013_ts_data *bu21013_data;
> +     struct input_dev *in_dev;
> +     short x_max;
> +     short y_max;
> +     struct bu21013_platform_device *pdata = i2c->dev.platform_data;
> +
> +     if (!i2c) {
> +             dev_err(&i2c->dev, "i2c client not defined\n");
> +             retval = -EINVAL;
> +             return retval;
> +     }
> +
> +     if (!pdata) {
> +             dev_err(&i2c->dev, "platform data not defined\n");
> +             retval = -EINVAL;
> +             return retval;
> +     }
> +
> +     bu21013_data = kzalloc(sizeof(struct bu21013_ts_data),
> GFP_KERNEL);
> +     if (!bu21013_data) {
> +             dev_err(&i2c->dev, "device memory alloc failed\n");
> +             retval = -ENOMEM;
> +             return retval;
> +     }
> +
> +     /* allocate input device */
> +     in_dev = input_allocate_device();
> +     if (!in_dev) {
> +             dev_err(&i2c->dev, "input device memory alloc failed\n");
> +             retval = -ENOMEM;
> +             goto err_alloc;
> +     }
> +     bu21013_data->in_dev = in_dev;
> +
> +     bu21013_data->chip = pdata;
> +     bu21013_data->client = i2c;
> +     bu21013_data->touch_stopped = false;
> +
> +     x_max = pdata->x_max_res;
> +     y_max = pdata->y_max_res;
> +
> +     init_waitqueue_head(&bu21013_data->wait);
> +
> +     i2c_set_clientdata(i2c, bu21013_data);
> +
> +     /* configure the gpio pins */
> +     if (pdata->cs_en) {
> +             retval = pdata->cs_en(pdata->cs_pin);
> +             if (retval < 0) {
> +                     dev_err(&i2c->dev, "chip init failed\n");
> +                     goto err_init_cs;
> +             }
> +     }
> +
> +     /* configure the touch panel controller */
> +     retval = bu21013_init_chip(bu21013_data);
> +     if (retval < 0) {
> +             dev_err(&i2c->dev, "error in bu21013 config\n");
> +             goto err_init_config;
> +     }
> +
> +     /* register the device to input subsystem */
> +     in_dev->name = DRIVER_TP;
> +     in_dev->id.bustype = BUS_I2C;
> +     in_dev->dev.parent = &i2c->dev;
> +     __set_bit(EV_SYN, in_dev->evbit);
> +     __set_bit(EV_KEY, in_dev->evbit);
> +     __set_bit(EV_ABS, in_dev->evbit);
> +     __set_bit(BTN_TOUCH, in_dev->keybit);
> +
> +     input_set_abs_params(in_dev, ABS_X, 0, x_max, 0, 0);
> +     input_set_abs_params(in_dev, ABS_Y, 0, y_max, 0, 0);
> +     input_set_abs_params(in_dev, ABS_PRESSURE, 0, MAX_PRESSURE, 0,
> 0);
> +     input_set_abs_params(in_dev, ABS_TOOL_WIDTH, 0, MAX_TOOL_WIDTH,
> 0, 0);
> +
> +     if (pdata->multi_touch) {
> +             input_set_abs_params(in_dev, ABS_MT_POSITION_X, 0, x_max,
> +                                                                     0, 0);
> +             input_set_abs_params(in_dev, ABS_MT_POSITION_Y, 0, y_max,
> +                                                                     0, 0);
> +             input_set_abs_params(in_dev, ABS_MT_TOUCH_MAJOR, 0,
> +                                             MAX_TOUCH_MAJOR, 0, 0);
> +             input_set_abs_params(in_dev, ABS_MT_TOUCH_MINOR, 0,
> +                                             MAX_TOUCH_MINOR, 0, 0);
> +             input_set_abs_params(in_dev, ABS_MT_ORIENTATION, 0, 1, 0,
> 0);
> +             input_set_abs_params(in_dev, ABS_MT_WIDTH_MAJOR, 0,
> +                                             MAX_TOOL_WIDTH, 0, 0);
> +     }
> +     retval = input_register_device(in_dev);
> +     if (retval)
> +             goto err_input_register;
> +
> +     retval = request_threaded_irq(pdata->irq, NULL, bu21013_gpio_irq,
> +                                     (IRQF_TRIGGER_FALLING | IRQF_SHARED),
> +                                     DRIVER_TP, bu21013_data);
> +     if (retval) {
> +             dev_err(&i2c->dev, "request irq %d failed\n", pdata->irq);
> +             goto err_init_irq;
> +     }
> +
> +     return retval;
> +
> +err_init_irq:
> +     input_unregister_device(bu21013_data->in_dev);
> +     bu21013_data->in_dev = NULL;
> +err_input_register:
> +err_init_config:
> +     pdata->cs_dis(pdata->cs_pin);
> +err_init_cs:
> +     input_free_device(bu21013_data->in_dev);
> +err_alloc:
> +     kfree(bu21013_data);
> +
> +     return retval;
> +}
> +/**
> + * bu21013_remove() - removes the i2c-client touchscreen driver
> + * @client: i2c client structure pointer
> + *
> + * This function uses to remove the i2c-client
> + * touchscreen driver and returns integer.
> + */
> +static int __devexit bu21013_remove(struct i2c_client *client)
> +{
> +     struct bu21013_ts_data *data = i2c_get_clientdata(client);
> +
> +     data->touch_stopped = true;
> +     free_irq(data->chip->irq, data);
> +     data->chip->cs_dis(data->chip->cs_pin);
> +     input_unregister_device(data->in_dev);
> +     kfree(data);
> +
> +     return 0;
> +}
> +
> +static const struct i2c_device_id bu21013_id[] = {
> +     { DRIVER_TP, 0 },
> +     { }
> +};
> +MODULE_DEVICE_TABLE(i2c, bu21013_id);
> +
> +static struct i2c_driver bu21013_driver = {
> +     .driver = {
> +             .name   =       DRIVER_TP,
> +             .owner  =       THIS_MODULE,
> +     },
> +     .probe          =       bu21013_probe,
> +#ifdef CONFIG_PM
> +     .suspend        =       bu21013_suspend,
> +     .resume         =       bu21013_resume,
> +#endif
> +     .remove         =       __devexit_p(bu21013_remove),
> +     .id_table       =       bu21013_id,
> +};
> +
> +/**
> + * bu21013_init() - initializes the bu21013 touchscreen driver
> + *
> + * This function used to initializes the bu21013
> + * touchscreen driver and returns integer.
> + */
> +static int __init bu21013_init(void)
> +{
> +     return i2c_add_driver(&bu21013_driver);
> +}
> +
> +/**
> + * bu21013_exit() - de-initializes the bu21013 touchscreen driver
> + *
> + * This function uses to de-initializes the bu21013
> + * touchscreen driver and returns none.
> + */
> +static void __exit bu21013_exit(void)
> +{
> +     i2c_del_driver(&bu21013_driver);
> +}
> +
> +module_init(bu21013_init);
> +module_exit(bu21013_exit);
> +
> +MODULE_LICENSE("GPL v2");
> +MODULE_AUTHOR("NAVEEN KUMAR G<naveen.gaddipati@stericsson.com");
> +MODULE_DESCRIPTION("bu21013 touch screen controller driver");
> diff --git a/include/linux/input/bu21013.h
> b/include/linux/input/bu21013.h
> new file mode 100644
> index 0000000..6c64bda
> --- /dev/null
> +++ b/include/linux/input/bu21013.h
> @@ -0,0 +1,44 @@
> +/*
> + * Copyright (C) ST-Ericsson SA 2010
> + * Author: Naveen Kumar G <naveen.gaddipati@stericsson.com> for ST-
> Ericsson
> + * License terms:GNU General Public License (GPL) version 2
> + */
> +
> +#ifndef _BU21013_H
> +#define _BU21013_H
> +
> +/**
> + * struct bu21013_platform_device - Handle the platform data
> + * @cs_en:   pointer to the cs enable function
> + * @cs_dis:  pointer to the cs disable function
> + * @irq_read_val:    pointer to read the pen irq value function
> + * @x_max_res: xmax resolution
> + * @y_max_res: ymax resolution
> + * @touch_x_max: touch x max
> + * @touch_y_max: touch y max
> + * @cs_pin: chip select pin
> + * @irq: irq pin
> + * @ext_clk: external clock flag
> + * @x_flip: x flip flag
> + * @y_flip: y flip flag
> + * @multi_touch: multi_touch flag
> + *
> + * This is used to handle the platform data
> + */
> +struct bu21013_platform_device {
> +     int (*cs_en)(int reset_pin);
> +     int (*cs_dis)(int reset_pin);
> +     int (*irq_read_val)(void);
> +     int x_max_res;
> +     int y_max_res;
> +     int touch_x_max;
> +     int touch_y_max;
> +     unsigned int cs_pin;
> +     unsigned int irq;
> +     bool ext_clk;
> +     bool x_flip;
> +     bool y_flip;
> +     bool multi_touch;
> +};
> +
> +#endif
> --
> 1.7.2.dirty
>
> Thanks & Regards,
> Naveen

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2010-09-13 14:37 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-09 11:36 [Patch v2] input:rohm based bu21013 touch panel controller driver support Naveen Kumar GADDIPATI
2010-09-09 12:10 ` Trilok Soni
2010-09-09 13:20   ` Naveen Kumar GADDIPATI
2010-09-09 13:46     ` Trilok Soni
2010-09-09 15:39       ` Dmitry Torokhov
2010-09-09 18:44         ` Trilok Soni
2010-09-09 15:59 ` Dmitry Torokhov
2010-09-09 16:01   ` Dmitry Torokhov
2010-09-09 18:26     ` Henrik Rydberg
2010-09-13 10:12       ` Naveen Kumar GADDIPATI
2010-09-13 14:36         ` Henrik Rydberg
2010-09-09 12:12 Naveen Kumar GADDIPATI
2010-09-09 12:13 ` Trilok Soni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.