lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANLzEktyaLeXw_CJPTZnvOO1EA=4AKcyRVrR6GE9YdjSYFYkMg@mail.gmail.com>
Date:	Thu, 13 Dec 2012 14:46:44 -0800
From:	Benson Leung <bleung@...omium.org>
To:	Henrik Rydberg <rydberg@...omail.se>
Cc:	khali@...ux-fr.org, Dmitry Torokhov <dmitry.torokhov@...il.com>,
	linux-input@...r.kernel.org,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Daniel Kurtz <djkurtz@...omium.org>,
	Olof Johansson <olofj@...omium.org>,
	Dudley Du <dudl@...ress.com>
Subject: Re: [PATCH v3 1/1] Input: add driver for Cypress APA I2C Trackpad

Thanks Henrik.

Hello Dmitry, do you have any comments on V3?

On Sat, Dec 8, 2012 at 4:42 AM, Henrik Rydberg <rydberg@...omail.se> wrote:
> On Fri, Dec 07, 2012 at 03:24:16PM -0800, Benson Leung wrote:
>> This patch introduces a driver for Cypress All Points Addressable
>> I2C Trackpad, including the ones in 2012 Samsung Chromebooks.
>>
>> This device is compatible with MT protocol type B, providing identifiable
>> contacts.
>>
>> Signed-off-by: Dudley Du <dudl@...ress.com>
>> Signed-off-by: Daniel Kurtz <djkurtz@...omium.org>
>> Signed-off-by: Benson Leung <bleung@...omium.org>
>> ---
>> Version history :
>> v3 : * Handle pointer emulation and unused slots in core
>>      * Fix compile fail on 3.7 for input_mt_init_slots (Thanks Rydberg)
>>      * Set BUTTONPAD property only if is a buttonpad based on capabilities.
>>      * Removed __devinit/__devexit/__devexit_p
>>      * Set to OFF power mode on cyapa_remove
>>      * Got rid of kasprintf'd physical location string for static one.
>>
>> v2 : * Removed firmware update.
>>      * Removed sysfs properties related to firmware update and power mode.
>>      * Folded cyapa_detect into cyapa_probe.
>>      * Added support for middle and right mechanical buttons, if they exist.
>>      * Rearranged disable_irq/enable_irq in suspend and resume to prevent
>>          a power mode change from colliding with a read of tracking data.
>>      * Made cyapa_get_state more reliable.
>>      * Use IRQF_ONESHOT for threaded irq
>>      * Simplified cyapa_set_power_mode.
>>      * Removed extra kernel-doc style comments
>>      * Removed dev_dbg messages.
>>      * Cleaned up unused includes.
>>      * Cleaned up unused #defines
>>
>> v1 : Initial
>> ---
>>  drivers/input/mouse/Kconfig  |   12 +
>>  drivers/input/mouse/Makefile |    1 +
>>  drivers/input/mouse/cyapa.c  |  803 ++++++++++++++++++++++++++++++++++++++++++
>>  3 files changed, 816 insertions(+), 0 deletions(-)
>>  create mode 100644 drivers/input/mouse/cyapa.c
>>
>> diff --git a/drivers/input/mouse/Kconfig b/drivers/input/mouse/Kconfig
>> index cd6268c..23db30a 100644
>> --- a/drivers/input/mouse/Kconfig
>> +++ b/drivers/input/mouse/Kconfig
>> @@ -193,6 +193,18 @@ config MOUSE_BCM5974
>>         To compile this driver as a module, choose M here: the
>>         module will be called bcm5974.
>>
>> +config MOUSE_CYAPA
>> +     tristate "Cypress APA I2C Trackpad support"
>> +     depends on I2C
>> +     help
>> +       This driver adds support for Cypress All Points Addressable (APA)
>> +       I2C Trackpads, including the ones used in 2012 Samsung Chromebooks.
>> +
>> +       Say Y here if you have a Cypress APA I2C Trackpad.
>> +
>> +       To compile this driver as a module, choose M here: the module will be
>> +       called cyapa.
>> +
>>  config MOUSE_INPORT
>>       tristate "InPort/MS/ATIXL busmouse"
>>       depends on ISA
>> diff --git a/drivers/input/mouse/Makefile b/drivers/input/mouse/Makefile
>> index 46ba755..10b4773 100644
>> --- a/drivers/input/mouse/Makefile
>> +++ b/drivers/input/mouse/Makefile
>> @@ -8,6 +8,7 @@ obj-$(CONFIG_MOUSE_AMIGA)             += amimouse.o
>>  obj-$(CONFIG_MOUSE_APPLETOUCH)               += appletouch.o
>>  obj-$(CONFIG_MOUSE_ATARI)            += atarimouse.o
>>  obj-$(CONFIG_MOUSE_BCM5974)          += bcm5974.o
>> +obj-$(CONFIG_MOUSE_CYAPA)            += cyapa.o
>>  obj-$(CONFIG_MOUSE_GPIO)             += gpio_mouse.o
>>  obj-$(CONFIG_MOUSE_INPORT)           += inport.o
>>  obj-$(CONFIG_MOUSE_LOGIBM)           += logibm.o
>> diff --git a/drivers/input/mouse/cyapa.c b/drivers/input/mouse/cyapa.c
>> new file mode 100644
>> index 0000000..0964583
>> --- /dev/null
>> +++ b/drivers/input/mouse/cyapa.c
>> @@ -0,0 +1,803 @@
>> +/*
>> + * Cypress APA trackpad with I2C interface
>> + *
>> + * Author: Dudley Du <dudl@...ress.com>
>> + * Further cleanup and restructuring by:
>> + *   Daniel Kurtz <djkurtz@...omium.org>
>> + *   Benson Leung <bleung@...omium.org>
>> + *
>> + * Copyright (C) 2011-2012 Cypress Semiconductor, Inc.
>> + * Copyright (C) 2011-2012 Google, Inc.
>> + *
>> + * This file is subject to the terms and conditions of the GNU General Public
>> + * License.  See the file COPYING in the main directory of this archive for
>> + * more details.
>> + */
>> +
>> +#include <linux/delay.h>
>> +#include <linux/i2c.h>
>> +#include <linux/input.h>
>> +#include <linux/input/mt.h>
>> +#include <linux/interrupt.h>
>> +#include <linux/module.h>
>> +#include <linux/slab.h>
>> +
>> +/* APA trackpad firmware generation */
>> +#define CYAPA_GEN3   0x03   /* support MT-protocol B with tracking ID. */
>> +
>> +#define CYAPA_NAME   "Cypress APA Trackpad (cyapa)"
>> +
>> +/* commands for read/write registers of Cypress trackpad */
>> +#define CYAPA_CMD_SOFT_RESET       0x00
>> +#define CYAPA_CMD_POWER_MODE       0x01
>> +#define CYAPA_CMD_DEV_STATUS       0x02
>> +#define CYAPA_CMD_GROUP_DATA       0x03
>> +#define CYAPA_CMD_GROUP_CMD        0x04
>> +#define CYAPA_CMD_GROUP_QUERY      0x05
>> +#define CYAPA_CMD_BL_STATUS        0x06
>> +#define CYAPA_CMD_BL_HEAD          0x07
>> +#define CYAPA_CMD_BL_CMD           0x08
>> +#define CYAPA_CMD_BL_DATA          0x09
>> +#define CYAPA_CMD_BL_ALL           0x0a
>> +#define CYAPA_CMD_BLK_PRODUCT_ID   0x0b
>> +#define CYAPA_CMD_BLK_HEAD         0x0c
>> +
>> +/* report data start reg offset address. */
>> +#define DATA_REG_START_OFFSET  0x0000
>> +
>> +#define BL_HEAD_OFFSET 0x00
>> +#define BL_DATA_OFFSET 0x10
>> +
>> +/*
>> + * Operational Device Status Register
>> + *
>> + * bit 7: Valid interrupt source
>> + * bit 6 - 4: Reserved
>> + * bit 3 - 2: Power status
>> + * bit 1 - 0: Device status
>> + */
>> +#define REG_OP_STATUS     0x00
>> +#define OP_STATUS_SRC     0x80
>> +#define OP_STATUS_POWER   0x0c
>> +#define OP_STATUS_DEV     0x03
>> +#define OP_STATUS_MASK (OP_STATUS_SRC | OP_STATUS_POWER | OP_STATUS_DEV)
>> +
>> +/*
>> + * Operational Finger Count/Button Flags Register
>> + *
>> + * bit 7 - 4: Number of touched finger
>> + * bit 3: Valid data
>> + * bit 2: Middle Physical Button
>> + * bit 1: Right Physical Button
>> + * bit 0: Left physical Button
>> + */
>> +#define REG_OP_DATA1       0x01
>> +#define OP_DATA_VALID      0x08
>> +#define OP_DATA_MIDDLE_BTN 0x04
>> +#define OP_DATA_RIGHT_BTN  0x02
>> +#define OP_DATA_LEFT_BTN   0x01
>> +#define OP_DATA_BTN_MASK (OP_DATA_MIDDLE_BTN | OP_DATA_RIGHT_BTN | \
>> +                       OP_DATA_LEFT_BTN)
>> +
>> +/*
>> + * Bootloader Status Register
>> + *
>> + * bit 7: Busy
>> + * bit 6 - 5: Reserved
>> + * bit 4: Bootloader running
>> + * bit 3 - 1: Reserved
>> + * bit 0: Checksum valid
>> + */
>> +#define REG_BL_STATUS        0x01
>> +#define BL_STATUS_BUSY       0x80
>> +#define BL_STATUS_RUNNING    0x10
>> +#define BL_STATUS_DATA_VALID 0x08
>> +#define BL_STATUS_CSUM_VALID 0x01
>> +
>> +/*
>> + * Bootloader Error Register
>> + *
>> + * bit 7: Invalid
>> + * bit 6: Invalid security key
>> + * bit 5: Bootloading
>> + * bit 4: Command checksum
>> + * bit 3: Flash protection error
>> + * bit 2: Flash checksum error
>> + * bit 1 - 0: Reserved
>> + */
>> +#define REG_BL_ERROR         0x02
>> +#define BL_ERROR_INVALID     0x80
>> +#define BL_ERROR_INVALID_KEY 0x40
>> +#define BL_ERROR_BOOTLOADING 0x20
>> +#define BL_ERROR_CMD_CSUM    0x10
>> +#define BL_ERROR_FLASH_PROT  0x08
>> +#define BL_ERROR_FLASH_CSUM  0x04
>> +
>> +#define BL_STATUS_SIZE  3  /* length of bootloader status registers */
>> +#define BLK_HEAD_BYTES 32
>> +
>> +#define PRODUCT_ID_SIZE  16
>> +#define QUERY_DATA_SIZE  27
>> +#define REG_PROTOCOL_GEN_QUERY_OFFSET  20
>> +
>> +#define REG_OFFSET_DATA_BASE     0x0000
>> +#define REG_OFFSET_COMMAND_BASE  0x0028
>> +#define REG_OFFSET_QUERY_BASE    0x002a
>> +
>> +#define CAPABILITY_LEFT_BTN_MASK     (0x01 << 3)
>> +#define CAPABILITY_RIGHT_BTN_MASK    (0x01 << 4)
>> +#define CAPABILITY_MIDDLE_BTN_MASK   (0x01 << 5)
>> +#define CAPABILITY_BTN_MASK  (CAPABILITY_LEFT_BTN_MASK | \
>> +                           CAPABILITY_RIGHT_BTN_MASK | \
>> +                           CAPABILITY_MIDDLE_BTN_MASK)
>> +
>> +#define CYAPA_OFFSET_SOFT_RESET  REG_OFFSET_COMMAND_BASE
>> +
>> +#define REG_OFFSET_POWER_MODE (REG_OFFSET_COMMAND_BASE + 1)
>> +
>> +#define PWR_MODE_MASK   0xfc
>> +#define PWR_MODE_FULL_ACTIVE (0x3f << 2)
>> +#define PWR_MODE_IDLE        (0x05 << 2) /* default sleep time is 50 ms. */
>> +#define PWR_MODE_OFF         (0x00 << 2)
>> +
>> +#define PWR_STATUS_MASK      0x0c
>> +#define PWR_STATUS_ACTIVE    (0x03 << 2)
>> +#define PWR_STATUS_IDLE      (0x02 << 2)
>> +#define PWR_STATUS_OFF       (0x00 << 2)
>> +
>> +/*
>> + * CYAPA trackpad device states.
>> + * Used in register 0x00, bit1-0, DeviceStatus field.
>> + * Other values indicate device is in an abnormal state and must be reset.
>> + */
>> +#define CYAPA_DEV_NORMAL  0x03
>> +#define CYAPA_DEV_BUSY    0x01
>> +
>> +enum cyapa_state {
>> +     CYAPA_STATE_OP,
>> +     CYAPA_STATE_BL_IDLE,
>> +     CYAPA_STATE_BL_ACTIVE,
>> +     CYAPA_STATE_BL_BUSY,
>> +     CYAPA_STATE_NO_DEVICE,
>> +};
>> +
>> +
>> +struct cyapa_touch {
>> +     /*
>> +      * high bits or x/y position value
>> +      * bit 7 - 4: high 4 bits of x position value
>> +      * bit 3 - 0: high 4 bits of y position value
>> +      */
>> +     u8 xy_hi;
>> +     u8 x_lo;  /* low 8 bits of x position value. */
>> +     u8 y_lo;  /* low 8 bits of y position value. */
>> +     u8 pressure;
>> +     /* id range is 1 - 15.  It is incremented with every new touch. */
>> +     u8 id;
>> +} __packed;
>> +
>> +/* The touch.id is used as the MT slot id, thus max MT slot is 15 */
>> +#define CYAPA_MAX_MT_SLOTS  15
>> +
>> +struct cyapa_reg_data {
>> +     /*
>> +      * bit 0 - 1: device status
>> +      * bit 3 - 2: power mode
>> +      * bit 6 - 4: reserved
>> +      * bit 7: interrupt valid bit
>> +      */
>> +     u8 device_status;
>> +     /*
>> +      * bit 7 - 4: number of fingers currently touching pad
>> +      * bit 3: valid data check bit
>> +      * bit 2: middle mechanism button state if exists
>> +      * bit 1: right mechanism button state if exists
>> +      * bit 0: left mechanism button state if exists
>> +      */
>> +     u8 finger_btn;
>> +     /* CYAPA reports up to 5 touches per packet. */
>> +     struct cyapa_touch touches[5];
>> +} __packed;
>> +
>> +/* The main device structure */
>> +struct cyapa {
>> +     enum cyapa_state state;
>> +
>> +     struct i2c_client *client;
>> +     struct input_dev *input;
>> +     char phys[32];  /* device physical location */
>> +     int irq;
>> +     bool irq_wake;  /* irq wake is enabled */
>> +
>> +     /* read from query data region. */
>> +     char product_id[16];
>> +     u8 btn_capability;
>> +     u8 gen;
>> +     int max_abs_x;
>> +     int max_abs_y;
>> +     int physical_size_x;
>> +     int physical_size_y;
>> +};
>> +
>> +static const u8 bl_deactivate[] = { 0x00, 0xff, 0x3b, 0x00, 0x01, 0x02, 0x03,
>> +             0x04, 0x05, 0x06, 0x07 };
>> +static const u8 bl_exit[] = { 0x00, 0xff, 0xa5, 0x00, 0x01, 0x02, 0x03, 0x04,
>> +             0x05, 0x06, 0x07 };
>> +
>> +struct cyapa_cmd_len {
>> +     u8 cmd;
>> +     u8 len;
>> +};
>> +
>> +static const struct cyapa_cmd_len cyapa_i2c_cmds[] = {
>> +     {CYAPA_OFFSET_SOFT_RESET, 1},
>> +     {REG_OFFSET_COMMAND_BASE + 1, 1},
>> +     {REG_OFFSET_DATA_BASE, 1},
>> +     {REG_OFFSET_DATA_BASE, sizeof(struct cyapa_reg_data)},
>> +     {REG_OFFSET_COMMAND_BASE, 0},
>> +     {REG_OFFSET_QUERY_BASE, QUERY_DATA_SIZE},
>> +     {BL_HEAD_OFFSET, 3},
>> +     {BL_HEAD_OFFSET, 16},
>> +     {BL_HEAD_OFFSET, 16},
>> +     {BL_DATA_OFFSET, 16},
>> +     {BL_HEAD_OFFSET, 32},
>> +     {REG_OFFSET_QUERY_BASE, PRODUCT_ID_SIZE},
>> +     {REG_OFFSET_DATA_BASE, 32}
>> +};
>> +
>> +static ssize_t cyapa_i2c_reg_read_block(struct cyapa *cyapa, u8 reg, size_t len,
>> +                                     u8 *values)
>> +{
>> +     return i2c_smbus_read_i2c_block_data(cyapa->client, reg, len, values);
>> +}
>> +
>> +static ssize_t cyapa_i2c_reg_write_block(struct cyapa *cyapa, u8 reg,
>> +                                      size_t len, const u8 *values)
>> +{
>> +     return i2c_smbus_write_i2c_block_data(cyapa->client, reg, len, values);
>> +}
>> +
>> +static s32 cyapa_read_byte(struct cyapa *cyapa, u8 cmd_idx)
>> +{
>> +     u8 cmd = cyapa_i2c_cmds[cmd_idx].cmd;
>> +
>> +     return i2c_smbus_read_byte_data(cyapa->client, cmd);
>> +}
>> +
>> +static s32 cyapa_write_byte(struct cyapa *cyapa, u8 cmd_idx, u8 value)
>> +{
>> +     u8 cmd = cyapa_i2c_cmds[cmd_idx].cmd;
>> +
>> +     return i2c_smbus_write_byte_data(cyapa->client, cmd, value);
>> +}
>> +
>> +static ssize_t cyapa_read_block(struct cyapa *cyapa, u8 cmd_idx, u8 *values)
>> +{
>> +     u8 cmd = cyapa_i2c_cmds[cmd_idx].cmd;
>> +     size_t len = cyapa_i2c_cmds[cmd_idx].len;
>> +
>> +     return cyapa_i2c_reg_read_block(cyapa, cmd, len, values);
>> +}
>> +
>> +/*
>> + * Query device for its current operating state.
>> + *
>> + */
>> +static int cyapa_get_state(struct cyapa *cyapa)
>> +{
>> +     int ret;
>> +     u8 status[BL_STATUS_SIZE];
>> +
>> +     cyapa->state = CYAPA_STATE_NO_DEVICE;
>> +
>> +     /*
>> +      * Get trackpad status by reading 3 registers starting from 0.
>> +      * If the device is in the bootloader, this will be BL_HEAD.
>> +      * If the device is in operation mode, this will be the DATA regs.
>> +      *
>> +      */
>> +     ret = cyapa_i2c_reg_read_block(cyapa, BL_HEAD_OFFSET, BL_STATUS_SIZE,
>> +                                    status);
>> +     if (ret != BL_STATUS_SIZE)
>> +             goto error;
>> +
>> +     if ((status[REG_OP_STATUS] & OP_STATUS_SRC) == OP_STATUS_SRC) {
>> +             switch (status[REG_OP_STATUS] & OP_STATUS_DEV) {
>> +             case CYAPA_DEV_NORMAL:
>> +             case CYAPA_DEV_BUSY:
>> +                     cyapa->state = CYAPA_STATE_OP;
>> +                     break;
>> +             default:
>> +                     ret = -EAGAIN;
>> +                     goto error;
>> +             }
>> +     } else {
>> +             if (status[REG_BL_STATUS] & BL_STATUS_BUSY)
>> +                     cyapa->state = CYAPA_STATE_BL_BUSY;
>> +             else if (status[REG_BL_ERROR] & BL_ERROR_BOOTLOADING)
>> +                     cyapa->state = CYAPA_STATE_BL_ACTIVE;
>> +             else
>> +                     cyapa->state = CYAPA_STATE_BL_IDLE;
>> +     }
>> +
>> +     return 0;
>> +error:
>> +     return (ret < 0) ? ret : -EAGAIN;
>> +}
>> +
>> +/*
>> + * Poll device for its status in a loop, waiting up to timeout for a response.
>> + *
>> + * When the device switches state, it usually takes ~300 ms.
>> + * However, when running a new firmware image, the device must calibrate its
>> + * sensors, which can take as long as 2 seconds.
>> + *
>> + * Note: The timeout has granularity of the polling rate, which is 100 ms.
>> + *
>> + * Returns:
>> + *   0 when the device eventually responds with a valid non-busy state.
>> + *   -ETIMEDOUT if device never responds (too many -EAGAIN)
>> + *   < 0    other errors
>> + */
>> +static int cyapa_poll_state(struct cyapa *cyapa, unsigned int timeout)
>> +{
>> +     int ret;
>> +     int tries = timeout / 100;
>> +
>> +     ret = cyapa_get_state(cyapa);
>> +     while ((ret || cyapa->state >= CYAPA_STATE_BL_BUSY) && tries--) {
>> +             msleep(100);
>> +             ret = cyapa_get_state(cyapa);
>> +     }
>> +     return (ret == -EAGAIN || ret == -ETIMEDOUT) ? -ETIMEDOUT : ret;
>> +}
>> +
>> +static int cyapa_bl_deactivate(struct cyapa *cyapa)
>> +{
>> +     int ret;
>> +
>> +     ret = cyapa_i2c_reg_write_block(cyapa, 0, sizeof(bl_deactivate),
>> +                                     bl_deactivate);
>> +     if (ret < 0)
>> +             return ret;
>> +
>> +     /* wait for bootloader to switch to idle state; should take < 100ms */
>> +     msleep(100);
>> +     ret = cyapa_poll_state(cyapa, 500);
>> +     if (ret < 0)
>> +             return ret;
>> +     if (cyapa->state != CYAPA_STATE_BL_IDLE)
>> +             return -EAGAIN;
>> +     return 0;
>> +}
>> +
>> +/*
>> + * Exit bootloader
>> + *
>> + * Send bl_exit command, then wait 50 - 100 ms to let device transition to
>> + * operational mode.  If this is the first time the device's firmware is
>> + * running, it can take up to 2 seconds to calibrate its sensors.  So, poll
>> + * the device's new state for up to 2 seconds.
>> + *
>> + * Returns:
>> + *   -EIO    failure while reading from device
>> + *   -EAGAIN device is stuck in bootloader, b/c it has invalid firmware
>> + *   0       device is supported and in operational mode
>> + */
>> +static int cyapa_bl_exit(struct cyapa *cyapa)
>> +{
>> +     int ret;
>> +
>> +     ret = cyapa_i2c_reg_write_block(cyapa, 0, sizeof(bl_exit), bl_exit);
>> +     if (ret < 0)
>> +             return ret;
>> +
>> +     /*
>> +      * Wait for bootloader to exit, and operation mode to start.
>> +      * Normally, this takes at least 50 ms.
>> +      */
>> +     usleep_range(50000, 100000);
>> +     /*
>> +      * In addition, when a device boots for the first time after being
>> +      * updated to new firmware, it must first calibrate its sensors, which
>> +      * can take up to an additional 2 seconds.
>> +      */
>> +     ret = cyapa_poll_state(cyapa, 2000);
>> +     if (ret < 0)
>> +             return ret;
>> +     if (cyapa->state != CYAPA_STATE_OP)
>> +             return -EAGAIN;
>> +
>> +     return 0;
>> +}
>> +
>> +/*
>> + * Set device power mode
>> + *
>> + */
>> +static int cyapa_set_power_mode(struct cyapa *cyapa, u8 power_mode)
>> +{
>> +     struct device *dev = &cyapa->client->dev;
>> +     int ret;
>> +     u8 power;
>> +
>> +     if (cyapa->state != CYAPA_STATE_OP)
>> +             return 0;
>> +
>> +     ret = cyapa_read_byte(cyapa, CYAPA_CMD_POWER_MODE);
>> +     if (ret < 0)
>> +             return ret;
>> +
>> +     power = ret & ~PWR_MODE_MASK;
>> +     power |= power_mode & PWR_MODE_MASK;
>> +     ret = cyapa_write_byte(cyapa, CYAPA_CMD_POWER_MODE, power);
>> +     if (ret < 0)
>> +             dev_err(dev, "failed to set power_mode 0x%02x err = %d\n",
>> +                     power_mode, ret);
>> +     return ret;
>> +}
>> +
>> +static int cyapa_get_query_data(struct cyapa *cyapa)
>> +{
>> +     u8 query_data[QUERY_DATA_SIZE];
>> +     int ret;
>> +
>> +     if (cyapa->state != CYAPA_STATE_OP)
>> +             return -EBUSY;
>> +
>> +     ret = cyapa_read_block(cyapa, CYAPA_CMD_GROUP_QUERY, query_data);
>> +     if (ret < 0)
>> +             return ret;
>> +     if (ret != QUERY_DATA_SIZE)
>> +             return -EIO;
>> +
>> +     memcpy(&cyapa->product_id[0], &query_data[0], 5);
>> +     cyapa->product_id[5] = '-';
>> +     memcpy(&cyapa->product_id[6], &query_data[5], 6);
>> +     cyapa->product_id[12] = '-';
>> +     memcpy(&cyapa->product_id[13], &query_data[11], 2);
>> +     cyapa->product_id[15] = '\0';
>> +
>> +     cyapa->btn_capability = query_data[19] & CAPABILITY_BTN_MASK;
>> +
>> +     cyapa->gen = query_data[20] & 0x0f;
>> +
>> +     cyapa->max_abs_x = ((query_data[21] & 0xf0) << 4) | query_data[22];
>> +     cyapa->max_abs_y = ((query_data[21] & 0x0f) << 8) | query_data[23];
>> +
>> +     cyapa->physical_size_x =
>> +             ((query_data[24] & 0xf0) << 4) | query_data[25];
>> +     cyapa->physical_size_y =
>> +             ((query_data[24] & 0x0f) << 8) | query_data[26];
>> +
>> +     return 0;
>> +}
>> +
>> +/*
>> + * Check if device is operational.
>> + *
>> + * An operational device is responding, has exited bootloader, and has
>> + * firmware supported by this driver.
>> + *
>> + * Returns:
>> + *   -EBUSY  no device or in bootloader
>> + *   -EIO    failure while reading from device
>> + *   -EAGAIN device is still in bootloader
>> + *           if ->state = CYAPA_STATE_BL_IDLE, device has invalid firmware
>> + *   -EINVAL device is in operational mode, but not supported by this driver
>> + *   0       device is supported
>> + */
>> +static int cyapa_check_is_operational(struct cyapa *cyapa)
>> +{
>> +     struct device *dev = &cyapa->client->dev;
>> +     static const char unique_str[] = "CYTRA";
>> +     int ret;
>> +
>> +     ret = cyapa_poll_state(cyapa, 2000);
>> +     if (ret < 0)
>> +             return ret;
>> +     switch (cyapa->state) {
>> +     case CYAPA_STATE_BL_ACTIVE:
>> +             ret = cyapa_bl_deactivate(cyapa);
>> +             if (ret)
>> +                     return ret;
>> +
>> +     /* Fallthrough state */
>> +     case CYAPA_STATE_BL_IDLE:
>> +             ret = cyapa_bl_exit(cyapa);
>> +             if (ret)
>> +                     return ret;
>> +
>> +     /* Fallthrough state */
>> +     case CYAPA_STATE_OP:
>> +             ret = cyapa_get_query_data(cyapa);
>> +             if (ret < 0)
>> +                     return ret;
>> +
>> +             /* only support firmware protocol gen3 */
>> +             if (cyapa->gen != CYAPA_GEN3) {
>> +                     dev_err(dev, "unsupported protocol version (%d)",
>> +                             cyapa->gen);
>> +                     return -EINVAL;
>> +             }
>> +
>> +             /* only support product ID starting with CYTRA */
>> +             if (memcmp(cyapa->product_id, unique_str,
>> +                        sizeof(unique_str) - 1) != 0) {
>> +                     dev_err(dev, "unsupported product ID (%s)\n",
>> +                             cyapa->product_id);
>> +                     return -EINVAL;
>> +             }
>> +             return 0;
>> +
>> +     default:
>> +             return -EIO;
>> +     }
>> +     return 0;
>> +}
>> +
>> +static irqreturn_t cyapa_irq(int irq, void *dev_id)
>> +{
>> +     struct cyapa *cyapa = dev_id;
>> +     struct device *dev = &cyapa->client->dev;
>> +     struct input_dev *input = cyapa->input;
>> +     struct cyapa_reg_data data;
>> +     int i;
>> +     int ret;
>> +     int num_fingers;
>> +
>> +     if (device_may_wakeup(dev))
>> +             pm_wakeup_event(dev, 0);
>> +
>> +     ret = cyapa_read_block(cyapa, CYAPA_CMD_GROUP_DATA, (u8 *)&data);
>> +     if (ret != sizeof(data))
>> +             goto irqhandled;
>> +
>> +     if ((data.device_status & OP_STATUS_SRC) != OP_STATUS_SRC ||
>> +         (data.device_status & OP_STATUS_DEV) != CYAPA_DEV_NORMAL ||
>> +         (data.finger_btn & OP_DATA_VALID) != OP_DATA_VALID) {
>> +             goto irqhandled;
>> +     }
>> +
>> +     num_fingers = (data.finger_btn >> 4) & 0x0f;
>> +     for (i = 0; i < num_fingers; i++) {
>> +             const struct cyapa_touch *touch = &data.touches[i];
>> +             /* Note: touch->id range is 1 to 15; slots are 0 to 14. */
>> +             int slot = touch->id - 1;
>> +
>> +             input_mt_slot(input, slot);
>> +             input_mt_report_slot_state(input, MT_TOOL_FINGER, true);
>> +             input_report_abs(input, ABS_MT_POSITION_X,
>> +                              ((touch->xy_hi & 0xf0) << 4) | touch->x_lo);
>> +             input_report_abs(input, ABS_MT_POSITION_Y,
>> +                              ((touch->xy_hi & 0x0f) << 8) | touch->y_lo);
>> +             input_report_abs(input, ABS_MT_PRESSURE, touch->pressure);
>> +     }
>> +
>> +     input_mt_sync_frame(input);
>> +
>> +     if (cyapa->btn_capability & CAPABILITY_LEFT_BTN_MASK) {
>> +             input_report_key(input, BTN_LEFT,
>> +                              !!(data.finger_btn & OP_DATA_LEFT_BTN));
>> +     }
>> +     if (cyapa->btn_capability & CAPABILITY_MIDDLE_BTN_MASK) {
>> +             input_report_key(input, BTN_MIDDLE,
>> +                              !!(data.finger_btn & OP_DATA_MIDDLE_BTN));
>> +     }
>> +     if (cyapa->btn_capability & CAPABILITY_RIGHT_BTN_MASK) {
>> +             input_report_key(input, BTN_RIGHT,
>> +                              !!(data.finger_btn & OP_DATA_RIGHT_BTN));
>> +     }
>> +     input_sync(input);
>> +
>> +irqhandled:
>> +     return IRQ_HANDLED;
>> +}
>> +
>> +static int cyapa_create_input_dev(struct cyapa *cyapa)
>> +{
>> +     struct device *dev = &cyapa->client->dev;
>> +     int ret;
>> +     struct input_dev *input;
>> +
>> +     if (!cyapa->physical_size_x || !cyapa->physical_size_y)
>> +             return -EINVAL;
>> +
>> +     input = cyapa->input = input_allocate_device();
>> +     if (!input) {
>> +             dev_err(dev, "allocate memory for input device failed\n");
>> +             return -ENOMEM;
>> +     }
>> +
>> +     input->name = CYAPA_NAME;
>> +     input->phys = cyapa->phys;
>> +     input->id.bustype = BUS_I2C;
>> +     input->id.version = 1;
>> +     input->id.product = 0;  /* means any product in eventcomm. */
>> +     input->dev.parent = &cyapa->client->dev;
>> +
>> +     input_set_drvdata(input, cyapa);
>> +
>> +     __set_bit(EV_ABS, input->evbit);
>> +
>> +     /* finger position */
>> +     input_set_abs_params(input, ABS_MT_POSITION_X, 0, cyapa->max_abs_x, 0,
>> +                          0);
>> +     input_set_abs_params(input, ABS_MT_POSITION_Y, 0, cyapa->max_abs_y, 0,
>> +                          0);
>> +     input_set_abs_params(input, ABS_MT_PRESSURE, 0, 255, 0, 0);
>> +
>> +     input_abs_set_res(input, ABS_MT_POSITION_X,
>> +                       cyapa->max_abs_x / cyapa->physical_size_x);
>> +     input_abs_set_res(input, ABS_MT_POSITION_Y,
>> +                       cyapa->max_abs_y / cyapa->physical_size_y);
>> +
>> +     if (cyapa->btn_capability & CAPABILITY_LEFT_BTN_MASK)
>> +             __set_bit(BTN_LEFT, input->keybit);
>> +     if (cyapa->btn_capability & CAPABILITY_MIDDLE_BTN_MASK)
>> +             __set_bit(BTN_MIDDLE, input->keybit);
>> +     if (cyapa->btn_capability & CAPABILITY_RIGHT_BTN_MASK)
>> +             __set_bit(BTN_RIGHT, input->keybit);
>> +     if (cyapa->btn_capability == CAPABILITY_LEFT_BTN_MASK)
>> +             __set_bit(INPUT_PROP_BUTTONPAD, input->propbit);
>> +
>> +     /* handle pointer emulation and unused slots in core */
>> +     ret = input_mt_init_slots(input, CYAPA_MAX_MT_SLOTS,
>> +                               INPUT_MT_POINTER | INPUT_MT_DROP_UNUSED);
>> +     if (ret) {
>> +             dev_err(dev, "allocate memory for MT slots failed, %d\n", ret);
>> +             goto err_free_device;
>> +     }
>> +
>> +     /* Register the device in input subsystem */
>> +     ret = input_register_device(input);
>> +     if (ret) {
>> +             dev_err(dev, "input device register failed, %d\n", ret);
>> +             goto err_free_device;
>> +     }
>> +     return 0;
>> +
>> +err_free_device:
>> +     input_free_device(input);
>> +     cyapa->input = NULL;
>> +     return ret;
>> +}
>> +
>> +static int cyapa_probe(struct i2c_client *client,
>> +                    const struct i2c_device_id *dev_id)
>> +{
>> +     int ret;
>> +     struct cyapa *cyapa;
>> +     struct device *dev = &client->dev;
>> +
>> +     cyapa = kzalloc(sizeof(struct cyapa), GFP_KERNEL);
>> +     if (!cyapa) {
>> +             dev_err(dev, "allocate memory for cyapa failed\n");
>> +             return -ENOMEM;
>> +     }
>> +
>> +     cyapa->gen = CYAPA_GEN3;
>> +     cyapa->client = client;
>> +     i2c_set_clientdata(client, cyapa);
>> +     sprintf(cyapa->phys, "i2c-%d-%04x/input0", client->adapter->nr,
>> +             client->addr);
>> +
>> +     cyapa->state = CYAPA_STATE_NO_DEVICE;
>> +     ret = cyapa_check_is_operational(cyapa);
>> +     if (ret) {
>> +             dev_err(dev, "device not operational, %d\n", ret);
>> +             goto err_mem_free;
>> +     }
>> +
>> +     ret = cyapa_create_input_dev(cyapa);
>> +     if (ret) {
>> +             dev_err(dev, "create input_dev instance failed, %d\n", ret);
>> +             goto err_mem_free;
>> +     }
>> +
>> +     ret = cyapa_set_power_mode(cyapa, PWR_MODE_FULL_ACTIVE);
>> +     if (ret) {
>> +             dev_err(dev, "set active power failed, %d\n", ret);
>> +             goto err_unregister_device;
>> +     }
>> +
>> +     cyapa->irq = client->irq;
>> +     ret = request_threaded_irq(cyapa->irq,
>> +                                NULL,
>> +                                cyapa_irq,
>> +                                IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
>> +                                "cyapa",
>> +                                cyapa);
>> +     if (ret) {
>> +             dev_err(dev, "IRQ request failed: %d\n, ", ret);
>> +             goto err_unregister_device;
>> +     }
>> +
>> +     return 0;
>> +
>> +err_unregister_device:
>> +     input_unregister_device(cyapa->input);
>> +err_mem_free:
>> +     kfree(cyapa);
>> +
>> +     return ret;
>> +}
>> +
>> +static int cyapa_remove(struct i2c_client *client)
>> +{
>> +     struct cyapa *cyapa = i2c_get_clientdata(client);
>> +
>> +     free_irq(cyapa->irq, cyapa);
>> +     input_unregister_device(cyapa->input);
>> +     cyapa_set_power_mode(cyapa, PWR_MODE_OFF);
>> +     kfree(cyapa);
>> +
>> +     return 0;
>> +}
>> +
>> +#ifdef CONFIG_PM_SLEEP
>> +static int cyapa_suspend(struct device *dev)
>> +{
>> +     int ret;
>> +     u8 power_mode;
>> +     struct cyapa *cyapa = dev_get_drvdata(dev);
>> +
>> +     disable_irq(cyapa->irq);
>> +
>> +     /*
>> +      * Set trackpad device to idle mode if wakeup is allowed,
>> +      * otherwise turn off.
>> +      */
>> +     power_mode = device_may_wakeup(dev) ? PWR_MODE_IDLE
>> +                                         : PWR_MODE_OFF;
>> +     ret = cyapa_set_power_mode(cyapa, power_mode);
>> +     if (ret < 0)
>> +             dev_err(dev, "set power mode failed, %d\n", ret);
>> +
>> +     if (device_may_wakeup(dev))
>> +             cyapa->irq_wake = (enable_irq_wake(cyapa->irq) == 0);
>> +     return 0;
>> +}
>> +
>> +static int cyapa_resume(struct device *dev)
>> +{
>> +     int ret;
>> +     struct cyapa *cyapa = dev_get_drvdata(dev);
>> +
>> +     if (device_may_wakeup(dev) && cyapa->irq_wake)
>> +             disable_irq_wake(cyapa->irq);
>> +
>> +     ret = cyapa_set_power_mode(cyapa, PWR_MODE_FULL_ACTIVE);
>> +     if (ret)
>> +             dev_warn(dev, "resume active power failed, %d\n", ret);
>> +
>> +     enable_irq(cyapa->irq);
>> +     return 0;
>> +}
>> +#endif /* CONFIG_PM_SLEEP */
>> +
>> +static SIMPLE_DEV_PM_OPS(cyapa_pm_ops, cyapa_suspend, cyapa_resume);
>> +
>> +static const struct i2c_device_id cyapa_id_table[] = {
>> +     { "cyapa", 0 },
>> +     { },
>> +};
>> +MODULE_DEVICE_TABLE(i2c, cyapa_id_table);
>> +
>> +static struct i2c_driver cyapa_driver = {
>> +     .driver = {
>> +             .name = "cyapa",
>> +             .owner = THIS_MODULE,
>> +             .pm = &cyapa_pm_ops,
>> +     },
>> +
>> +     .probe = cyapa_probe,
>> +     .remove = cyapa_remove,
>> +     .id_table = cyapa_id_table,
>> +};
>> +
>> +module_i2c_driver(cyapa_driver);
>> +
>> +MODULE_DESCRIPTION("Cypress APA I2C Trackpad Driver");
>> +MODULE_AUTHOR("Dudley Du <dudl@...ress.com>");
>> +MODULE_LICENSE("GPL");
>> --
>> 1.7.7.3
>>
>
>     Reviewed-by: Henrik Rydberg <rydberg@...omail.se>
>
> Thanks,
> Henrik



-- 
Benson Leung
Software Engineer, Chrom* OS
bleung@...omium.org
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ