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] [day] [month] [year] [list]
Date:	Fri, 16 Aug 2013 12:36:13 +0100
From:	Jonathan Cameron <jic23@...nel.org>
To:	Oleksandr Kravchenko <o.v.kravchenko@...ballogic.com>
CC:	devicetree@...r.kernel.org, linux-doc@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-iio@...r.kernel.org,
	Rob Herring <rob.herring@...xeda.com>, pawel.moll@....com,
	mark.rutland@....com, Stephen Warren <swarren@...dotorg.org>,
	Ian Campbell <ian.campbell@...rix.com>,
	Rob Landley <rob@...dley.net>, jic23@....ac.uk,
	Denis Ciocca <denis.ciocca@...com>,
	srinivas.pandruvada@...ux.intel.com,
	Lars-Peter Clausen <lars@...afoo.de>
Subject: Re: [PATCH v3] iio: add Bosch BMA180 acceleration sensor driver

On 08/16/13 10:29, Oleksandr Kravchenko wrote:
> On Thu, Aug 15, 2013 at 8:57 PM, Jonathan Cameron <jic23@...nel.org> wrote:
>> On 08/15/13 14:41, Oleksandr Kravchenko wrote:
>>> On Thu, Aug 15, 2013 at 1:22 PM, Jonathan Cameron <jic23@...nel.org> wrote:
>>>> On 08/13/13 16:44, Oleksandr Kravchenko wrote:
>>>>> This patch adds IIO driver for Bosch BMA180 triaxial
>>>>> acceleration sensor.
>>>>> http://dlnmh9ip6v2uc.cloudfront.net/datasheets/
>>>>>               Sensors/Accelerometers/BST-BMA180-DS000-07_2.pdf
>>>>>
>>>>> Signed-off-by: Oleksandr Kravchenko <o.v.kravchenko@...ballogic.com>
>>>> Hi,
>>>>
>>>> Firstly, just a thought (I haven't made up my mind whether it is a good idea or
>>>> not!) but you could use devm_kzalloc for data->buff and hence avoid having
>>>> to care about freeing it in the remove function.  Update scan mode isn't
>>>> a particularly fast path so I doubt the small overhead will matter.
>>> Thanks, it is a good idea.
>>>
>>>>
>>>> Otherwise, the abi additions need documenting iff they don't fit within
>>>> existing ABI. If they do then they need to match existing ABI.
>>>>
>>>> Of those, 'mode' is rather device specific and could probably do with
>>>> a more detailed name.  The only 'generic' way to do it would be to make the
>>>> channel type attributes writable and treat it as a resolution control rather
>>>> than a noise/power trade off.  Uggly though so perhaps what you have makes
>>>> sense.  We can always add a more generic way of controlling this if it
>>>> starts turning up in lots of devices (keeping this method of course to avoid
>>>> breaking ABI).
>>> Ok, I'll change 'mode' to 'power_mode' and document it in ABI.
> Can I use ext_info and iio_chan_spec_ext_info to provide my "power_mode"
> in my driver like "powerdown_mode" in some other drivers?
Yes.
> 
>>>>
>>>> 'bandwidth' is already (I think) covered by the ABI.  For the bandwidth optoins you have enabled we are talking low pass
>>>> filters and presumably their 3db point.
>>>> In Documentation/ABI/testing/sysfs-bus-iio you will find in_accel_filter_low_pass_3db_frequency. My filter theory is
>>>> rather rusty (not sure it was
>>>> every anything else ;). Hence I'm not entirely sure how to work
>>>> out the 3db frequency from the 'pole frequency' if they are not infact
>>>> the same thing..
>>> You are right. In my case 'bandwidth' is low path filter. I'll change my driver
>>> accordingly. But how can I declare available values?
>>> If I do it in next way:
>>>     static IIO_CONST_ATTR(in_accel0_filter_low_pass_3db_frequency_available,
>>> BMA180_BW_AVAILABLE);
>> Not shared across all channels?  If it is then
>> in_accel_filter_low_pass_3db_frequency_available it is...
>>> it will lock so ugly in the code and sysfs...
>>
>> yeah.  There are plans to clean that sort of thing up.   I keep mentioning them
>> and never getting them done though.
>>
>> In brief drivers will  be able to provide and additional callback
>> which will then return pointers to arrays containing all the numbers to
>> specify the 'range' of values any parameter in info_mask can take
>> (note this includes the raw and processed values as well as the other bits).
>>
>> I sketch out how we could do this from time to time and dont' quite get around to
>> doing it.
>>>
>>>>
>>>> Otherwise, the driver looks good to me.
>>>>
>>>> Jonathan
>>>>> ---
>>>>>  .../devicetree/bindings/iio/accel/bma180.txt       |   24 +
>>>>>  drivers/iio/accel/Kconfig                          |   12 +
>>>>>  drivers/iio/accel/Makefile                         |    2 +
>>>>>  drivers/iio/accel/bma180.c                         |  689 ++++++++++++++++++++
>>>>>  4 files changed, 727 insertions(+)
>>>>>  create mode 100644 Documentation/devicetree/bindings/iio/accel/bma180.txt
>>>>>  create mode 100644 drivers/iio/accel/bma180.c
>>>>>
>>>>> diff --git a/Documentation/devicetree/bindings/iio/accel/bma180.txt b/Documentation/devicetree/bindings/iio/accel/bma180.txt
>>>>> new file mode 100644
>>>>> index 0000000..c593357
>>>>> --- /dev/null
>>>>> +++ b/Documentation/devicetree/bindings/iio/accel/bma180.txt
>>>>> @@ -0,0 +1,24 @@
>>>>> +* Bosch BMA180 triaxial acceleration sensor
>>>>> +
>>>>> +http://omapworld.com/BMA180_111_1002839.pdf
>>>>> +
>>>>> +Required properties:
>>>>> +
>>>>> +  - compatible : should be "bosch,bma180"
>>>>> +  - reg : the I2C address of the sensor
>>>>> +
>>>>> +Optional properties:
>>>>> +
>>>>> +  - interrupt-parent : should be the phandle for the interrupt controller
>>>>> +
>>>>> +  - interrupts : interrupt mapping for GPIO IRQ, it should by configured with
>>>>> +             flags IRQ_TYPE_LEVEL_HIGH | IRQ_TYPE_EDGE_RISING
>>>>> +
>>>>> +Example:
>>>>> +
>>>>> +bma180@40 {
>>>>> +     compatible = "bosch,bma180";
>>>>> +     reg = <0x40>;
>>>>> +     interrupt-parent = <&gpio6>;
>>>>> +     interrupts = <18 (IRQ_TYPE_LEVEL_HIGH | IRQ_TYPE_EDGE_RISING)>;
>>>>> +};
>>>>> diff --git a/drivers/iio/accel/Kconfig b/drivers/iio/accel/Kconfig
>>>>> index 719d83f..bd9d581 100644
>>>>> --- a/drivers/iio/accel/Kconfig
>>>>> +++ b/drivers/iio/accel/Kconfig
>>>>> @@ -3,6 +3,18 @@
>>>>>  #
>>>>>  menu "Accelerometers"
>>>>>
>>>>> +config BMA180
>>>>> +     tristate "Bosch BMA180 3-Axis Accelerometer Driver"
>>>>> +     depends on I2C
>>>>> +     select IIO_BUFFER
>>>>> +     select IIO_TRIGGERED_BUFFER
>>>>> +     help
>>>>> +       Say Y here if you want to build a driver for the Bosch BMA180
>>>>> +       triaxial acceleration sensor.
>>>>> +
>>>>> +       To compile this driver as a module, choose M here: the
>>>>> +       module will be called bma180.
>>>>> +
>>>>>  config HID_SENSOR_ACCEL_3D
>>>>>       depends on HID_SENSOR_HUB
>>>>>       select IIO_BUFFER
>>>>> diff --git a/drivers/iio/accel/Makefile b/drivers/iio/accel/Makefile
>>>>> index 87d8fa2..eb09d72 100644
>>>>> --- a/drivers/iio/accel/Makefile
>>>>> +++ b/drivers/iio/accel/Makefile
>>>>> @@ -2,6 +2,8 @@
>>>>>  # Makefile for industrial I/O accelerometer drivers
>>>>>  #
>>>>>
>>>>> +obj-$(CONFIG_BMA180) += bma180.o
>>>>> +
>>>>>  obj-$(CONFIG_HID_SENSOR_ACCEL_3D) += hid-sensor-accel-3d.o
>>>>>
>>>>>  obj-$(CONFIG_IIO_ST_ACCEL_3AXIS) += st_accel.o
>>>>> diff --git a/drivers/iio/accel/bma180.c b/drivers/iio/accel/bma180.c
>>>>> new file mode 100644
>>>>> index 0000000..23c738f
>>>>> --- /dev/null
>>>>> +++ b/drivers/iio/accel/bma180.c
>>>>> @@ -0,0 +1,689 @@
>>>>> +/*
>>>>> + * bma180.c - IIO driver for Bosch BMA180 triaxial acceleration sensor
>>>>> + *
>>>>> + * Copyright 2013 Oleksandr Kravchenko <x0199363@...com>
>>>>> + *
>>>>> + * This file is subject to the terms and conditions of version 2 of
>>>>> + * the GNU General Public License.  See the file COPYING in the main
>>>>> + * directory of this archive for more details.
>>>>> + */
>>>>> +
>>>>> +#include <linux/module.h>
>>>>> +#include <linux/i2c.h>
>>>>> +#include <linux/interrupt.h>
>>>>> +#include <linux/delay.h>
>>>>> +#include <linux/of.h>
>>>>> +#include <linux/bitops.h>
>>>>> +#include <linux/slab.h>
>>>>> +#include <linux/string.h>
>>>>> +#include <linux/iio/iio.h>
>>>>> +#include <linux/iio/sysfs.h>
>>>>> +#include <linux/iio/buffer.h>
>>>>> +#include <linux/iio/trigger.h>
>>>>> +#include <linux/iio/trigger_consumer.h>
>>>>> +#include <linux/iio/triggered_buffer.h>
>>>>> +
>>>>> +#define BMA180_DRV_NAME "bma180"
>>>>> +#define BMA180_IRQ_NAME "bma180_event"
>>>>> +
>>>>> +/* Register set */
>>>>> +#define BMA180_CHIP_ID               0x00 /* Need to distinguish BMA180 from other */
>>>>> +#define BMA180_ACC_X_LSB     0x02 /* First of 6 registers of accel data */
>>>>> +#define BMA180_CTRL_REG0     0x0d
>>>>> +#define BMA180_RESET         0x10
>>>>> +#define BMA180_BW_TCS                0x20
>>>>> +#define BMA180_CTRL_REG3     0x21
>>>>> +#define BMA180_TCO_Z         0x30
>>>>> +#define BMA180_OFFSET_LSB1   0x35
>>>>> +
>>>>> +/* BMA180_CTRL_REG0 bits */
>>>>> +#define BMA180_DIS_WAKE_UP   BIT(0) /* Disable wake up mode */
>>>>> +#define BMA180_SLEEP         BIT(1) /* 1 - chip will sleep */
>>>>> +#define BMA180_EE_W          BIT(4) /* Unlock writing to addr from 0x20 */
>>>>> +#define BMA180_RESET_INT     BIT(6) /* Reset pending interrupts */
>>>>> +
>>>>> +/* BMA180_CTRL_REG3 bits */
>>>>> +#define BMA180_NEW_DATA_INT  BIT(1) /* Intr every new accel data is ready */
>>>>> +
>>>>> +/* BMA180_OFFSET_LSB1 skipping mode bit */
>>>>> +#define BMA180_SMP_SKIP              BIT(0)
>>>>> +
>>>>> +/* Bit masks for registers bit fields */
>>>>> +#define BMA180_RANGE         0x0e /* Range of measured accel values*/
>>>>> +#define BMA180_BW            0xf0 /* Accel bandwidth */
>>>>> +#define BMA180_MODE_CONFIG   0x03 /* Config operation modes */
>>>>> +
>>>>> +/* We have to write this value in reset register to do soft reset */
>>>>> +#define BMA180_RESET_VAL     0xb6
>>>>> +
>>>>> +#define BMA_180_ID_REG_VAL   0x03
>>>>> +
>>>>> +/* Chip operation modes */
>>>>> +#define BMA180_LOW_NOISE     0x00
>>>>> +#define BMA180_LOW_POWER     0x03
>>>>> +
>>>>> +#define BMA180_LOW_NOISE_STR "low_noise"
>>>>> +#define BMA180_LOW_POWER_STR "low_power"
>>>>> +
>>>>> +/* Defaults values */
>>>>> +#define BMA180_DEF_MODE              BMA180_LOW_NOISE_STR
>>>>> +#define BMA180_DEF_BW                20
>>>>> +#define BMA180_DEF_SCALE     250
>>>>> +
>>>>> +/* Available values */
>>>>> +#define BMA180_MODE_AVAILABLE \
>>>>> +     BMA180_LOW_NOISE_STR " " BMA180_LOW_POWER_STR
>>>>> +#define BMA180_BW_AVAILABLE  "10 20 40 75 150 300"
>>>>> +#define BMA180_SCALE_AVAILABLE \
>>>>> +     "0.000130 0.000190 0.000250 0.000380 0.000500 0.000990 0.001980"
>>>>> +
>>>>> +struct bma180_data {
>>>>> +     struct i2c_client *client;
>>>>> +     struct iio_trigger *trig;
>>>>> +     struct mutex mutex;
>>>>> +     int sleep_state;
>>>>> +     int scale;
>>>>> +     int bw;
>>>>> +     int mode;
>>>>> +     char *buff;
>>>>> +};
>>>>> +
>>>>> +enum bma180_axis {
>>>>> +     AXIS_X,
>>>>> +     AXIS_Y,
>>>>> +     AXIS_Z,
>>>>> +};
>>>>> +
>>>>> +static int bw_table[] = { 10, 20, 40, 75, 150, 300 }; /* Hz */
>>>>> +static int scale_table[] = { 130, 190, 250, 380, 500, 990, 1980 };
>>>>> +
>>>>> +static int bma180_get_acc_reg(struct bma180_data *data, enum bma180_axis axis)
>>>>> +{
>>>>> +     u8 reg = BMA180_ACC_X_LSB + axis * 2;
>>>>> +     int ret;
>>>>> +
>>>>> +     if (data->sleep_state)
>>>>> +             return -EBUSY;
>>>>> +
>>>>> +     ret = i2c_smbus_read_word_data(data->client, reg);
>>>>> +     if (ret < 0)
>>>>> +             dev_err(&data->client->dev,
>>>>> +                     "failed to read accel_%c registers\n", 'x' + axis);
>>>>> +
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static int bma180_set_bits(struct bma180_data *data, u8 reg, u8 mask, u8 val)
>>>>> +{
>>>>> +     int ret = i2c_smbus_read_byte_data(data->client, reg);
>>>>> +     u8 reg_val = (ret & ~mask) | (val << (ffs(mask) - 1));
>>>>> +
>>>>> +     if (ret < 0)
>>>>> +             return ret;
>>>>> +
>>>>> +     return i2c_smbus_write_byte_data(data->client, reg, reg_val);
>>>>> +}
>>>>> +
>>>>> +static int bma180_reset_intr(struct bma180_data *data)
>>>>> +{
>>>>> +     int ret = bma180_set_bits(data, BMA180_CTRL_REG0, BMA180_RESET_INT, 1);
>>>>> +
>>>>> +     if (ret)
>>>>> +             dev_err(&data->client->dev, "failed to reset interrupt\n");
>>>>> +
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static int bma180_set_new_data_intr_state(struct bma180_data *data, int state)
>>>>> +{
>>>>> +     u8 reg_val = state ? BMA180_NEW_DATA_INT : 0x00;
>>>>> +     int ret = i2c_smbus_write_byte_data(data->client, BMA180_CTRL_REG3,
>>>>> +                     reg_val);
>>>>> +
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +     ret = bma180_reset_intr(data);
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +
>>>>> +     return 0;
>>>>> +
>>>>> +err:
>>>>> +     dev_err(&data->client->dev,
>>>>> +             "failed to set new data interrupt state %d\n", state);
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static int bma180_set_sleep_state(struct bma180_data *data, int state)
>>>>> +{
>>>>> +     int ret = bma180_set_bits(data, BMA180_CTRL_REG0, BMA180_SLEEP, state);
>>>>> +
>>>>> +     if (ret) {
>>>>> +             dev_err(&data->client->dev,
>>>>> +                     "failed to set sleep state %d\n", state);
>>>>> +             return ret;
>>>>> +     }
>>>>> +     data->sleep_state = state;
>>>>> +
>>>>> +     return 0;
>>>>> +}
>>>>> +
>>>>> +static int bma180_set_ee_writing_state(struct bma180_data *data, int state)
>>>>> +{
>>>>> +     int ret = bma180_set_bits(data, BMA180_CTRL_REG0, BMA180_EE_W, state);
>>>>> +
>>>>> +     if (ret)
>>>>> +             dev_err(&data->client->dev,
>>>>> +                     "failed to set ee writing state %d\n", state);
>>>>> +
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static int bma180_set_bw(struct bma180_data *data, int val)
>>>>> +{
>>>>> +     int ret, i;
>>>>> +
>>>>> +     if (data->sleep_state)
>>>>> +             return -EBUSY;
>>>>> +
>>>>> +     for (i = 0; i < ARRAY_SIZE(bw_table); ++i) {
>>>>> +             if (bw_table[i] == val) {
>>>>> +                     ret = bma180_set_bits(data,
>>>>> +                                     BMA180_BW_TCS, BMA180_BW, i);
>>>>> +                     if (ret) {
>>>>> +                             dev_err(&data->client->dev,
>>>>> +                                     "failed to set bandwidth\n");
>>>>> +                             return ret;
>>>>> +                     }
>>>>> +                     data->bw = val;
>>>>> +                     return 0;
>>>>> +             }
>>>>> +     }
>>>>> +
>>>>> +     return -EINVAL;
>>>>> +}
>>>>> +
>>>>> +static int bma180_set_scale(struct bma180_data *data, int val)
>>>>> +{
>>>>> +     int ret, i;
>>>>> +
>>>>> +     if (data->sleep_state)
>>>>> +             return -EBUSY;
>>>>> +
>>>>> +     for (i = 0; i < ARRAY_SIZE(scale_table); ++i)
>>>>> +             if (scale_table[i] == val) {
>>>>> +                     ret = bma180_set_bits(data,
>>>>> +                                     BMA180_OFFSET_LSB1, BMA180_RANGE, i);
>>>>> +                     if (ret) {
>>>>> +                             dev_err(&data->client->dev,
>>>>> +                                     "failed to set scale\n");
>>>>> +                             return ret;
>>>>> +                     }
>>>>> +                     data->scale = val;
>>>>> +                     return 0;
>>>>> +             }
>>>>> +
>>>>> +     return -EINVAL;
>>>>> +}
>>>>> +
>>>>> +static int bma180_set_mode(struct bma180_data *data, const char *str)
>>>>> +{
>>>>> +     u8 reg_val;
>>>>> +     int ret;
>>>>> +
>>>>> +     if (!strcmp(str, BMA180_LOW_NOISE_STR "\n"))
>>>>> +             reg_val = BMA180_LOW_NOISE;
>>>>> +     else if (!strcmp(str, BMA180_LOW_POWER_STR "\n"))
>>>>> +             reg_val = BMA180_LOW_POWER;
>>>>> +     else
>>>>> +             return -EINVAL;
>>>>> +
>>>>> +     ret = bma180_set_bits(data, BMA180_TCO_Z, BMA180_MODE_CONFIG, reg_val);
>>>>> +     if (ret)
>>>>> +             dev_err(&data->client->dev, "failed to set mode\n");
>>>>> +
>>>>> +     data->mode = reg_val;
>>>>> +
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static int bma180_soft_reset(struct bma180_data *data)
>>>>> +{
>>>>> +     int ret = i2c_smbus_write_byte_data(data->client,
>>>>> +                     BMA180_RESET, BMA180_RESET_VAL);
>>>>> +
>>>>> +     if (ret)
>>>>> +             dev_err(&data->client->dev, "failed to reset the chip\n");
>>>>> +
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static int bma180_chip_init(struct bma180_data *data)
>>>>> +{
>>>>> +     /* Try to read chip_id register. It must return 0x03. */
>>>>> +     int ret = i2c_smbus_read_byte_data(data->client, BMA180_CHIP_ID);
>>>>> +
>>>>> +     if (ret < 0)
>>>>> +             goto err;
>>>>> +     if (ret != BMA_180_ID_REG_VAL) {
>>>>> +             ret = -ENODEV;
>>>>> +             goto err;
>>>>> +     }
>>>>> +
>>>>> +     ret = bma180_soft_reset(data);
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +     /*
>>>>> +      * No serial transaction should occur within minimum 10 us
>>>>> +      * after soft_reset command
>>>>> +      */
>>>>> +     msleep(20);
>>>>> +
>>>>> +     ret = bma180_set_bits(data, BMA180_CTRL_REG0, BMA180_DIS_WAKE_UP, 1);
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +     ret = bma180_set_ee_writing_state(data, 1);
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +     ret = bma180_set_new_data_intr_state(data, 0);
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +     ret = bma180_set_bits(data, BMA180_OFFSET_LSB1, BMA180_SMP_SKIP, 1);
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +     ret = bma180_set_mode(data, BMA180_DEF_MODE "\n");
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +     ret = bma180_set_bw(data, BMA180_DEF_BW);
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +     ret = bma180_set_scale(data, BMA180_DEF_SCALE);
>>>>> +     if (ret)
>>>>> +             goto err;
>>>>> +
>>>>> +     return 0;
>>>>> +
>>>>> +err:
>>>>> +     dev_err(&data->client->dev, "failed to init the chip\n");
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static void bma180_chip_disable(struct bma180_data *data)
>>>>> +{
>>>>> +     if (bma180_set_new_data_intr_state(data, 0))
>>>>> +             goto err;
>>>>> +     if (bma180_set_ee_writing_state(data, 0))
>>>>> +             goto err;
>>>>> +     if (bma180_set_sleep_state(data, 1))
>>>>> +             goto err;
>>>>> +
>>>>> +     return;
>>>>> +
>>>>> +err:
>>>>> +     dev_err(&data->client->dev, "failed to disable the chip\n");
>>>>> +}
>>>>> +
>>>>> +static ssize_t bma180_read_mode_config(struct device *dev,
>>>>> +             struct device_attribute *attr, char *buf)
>>>>> +{
>>>>> +     struct iio_dev *indio_dev = dev_to_iio_dev(dev);
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +
>>>>> +     return sprintf(buf, "%s\n",
>>>>> +                     data->mode ?
>>>>> +                     BMA180_LOW_POWER_STR : BMA180_LOW_NOISE_STR);
>>>>> +}
>>>>> +
>>>>> +static ssize_t bma180_write_mode_config(struct device *dev,
>>>>> +             struct device_attribute *attr, const char *buf, size_t len)
>>>>> +{
>>>>> +     struct iio_dev *indio_dev = dev_to_iio_dev(dev);
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +     int ret;
>>>>> +
>>>>> +     mutex_lock(&data->mutex);
>>>>> +     ret = bma180_set_mode(data, buf);
>>>>> +     mutex_unlock(&data->mutex);
>>>>> +
>>>>> +     return ret ? ret : len;
>>>>> +}
>>>>> +
>>>>> +static ssize_t bma180_read_bw_config(struct device *dev,
>>>>> +             struct device_attribute *attr, char *buf)
>>>>> +{
>>>>> +     struct iio_dev *indio_dev = dev_to_iio_dev(dev);
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +
>>>>> +     return sprintf(buf, "%d\n", data->bw);
>>>>> +}
>>>>> +
>>>>> +static ssize_t bma180_write_bw_config(struct device *dev,
>>>>> +             struct device_attribute *attr, const char *buf, size_t len)
>>>>> +{
>>>>> +     struct iio_dev *indio_dev = dev_to_iio_dev(dev);
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +     long val;
>>>>> +     int ret = kstrtol(buf, 10, &val);
>>>>> +
>>>>> +     if (ret)
>>>>> +             return ret;
>>>>> +
>>>>> +     mutex_lock(&data->mutex);
>>>>> +     ret = bma180_set_bw(data, val);
>>>>> +     mutex_unlock(&data->mutex);
>>>>> +
>>>>> +     return ret ? ret : len;
>>>>> +}
>>>>> +
>>>>> +static IIO_DEVICE_ATTR(mode, S_IRUGO | S_IWUSR, bma180_read_mode_config,
>>>>> +             bma180_write_mode_config, 0);
>>>>> +static IIO_CONST_ATTR(mode_available, BMA180_MODE_AVAILABLE);
>>>>> +static IIO_DEVICE_ATTR(bandwidth, S_IRUGO | S_IWUSR, bma180_read_bw_config,
>>>>> +             bma180_write_bw_config, 0);
>>>>> +static IIO_CONST_ATTR(bandwidth_available, BMA180_BW_AVAILABLE);
>>>>> +static IIO_CONST_ATTR(scale_available, BMA180_SCALE_AVAILABLE);
>>>>> +
>>>>> +static struct attribute *bma180_attributes[] = {
>>>>> +     &iio_dev_attr_mode.dev_attr.attr,
>>>>> +     &iio_const_attr_mode_available.dev_attr.attr,
>>>> These need documenting if they don't fit within standard abi options.
>>>>> +     &iio_dev_attr_bandwidth.dev_attr.attr,
>>>>> +     &iio_const_attr_bandwidth_available.dev_attr.attr,
>>>> bandwidth = 3db frequency?
>>>>> +     &iio_const_attr_scale_available.dev_attr.attr,
>>>>> +     NULL,
>>>>> +};
>>>>> +
>>>>> +static const struct attribute_group bma180_attrs_group = {
>>>>> +     .attrs = bma180_attributes,
>>>>> +};
>>>>> +
>>>>> +static int bma180_read_raw(struct iio_dev *indio_dev,
>>>>> +             struct iio_chan_spec const *chan, int *val, int *val2,
>>>>> +             long mask)
>>>>> +{
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +     int ret;
>>>>> +
>>>>> +     switch (mask) {
>>>>> +     case IIO_CHAN_INFO_RAW:
>>>>> +             mutex_lock(&data->mutex);
>>>>> +             if (iio_buffer_enabled(indio_dev))
>>>>> +                     ret = -EBUSY;
>>>>> +             else
>>>>> +                     ret = bma180_get_acc_reg(data, chan->scan_index);
>>>>> +             mutex_unlock(&data->mutex);
>>>>> +             if (ret < 0)
>>>>> +                     return ret;
>>>>> +             *val = (s16)ret >> chan->scan_type.shift;
>>>>> +             return IIO_VAL_INT;
>>>>> +     case IIO_CHAN_INFO_SCALE:
>>>>> +             *val = 0;
>>>>> +             *val2 = data->scale;
>>>>> +             return IIO_VAL_INT_PLUS_MICRO;
>>>>> +     default:
>>>>> +             return -EINVAL;
>>>>> +     }
>>>>> +}
>>>>> +
>>>>> +static int bma180_write_raw(struct iio_dev *indio_dev,
>>>>> +             struct iio_chan_spec const *chan, int val, int val2, long mask)
>>>>> +{
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +     int ret;
>>>>> +
>>>>> +     if (mask != IIO_CHAN_INFO_SCALE || val)
>>>>> +             return -EINVAL;
>>>>> +
>>>>> +     mutex_lock(&data->mutex);
>>>>> +     ret = bma180_set_scale(data, val2);
>>>>> +     mutex_unlock(&data->mutex);
>>>>> +
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static int bma180_update_scan_mode(struct iio_dev *indio_dev,
>>>>> +             const unsigned long *scan_mask)
>>>>> +{
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +
>>>>> +     kfree(data->buff);
>>>>> +     data->buff = kmalloc(indio_dev->scan_bytes, GFP_KERNEL);
>>>>> +     if (!data->buff)
>>>>> +             return -ENOMEM;
>>>>> +
>>>>> +     return 0;
>>>>> +}
>>>>> +
>>>>> +static const struct iio_info bma180_info = {
>>>>> +     .attrs                  = &bma180_attrs_group,
>>>>> +     .read_raw               = bma180_read_raw,
>>>>> +     .write_raw              = bma180_write_raw,
>>>>> +     .update_scan_mode       = bma180_update_scan_mode,
>>>>> +     .driver_module          = THIS_MODULE,
>>>>> +};
>>>>> +
>>>>> +#define BMA180_CHANNEL(_index) {                             \
>>>>> +     .type = IIO_ACCEL,                                      \
>>>>> +     .indexed = 1,                                           \
>>>>> +     .channel = (_index),                                    \
>>>>> +     .info_mask_separate = BIT(IIO_CHAN_INFO_RAW),           \
>>>>> +     .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE),   \
>>>>> +     .scan_index = (_index),                                 \
>>>>> +     .scan_type = IIO_ST('s', 14, 16, 2),                    \
>>>>> +}
>>>>> +
>>>>> +static const struct iio_chan_spec bma180_channels[] = {
>>>>> +     BMA180_CHANNEL(AXIS_X),
>>>>> +     BMA180_CHANNEL(AXIS_Y),
>>>>> +     BMA180_CHANNEL(AXIS_Z),
>>>>> +     IIO_CHAN_SOFT_TIMESTAMP(4),
>>>>> +};
>>>>> +
>>>>> +static irqreturn_t bma180_trigger_handler(int irq, void *p)
>>>>> +{
>>>>> +     struct iio_poll_func *pf = p;
>>>>> +     struct iio_dev *indio_dev = pf->indio_dev;
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +     int bit, ret, i = 0;
>>>>> +
>>>>> +     mutex_lock(&data->mutex);
>>>>> +     if (indio_dev->scan_timestamp) {
>>>>> +             ret = indio_dev->scan_bytes / sizeof(s64) - 1;
>>>>> +             ((s64 *)data->buff)[ret] = iio_get_time_ns();
>>>>> +     }
>>>>> +
>>>>> +     for_each_set_bit(bit, indio_dev->buffer->scan_mask,
>>>>> +                      indio_dev->masklength) {
>>>>> +             ret = bma180_get_acc_reg(data, bit);
>>>>> +             if (ret < 0) {
>>>>> +                     mutex_unlock(&data->mutex);
>>>>> +                     goto err;
>>>>> +             }
>>>>> +             ((s16 *)data->buff)[i++] = ret;
>>>>> +     }
>>>>> +     mutex_unlock(&data->mutex);
>>>>> +
>>>>> +     iio_push_to_buffers(indio_dev, (u8 *)data->buff);
>>>>> +err:
>>>>> +     iio_trigger_notify_done(indio_dev->trig);
>>>>> +
>>>>> +     return IRQ_HANDLED;
>>>>> +}
>>>>> +
>>>>> +static int bma180_data_rdy_trigger_set_state(struct iio_trigger *trig,
>>>>> +             bool state)
>>>>> +{
>>>>> +     struct iio_dev *indio_dev = iio_trigger_get_drvdata(trig);
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +
>>>>> +     return bma180_set_new_data_intr_state(data, state);
>>>>> +}
>>>>> +
>>>>> +static int bma180_trig_try_reen(struct iio_trigger *trig)
>>>>> +{
>>>>> +     struct iio_dev *indio_dev = iio_trigger_get_drvdata(trig);
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +
>>>>> +     return bma180_reset_intr(data);
>>>>> +}
>>>>> +
>>>>> +static const struct iio_trigger_ops bma180_trigger_ops = {
>>>>> +     .set_trigger_state = bma180_data_rdy_trigger_set_state,
>>>>> +     .try_reenable = bma180_trig_try_reen,
>>>>> +     .owner = THIS_MODULE,
>>>>> +};
>>>>> +
>>>>> +static int bma180_probe(struct i2c_client *client,
>>>>> +             const struct i2c_device_id *id)
>>>>> +{
>>>>> +     struct bma180_data *data;
>>>>> +     struct iio_dev *indio_dev;
>>>>> +     struct iio_trigger *trig;
>>>>> +     int ret;
>>>>> +
>>>>> +     indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
>>>>> +     if (!indio_dev)
>>>>> +             return -ENOMEM;
>>>>> +
>>>>> +     data = iio_priv(indio_dev);
>>>>> +     i2c_set_clientdata(client, indio_dev);
>>>>> +     data->client = client;
>>>>> +
>>>>> +     ret = bma180_chip_init(data);
>>>>> +     if (ret < 0)
>>>>> +             goto err_chip_disable;
>>>>> +
>>>>> +     mutex_init(&data->mutex);
>>>>> +
>>>>> +     indio_dev->dev.parent = &client->dev;
>>>>> +     indio_dev->channels = bma180_channels;
>>>>> +     indio_dev->num_channels = ARRAY_SIZE(bma180_channels);
>>>>> +     indio_dev->name = BMA180_DRV_NAME;
>>>>> +     indio_dev->modes = INDIO_DIRECT_MODE;
>>>>> +     indio_dev->info = &bma180_info;
>>>>> +
>>>>> +     trig = iio_trigger_alloc("%s-dev%d", indio_dev->name, indio_dev->id);
>>>>> +     if (!trig) {
>>>>> +             ret = -ENOMEM;
>>>>> +             goto err_chip_disable;
>>>>> +     }
>>>>> +
>>>>> +     ret = devm_request_irq(&client->dev, client->irq,
>>>>> +                     iio_trigger_generic_data_rdy_poll,
>>>>> +                     IRQF_TRIGGER_RISING, BMA180_IRQ_NAME, trig);
>>>>> +     if (ret) {
>>>>> +             dev_err(&client->dev, "unable to request IRQ\n");
>>>>> +             goto err_trigger_free;
>>>>> +     }
>>>>> +
>>>>> +     trig->dev.parent = &client->dev;
>>>>> +     trig->ops = &bma180_trigger_ops;
>>>>> +     iio_trigger_set_drvdata(trig, indio_dev);
>>>>> +     data->trig = trig;
>>>>> +     indio_dev->trig = trig;
>>>>> +
>>>>> +     ret = iio_trigger_register(trig);
>>>>> +     if (ret)
>>>>> +             goto err_trigger_free;
>>>>> +
>>>>> +     ret = iio_triggered_buffer_setup(indio_dev, NULL,
>>>>> +                     bma180_trigger_handler, NULL);
>>>>> +     if (ret < 0) {
>>>>> +             dev_err(&client->dev, "unable to setup iio triggered buffer\n");
>>>>> +             goto err_trigger_unregister;
>>>>> +     }
>>>>> +
>>>>> +     ret = iio_device_register(indio_dev);
>>>>> +     if (ret < 0) {
>>>>> +             dev_err(&client->dev, "unable to register iio device\n");
>>>>> +             goto err_buffer_cleanup;
>>>>> +     }
>>>>> +
>>>>> +     return 0;
>>>>> +
>>>>> +err_buffer_cleanup:
>>>>> +     iio_triggered_buffer_cleanup(indio_dev);
>>>>> +err_trigger_unregister:
>>>>> +     iio_trigger_unregister(trig);
>>>>> +err_trigger_free:
>>>>> +     iio_trigger_free(trig);
>>>>> +err_chip_disable:
>>>>> +     bma180_chip_disable(data);
>>>>> +
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static int bma180_remove(struct i2c_client *client)
>>>>> +{
>>>>> +     struct iio_dev *indio_dev = i2c_get_clientdata(client);
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +
>>>>> +     iio_device_unregister(indio_dev);
>>>>> +     iio_triggered_buffer_cleanup(indio_dev);
>>>>> +     iio_trigger_unregister(data->trig);
>>>>> +     iio_trigger_free(data->trig);
>>>>> +     kfree(data->buff);
>>>>> +
>>>>> +     mutex_lock(&data->mutex);
>>>>> +     bma180_chip_disable(data);
>>>>> +     mutex_unlock(&data->mutex);
>>>>> +
>>>>> +     return 0;
>>>>> +}
>>>>> +
>>>>> +#ifdef CONFIG_PM_SLEEP
>>>>> +static int bma180_suspend(struct device *dev)
>>>>> +{
>>>>> +     struct iio_dev *indio_dev = dev_to_iio_dev(dev);
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +     int ret;
>>>>> +
>>>>> +     mutex_lock(&data->mutex);
>>>>> +     ret = bma180_set_sleep_state(data, 1);
>>>>> +     mutex_unlock(&data->mutex);
>>>>> +
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static int bma180_resume(struct device *dev)
>>>>> +{
>>>>> +     struct iio_dev *indio_dev = dev_to_iio_dev(dev);
>>>>> +     struct bma180_data *data = iio_priv(indio_dev);
>>>>> +     int ret;
>>>>> +
>>>>> +     mutex_lock(&data->mutex);
>>>>> +     ret = bma180_set_sleep_state(data, 0);
>>>>> +     mutex_unlock(&data->mutex);
>>>>> +
>>>>> +     return ret;
>>>>> +}
>>>>> +
>>>>> +static SIMPLE_DEV_PM_OPS(bma180_pm_ops, bma180_suspend, bma180_resume);
>>>>> +#define BMA180_PM_OPS (&bma180_pm_ops)
>>>>> +#else
>>>>> +#define BMA180_PM_OPS NULL
>>>>> +#endif
>>>>> +
>>>>> +static struct i2c_device_id bma180_id[] = {
>>>>> +     { BMA180_DRV_NAME, 0 },
>>>>> +     { }
>>>>> +};
>>>>> +
>>>>> +MODULE_DEVICE_TABLE(i2c, bma180_id);
>>>>> +
>>>>> +static struct i2c_driver bma180_driver = {
>>>>> +     .driver = {
>>>>> +             .name   = BMA180_DRV_NAME,
>>>>> +             .owner  = THIS_MODULE,
>>>>> +             .pm     = BMA180_PM_OPS,
>>>>> +     },
>>>>> +     .probe          = bma180_probe,
>>>>> +     .remove         = bma180_remove,
>>>>> +     .id_table       = bma180_id,
>>>>> +};
>>>>> +
>>>>> +module_i2c_driver(bma180_driver);
>>>>> +
>>>>> +MODULE_AUTHOR("Kravchenko Oleksandr <x0199363@...com>");
>>>>> +MODULE_AUTHOR("Texas Instruments, Inc.");
>>>>> +MODULE_DESCRIPTION("Bosch BMA180 triaxial acceleration sensor");
>>>>> +MODULE_LICENSE("GPL");
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>>> the body of a message to majordomo@...r.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
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