From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: Naveen Kumar GADDIPATI <naveen.gaddipati@stericsson.com>
Cc: Trilok Soni <tsoni@codeaurora.org>,
"linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
STEricsson_nomadik_linux <STEricsson_nomadik_linux@list.st.com>,
Henrik Rydberg <rydberg@euromail.se>
Subject: Re: [Patch v2] input:rohm based bu21013 touch panel controller driver support
Date: Thu, 9 Sep 2010 09:01:40 -0700 [thread overview]
Message-ID: <20100909160140.GC19937@core.coreip.homeip.net> (raw)
In-Reply-To: <20100909155918.GB19937@core.coreip.homeip.net>
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
next prev parent reply other threads:[~2010-09-09 16:01 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
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 [this message]
2010-09-09 18:26 ` Henrik Rydberg
2010-09-13 10:12 ` Naveen Kumar GADDIPATI
2010-09-13 14:36 ` Henrik Rydberg
-- strict thread matches above, loose matches on Subject: below --
2010-09-09 12:12 Naveen Kumar GADDIPATI
2010-09-09 12:13 ` Trilok Soni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100909160140.GC19937@core.coreip.homeip.net \
--to=dmitry.torokhov@gmail.com \
--cc=STEricsson_nomadik_linux@list.st.com \
--cc=linux-input@vger.kernel.org \
--cc=naveen.gaddipati@stericsson.com \
--cc=rydberg@euromail.se \
--cc=tsoni@codeaurora.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).