On March 18, 2014 9:51:06 PM GMT+00:00, Kevin Tsai <[email protected]> 
wrote:
>Add Capella Microsystem CM3218x family Ambient Light Sensor IIO driver.
>This driver will convert raw data to lux value.  Default parameters are
>for reference only.  It will detect ACPI table to load per-system
>manufacturing
>parameters.
Hi Kevin 

Looks good. Wildcards in bindings or driver names are generally a bad idea. Too 
high a chance of an incompatible matching device.

As such name everything after one part and add additional compatible strings 
etc.

Otherwise I will want to do a proper review on a bigger screen than my phone 
but I am pretty much happy.

Jonathan


>
>Signed-off-by: Kevin Tsai <[email protected]>
>---
> .../devicetree/bindings/i2c/trivial-devices.txt    |   1 +
> drivers/iio/light/Kconfig                          |  11 +
> drivers/iio/light/Makefile                         |   1 +
>drivers/iio/light/cm3218x.c                        | 769
>+++++++++++++++++++++
> 4 files changed, 782 insertions(+)
> create mode 100644 drivers/iio/light/cm3218x.c
>
>diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
>b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
>index 1a1ac2e..29e7eae 100644
>--- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt
>+++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt
>@@ -17,6 +17,7 @@ at,24c08             i2c serial eeprom  (24cxx)
> atmel,24c02           i2c serial eeprom  (24cxx)
> atmel,at97sc3204t     i2c trusted platform module (TPM)
> capella,cm32181               CM32181: Ambient Light Sensor
>+capella,cm3218x               CM3218x: Ambient Light Sensor
> catalyst,24c32                i2c serial eeprom
> dallas,ds1307         64 x 8, Serial, I2C Real-Time Clock
> dallas,ds1338         I2C RTC with 56-Byte NV RAM
>diff --git a/drivers/iio/light/Kconfig b/drivers/iio/light/Kconfig
>index d12b2a0..45a22a6 100644
>--- a/drivers/iio/light/Kconfig
>+++ b/drivers/iio/light/Kconfig
>@@ -38,6 +38,17 @@ config CM32181
>        To compile this driver as a module, choose M here:
>        the module will be called cm32181.
> 
>+config CM3218X
>+      depends on I2C
>+      tristate "CM3218x driver"
>+      help
>+       Say Y here if you use cm3218x.
>+       This option enables ambient light sensor using
>+       Capella cm3218x device driver.
>+
>+       To compile this driver as a module, choose M here:
>+       the module will be called cm3218x.
>+
> config CM36651
>       depends on I2C
>       tristate "CM36651 driver"
>diff --git a/drivers/iio/light/Makefile b/drivers/iio/light/Makefile
>index 60e35ac..a506c23 100644
>--- a/drivers/iio/light/Makefile
>+++ b/drivers/iio/light/Makefile
>@@ -6,6 +6,7 @@
> obj-$(CONFIG_ADJD_S311)               += adjd_s311.o
> obj-$(CONFIG_APDS9300)                += apds9300.o
> obj-$(CONFIG_CM32181)         += cm32181.o
>+obj-$(CONFIG_CM3218X)         += cm3218x.o
> obj-$(CONFIG_CM36651)         += cm36651.o
> obj-$(CONFIG_GP2AP020A00F)    += gp2ap020a00f.o
> obj-$(CONFIG_HID_SENSOR_ALS)  += hid-sensor-als.o
>diff --git a/drivers/iio/light/cm3218x.c b/drivers/iio/light/cm3218x.c
>new file mode 100644
>index 0000000..e422f68
>--- /dev/null
>+++ b/drivers/iio/light/cm3218x.c
>@@ -0,0 +1,769 @@
>+/*
>+ * Copyright (C) 2014 Capella Microsystems Inc.
>+ * Author: Kevin Tsai <[email protected]>
>+ *
>+ * This program is free software; you can redistribute it and/or
>modify it
>+ * under the terms of the GNU General Public License version 2, as
>published
>+ * by the Free Software Foundation.
>+ *
>+ * Special thanks Srinivas Pandruvada
><[email protected]>
>+ * help to add ACPI support.
>+ *
>+ */
>+
>+#include <linux/delay.h>
>+#include <linux/err.h>
>+#include <linux/i2c.h>
>+#include <linux/mutex.h>
>+#include <linux/module.h>
>+#include <linux/interrupt.h>
>+#include <linux/regulator/consumer.h>
>+#include <linux/iio/iio.h>
>+#include <linux/iio/sysfs.h>
>+#include <linux/iio/events.h>
>+#include <linux/init.h>
>+#include <linux/acpi.h>
>+
>+/* Registers Address */
>+#define CM3218x_REG_ADDR_CMD          0x00
>+#define       CM3218x_REG_ADDR_WH             0x01
>+#define       CM3218x_REG_ADDR_WL             0x02
>+#define       CM3218x_REG_ADDR_TEST           0x03
>+#define CM3218x_REG_ADDR_ALS          0x04
>+#define CM3218x_REG_ADDR_STATUS               0x06
>+#define CM3218x_REG_ADDR_ID           0x07
>+
>+/* Number of Configurable Registers */
>+#define CM3218x_CONF_REG_NUM          16
>+
>+/* CMD register */
>+#define CM3218x_CMD_ALS_DISABLE               0x01
>+#define CM3218x_CMD_ALS_INT_EN                0x02
>+#define       CM3218x_CMD_ALS_THRES_WINDOW    0x04
>+
>+#define       CM3218x_CMD_ALS_PERS_SHIFT      4
>+#define       CM3218x_CMD_ALS_PERS_MASK       (0x03 << 
>CM3218x_CMD_ALS_PERS_SHIFT)
>+#define       CM3218x_CMD_ALS_PERS_DEFAULT    (0x01 <<
>CM3218x_CMD_ALS_PERS_SHIFT)
>+
>+#define CM3218x_CMD_ALS_IT_SHIFT      6
>+#define CM3218x_CMD_ALS_IT_MASK               (0x0F << 
>CM3218x_CMD_ALS_IT_SHIFT)
>+#define CM3218x_CMD_ALS_IT_DEFAULT    (0x01 << CM3218x_CMD_ALS_IT_SHIFT)
>+
>+#define CM3218x_CMD_ALS_HS_SHIFT      11
>+#define CM3218x_CMD_ALS_HS_MASK               (0x01 << 
>CM3218x_CMD_ALS_HS_SHIFT)
>+#define CM3218x_CMD_ALS_HS_DEFAULT    (0x00 << CM3218x_CMD_ALS_HS_SHIFT)
>+
>+#define       CM3218x_CMD_DEFAULT             (CM3218x_CMD_ALS_THRES_WINDOW |\
>+                                      CM3218x_CMD_ALS_PERS_DEFAULT |\
>+                                      CM3218x_CMD_ALS_IT_DEFAULT |\
>+                                      CM3218x_CMD_ALS_HS_DEFAULT)
>+
>+#define       CM3218x_WH_DEFAULT              0xFFFF
>+#define       CM3218x_WL_DEFAULT              0x0000
>+
>+#define CM3218x_CALIBSCALE_DEFAULT    100000
>+#define CM3218x_CALIBSCALE_RESOLUTION 100000
>+#define MLUX_PER_LUX                  1000
>+#define CM3218x_THRESHOLD_PERCENT     10      /* 10 percent */
>+
>+/* CM3218 Family */
>+#define CM3218_MLUX_PER_BIT_DEFAULT   5       /* Depend on system */
>+#define CM3218_MLUX_PER_BIT_BASE_IT   800000
>+static int CM3218_als_it_bits[] = {0, 1, 2, 3};
>+static int CM3218_als_it_values[] = {100000, 200000, 400000, 800000};
>+
>+/* CM32181 Family */
>+#define CM32181_MLUX_PER_BIT_DEFAULT  5
>+#define CM32181_MLUX_PER_BIT_BASE_IT  800000
>+static int CM32181_als_it_bits[] = {12, 8, 0, 1, 2, 3};
>+static int CM32181_als_it_values[] = {
>+                      25000, 50000, 100000, 200000, 400000, 800000};
>+
>+/* CM32182 Family */
>+#define CM32182_MLUX_PER_BIT_DEFAULT  5
>+#define CM32182_MLUX_PER_BIT_BASE_IT  800000
>+static int CM32182_als_it_bits[] = {12, 8, 0, 1, 2, 3};
>+static int CM32182_als_it_values[] = {
>+                      25000, 50000, 100000, 200000, 400000, 800000};
>+
>+struct cmdev {
>+      u32 family_id;
>+      int int_type;
>+#define       INT_TYPE_SMBUS                  0
>+#define       INT_TYPE_I2C                    1
>+      int regs_bmp;
>+      int calibscale;
>+      int mlux_per_bit;
>+      int mlux_per_bit_base_it;
>+      int *als_it_bits;
>+      int *als_it_values;
>+      int num_als_it;
>+};
>+
>+static struct cmdev cm3218 = {
>+      3218, INT_TYPE_SMBUS, 0x0F, CM3218x_CALIBSCALE_DEFAULT,
>+      CM3218_MLUX_PER_BIT_DEFAULT, CM3218_MLUX_PER_BIT_BASE_IT,
>+      CM3218_als_it_bits, CM3218_als_it_values,
>+      ARRAY_SIZE(CM3218_als_it_bits)};
>+
>+static struct cmdev cm32181 = {
>+      32181, INT_TYPE_I2C, 0x0F, CM3218x_CALIBSCALE_DEFAULT,
>+      CM32181_MLUX_PER_BIT_DEFAULT, CM32181_MLUX_PER_BIT_BASE_IT,
>+      CM32181_als_it_bits, CM32181_als_it_values,
>+      ARRAY_SIZE(CM32181_als_it_bits)};
>+
>+static struct cmdev cm32182 = {
>+      32182, INT_TYPE_I2C, 0x0F, CM3218x_CALIBSCALE_DEFAULT,
>+      CM32182_MLUX_PER_BIT_DEFAULT, CM32182_MLUX_PER_BIT_BASE_IT,
>+      CM32182_als_it_bits, CM32182_als_it_values,
>+      ARRAY_SIZE(CM32182_als_it_bits)};
>+
>+struct cm3218x_chip {
>+      struct i2c_client *client;
>+      struct cmdev *cmdev;
>+      struct mutex lock;
>+      u16 conf_regs[CM3218x_CONF_REG_NUM];
>+      int als_raw;
>+};
>+
>+static int cm3218x_get_lux(struct cm3218x_chip *chip);
>+static int cm3218x_threshold_update(struct cm3218x_chip *chip, int
>percent);
>+static int cm3218x_read_als_it(struct cm3218x_chip *chip, int *val2);
>+
>+/**
>+ * cm3218x_read_ara() - Read ARA register
>+ * @cm3218x:  pointer of struct cm3218x.
>+ *
>+ * Following SMBus protocol, ARA register is available only when
>interrupt
>+ * event happened.  Read it to clean interrupt event.  Otherwise,
>other
>+ * device address/registers will be blocked during interrupt event.
>+ *
>+ * Return: 0 for success; otherwise for error code.
>+ */
>+static int cm3218x_read_ara(struct cm3218x_chip *chip)
>+{
>+      struct i2c_client *client = chip->client;
>+      int status;
>+      unsigned short addr;
>+
>+      addr = client->addr;
>+      client->addr = 0x0C;
>+      status = i2c_smbus_read_byte(client);
>+      client->addr = addr;
>+
>+      if (status < 0)
>+              return -ENODEV;
>+
>+      return status;
>+}
>+
>+/**
>+ * cm3218x_interrupt_config() - Enable/Disable CM3218x interrupt
>+ * @cm3218x:  pointer of struct cm3218x.
>+ * @enable:   0 to disable; otherwise to enable
>+ *
>+ * Config CM3218x interrupt control bit.
>+ *
>+ * Return: 0 for success; otherwise for error code.
>+ */
>+static int cm3218x_interrupt_config(struct cm3218x_chip *chip, int
>enable)
>+{
>+      struct i2c_client *client = chip->client;
>+      struct cmdev *cmdev = chip->cmdev;
>+      int status;
>+
>+      if (!cmdev)
>+              return -ENODEV;
>+
>+      /* Force to clean interrupt */
>+      if (cmdev->int_type == INT_TYPE_I2C) {
>+              status = i2c_smbus_read_word_data(client,
>+                              CM3218x_REG_ADDR_STATUS);
>+              if (status < 0)
>+                      cmdev->int_type = INT_TYPE_SMBUS;
>+      }
>+      if (cmdev->int_type == INT_TYPE_SMBUS)
>+              cm3218x_read_ara(chip);
>+
>+      if (enable)
>+              chip->conf_regs[CM3218x_REG_ADDR_CMD] |=
>+                      CM3218x_CMD_ALS_INT_EN;
>+      else
>+              chip->conf_regs[CM3218x_REG_ADDR_CMD] &=
>+                      ~CM3218x_CMD_ALS_INT_EN;
>+
>+      status = i2c_smbus_write_word_data(client, CM3218x_REG_ADDR_CMD,
>+              chip->conf_regs[CM3218x_REG_ADDR_CMD]);
>+
>+      if (status < 0)
>+              return -ENODEV;
>+
>+      return status;
>+}
>+
>+/**
>+ * cm3218x_acpi_get_cpm_info() - Get CPM object from ACPI
>+ * @client    pointer of struct i2c_client.
>+ * @obj_name  pointer of ACPI object name.
>+ * @count     maximum size of return array.
>+ * @vals      pointer of array for return elements.
>+ *
>+ * Convert ACPI CPM table to array. Special thanks Srinivas
>Pandruvada's
>+ * help to implement this routine.
>+ *
>+ * Return: -ENODEV for fail.  Otherwise is number of elements.
>+ */
>+static int cm3218x_acpi_get_cpm_info(struct i2c_client *client, char
>*obj_name,
>+                                                      int count, u64 *vals)
>+{
>+      acpi_handle handle;
>+      struct acpi_buffer buffer = {ACPI_ALLOCATE_BUFFER, NULL};
>+      int i;
>+      acpi_status status;
>+      union acpi_object *cpm = NULL;
>+
>+      handle = ACPI_HANDLE(&client->dev);
>+      if (!handle)
>+              return -ENODEV;
>+
>+      status = acpi_evaluate_object(handle, obj_name, NULL, &buffer);
>+      if (ACPI_FAILURE(status)) {
>+              dev_err(&client->dev, "object %s not found\n", obj_name);
>+              return -ENODEV;
>+      }
>+
>+      cpm = buffer.pointer;
>+      for (i = 0; i < cpm->package.count && i < count; ++i) {
>+              union acpi_object *elem;
>+              elem = &(cpm->package.elements[i]);
>+              vals[i] = elem->integer.value;
>+      }
>+
>+      kfree(buffer.pointer);
>+
>+      return cpm->package.count;
>+}
>+
>+/**
>+ * cm3218x_reg_init() - Initialize CM3218x registers
>+ * @cm3218x:  pointer of struct cm3218x.
>+ *
>+ * Initialize CM3218x ambient light sensor register to default values.
>+ *
>+  Return: 0 for success; otherwise for error code.
>+ */
>+static int cm3218x_reg_init(struct cm3218x_chip *chip)
>+{
>+      struct i2c_client *client = chip->client;
>+      int i;
>+      s32 ret;
>+      int cpm_elem_count;
>+      u64 cpm_elems[20];
>+      struct cmdev *cmdev;
>+
>+      /* Default device */
>+      chip->cmdev = &cm3218;
>+      chip->conf_regs[CM3218x_REG_ADDR_CMD] = CM3218x_CMD_DEFAULT;
>+      chip->conf_regs[CM3218x_REG_ADDR_WH] = CM3218x_WH_DEFAULT;
>+      chip->conf_regs[CM3218x_REG_ADDR_WL] = CM3218x_WL_DEFAULT;
>+
>+      /* Disable interrupt */
>+      cm3218x_interrupt_config(chip, 0);
>+
>+      /* Disable Test Mode */
>+      i2c_smbus_write_word_data(client, CM3218x_REG_ADDR_TEST, 0x0000);
>+
>+      /* Disable device */
>+      i2c_smbus_write_word_data(client, CM3218x_REG_ADDR_CMD,
>+              CM3218x_CMD_ALS_DISABLE);
>+
>+      /* Identify device */
>+      ret = i2c_smbus_read_word_data(client, CM3218x_REG_ADDR_ID);
>+      if (ret < 0)
>+              return ret;
>+      switch (ret & 0xFF) {
>+      case 0x18:
>+              cmdev = chip->cmdev = &cm3218;
>+              if (ret & 0x0800)
>+                      cmdev->int_type = INT_TYPE_I2C;
>+              else
>+                      cmdev->int_type = INT_TYPE_SMBUS;
>+              break;
>+      case 0x81:
>+              cmdev = chip->cmdev = &cm32181;
>+              break;
>+      case 0x82:
>+              cmdev = chip->cmdev = &cm32182;
>+              break;
>+      default:
>+              return -ENODEV;
>+      }
>+
>+      if (ACPI_HANDLE(&client->dev)) {
>+              /* Load from ACPI */
>+              cpm_elem_count = cm3218x_acpi_get_cpm_info(client, "CPM0",
>+                                                      ARRAY_SIZE(cpm_elems),
>+                                                      cpm_elems);
>+              if (cpm_elem_count > 0) {
>+                      int header_num = 3;
>+                      int reg_num = cpm_elem_count - header_num;
>+
>+                      cmdev->regs_bmp = cpm_elems[2];
>+                      for (i = 0; i < reg_num; i++)
>+                              if (cmdev->regs_bmp & (1<<i))
>+                                      chip->conf_regs[i] =
>+                                              cpm_elems[header_num+i];
>+              }
>+
>+              cpm_elem_count = cm3218x_acpi_get_cpm_info(client, "CPM1",
>+                                                      ARRAY_SIZE(cpm_elems),
>+                                                      cpm_elems);
>+              if (cpm_elem_count > 0) {
>+                      cmdev->mlux_per_bit = (int)cpm_elems[0] / 100;
>+                      cmdev->calibscale = (int)cpm_elems[1];
>+              }
>+      }
>+
>+      /* Force to disable interrupt */
>+      chip->conf_regs[CM3218x_REG_ADDR_CMD] &= ~CM3218x_CMD_ALS_INT_EN;
>+
>+      /* Initialize registers*/
>+      for (i = 0; i < CM3218x_CONF_REG_NUM; i++) {
>+              if (cmdev->regs_bmp & (1<<i)) {
>+                      ret = i2c_smbus_write_word_data(client, i,
>+                                              chip->conf_regs[i]);
>+                      if (ret < 0)
>+                              return ret;
>+              }
>+      }
>+
>+      return 0;
>+}
>+
>+/**
>+ *  cm3218x_read_als_it() - Get sensor integration time (ms)
>+ *  @cm3218x: pointer of struct cm3218x
>+ *  @val2:    pointer of int to load the als_it value.
>+ *
>+ *  Report the current integartion time by millisecond.
>+ *
>+ *  Return: IIO_VAL_INT_PLUS_MICRO for success, otherwise -EINVAL.
>+ */
>+static int cm3218x_read_als_it(struct cm3218x_chip *chip, int *val2)
>+{
>+      struct cmdev *cmdev = chip->cmdev;
>+      u16 als_it;
>+      int i;
>+
>+      als_it = chip->conf_regs[CM3218x_REG_ADDR_CMD];
>+      als_it &= CM3218x_CMD_ALS_IT_MASK;
>+      als_it >>= CM3218x_CMD_ALS_IT_SHIFT;
>+      for (i = 0; i < cmdev->num_als_it; i++) {
>+              if (als_it == cmdev->als_it_bits[i]) {
>+                      *val2 = cmdev->als_it_values[i];
>+                      return IIO_VAL_INT_PLUS_MICRO;
>+              }
>+      }
>+
>+      return -EINVAL;
>+}
>+
>+/**
>+ * cm3218x_write_als_it() - Write sensor integration time
>+ * @cm3218x:  pointer of struct cm3218x.
>+ * @val:      integration time by millisecond.
>+ *
>+ * Convert integration time (ms) to sensor value.
>+ *
>+ * Return: i2c_smbus_write_word_data command return value.
>+ */
>+static int cm3218x_write_als_it(struct cm3218x_chip *chip, int val)
>+{
>+      struct i2c_client *client = chip->client;
>+      struct cmdev *cmdev = chip->cmdev;
>+      u16 als_it;
>+      int ret, i;
>+
>+      for (i = 0; i < cmdev->num_als_it; i++)
>+              if (val <= cmdev->als_it_values[i])
>+                      break;
>+      if (i >= cmdev->num_als_it)
>+              i = cmdev->num_als_it - 1;
>+
>+      als_it = cmdev->als_it_bits[i];
>+      als_it <<= CM3218x_CMD_ALS_IT_SHIFT;
>+
>+      mutex_lock(&chip->lock);
>+      chip->conf_regs[CM3218x_REG_ADDR_CMD] &=
>+                      ~CM3218x_CMD_ALS_IT_MASK;
>+      chip->conf_regs[CM3218x_REG_ADDR_CMD] |=
>+                      als_it;
>+      ret = i2c_smbus_write_word_data(client, CM3218x_REG_ADDR_CMD,
>+                      chip->conf_regs[CM3218x_REG_ADDR_CMD]);
>+      mutex_unlock(&chip->lock);
>+
>+      return ret;
>+}
>+
>+/**
>+ * cm3218x_get_lux() - report current lux value
>+ * @cm3218x:  pointer of struct cm3218x.
>+ *
>+ * Convert sensor raw data to lux.  It depends on integration
>+ * time and claibscale variable.
>+ *
>+ * Return: Positive value is lux, otherwise is error code.
>+ */
>+static int cm3218x_get_lux(struct cm3218x_chip *chip)
>+{
>+      struct i2c_client *client = chip->client;
>+      struct cmdev *cmdev = chip->cmdev;
>+      int ret;
>+      int als_it;
>+      int lux;
>+      u64 tmp;
>+
>+      /* Calculate mlux per bit based on als_it */
>+      ret = cm3218x_read_als_it(chip, &als_it);
>+      if (ret < 0)
>+              return -EINVAL;
>+      tmp = (__force u64)cmdev->mlux_per_bit;
>+      tmp *= cmdev->mlux_per_bit_base_it;
>+      tmp = div_u64(tmp, als_it);
>+
>+      /* Get als_raw */
>+      if (!(chip->conf_regs[CM3218x_REG_ADDR_CMD] &
>CM3218x_CMD_ALS_INT_EN))
>+              chip->als_raw = i2c_smbus_read_word_data(
>+                                      client,
>+                                      CM3218x_REG_ADDR_ALS);
>+      if (chip->als_raw < 0)
>+              return chip->als_raw;
>+
>+      tmp *= chip->als_raw;
>+      tmp *= cmdev->calibscale;
>+      tmp = div_u64(tmp, CM3218x_CALIBSCALE_RESOLUTION);
>+      tmp = div_u64(tmp, MLUX_PER_LUX);
>+
>+      if (tmp > 0xFFFF)
>+              tmp = 0xFFFF;
>+
>+      lux = (int)tmp;
>+      return lux;
>+}
>+
>+static int cm3218x_read_raw(struct iio_dev *indio_dev,
>+                      struct iio_chan_spec const *chan,
>+                      int *val, int *val2, long mask)
>+{
>+      struct cm3218x_chip *chip = iio_priv(indio_dev);
>+      struct cmdev *cmdev = chip->cmdev;
>+      int ret;
>+
>+      switch (mask) {
>+      case IIO_CHAN_INFO_PROCESSED:
>+              ret = cm3218x_get_lux(chip);
>+              if (ret < 0)
>+                      return ret;
>+              *val = ret;
>+              return IIO_VAL_INT;
>+      case IIO_CHAN_INFO_CALIBSCALE:
>+              *val = cmdev->calibscale;
>+              return IIO_VAL_INT;
>+      case IIO_CHAN_INFO_INT_TIME:
>+              *val = 0;
>+              ret = cm3218x_read_als_it(chip, val2);
>+              return ret;
>+      }
>+
>+      return -EINVAL;
>+}
>+
>+static int cm3218x_write_raw(struct iio_dev *indio_dev,
>+                      struct iio_chan_spec const *chan,
>+                      int val, int val2, long mask)
>+{
>+      struct cm3218x_chip *chip = iio_priv(indio_dev);
>+      struct cmdev *cmdev = chip->cmdev;
>+      int ret;
>+
>+      switch (mask) {
>+      case IIO_CHAN_INFO_CALIBSCALE:
>+              cmdev->calibscale = val;
>+              return val;
>+      case IIO_CHAN_INFO_INT_TIME:
>+              ret = cm3218x_write_als_it(chip, val2);
>+              return ret;
>+      }
>+
>+      return -EINVAL;
>+}
>+
>+/**
>+ * cm3218x_get_it_available() - Get available ALS IT value
>+ * @dev:      pointer of struct device.
>+ * @attr:     pointer of struct device_attribute.
>+ * @buf:      pointer of return string buffer.
>+ *
>+ * Display the available integration time values by millisecond.
>+ *
>+ * Return: string length.
>+ */
>+static ssize_t cm3218x_get_it_available(struct device *dev,
>+                      struct device_attribute *attr, char *buf)
>+{
>+      struct cm3218x_chip *chip = iio_priv(dev_to_iio_dev(dev));
>+      struct cmdev *cmdev = chip->cmdev;
>+      int i, len;
>+
>+      for (i = 0, len = 0; i < cmdev->num_als_it; i++)
>+              len += sprintf(buf + len, "0.%06u ", cmdev->als_it_values[i]);
>+      return len + sprintf(buf + len, "\n");
>+}
>+
>+/**
>+ * cm3218x_threshold_update() - Update the threshold registers.
>+ * @dev:      pointer of struct cm3218x_chip.
>+ * @percent:  +/- percent.
>+ *
>+ * Based on the current ALS value, tupdate the hi and low threshold
>registers.
>+ *
>+ * Return: string length.
>+ */
>+static int cm3218x_threshold_update(struct cm3218x_chip *chip, int
>percent)
>+{
>+      struct i2c_client *client = chip->client;
>+      int ret;
>+      int wh, wl;
>+
>+      ret = chip->als_raw = i2c_smbus_read_word_data(client,
>+                                      CM3218x_REG_ADDR_ALS);
>+      if (ret < 0)
>+              return ret;
>+
>+      wh = wl = ret;
>+      ret *= percent;
>+      ret /= 100;
>+      if (ret < 1)
>+              ret = 1;
>+      wh += ret;
>+      wl -= ret;
>+      if (wh > 65535)
>+              wh = 65535;
>+      if (wl < 0)
>+              wl = 0;
>+
>+      chip->conf_regs[CM3218x_REG_ADDR_WH] = wh;
>+      ret = i2c_smbus_write_word_data(
>+              client,
>+              CM3218x_REG_ADDR_WH,
>+              chip->conf_regs[CM3218x_REG_ADDR_WH]);
>+      if (ret < 0)
>+              return ret;
>+
>+      chip->conf_regs[CM3218x_REG_ADDR_WL] = wl;
>+      ret = i2c_smbus_write_word_data(
>+              client,
>+              CM3218x_REG_ADDR_WL,
>+              chip->conf_regs[CM3218x_REG_ADDR_WL]);
>+      if (ret < 0)
>+              return ret;
>+
>+      return 0;
>+}
>+
>+/**
>+ * cm3218x_event_handler() - Interrupt handling routine.
>+ * @irq:      irq number.
>+ * @private:  pointer of void.
>+ *
>+ * Clean interrupt and reset threshold registers.
>+ *
>+ * Return: string length.
>+ */
>+static irqreturn_t cm3218x_event_handler(int irq, void *private)
>+{
>+      struct iio_dev *dev_info = private;
>+      struct cm3218x_chip *chip = iio_priv(dev_info);
>+      int ret;
>+
>+      mutex_lock(&chip->lock);
>+
>+      /* Disable interrupt */
>+      ret = cm3218x_interrupt_config(chip, 0);
>+      if (ret < 0)
>+              goto error_handler_unlock;
>+
>+      /* Update Hi/Lo windows */
>+      ret = cm3218x_threshold_update(chip, CM3218x_THRESHOLD_PERCENT);
>+      if (ret < 0)
>+              goto error_handler_unlock;
>+
>+      /* Enable interrupt */
>+      ret = cm3218x_interrupt_config(chip, 1);
>+      if (ret < 0)
>+              goto error_handler_unlock;
>+
>+      mutex_unlock(&chip->lock);
>+
>+      return IRQ_HANDLED;
>+
>+error_handler_unlock:
>+      mutex_unlock(&chip->lock);
>+      return IRQ_NONE;
>+}
>+
>+static const struct iio_chan_spec cm3218x_channels[] = {
>+      {
>+              .type = IIO_LIGHT,
>+              .info_mask_separate =
>+                      BIT(IIO_CHAN_INFO_PROCESSED) |
>+                      BIT(IIO_CHAN_INFO_CALIBSCALE) |
>+                      BIT(IIO_CHAN_INFO_INT_TIME),
>+      }
>+};
>+
>+static IIO_DEVICE_ATTR(in_illuminance_integration_time_available,
>+                      S_IRUGO, cm3218x_get_it_available, NULL, 0);
>+
>+static struct attribute *cm3218x_attributes[] = {
>+      &iio_dev_attr_in_illuminance_integration_time_available.dev_attr.attr,
>+      NULL,
>+};
>+
>+static const struct attribute_group cm3218x_attribute_group = {
>+      .attrs = cm3218x_attributes
>+};
>+
>+static const struct iio_info cm3218x_info = {
>+      .driver_module          = THIS_MODULE,
>+      .read_raw               = &cm3218x_read_raw,
>+      .write_raw              = &cm3218x_write_raw,
>+      .attrs                  = &cm3218x_attribute_group,
>+};
>+
>+static int cm3218x_probe(struct i2c_client *client,
>+                      const struct i2c_device_id *id)
>+{
>+      struct cm3218x_chip *chip;
>+      struct iio_dev *indio_dev;
>+      int ret;
>+
>+      indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*chip));
>+      if (!indio_dev) {
>+              dev_err(&client->dev, "devm_iio_device_alloc failed\n");
>+              return -ENOMEM;
>+      }
>+
>+      chip = iio_priv(indio_dev);
>+      i2c_set_clientdata(client, indio_dev);
>+      chip->client = client;
>+
>+      mutex_init(&chip->lock);
>+      indio_dev->dev.parent = &client->dev;
>+      indio_dev->channels = cm3218x_channels;
>+      indio_dev->num_channels = ARRAY_SIZE(cm3218x_channels);
>+      indio_dev->info = &cm3218x_info;
>+      if (id && id->name)
>+              indio_dev->name = id->name;
>+      else
>+              indio_dev->name = (char *)dev_name(&client->dev);
>+      indio_dev->modes = INDIO_DIRECT_MODE;
>+
>+      ret = cm3218x_reg_init(chip);
>+      if (ret) {
>+              dev_err(&client->dev,
>+                      "%s: register init failed\n",
>+                      __func__);
>+              return ret;
>+      }
>+
>+      if (client->irq) {
>+              ret = request_threaded_irq(client->irq,
>+                                      NULL,
>+                                      cm3218x_event_handler,
>+                                      IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
>+                                      "cm3218x_event",
>+                                      indio_dev);
>+
>+              if (ret < 0) {
>+                      dev_err(&client->dev, "irq request error %d\n",
>+                              -ret);
>+                      goto error_disable_int;
>+              }
>+      }
>+
>+      ret = iio_device_register(indio_dev);
>+      if (ret < 0) {
>+              dev_err(&client->dev,
>+                      "%s: regist device failed\n",
>+                      __func__);
>+              goto error_free_irq;
>+      }
>+
>+      if (client->irq) {
>+              ret = cm3218x_threshold_update(chip, CM3218x_THRESHOLD_PERCENT);
>+              if (ret < 0)
>+                      goto error_free_irq;
>+
>+              ret = cm3218x_interrupt_config(chip, 1);
>+              if (ret < 0)
>+                      goto error_free_irq;
>+      }
>+
>+      return 0;
>+
>+error_free_irq:
>+      free_irq(client->irq, indio_dev);
>+error_disable_int:
>+      cm3218x_interrupt_config(chip, 0);
>+      return ret;
>+}
>+
>+static int cm3218x_remove(struct i2c_client *client)
>+{
>+      struct iio_dev *indio_dev = i2c_get_clientdata(client);
>+      struct cm3218x_chip *chip = iio_priv(indio_dev);
>+
>+      cm3218x_interrupt_config(chip, 0);
>+      if (client->irq)
>+              free_irq(client->irq, indio_dev);
>+      iio_device_unregister(indio_dev);
>+      return 0;
>+}
>+
>+static const struct i2c_device_id cm3218x_id[] = {
>+      { "cm3218x", 0},
>+      { }
>+};
>+
>+MODULE_DEVICE_TABLE(i2c, cm3218x_id);
>+
>+static const struct of_device_id cm3218x_of_match[] = {
>+      { .compatible = "capella,cm3218x" },
>+      { }
>+};
>+
>+static const struct acpi_device_id cm3218x_acpi_match[] = {
>+      { "CPLM3218", 0},
>+      {},
>+};
>+
>+MODULE_DEVICE_TABLE(acpi, cm3218x_acpi_match);
>+
>+static struct i2c_driver cm3218x_driver = {
>+      .driver = {
>+              .name   = "cm3218x",
>+              .acpi_match_table = ACPI_PTR(cm3218x_acpi_match),
>+              .of_match_table = of_match_ptr(cm3218x_of_match),
>+              .owner  = THIS_MODULE,
>+      },
>+      .id_table       = cm3218x_id,
>+      .probe          = cm3218x_probe,
>+      .remove         = cm3218x_remove,
>+};
>+
>+module_i2c_driver(cm3218x_driver);
>+
>+MODULE_AUTHOR("Kevin Tsai <[email protected]>");
>+MODULE_DESCRIPTION("CM3218x ambient light sensor driver");
>+MODULE_LICENSE("GPL");

-- 
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to