RE: [patch 03/22] input: AD7879 Touchscreen driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dmitry,

Is there a schedule when this and my other patch get merged mainline?

[patch 01/22] input/touchscreen driver: add support AD7877 touchscreen
driver

Do you have any concerns or is there something that should be done
differently?

Best regards,
Michael 


>From: akpm@xxxxxxxxxxxxxxxxxxxx [mailto:akpm@xxxxxxxxxxxxxxxxxxxx]
>Subject: [patch 03/22] input: AD7879 Touchscreen driver
>
>From: Michael Hennerich <michael.hennerich@xxxxxxxxxx>
>
>[randy.dunlap@xxxxxxxxxx: touchscreen/ad787x: don't use bus_id]
>Signed-off-by: Michael Hennerich <michael.hennerich@xxxxxxxxxx>
>Signed-off-by: Bryan Wu <cooloney@xxxxxxxxxx>
>Cc: Dmitry Torokhov <dtor@xxxxxxx>
>Cc: Michael Hennerich <michael.hennerich@xxxxxxxxxx>
>Signed-off-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
>Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
>---
>
> drivers/input/touchscreen/Kconfig  |   32 +
> drivers/input/touchscreen/Makefile |    1
> drivers/input/touchscreen/ad7877.c |    6
> drivers/input/touchscreen/ad7879.c |  797 +++++++++++++++++++++++++++
> include/linux/spi/ad7879.h         |   35 +
> 5 files changed, 868 insertions(+), 3 deletions(-)
>
>diff -puN
drivers/input/touchscreen/Kconfig~input-ad7879-touchscreen-driver
>drivers/input/touchscreen/Kconfig
>--- a/drivers/input/touchscreen/Kconfig~input-ad7879-touchscreen-driver
>+++ a/drivers/input/touchscreen/Kconfig
>@@ -42,6 +42,38 @@ config TOUCHSCREEN_AD7877
> 	  To compile this driver as a module, choose M here: the
> 	  module will be called ad7877.
>
>+config TOUCHSCREEN_AD7879_I2C
>+	tristate "AD7879 based touchscreens: AD7879-1 I2C Interface"
>+	depends on I2C
>+	select TOUCHSCREEN_AD7879
>+	help
>+	  Say Y here if you have a touchscreen interface using the
>+	  AD7879-1 controller, and your board-specific initialization
>+	  code includes that in its table of I2C devices.
>+
>+	  If unsure, say N (but it's safe to say "Y").
>+
>+	  To compile this driver as a module, choose M here: the
>+	  module will be called ad7879.
>+
>+config TOUCHSCREEN_AD7879_SPI
>+	tristate "AD7879 based touchscreens: AD7879 SPI Interface"
>+	depends on SPI_MASTER && TOUCHSCREEN_AD7879_I2C = n
>+	select TOUCHSCREEN_AD7879
>+	help
>+	  Say Y here if you have a touchscreen interface using the
>+	  AD7879 controller, and your board-specific initialization
>+	  code includes that in its table of SPI devices.
>+
>+	  If unsure, say N (but it's safe to say "Y").
>+
>+	  To compile this driver as a module, choose M here: the
>+	  module will be called ad7879.
>+
>+config TOUCHSCREEN_AD7879
>+	tristate
>+	default n
>+
> config TOUCHSCREEN_BITSY
> 	tristate "Compaq iPAQ H3600 (Bitsy) touchscreen"
> 	depends on SA1100_BITSY
>diff -puN drivers/input/touchscreen/Makefile~input-ad7879-touchscreen-
>driver drivers/input/touchscreen/Makefile
>---
a/drivers/input/touchscreen/Makefile~input-ad7879-touchscreen-driver
>+++ a/drivers/input/touchscreen/Makefile
>@@ -7,6 +7,7 @@
> wm97xx-ts-y := wm97xx-core.o
>
> obj-$(CONFIG_TOUCHSCREEN_AD7877)	+= ad7877.o
>+obj-$(CONFIG_TOUCHSCREEN_AD7879)	+= ad7879.o
> obj-$(CONFIG_TOUCHSCREEN_ADS7846)	+= ads7846.o
> obj-$(CONFIG_TOUCHSCREEN_ATMEL_TSADCC)	+= atmel_tsadcc.o
> obj-$(CONFIG_TOUCHSCREEN_BITSY)		+= h3600_ts_input.o
>diff -puN drivers/input/touchscreen/ad7877.c~input-ad7879-touchscreen-
>driver drivers/input/touchscreen/ad7877.c
>---
a/drivers/input/touchscreen/ad7877.c~input-ad7879-touchscreen-driver
>+++ a/drivers/input/touchscreen/ad7877.c
>@@ -713,7 +713,7 @@ static int __devinit ad7877_probe(struct
> 	ts->averaging = pdata->averaging;
> 	ts->pen_down_acc_interval = pdata->pen_down_acc_interval;
>
>-	snprintf(ts->phys, sizeof(ts->phys), "%s/inputX",
spi->dev.bus_id);
>+	snprintf(ts->phys, sizeof(ts->phys), "%s/inputX", dev_name(&spi-
>>dev));
>
> 	input_dev->name = "AD7877 Touchscreen";
> 	input_dev->phys = ts->phys;
>@@ -740,8 +740,8 @@ static int __devinit ad7877_probe(struct
> 	verify = ad7877_read(spi, AD7877_REG_SEQ1);
>
> 	if (verify != AD7877_MM_SEQUENCE){
>-		dev_err(&spi->dev, "%s: Failed to probe %s\n",
spi->dev.bus_id,
>-			 input_dev->name);
>+		dev_err(&spi->dev, "%s: Failed to probe %s\n",
>+			dev_name(&spi->dev), input_dev->name);
> 		err = -ENODEV;
> 		goto err_free_mem;
> 	}
>diff -puN /dev/null drivers/input/touchscreen/ad7879.c
>--- /dev/null
>+++ a/drivers/input/touchscreen/ad7879.c
>@@ -0,0 +1,797 @@
>+/*
>+ * File:        drivers/input/touchscreen/ad7879.c
>+ *
>+ *		Copyright (C) 2008 Michael Hennerich, Analog Devices
Inc.
>+ *
>+ * Description:	AD7879 based touchscreen, and GPIO driver
(I2C/SPI
>Interface)
>+ *
>+ * Bugs:        Enter bugs at http://blackfin.uclinux.org/
>+ *
>+ * This program is free software; you can redistribute it and/or
modify
>+ * it under the terms of the GNU General Public License as published
by
>+ * the Free Software Foundation; either version 2 of the License, or
>+ * (at your option) any later version.
>+ *
>+ * This program is distributed in the hope that it will be useful,
>+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
>+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>+ * GNU General Public License for more details.
>+ *
>+ * You should have received a copy of the GNU General Public License
>+ * along with this program; if not, see the file COPYING, or write
>+ * to the Free Software Foundation, Inc.,
>+ * 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
>+ *
>+ * History:
>+ * Copyright (c) 2005 David Brownell
>+ * Copyright (c) 2006 Nokia Corporation
>+ * Various changes: Imre Deak <imre.deak@xxxxxxxxx>
>+ *
>+ * Using code from:
>+ *  - corgi_ts.c
>+ *	Copyright (C) 2004-2005 Richard Purdie
>+ *  - omap_ts.[hc], ads7846.h, ts_osk.c
>+ *	Copyright (C) 2002 MontaVista Software
>+ *	Copyright (C) 2004 Texas Instruments
>+ *	Copyright (C) 2005 Dirk Behme
>+ *  - ad7877.c
>+ * 	Copyright (C) 2006-2008 Analog Devices Inc.
>+ */
>+
>+#include <linux/device.h>
>+#include <linux/init.h>
>+#include <linux/delay.h>
>+#include <linux/input.h>
>+#include <linux/interrupt.h>
>+#include <linux/irq.h>
>+#include <linux/slab.h>
>+#include <linux/workqueue.h>
>+#include <linux/spi/spi.h>
>+#include <linux/i2c.h>
>+
>+#include <linux/spi/ad7879.h>
>+
>+#define AD7879_REG_ZEROS		0
>+#define AD7879_REG_CTRL1		1
>+#define AD7879_REG_CTRL2		2
>+#define AD7879_REG_CTRL3		3
>+#define AD7879_REG_AUX1HIGH		4
>+#define AD7879_REG_AUX1LOW		5
>+#define AD7879_REG_TEMP1HIGH		6
>+#define AD7879_REG_TEMP1LOW		7
>+#define AD7879_REG_XPLUS		8
>+#define AD7879_REG_YPLUS		9
>+#define AD7879_REG_Z1			10
>+#define AD7879_REG_Z2			11
>+#define AD7879_REG_AUXVBAT		12
>+#define AD7879_REG_TEMP			13
>+#define AD7879_REG_REVID		14
>+
>+/* Control REG 1 */
>+#define AD7879_TMR(x)			((x & 0xFF) << 0)
>+#define AD7879_ACQ(x)			((x & 0x3) << 8)
>+#define AD7879_MODE_NOC  		(0 << 10)	/* Do not
convert */
>+#define AD7879_MODE_SCC  		(1 << 10)	/* Single
channel
>conversion */
>+#define AD7879_MODE_SEQ0 		(2 << 10)	/* Sequence 0 in
Slave Mode
>*/
>+#define AD7879_MODE_SEQ1 		(3 << 10)	/* Sequence 1 in
Master
>Mode */
>+#define AD7879_MODE_INT 		(1 << 15)	/* PENIRQ
disabled INT
>enabled */
>+
>+/* Control REG 2 */
>+#define AD7879_FCD(x)			((x & 0x3) << 0)
>+#define AD7879_RESET			(1 << 4)
>+#define AD7879_MFS(x)			((x & 0x3) << 5)
>+#define AD7879_AVG(x)			((x & 0x3) << 7)
>+#define	AD7879_SER			(1 << 9)	/*
non-differential */
>+#define	AD7879_DFR			(0 << 9)	/*
differential */
>+#define AD7879_GPIOPOL			(1 << 10)
>+#define AD7879_GPIODIR			(1 << 11)
>+#define AD7879_GPIO_DATA		(1 << 12)
>+#define AD7879_GPIO_EN			(1 << 13)
>+#define AD7879_PM(x)			((x & 0x3) << 14)
>+#define AD7879_PM_SHUTDOWN		(0)
>+#define AD7879_PM_DYN			(1)
>+#define AD7879_PM_FULLON		(2)
>+
>+/* Control REG 3 */
>+#define AD7879_TEMPMASK_BIT		(1<<15)
>+#define AD7879_AUXVBATMASK_BIT		(1<<14)
>+#define AD7879_INTMODE_BIT		(1<<13)
>+#define AD7879_GPIOALERTMASK_BIT	(1<<12)
>+#define AD7879_AUXLOW_BIT		(1<<11)
>+#define AD7879_AUXHIGH_BIT		(1<<10)
>+#define AD7879_TEMPLOW_BIT		(1<<9)
>+#define AD7879_TEMPHIGH_BIT		(1<<8)
>+#define AD7879_YPLUS_BIT		(1<<7)
>+#define AD7879_XPLUS_BIT		(1<<6)
>+#define AD7879_Z1_BIT			(1<<5)
>+#define AD7879_Z2_BIT			(1<<4)
>+#define AD7879_AUX_BIT			(1<<3)
>+#define AD7879_VBAT_BIT			(1<<2)
>+#define AD7879_TEMP_BIT			(1<<1)
>+
>+enum {
>+	AD7879_SEQ_XPOS  = 0,
>+	AD7879_SEQ_YPOS  = 1,
>+	AD7879_SEQ_Z1    = 2,
>+	AD7879_SEQ_Z2    = 3,
>+	AD7879_NR_SENSE  = 4,
>+};
>+
>+#define	MAX_12BIT			((1<<12)-1)
>+#define	TS_PEN_UP_TIMEOUT		msecs_to_jiffies(50)
>+
>+#if defined(CONFIG_TOUCHSCREEN_AD7879_SPI) ||
>defined(CONFIG_TOUCHSCREEN_AD7879_SPI_MODULE)
>+#define AD7879_DEVID		0x7A
>+typedef struct spi_device	bus_device;
>+#elif defined(CONFIG_TOUCHSCREEN_AD7879_I2C) ||
>defined(CONFIG_TOUCHSCREEN_AD7879_I2C_MODULE)
>+#define AD7879_DEVID		0x79
>+typedef struct i2c_client	bus_device;
>+#endif
>+
>+struct ad7879 {
>+	bus_device 		*bus;
>+	struct input_dev	*input;
>+	struct work_struct	work;
>+	struct timer_list	timer;
>+	spinlock_t		lock;
>+
>+#if defined(CONFIG_TOUCHSCREEN_AD7879_SPI) ||
>defined(CONFIG_TOUCHSCREEN_AD7879_SPI_MODULE)
>+	struct spi_message	msg;
>+	struct spi_transfer	xfer[AD7879_NR_SENSE + 1];
>+	u16			cmd;
>+#endif
>+	u16 			conversion_data[AD7879_NR_SENSE];
>+	char			phys[32];
>+	u8			first_conversion_delay;
>+	u8			acquisition_time;
>+	u8			averaging;
>+	u8			pen_down_acc_interval;
>+	u8			median;
>+	u16			x_plate_ohms;
>+	u16			pressure_max;
>+	u16			gpio_init;
>+	u16			cmd_crtl1;
>+	u16			cmd_crtl2;
>+	u16			cmd_crtl3;
>+	unsigned		disabled:1;	/* P: lock */
>+	unsigned		gpio:1;
>+};
>+
>+static int ad7879_read(bus_device *, u8);
>+static int ad7879_write(bus_device *, u8, u16);
>+static void ad7879_collect(struct ad7879 *);
>+
>+static void ad7879_report(struct ad7879 *ts)
>+{
>+	struct input_dev	*input_dev = ts->input;
>+	unsigned		Rt;
>+	u16			x, y, z1, z2;
>+
>+	x = ts->conversion_data[AD7879_SEQ_XPOS] & MAX_12BIT;
>+	y = ts->conversion_data[AD7879_SEQ_YPOS] & MAX_12BIT;
>+	z1 = ts->conversion_data[AD7879_SEQ_Z1] & MAX_12BIT;
>+	z2 = ts->conversion_data[AD7879_SEQ_Z2] & MAX_12BIT;
>+
>+	/*
>+	 * The samples processed here are already preprocessed by the
AD7879.
>+	 * The preprocessing function consists of a median and an
averaging
>filter.
>+	 * The combination of these two techniques provides a robust
>solution,
>+	 * discarding the spurious noise in the signal and keeping only
the
>data of interest.
>+	 * The size of both filters is programmable. (dev.platform_data,
see
>linux/spi/ad7879.h)
>+	 * Other user-programmable conversion controls include variable
>acquisition time,
>+	 * and first conversion delay. Up to 16 averages can be taken
per
>conversion.
>+	 */
>+
>+	if (likely(x && z1)) {
>+		/* compute touch pressure resistance using equation #1
*/
>+		Rt = (z2 - z1) * x * ts->x_plate_ohms;
>+		Rt /= z1;
>+		Rt = (Rt + 2047) >> 12;
>+	} else
>+		Rt = 0;
>+
>+	if (Rt) {
>+		input_report_abs(input_dev, ABS_X, x);
>+		input_report_abs(input_dev, ABS_Y, y);
>+		input_report_abs(input_dev, ABS_PRESSURE, Rt);
>+		input_sync(input_dev);
>+	}
>+}
>+
>+static void ad7879_work(struct work_struct *work)
>+{
>+	struct ad7879 *ts = container_of(work, struct ad7879, work);
>+
>+	/* use keventd context to read the result registers */
>+	ad7879_collect(ts);
>+	ad7879_report(ts);
>+	mod_timer(&ts->timer, jiffies + TS_PEN_UP_TIMEOUT);
>+}
>+
>+static void ad7879_ts_event_release(struct ad7879 *ts)
>+{
>+	struct input_dev *input_dev = ts->input;
>+
>+	input_report_abs(input_dev, ABS_PRESSURE, 0);
>+	input_sync(input_dev);
>+}
>+
>+static void ad7879_timer(unsigned long handle)
>+{
>+	struct ad7879	*ts = (void *)handle;
>+
>+	ad7879_ts_event_release(ts);
>+}
>+
>+static irqreturn_t ad7879_irq(int irq, void *handle)
>+{
>+	struct ad7879 *ts = handle;
>+
>+	/* The repeated conversion sequencer controlled by TMR kicked
off too
>fast.
>+	 * We ignore the last and process the sample sequence currently
in
>the queue.
>+	 * It can't be older than 9.4ms
>+	 */
>+
>+	if (!work_pending(&ts->work))
>+		schedule_work(&ts->work);
>+
>+	return IRQ_HANDLED;
>+}
>+
>+static void ad7879_disable(struct ad7879 *ts)
>+{
>+	unsigned long flags;
>+
>+	spin_lock_irqsave(&ts->lock, flags);
>+	if (ts->disabled) {
>+		spin_unlock_irqrestore(&ts->lock, flags);
>+		return;
>+	}
>+
>+	ts->disabled = 1;
>+	disable_irq(ts->bus->irq);
>+	spin_unlock_irqrestore(&ts->lock, flags);
>+
>+	cancel_work_sync(&ts->work);
>+
>+	if (del_timer_sync(&ts->timer))
>+		ad7879_ts_event_release(ts);
>+
>+	/* we know the chip's in lowpower mode since we always
>+	 * leave it that way after every request
>+	 */
>+}
>+
>+static void ad7879_enable(struct ad7879 *ts)
>+{
>+	unsigned long flags;
>+
>+	spin_lock_irqsave(&ts->lock, flags);
>+	if (ts->disabled) {
>+		spin_unlock_irqrestore(&ts->lock, flags);
>+		return;
>+	}
>+	ts->disabled = 0;
>+	enable_irq(ts->bus->irq);
>+	spin_unlock_irqrestore(&ts->lock, flags);
>+}
>+
>+static ssize_t ad7879_disable_show(struct device *dev,
>+				     struct device_attribute *attr, char
*buf)
>+{
>+	struct ad7879 *ts = dev_get_drvdata(dev);
>+
>+	return sprintf(buf, "%u\n", ts->disabled);
>+}
>+
>+static ssize_t ad7879_disable_store(struct device *dev,
>+				     struct device_attribute *attr,
>+				     const char *buf, size_t count)
>+{
>+	struct ad7879 *ts = dev_get_drvdata(dev);
>+	unsigned long val;
>+	int ret;
>+
>+	ret = strict_strtoul(buf, 10, &val);
>+
>+	if (ret)
>+		return ret;
>+
>+	if (val)
>+		ad7879_disable(ts);
>+	else
>+		ad7879_enable(ts);
>+
>+	return count;
>+}
>+
>+static DEVICE_ATTR(disable, 0664, ad7879_disable_show,
>ad7879_disable_store);
>+
>+static ssize_t ad7879_gpio_show(struct device *dev,
>+				     struct device_attribute *attr, char
*buf)
>+{
>+	struct ad7879 *ts = dev_get_drvdata(dev);
>+
>+	return sprintf(buf, "%u\n", ts->gpio);
>+}
>+
>+static ssize_t ad7879_gpio_store(struct device *dev,
>+				     struct device_attribute *attr,
>+				     const char *buf, size_t count)
>+{
>+	struct ad7879 *ts = dev_get_drvdata(dev);
>+	unsigned long val;
>+	int ret;
>+
>+	ret = strict_strtoul(buf, 10, &val);
>+	if (ret)
>+		return ret;
>+
>+	ts->gpio = !!val;
>+
>+	ret = ad7879_write(ts->bus, AD7879_REG_CTRL2,
>+			ts->gpio ? ts->cmd_crtl2 & ~AD7879_GPIO_DATA
>+			: ts->cmd_crtl2 | AD7879_GPIO_DATA);
>+
>+	if (ret)
>+		return ret;
>+
>+	return count;
>+}
>+
>+static DEVICE_ATTR(gpio, 0664, ad7879_gpio_show, ad7879_gpio_store);
>+
>+static struct attribute *ad7879_attributes[] = {
>+	&dev_attr_disable.attr,
>+	&dev_attr_gpio.attr,
>+	NULL
>+};
>+
>+static const struct attribute_group ad7879_attr_group = {
>+	.attrs = ad7879_attributes,
>+};
>+
>+static void ad7879_setup(bus_device *bus, struct ad7879 *ts)
>+{
>+	ts->cmd_crtl3 = AD7879_YPLUS_BIT |
>+			AD7879_XPLUS_BIT |
>+			AD7879_Z2_BIT |
>+			AD7879_Z1_BIT |
>+			AD7879_TEMPMASK_BIT |
>+			AD7879_AUXVBATMASK_BIT |
>+			AD7879_GPIOALERTMASK_BIT;
>+
>+	ts->cmd_crtl2 = AD7879_PM(AD7879_PM_DYN) | AD7879_DFR |
>+			AD7879_AVG(ts->averaging) |
>+			AD7879_MFS(ts->median) |
>+			AD7879_FCD(ts->first_conversion_delay) |
>+			ts->gpio_init;
>+
>+	ts->cmd_crtl1 = AD7879_MODE_INT | AD7879_MODE_SEQ1 |
>+			AD7879_ACQ(ts->acquisition_time) |
>+			AD7879_TMR(ts->pen_down_acc_interval);
>+
>+	ad7879_write(bus, AD7879_REG_CTRL2, ts->cmd_crtl2);
>+	ad7879_write(bus, AD7879_REG_CTRL3, ts->cmd_crtl3);
>+	ad7879_write(bus, AD7879_REG_CTRL1, ts->cmd_crtl1);
>+}
>+
>+static int __devinit ad7879_construct(bus_device *bus, struct ad7879
*ts)
>+{
>+	struct input_dev *input_dev;
>+	struct ad7879_platform_data *pdata = bus->dev.platform_data;
>+	int err;
>+	u16 revid;
>+
>+	if (!bus->irq) {
>+		dev_err(&bus->dev, "no IRQ?\n");
>+		return -ENODEV;
>+	}
>+
>+	if (!pdata) {
>+		dev_err(&bus->dev, "no platform data?\n");
>+		return -ENODEV;
>+	}
>+
>+	input_dev = input_allocate_device();
>+	if (!input_dev)
>+		return -ENOMEM;
>+
>+	ts->input = input_dev;
>+
>+	setup_timer(&ts->timer, ad7879_timer, (unsigned long) ts);
>+	INIT_WORK(&ts->work, ad7879_work);
>+	spin_lock_init(&ts->lock);
>+
>+	ts->x_plate_ohms = pdata->x_plate_ohms ? : 400;
>+	ts->pressure_max = pdata->pressure_max ? : ~0;
>+
>+	ts->first_conversion_delay = pdata->first_conversion_delay;
>+	ts->acquisition_time = pdata->acquisition_time;
>+	ts->averaging = pdata->averaging;
>+	ts->pen_down_acc_interval = pdata->pen_down_acc_interval;
>+	ts->median = pdata->median;
>+
>+	if (pdata->gpio_output)
>+		ts->gpio_init = AD7879_GPIO_EN |
>+				(pdata->gpio_default ? 0 :
AD7879_GPIO_DATA);
>+	else
>+		ts->gpio_init = AD7879_GPIO_EN | AD7879_GPIODIR;
>+
>+	snprintf(ts->phys, sizeof(ts->phys), "%s/inputX", dev_name(&bus-
>>dev));
>+
>+	input_dev->name = "AD7879 Touchscreen";
>+	input_dev->phys = ts->phys;
>+	input_dev->dev.parent = &bus->dev;
>+
>+	__set_bit(EV_ABS, input_dev->evbit);
>+	__set_bit(ABS_X, input_dev->absbit);
>+	__set_bit(ABS_Y, input_dev->absbit);
>+	__set_bit(ABS_PRESSURE, input_dev->absbit);
>+
>+	input_set_abs_params(input_dev, ABS_X,
>+			pdata->x_min ? : 0,
>+			pdata->x_max ? : MAX_12BIT,
>+			0, 0);
>+	input_set_abs_params(input_dev, ABS_Y,
>+			pdata->y_min ? : 0,
>+			pdata->y_max ? : MAX_12BIT,
>+			0, 0);
>+	input_set_abs_params(input_dev, ABS_PRESSURE,
>+			pdata->pressure_min, pdata->pressure_max, 0, 0);
>+
>+	err = ad7879_write(bus, AD7879_REG_CTRL2, AD7879_RESET);
>+
>+	if (err < 0) {
>+		dev_err(&bus->dev, "Failed to write %s\n",
input_dev->name);
>+		goto err_free_mem;
>+	}
>+
>+	revid = ad7879_read(bus, AD7879_REG_REVID);
>+
>+	if ((revid & 0xFF) != AD7879_DEVID) {
>+		dev_err(&bus->dev, "Failed to probe %s\n",
input_dev->name);
>+		err = -ENODEV;
>+		goto err_free_mem;
>+	}
>+
>+	ad7879_setup(bus, ts);
>+
>+	err = request_irq(bus->irq, ad7879_irq, IRQF_TRIGGER_FALLING |
>+		IRQF_SAMPLE_RANDOM, bus->dev.driver->name, ts);
>+
>+	if (err) {
>+		dev_err(&bus->dev, "irq %d busy?\n", bus->irq);
>+		goto err_free_mem;
>+	}
>+
>+	err = sysfs_create_group(&bus->dev.kobj, &ad7879_attr_group);
>+	if (err)
>+		goto err_free_irq;
>+
>+	err = input_register_device(input_dev);
>+	if (err)
>+		goto err_remove_attr;
>+
>+	dev_info(&bus->dev, "Rev.%d touchscreen, irq %d\n",
>+		revid >> 8, bus->irq);
>+
>+	return 0;
>+
>+err_remove_attr:
>+	sysfs_remove_group(&bus->dev.kobj, &ad7879_attr_group);
>+err_free_irq:
>+	free_irq(bus->irq, ts);
>+err_free_mem:
>+	input_free_device(input_dev);
>+
>+	return err;
>+}
>+
>+static int __devexit ad7879_destroy(bus_device *bus, struct ad7879
*ts)
>+{
>+	ad7879_disable(ts);
>+	ad7879_write(ts->bus, AD7879_REG_CTRL2,
>+			AD7879_PM(AD7879_PM_SHUTDOWN));
>+	sysfs_remove_group(&ts->bus->dev.kobj, &ad7879_attr_group);
>+	free_irq(ts->bus->irq, ts);
>+	input_unregister_device(ts->input);
>+	dev_dbg(&bus->dev, "unregistered touchscreen\n");
>+
>+	return 0;
>+}
>+
>+#ifdef CONFIG_PM
>+static int ad7879_suspend(bus_device *bus, pm_message_t message)
>+{
>+	struct ad7879 *ts = dev_get_drvdata(&bus->dev);
>+
>+	ad7879_disable(ts);
>+	ad7879_write(bus, AD7879_REG_CTRL2,
>+			AD7879_PM(AD7879_PM_SHUTDOWN));
>+
>+	return 0;
>+}
>+
>+static int ad7879_resume(bus_device *bus)
>+{
>+	struct ad7879 *ts = dev_get_drvdata(&bus->dev);
>+
>+	ad7879_setup(bus, ts);
>+	ad7879_enable(ts);
>+
>+	return 0;
>+}
>+#else
>+#define ad7879_suspend NULL
>+#define ad7879_resume  NULL
>+#endif
>+
>+#if defined(CONFIG_TOUCHSCREEN_AD7879_SPI) ||
>defined(CONFIG_TOUCHSCREEN_AD7879_SPI_MODULE)
>+#define MAX_SPI_FREQ_HZ		5000000
>+#define AD7879_CMD_MAGIC	0xE000
>+#define AD7879_CMD_READ		(1 << 10)
>+#define AD7879_WRITECMD(reg)  	(AD7879_CMD_MAGIC | (reg & 0xF))
>+#define AD7879_READCMD(reg)  	(AD7879_CMD_MAGIC | AD7879_CMD_READ |
>(reg & 0xF))
>+
>+struct ser_req {
>+	u16			command;
>+	u16			data;
>+	struct spi_message	msg;
>+	struct spi_transfer	xfer[2];
>+};
>+
>+/*
>+ * ad7879_read/write are only used for initial setup and for sysfs
>controls.
>+ * The main traffic is done in ad7879_collect().
>+ */
>+
>+static int ad7879_read(struct spi_device *spi, u8 reg)
>+{
>+	struct ser_req *req = kzalloc(sizeof *req, GFP_KERNEL);
>+	int status, ret;
>+
>+	if (!req)
>+		return -ENOMEM;
>+
>+	spi_message_init(&req->msg);
>+
>+	req->command = (u16) AD7879_READCMD(reg);
>+	req->xfer[0].tx_buf = &req->command;
>+	req->xfer[0].len = 2;
>+
>+	req->xfer[1].rx_buf = &req->data;
>+	req->xfer[1].len = 2;
>+
>+	spi_message_add_tail(&req->xfer[0], &req->msg);
>+	spi_message_add_tail(&req->xfer[1], &req->msg);
>+
>+	status = spi_sync(spi, &req->msg);
>+
>+	if (status == 0)
>+		status = req->msg.status;
>+
>+	ret = status ? status : req->data;
>+	kfree(req);
>+
>+	return ret;
>+}
>+
>+static int ad7879_write(struct spi_device *spi, u8 reg, u16 val)
>+{
>+	struct ser_req *req = kzalloc(sizeof *req, GFP_KERNEL);
>+	int status;
>+
>+	if (!req)
>+		return -ENOMEM;
>+
>+	spi_message_init(&req->msg);
>+
>+	req->command = (u16) AD7879_WRITECMD(reg);
>+	req->xfer[0].tx_buf = &req->command;
>+	req->xfer[0].len = 2;
>+
>+	req->data = val;
>+	req->xfer[1].tx_buf = &req->data;
>+	req->xfer[1].len = 2;
>+
>+	spi_message_add_tail(&req->xfer[0], &req->msg);
>+	spi_message_add_tail(&req->xfer[1], &req->msg);
>+
>+	status = spi_sync(spi, &req->msg);
>+
>+	if (status == 0)
>+		status = req->msg.status;
>+
>+	kfree(req);
>+
>+	return status;
>+}
>+
>+static void ad7879_collect(struct ad7879 *ts)
>+{
>+	int status = spi_sync(ts->bus, &ts->msg);
>+	if (status)
>+		dev_err(&ts->bus->dev, "spi_sync --> %d\n", status);
>+}
>+
>+static void ad7879_setup_ts_def_msg(struct ad7879 *ts)
>+{
>+	struct spi_message *m;
>+	int i;
>+
>+	ts->cmd = (u16) AD7879_READCMD(AD7879_REG_XPLUS);
>+
>+	m = &ts->msg;
>+	spi_message_init(m);
>+	ts->xfer[0].tx_buf = &ts->cmd;
>+	ts->xfer[0].len = 2;
>+
>+	spi_message_add_tail(&ts->xfer[0], m);
>+
>+	for (i = 0; i < AD7879_NR_SENSE; i++) {
>+		ts->xfer[i + 1].rx_buf = &ts->conversion_data[i];
>+		ts->xfer[i + 1].len = 2;
>+		spi_message_add_tail(&ts->xfer[i + 1], m);
>+	}
>+}
>+
>+static int __devinit ad7879_probe(struct spi_device *spi)
>+{
>+	struct ad7879 *ts;
>+	int ret;
>+
>+	/* don't exceed max specified SPI CLK frequency */
>+	if (spi->max_speed_hz > MAX_SPI_FREQ_HZ) {
>+		dev_err(&spi->dev, "SPI CLK %d Hz?\n",
spi->max_speed_hz);
>+		return -EINVAL;
>+	}
>+
>+	ts = kzalloc(sizeof(struct ad7879), GFP_KERNEL);
>+	if (!ts)
>+		return -ENOMEM;
>+
>+	dev_set_drvdata(&spi->dev, ts);
>+	ts->bus = spi;
>+
>+	ad7879_setup_ts_def_msg(ts);
>+
>+	ret = ad7879_construct(spi, ts);
>+	if (!ret)
>+		return ret;
>+
>+	dev_set_drvdata(&spi->dev, NULL);
>+	kfree(ts);
>+
>+	return ret;
>+}
>+
>+static int __devexit ad7879_remove(struct spi_device *spi)
>+{
>+	struct ad7879 *ts = dev_get_drvdata(&spi->dev);
>+
>+	ad7879_destroy(spi, ts);
>+	dev_set_drvdata(&spi->dev, NULL);
>+	kfree(ts);
>+	return 0;
>+}
>+
>+static struct spi_driver ad7879_driver = {
>+	.driver = {
>+		.name	= "ad7879",
>+		.bus	= &spi_bus_type,
>+		.owner	= THIS_MODULE,
>+	},
>+	.probe		= ad7879_probe,
>+	.remove		= __devexit_p(ad7879_remove),
>+	.suspend	= ad7879_suspend,
>+	.resume		= ad7879_resume,
>+};
>+
>+static int __init ad7879_init(void)
>+{
>+	return spi_register_driver(&ad7879_driver);
>+}
>+module_init(ad7879_init);
>+
>+static void __exit ad7879_exit(void)
>+{
>+	spi_unregister_driver(&ad7879_driver);
>+}
>+module_exit(ad7879_exit);
>+
>+#elif defined(CONFIG_TOUCHSCREEN_AD7879_I2C) ||
>defined(CONFIG_TOUCHSCREEN_AD7879_I2C_MODULE)
>+
>+/* All registers are word-sized.
>+ * AD7879 uses a high-byte first convention.
>+ */
>+static int ad7879_read(struct i2c_client *client, u8 reg)
>+{
>+	return swab16(i2c_smbus_read_word_data(client, reg));
>+}
>+
>+static int ad7879_write(struct i2c_client *client, u8 reg, u16 val)
>+{
>+	return i2c_smbus_write_word_data(client, reg, swab16(val));
>+}
>+
>+static void ad7879_collect(struct ad7879 *ts)
>+{
>+	int i;
>+	for (i = 0; i < AD7879_NR_SENSE; i++)
>+		ts->conversion_data[i] =
>+			ad7879_read(ts->bus, AD7879_REG_XPLUS + i);
>+}
>+
>+static int __devinit ad7879_probe(struct i2c_client *client,
>+					const struct i2c_device_id *id)
>+{
>+	struct ad7879 *ts;
>+	int ret;
>+
>+	if (!i2c_check_functionality(client->adapter,
>+					I2C_FUNC_SMBUS_WORD_DATA)) {
>+		dev_err(&client->dev, "SMBUS Word Data not
Supported\n");
>+		return -EIO;
>+	}
>+
>+	ts = kzalloc(sizeof(struct ad7879), GFP_KERNEL);
>+	if (!ts)
>+		return -ENOMEM;
>+
>+	i2c_set_clientdata(client, ts);
>+	ts->bus = client;
>+
>+	ret = ad7879_construct(client, ts);
>+	if (!ret)
>+		return ret;
>+
>+	i2c_set_clientdata(client, NULL);
>+	kfree(ts);
>+
>+	return ret;
>+}
>+
>+static int __devexit ad7879_remove(struct i2c_client *client)
>+{
>+	struct ad7879 *ts = dev_get_drvdata(&client->dev);
>+
>+	ad7879_destroy(client, ts);
>+	i2c_set_clientdata(client, NULL);
>+	kfree(ts);
>+	return 0;
>+}
>+static const struct i2c_device_id ad7979_id[] = {
>+	{ "ad7879", 0 },
>+	{ }
>+};
>+MODULE_DEVICE_TABLE(i2c, ad7979_id);
>+
>+static struct i2c_driver ad7879_driver = {
>+	.driver = {
>+		.name	= "ad7879",
>+		.owner	= THIS_MODULE,
>+	},
>+	.probe		= ad7879_probe,
>+	.remove		= __devexit_p(ad7879_remove),
>+	.suspend	= ad7879_suspend,
>+	.resume		= ad7879_resume,
>+	.id_table 	= ad7979_id,
>+};
>+
>+static int __init ad7879_init(void)
>+{
>+	return i2c_add_driver(&ad7879_driver);
>+}
>+module_init(ad7879_init);
>+
>+static void __exit ad7879_exit(void)
>+{
>+	i2c_del_driver(&ad7879_driver);
>+}
>+module_exit(ad7879_exit);
>+#endif
>+
>+MODULE_AUTHOR("Michael Hennerich <hennerich@xxxxxxxxxxxxxxxxxxxx>");
>+MODULE_DESCRIPTION("AD7879(-1) touchscreen Driver");
>+MODULE_LICENSE("GPL");
>diff -puN /dev/null include/linux/spi/ad7879.h
>--- /dev/null
>+++ a/include/linux/spi/ad7879.h
>@@ -0,0 +1,35 @@
>+/* linux/spi/ad7879.h */
>+
>+/* Touchscreen characteristics vary between boards and models.  The
>+ * platform_data for the device's "struct device" holds this
information.
>+ *
>+ * It's OK if the min/max values are zero.
>+ */
>+struct ad7879_platform_data {
>+	u16	model;			/* 7879 */
>+	u16	x_plate_ohms;
>+	u16	x_min, x_max;
>+	u16	y_min, y_max;
>+	u16	pressure_min, pressure_max;
>+
>+	/* [0..255] 0=OFF Starts at 1=550us and goes
>+	 * all the way to 9.440ms in steps of 35us.
>+	 */
>+	u8	pen_down_acc_interval;
>+	/* [0..15] Starts at 0=128us and goes all the
>+	 * way to 4.096ms in steps of 128us.
>+	 */
>+	u8	first_conversion_delay;
>+	/* [0..3] 0 = 2us, 1 = 4us, 2 = 8us, 3 = 16us */
>+	u8	acquisition_time;
>+	/* [0..3] Average X middle samples 0 = 2, 1 = 4, 2 = 8, 3 = 16
*/
>+	u8	averaging;
>+	/* [0..3] Perform X measurements 0 = OFF,
>+	 * 1 = 4, 2 = 8, 3 = 16 (median > averaging)
>+	 */
>+	u8	median;
>+	/* 1 = AUX/VBAT/GPIO set to GPIO Output */
>+	u8	gpio_output;
>+	/* Initial GPIO pin state (valid if gpio_output = 1) */
>+	u8	gpio_default;
>+};
>_
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux