From patchwork Tue May 10 11:35:05 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Hennerich, Michael" X-Patchwork-Id: 774102 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.3) with ESMTP id p4ABii0o014173 for ; Tue, 10 May 2011 11:44:44 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755861Ab1EJLoi (ORCPT ); Tue, 10 May 2011 07:44:38 -0400 Received: from tx2ehsobe001.messaging.microsoft.com ([65.55.88.11]:42588 "EHLO TX2EHSOBE001.bigfish.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755389Ab1EJLog (ORCPT ); Tue, 10 May 2011 07:44:36 -0400 Received: from mail30-tx2-R.bigfish.com (10.9.14.237) by TX2EHSOBE001.bigfish.com (10.9.40.21) with Microsoft SMTP Server id 14.1.225.8; Tue, 10 May 2011 11:44:35 +0000 Received: from mail30-tx2 (localhost.localdomain [127.0.0.1]) by mail30-tx2-R.bigfish.com (Postfix) with ESMTP id 91DF64B011B; Tue, 10 May 2011 11:44:35 +0000 (UTC) X-SpamScore: 0 X-BigFish: VPS0(zzzz1202hzz8275bhz2dh2a8h668h839h62h) X-Spam-TCS-SCL: 1:0 X-Forefront-Antispam-Report: KIP:(null); UIP:(null); IPVD:NLI; H:NWD2EDG1.analog.com; RD:none; EFVD:NLI Received: from mail30-tx2 (localhost.localdomain [127.0.0.1]) by mail30-tx2 (MessageSwitch) id 130502787517173_17375; Tue, 10 May 2011 11:44:35 +0000 (UTC) Received: from TX2EHSMHS020.bigfish.com (unknown [10.9.14.247]) by mail30-tx2.bigfish.com (Postfix) with ESMTP id E939E13F8051; Tue, 10 May 2011 11:44:34 +0000 (UTC) Received: from NWD2EDG1.analog.com (137.71.25.53) by TX2EHSMHS020.bigfish.com (10.9.99.120) with Microsoft SMTP Server (TLS) id 14.1.225.8; Tue, 10 May 2011 11:44:31 +0000 Received: from NWD2HUBCAS1.ad.analog.com (10.64.73.29) by NWD2EDG1.analog.com (137.71.25.53) with Microsoft SMTP Server (TLS) id 8.1.240.5; Tue, 10 May 2011 07:43:59 -0400 Received: from zeus.spd.analog.com (10.64.82.11) by NWD2HUBCAS1.ad.analog.com (10.64.73.29) with Microsoft SMTP Server id 8.1.358.0; Tue, 10 May 2011 07:44:27 -0400 Received: from localhost.localdomain ([10.44.2.88]) by zeus.spd.analog.com (8.14.1/8.14.1) with ESMTP id p4ABiPtG023644; Tue, 10 May 2011 07:44:26 -0400 (EDT) From: To: CC: , , , , Michael Hennerich Subject: [PATCH 1/5] input: misc: AD714x: The interrupt status registers should be read in row. Date: Tue, 10 May 2011 13:35:05 +0200 Message-ID: <1305027309-17637-1-git-send-email-michael.hennerich@analog.com> X-Mailer: git-send-email 1.6.0.2 MIME-Version: 1.0 X-OriginatorOrg: analog.com Sender: linux-input-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Tue, 10 May 2011 11:44:45 +0000 (UTC) From: Michael Hennerich The interrupt status registers should be read in row to avoid invalid data. Implement read sequence function for both bus options. Always use it for reading the interrupt status registers. Read sequence saves 50% of bus transactions compared to single register reads. So use it also for the result registers, which are also located in row. Update copyright notice. Signed-off-by: Michael Hennerich --- drivers/input/misc/ad714x-i2c.c | 30 ++++++++++++++++++++++++++++-- drivers/input/misc/ad714x-spi.c | 14 ++++++++++++-- drivers/input/misc/ad714x.c | 35 +++++++++++++++++------------------ drivers/input/misc/ad714x.h | 7 +++++-- 4 files changed, 62 insertions(+), 24 deletions(-) -- 1.6.0.2 -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/input/misc/ad714x-i2c.c b/drivers/input/misc/ad714x-i2c.c index e21deb1..fb6564e 100644 --- a/drivers/input/misc/ad714x-i2c.c +++ b/drivers/input/misc/ad714x-i2c.c @@ -1,7 +1,7 @@ /* * AD714X CapTouch Programmable Controller driver (I2C bus) * - * Copyright 2009 Analog Devices Inc. + * Copyright 2009-2011 Analog Devices Inc. * * Licensed under the GPL-2 or later. */ @@ -77,13 +77,39 @@ static int ad714x_i2c_read(struct device *dev, unsigned short reg, return ret; } +static int ad714x_i2c_read_seq(struct device *dev, unsigned short reg, + unsigned short *data, unsigned len) +{ + struct i2c_client *client = to_i2c_client(dev); + int ret = 0, i; + u8 *_reg = (u8 *)® + + u8 tx[2] = { + _reg[1], + _reg[0] + }; + + ret = i2c_master_send(client, tx, 2); + if (ret >= 0) + ret = i2c_master_recv(client, (u8 *)data, len * 2); + + if (unlikely(ret < 0)) + dev_err(&client->dev, "I2C read error\n"); + else + for (i = 0; i < len; i++) + data[i] = be16_to_cpu(data[i]); + + return ret; +} + static int __devinit ad714x_i2c_probe(struct i2c_client *client, const struct i2c_device_id *id) { struct ad714x_chip *chip; chip = ad714x_probe(&client->dev, BUS_I2C, client->irq, - ad714x_i2c_read, ad714x_i2c_write); + ad714x_i2c_read, ad714x_i2c_read_seq, + ad714x_i2c_write); if (IS_ERR(chip)) return PTR_ERR(chip); diff --git a/drivers/input/misc/ad714x-spi.c b/drivers/input/misc/ad714x-spi.c index 4120dd5..a0cb115 100644 --- a/drivers/input/misc/ad714x-spi.c +++ b/drivers/input/misc/ad714x-spi.c @@ -1,7 +1,7 @@ /* * AD714X CapTouch Programmable Controller driver (SPI bus) * - * Copyright 2009 Analog Devices Inc. + * Copyright 2009-2011 Analog Devices Inc. * * Licensed under the GPL-2 or later. */ @@ -39,6 +39,15 @@ static int ad714x_spi_read(struct device *dev, unsigned short reg, return spi_write_then_read(spi, (u8 *)&tx, 2, (u8 *)data, 2); } +static int ad714x_spi_read_seq(struct device *dev, unsigned short reg, + unsigned short *data, unsigned len) +{ + struct spi_device *spi = to_spi_device(dev); + unsigned short tx = AD714x_SPI_CMD_PREFIX | AD714x_SPI_READ | reg; + + return spi_write_then_read(spi, (u8 *)&tx, 2, (u8 *)data, len * 2); +} + static int ad714x_spi_write(struct device *dev, unsigned short reg, unsigned short data) { @@ -56,7 +65,8 @@ static int __devinit ad714x_spi_probe(struct spi_device *spi) struct ad714x_chip *chip; chip = ad714x_probe(&spi->dev, BUS_SPI, spi->irq, - ad714x_spi_read, ad714x_spi_write); + ad714x_spi_read, ad714x_spi_read_seq, + ad714x_spi_write); if (IS_ERR(chip)) return PTR_ERR(chip); diff --git a/drivers/input/misc/ad714x.c b/drivers/input/misc/ad714x.c index c431d09..eb352fb 100644 --- a/drivers/input/misc/ad714x.c +++ b/drivers/input/misc/ad714x.c @@ -1,7 +1,7 @@ /* * AD714X CapTouch Programmable Controller driver supporting AD7142/3/7/8/7A * - * Copyright 2009 Analog Devices Inc. + * Copyright 2009-2011 Analog Devices Inc. * * Licensed under the GPL-2 or later. */ @@ -131,8 +131,8 @@ struct ad714x_driver_data { * of spi/i2c device */ struct ad714x_chip { - unsigned short h_state; unsigned short l_state; + unsigned short h_state; unsigned short c_state; unsigned short adc_reg[STAGE_NUM]; unsigned short amb_reg[STAGE_NUM]; @@ -144,6 +144,7 @@ struct ad714x_chip { int irq; struct device *dev; ad714x_read_t read; + ad714x_read_seq_t read_seq; ad714x_write_t write; struct mutex mutex; @@ -279,9 +280,11 @@ static void ad714x_slider_cal_sensor_val(struct ad714x_chip *ad714x, int idx) struct ad714x_slider_plat *hw = &ad714x->hw->slider[idx]; int i; + ad714x->read_seq(ad714x->dev, CDC_RESULT_S0 + hw->start_stage, + &ad714x->adc_reg[hw->start_stage], + hw->end_stage - hw->start_stage + 1); + for (i = hw->start_stage; i <= hw->end_stage; i++) { - ad714x->read(ad714x->dev, CDC_RESULT_S0 + i, - &ad714x->adc_reg[i]); ad714x->read(ad714x->dev, STAGE0_AMBIENT + i * PER_STAGE_REG_NUM, &ad714x->amb_reg[i]); @@ -451,9 +454,11 @@ static void ad714x_wheel_cal_sensor_val(struct ad714x_chip *ad714x, int idx) struct ad714x_wheel_plat *hw = &ad714x->hw->wheel[idx]; int i; + ad714x->read_seq(ad714x->dev, CDC_RESULT_S0 + hw->start_stage, + &ad714x->adc_reg[hw->start_stage], + hw->end_stage - hw->start_stage + 1); + for (i = hw->start_stage; i <= hw->end_stage; i++) { - ad714x->read(ad714x->dev, CDC_RESULT_S0 + i, - &ad714x->adc_reg[i]); ad714x->read(ad714x->dev, STAGE0_AMBIENT + i * PER_STAGE_REG_NUM, &ad714x->amb_reg[i]); @@ -1025,9 +1030,7 @@ static void ad714x_hw_init(struct ad714x_chip *ad714x) ad714x->write(ad714x->dev, AD714X_STG_CAL_EN_REG, 0xFFF); /* clear all interrupts */ - ad714x->read(ad714x->dev, STG_LOW_INT_STA_REG, &data); - ad714x->read(ad714x->dev, STG_HIGH_INT_STA_REG, &data); - ad714x->read(ad714x->dev, STG_COM_INT_STA_REG, &data); + ad714x->read_seq(ad714x->dev, STG_LOW_INT_STA_REG, &ad714x->l_state, 3); } static irqreturn_t ad714x_interrupt_thread(int irq, void *data) @@ -1037,9 +1040,7 @@ static irqreturn_t ad714x_interrupt_thread(int irq, void *data) mutex_lock(&ad714x->mutex); - ad714x->read(ad714x->dev, STG_LOW_INT_STA_REG, &ad714x->l_state); - ad714x->read(ad714x->dev, STG_HIGH_INT_STA_REG, &ad714x->h_state); - ad714x->read(ad714x->dev, STG_COM_INT_STA_REG, &ad714x->c_state); + ad714x->read_seq(ad714x->dev, STG_LOW_INT_STA_REG, &ad714x->l_state, 3); for (i = 0; i < ad714x->hw->button_num; i++) ad714x_button_state_machine(ad714x, i); @@ -1057,7 +1058,8 @@ static irqreturn_t ad714x_interrupt_thread(int irq, void *data) #define MAX_DEVICE_NUM 8 struct ad714x_chip *ad714x_probe(struct device *dev, u16 bus_type, int irq, - ad714x_read_t read, ad714x_write_t write) + ad714x_read_t read, ad714x_read_seq_t read_seq, + ad714x_write_t write) { int i, alloc_idx; int error; @@ -1110,6 +1112,7 @@ struct ad714x_chip *ad714x_probe(struct device *dev, u16 bus_type, int irq, drv_mem += sizeof(*bt_drv) * ad714x->hw->button_num; ad714x->read = read; + ad714x->read_seq = read_seq; ad714x->write = write; ad714x->irq = irq; ad714x->dev = dev; @@ -1316,8 +1319,6 @@ EXPORT_SYMBOL(ad714x_disable); int ad714x_enable(struct ad714x_chip *ad714x) { - unsigned short data; - dev_dbg(ad714x->dev, "%s enter\n", __func__); mutex_lock(&ad714x->mutex); @@ -1331,9 +1332,7 @@ int ad714x_enable(struct ad714x_chip *ad714x) * otherwise we will get no chance to enter falling-edge irq again */ - ad714x->read(ad714x->dev, STG_LOW_INT_STA_REG, &data); - ad714x->read(ad714x->dev, STG_HIGH_INT_STA_REG, &data); - ad714x->read(ad714x->dev, STG_COM_INT_STA_REG, &data); + ad714x->read_seq(ad714x->dev, STG_LOW_INT_STA_REG, &ad714x->l_state, 3); mutex_unlock(&ad714x->mutex); diff --git a/drivers/input/misc/ad714x.h b/drivers/input/misc/ad714x.h index 45c54fb..58d5e4e 100644 --- a/drivers/input/misc/ad714x.h +++ b/drivers/input/misc/ad714x.h @@ -1,7 +1,7 @@ /* * AD714X CapTouch Programmable Controller driver (bus interfaces) * - * Copyright 2009 Analog Devices Inc. + * Copyright 2009-2011 Analog Devices Inc. * * Licensed under the GPL-2 or later. */ @@ -15,12 +15,15 @@ struct device; struct ad714x_chip; typedef int (*ad714x_read_t)(struct device *, unsigned short, unsigned short *); +typedef int (*ad714x_read_seq_t)(struct device *, unsigned short, + unsigned short *, unsigned len); typedef int (*ad714x_write_t)(struct device *, unsigned short, unsigned short); int ad714x_disable(struct ad714x_chip *ad714x); int ad714x_enable(struct ad714x_chip *ad714x); struct ad714x_chip *ad714x_probe(struct device *dev, u16 bus_type, int irq, - ad714x_read_t read, ad714x_write_t write); + ad714x_read_t read, ad714x_read_seq_t read_seq, + ad714x_write_t write); void ad714x_remove(struct ad714x_chip *ad714x); #endif