| 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
| ||
|
Message-ID: <CAEnQRZBMD7968_ByRicN8bTUetoOHriQEpvex1GjTMgR6Qp2zw@mail.gmail.com>
Date: Mon, 25 May 2015 15:45:56 +0300
From: Daniel Baluta <daniel.baluta@...el.com>
To: Jonathan Cameron <jic23@...nel.org>
Cc: Daniel Baluta <daniel.baluta@...el.com>,
Peter Meerwald <pmeerw@...erw.net>,
Hartmut Knaack <knaack.h@....de>,
Lars-Peter Clausen <lars@...afoo.de>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
"linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>
Subject: Re: [PATCH] iio: light: Add support for ROHM RPR0521 sensor
On Sat, May 23, 2015 at 1:51 PM, Jonathan Cameron <jic23@...nel.org> wrote:
> On 22/05/15 13:36, Daniel Baluta wrote:
>> This patch adds support for ROHM RPR0521 ambient light and proximity
>> sensor. It offers raw readings for intensity and proximity.
>>
>> Signed-off-by: Daniel Baluta <daniel.baluta@...el.com>
> A simple driver with some fiddly power management!
> Mostly looks fine, but I'm not sure why there is an explicit enable in
> the set_power_state function as well as the runtime ones.
Mostly because I don't want to activate both ALS and PS, if for example
only ALS is read.
>
> Just to check, for the calibscale attributes... Does the device always report
> at the same scale whatever the calibscale value?
>
> E.g. Setting calibscale = 16 gives same answer from a raw read as calibscale = 2?
>
> Just would be a bit unusual for a device to do this over such a range so I thought I'd
> check.
# cat in_intensity_calibscale_available
1 2 64 128
# echo 1 > in_intensity_ir_calibscale
# cat in_intensity_ir_raw
79
# echo 64 > in_intensity_ir_calibscale
# cat in_intensity_ir_raw
5084
So, setting calibscale = 64 gives a raw read value x64 times greater
then read value
at calibscale = 1.
Driver doesn't expose yet processed values. I will later add an
illuminance channel
that will exposed processed lux value.
I'm still trying to "decode" the formula for lux calculation and will
send a patch on top
of this. Basically Lux = f(data0, data1, gain0, gain1, predefined_constants)
>> ---
>> drivers/iio/light/Kconfig | 11 +
>> drivers/iio/light/Makefile | 1 +
>> drivers/iio/light/rpr0521.c | 579 ++++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 591 insertions(+)
>> create mode 100644 drivers/iio/light/rpr0521.c
>>
>> diff --git a/drivers/iio/light/Kconfig b/drivers/iio/light/Kconfig
>> index e6198b7..cbc4677 100644
>> --- a/drivers/iio/light/Kconfig
>> +++ b/drivers/iio/light/Kconfig
>> @@ -168,6 +168,17 @@ config JSA1212
>> To compile this driver as a module, choose M here:
>> the module will be called jsa1212.
>>
>> +config RPR0521
>> + tristate "ROHM RPR0521 ALS and proximity sensor driver"
>> + depends on I2C
>> + select REGMAP_I2C
>> + help
>> + Say Y here if you want to build support for ROHM's RPR0521
>> + ambient light and proximity sensor device.
>> +
>> + To compile this driver as a module, choose M here:
>> + the module will be called rpr0521.
>> +
>> config SENSORS_LM3533
>> tristate "LM3533 ambient light sensor"
>> depends on MFD_LM3533
>> diff --git a/drivers/iio/light/Makefile b/drivers/iio/light/Makefile
>> index e2d50fd..66fc2fd 100644
>> --- a/drivers/iio/light/Makefile
>> +++ b/drivers/iio/light/Makefile
>> @@ -17,6 +17,7 @@ obj-$(CONFIG_HID_SENSOR_ALS) += hid-sensor-als.o
>> obj-$(CONFIG_HID_SENSOR_PROX) += hid-sensor-prox.o
>> obj-$(CONFIG_ISL29125) += isl29125.o
>> obj-$(CONFIG_JSA1212) += jsa1212.o
>> +obj-$(CONFIG_RPR0521) += rpr0521.o
>> obj-$(CONFIG_SENSORS_LM3533) += lm3533-als.o
>> obj-$(CONFIG_LTR501) += ltr501.o
>> obj-$(CONFIG_SENSORS_TSL2563) += tsl2563.o
>> diff --git a/drivers/iio/light/rpr0521.c b/drivers/iio/light/rpr0521.c
>> new file mode 100644
>> index 0000000..9d3d312
>> --- /dev/null
>> +++ b/drivers/iio/light/rpr0521.c
>> @@ -0,0 +1,579 @@
>> +/*
>> + * RPR-0521 ROHM Ambient Light and Proximity Sensor
>> + *
>> + * Copyright (c) 2015, Intel Corporation.
>> + *
>> + * 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.
>> + *
>> + * IIO driver for RPR-0521RS (7-bit I2C slave address 0x38).
>> + *
>> + * TODO: illuminance channel, PM support, buffer
>> + */
>> +
>> +#include <linux/module.h>
>> +#include <linux/init.h>
>> +#include <linux/i2c.h>
>> +#include <linux/regmap.h>
>> +#include <linux/delay.h>
>> +#include <linux/acpi.h>
>> +
>> +#include <linux/iio/iio.h>
>> +#include <linux/iio/sysfs.h>
>> +#include <linux/pm_runtime.h>
>> +
>> +#define RPR0521_REG_SYSTEM_CTRL 0x40
>> +#define RPR0521_REG_MODE_CTRL 0x41
>> +#define RPR0521_REG_ALS_CTRL 0x42
>> +#define RPR0521_REG_PXS_CTRL 0x43
>> +#define RPR0521_REG_PXS_DATA 0x44 /* 16-bit, little endian */
>> +#define RPR0521_REG_ALS_DATA0 0x46 /* 16-bit, little endian */
>> +#define RPR0521_REG_ALS_DATA1 0x48 /* 16-bit, little endian */
>> +#define RPR0521_REG_ID 0x92
>> +
>> +#define RPR0521_MODE_ALS_MASK BIT(7)
>> +#define RPR0521_MODE_PXS_MASK BIT(6)
>> +#define RPR0521_MODE_MEAS_TIME_MASK GENMASK(3, 0)
>> +#define RPR0521_ALS_DATA0_GAIN_MASK (BIT(4) | BIT(5))
>> +#define RPR0521_ALS_DATA0_GAIN_SHIFT 4
>> +#define RPR0521_ALS_DATA1_GAIN_MASK (BIT(2) | BIT(3))
>> +#define RPR0521_ALS_DATA1_GAIN_SHIFT 2
>> +#define RPR0521_PXS_GAIN_MASK (BIT(4) | BIT(5))
>> +#define RPR0521_PXS_GAIN_SHIFT 4
>> +
>> +#define RPR0521_MODE_ALS_ENABLE BIT(7)
>> +#define RPR0521_MODE_ALS_DISABLE 0x00
>> +#define RPR0521_MODE_PXS_ENABLE BIT(6)
>> +#define RPR0521_MODE_PXS_DISABLE 0x00
>> +
>> +#define RPR0521_PXS_MAX_VAL (BIT(13) - 1)
>> +#define RPR0521_MANUFACT_ID 0xE0
>> +#define RPR0521_DEFAULT_MEAS_TIME 0x06 /* ALS - 100ms, PXS - 100ms */
>> +
>> +#define RPR0521_DRV_NAME "RPR0521"
>> +#define RPR0521_REGMAP_NAME "rpr0521_regmap"
>> +
>> +#define RPR0521_SLEEP_DELAY_MS 2000
>> +
> static const
>> +int rpr0521_als_gain[4] = {1, 2, 64, 128};
>> +int rpr0521_pxs_gain[3] = {1, 2, 4};
>> +
>> +enum rpr0521_channel {
>> + RPR0521_CHAN_ALS_DATA0,
>> + RPR0521_CHAN_ALS_DATA1,
>> + RPR0521_CHAN_PXS,
>> +};
>> +
> static const
>> +int rpr0521_data_reg[] = {
>> + [RPR0521_CHAN_ALS_DATA0] = RPR0521_REG_ALS_DATA0,
>> + [RPR0521_CHAN_ALS_DATA1] = RPR0521_REG_ALS_DATA1,
>> + [RPR0521_CHAN_PXS] = RPR0521_REG_PXS_DATA,
>> +};
>> +
> const
>> +struct rpr0521_gain_info {
>> + u8 reg;
>> + u8 mask;
>> + u8 shift;
>> + int *gain;
>> + int size;
>> +} rpr0521_gain[] = {
>> + [RPR0521_CHAN_ALS_DATA0] = {
>> + .reg = RPR0521_REG_ALS_CTRL,
>> + .mask = RPR0521_ALS_DATA0_GAIN_MASK,
>> + .shift = RPR0521_ALS_DATA0_GAIN_SHIFT,
>> + .gain = rpr0521_als_gain,
>> + .size = ARRAY_SIZE(rpr0521_als_gain),
>> + },
>> + [RPR0521_CHAN_ALS_DATA1] = {
>> + .reg = RPR0521_REG_ALS_CTRL,
>> + .mask = RPR0521_ALS_DATA1_GAIN_MASK,
>> + .shift = RPR0521_ALS_DATA1_GAIN_SHIFT,
>> + .gain = rpr0521_als_gain,
>> + .size = ARRAY_SIZE(rpr0521_als_gain),
>> + },
>> + [RPR0521_CHAN_PXS] = {
>> + .reg = RPR0521_REG_PXS_CTRL,
>> + .mask = RPR0521_PXS_GAIN_MASK,
>> + .shift = RPR0521_PXS_GAIN_SHIFT,
>> + .gain = rpr0521_pxs_gain,
>> + .size = ARRAY_SIZE(rpr0521_pxs_gain),
>> + },
>> +};
>> +
>> +struct rpr0521_data {
>> + struct i2c_client *client;
>> +
>> + /* protect device params updates (e.g state, gain) */
>> + struct mutex lock;
>> +
>> + /* device active status */
>> + bool als_dev_en;
>> + bool pxs_dev_en;
>> +
>> + /* optimize runtime pm ops - enable device only if needed */
>> + bool als_ps_need_en;
>> + bool pxs_ps_need_en;
>> +
>> + struct regmap *regmap;
>> +};
>> +
>> +static const struct iio_chan_spec rpr0521_channels[] = {
>> + {
>> + .type = IIO_INTENSITY,
>> + .modified = 1,
>> + .address = RPR0521_CHAN_ALS_DATA0,
>> + .channel2 = IIO_MOD_LIGHT_BOTH,
>> + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |
>> + BIT(IIO_CHAN_INFO_CALIBSCALE),
>> + },
>> + {
>> + .type = IIO_INTENSITY,
>> + .modified = 1,
>> + .address = RPR0521_CHAN_ALS_DATA1,
>> + .channel2 = IIO_MOD_LIGHT_IR,
>> + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |
>> + BIT(IIO_CHAN_INFO_CALIBSCALE),
>> + },
>> + {
>> + .type = IIO_PROXIMITY,
>> + .address = RPR0521_CHAN_PXS,
>> + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |
>> + BIT(IIO_CHAN_INFO_CALIBSCALE),
>> + }
>> +};
>> +
>> +static int rpr0521_als_enable(struct rpr0521_data *data, u8 status)
>> +{
>> + int ret;
>> +
>> + ret = regmap_update_bits(data->regmap, RPR0521_REG_MODE_CTRL,
>> + RPR0521_MODE_ALS_MASK,
>> + status);
>> + if (ret < 0)
>> + return ret;
>> +
>> + data->als_dev_en = true;
>> +
>> + return 0;
>> +}
>> +
>> +static int rpr0521_pxs_enable(struct rpr0521_data *data, u8 status)
>> +{
>> + int ret;
>> +
>> + ret = regmap_update_bits(data->regmap, RPR0521_REG_MODE_CTRL,
>> + RPR0521_MODE_PXS_MASK,
>> + status);
>> + if (ret < 0)
>> + return ret;
>> +
>> + data->pxs_dev_en = true;
>> +
>> + return 0;
>> +}
>> +
>> +static int rpr0521_set_power_state(struct rpr0521_data *data, bool on,
>> + u8 device_mask)
>> +{
>> +#ifdef CONFIG_PM
>> + int ret;
>> + u8 update_mask = 0;
>> +
>> + if (device_mask & RPR0521_MODE_ALS_MASK) {
> This is a rather fiddly bit of power management isn't it!
Yes it is. I should have added some comments, it seemed obvious after
few hours of debugging :).
>
> If we are enabling the channel and als_ps_need_en is not set (which means we
> are not in runtime power saving?) and the channel is currently enabled then
> set the update mask that turns the channel on then carry on.
>
> If it's currently enabled, why would we want to set it at all?
... and the *other* channel is currently enabled (the check is for
pxs_dev_en here!),
this means that there was a previous call to set_power_state ->
get_sync -> runtime_resume,
and setting als_ps_need_en will not help because another call to
get_sync will not result in a call to runtime_resume.
So we need to enable the channel now.
>
>> + if (on && !data->als_ps_need_en && data->pxs_dev_en)
>> + update_mask |= RPR0521_MODE_ALS_MASK;
>> + else
> Otherwise we flag that we want the channel on when we do the get_sync below
> which makes sense.
Partially correct. get_sync -> calls runtime_resume only first time or
after the SLEEP_DELAY_MS
timeout expired.
So, if we have two to quick consecutive reads one for PS and one for
ALS. Only, PS will be enabled
because runtime_resume is called only once.
Of course, we can simplify this and enable both ALS and PS when one of
them is read, but we
want to optimize power consumption.
I think this function needs a really good comment on top. I will add it in v2.
>
>> + data->als_ps_need_en = on;
>> + }
>> +
>> + if (device_mask & RPR0521_MODE_PXS_MASK) {
>> + if (on && !data->pxs_ps_need_en && data->als_dev_en)
>> + update_mask |= RPR0521_MODE_PXS_MASK;
>> + else
>> + data->pxs_ps_need_en = on;
>> + }
>> +
>> + ret = regmap_update_bits(data->regmap, RPR0521_REG_MODE_CTRL,
>> + update_mask, update_mask);
>> + if (ret < 0)
>> + return ret;
>> +
>> + if (on) {
>> + ret = pm_runtime_get_sync(&data->client->dev);
>> + } else {
>> + pm_runtime_mark_last_busy(&data->client->dev);
>> + ret = pm_runtime_put_autosuspend(&data->client->dev);
>> + }
>> + if (ret < 0) {
>> + dev_err(&data->client->dev,
>> + "Failed: rpr0521_set_power_state for %d, ret %d\n",
>> + on, ret);
>> + if (on)
>> + pm_runtime_put_noidle(&data->client->dev);
>> +
>> + return ret;
>> + }
>> +#endif
>> + return 0;
>> +}
>> +
>> +int rpr0521_get_gain_index(int *gainv, int size, int gain)
>> +{
>> + int i;
>> +
>> + for (i = 0; i < size; i++)
>> + if (gain == gainv[i])
>> + return i;
> blank line here.
>> + return -EINVAL;
>> +}
>> +
>> +int rpr0521_get_gain(struct rpr0521_data *data, int chan, int *val)
>> +{
>> + int ret, reg, idx;
>> +
>> + ret = regmap_read(data->regmap, rpr0521_gain[chan].reg, ®);
>> + if (ret < 0)
>> + return ret;
>> +
>> + idx = (rpr0521_gain[chan].mask & reg) >> rpr0521_gain[chan].shift;
>> + *val = rpr0521_gain[chan].gain[idx];
>> +
>> + return 0;
>> +}
>> +
>> +int rpr0521_set_gain(struct rpr0521_data *data, int chan, int val)
>> +{
>> + int idx;
>> +
>> + idx = rpr0521_get_gain_index(rpr0521_gain[chan].gain,
>> + rpr0521_gain[chan].size, val);
>> + if (idx < 0)
>> + return idx;
>> +
>> + return regmap_update_bits(data->regmap, rpr0521_gain[chan].reg,
>> + rpr0521_gain[chan].mask,
>> + idx << rpr0521_gain[chan].shift);
>> +}
>> +
>> +static int rpr0521_read_raw(struct iio_dev *indio_dev,
>> + struct iio_chan_spec const *chan, int *val,
>> + int *val2, long mask)
>> +{
>> + struct rpr0521_data *data = iio_priv(indio_dev);
>> + int ret;
>> + u8 device_mask;
>> + __le16 raw_data;
>> +
>> + switch (mask) {
>> + case IIO_CHAN_INFO_RAW:
>> + switch (chan->type) {
>> + case IIO_INTENSITY:
>> + device_mask = RPR0521_MODE_ALS_MASK;
>> + break;
>> + case IIO_PROXIMITY:
>> + device_mask = RPR0521_MODE_PXS_MASK;
>> + break;
>> + default:
>> + return -EINVAL;
>> + }
>> + mutex_lock(&data->lock);
>> + ret = rpr0521_set_power_state(data, true, device_mask);
>> + if (ret < 0) {
>> + mutex_unlock(&data->lock);
>> + return ret;
>> + }
>> +
>> + ret = regmap_bulk_read(data->regmap,
>> + rpr0521_data_reg[chan->address],
>> + &raw_data, 2);
>> + if (ret < 0) {
>> + rpr0521_set_power_state(data, false, device_mask);
>> + mutex_unlock(&data->lock);
>> + return ret;
>> + }
>> +
>> + ret = rpr0521_set_power_state(data, false, device_mask);
>> + mutex_unlock(&data->lock);
>> + if (ret < 0)
>> + return ret;
>> +
>> + *val = le16_to_cpu(raw_data);
>> + /*
>> + * proximity uses 12 bits, with bits 7:4 of PXS MSB DATA
>> + * being always zero. Also, proximity MUST be exposed as
>> + * a distance with lower values for closer objects.
>> + */
>> + if (chan->type == IIO_PROXIMITY)
>> + *val = RPR0521_PXS_MAX_VAL - *val;
>> +
>> + return IIO_VAL_INT;
>> + case IIO_CHAN_INFO_CALIBSCALE:
>> + mutex_lock(&data->lock);
>> + ret = rpr0521_get_gain(data, chan->address, val);
>> + mutex_unlock(&data->lock);
>> + if (ret < 0)
>> + return ret;
>> +
>> + return IIO_VAL_INT;
>> + default:
>> + return -EINVAL;
>> + }
>> +}
>> +
>> +static int rpr0521_write_raw(struct iio_dev *indio_dev,
>> + struct iio_chan_spec const *chan, int val,
>> + int val2, long mask)
>> +{
>> + struct rpr0521_data *data = iio_priv(indio_dev);
>> + int ret;
>> +
>> + switch (mask) {
>> + case IIO_CHAN_INFO_CALIBSCALE:
>> + mutex_lock(&data->lock);
>> + ret = rpr0521_set_gain(data, chan->address, val);
>> + mutex_unlock(&data->lock);
>> +
>> + return ret;
>> + default:
>> + return -EINVAL;
>> + }
>> +}
>> +
>> +static IIO_CONST_ATTR(in_intensity_calibscale_available, "1 2 64 128");
>> +static IIO_CONST_ATTR(in_proximity_calibscale_available, "1 2 4");
>> +
>> +static struct attribute *rpr0521_attributes[] = {
>> + &iio_const_attr_in_intensity_calibscale_available.dev_attr.attr,
>> + &iio_const_attr_in_proximity_calibscale_available.dev_attr.attr,
>> + NULL
>> +};
>> +
>> +static const struct attribute_group rpr0521_attribute_group = {
>> + .attrs = rpr0521_attributes,
>> +};
>> +
>> +static const struct iio_info rpr0521_info = {
>> + .driver_module = THIS_MODULE,
>> + .read_raw = rpr0521_read_raw,
>> + .write_raw = rpr0521_write_raw,
>> + .attrs = &rpr0521_attribute_group,
>> +};
>> +
>> +static int rpr0521_init(struct rpr0521_data *data)
>> +{
>> + int ret;
>> + int id;
>> +
>> + ret = regmap_read(data->regmap, RPR0521_REG_ID, &id);
>> + if (ret < 0) {
>> + dev_err(&data->client->dev, "Failed to read REG_ID register\n");
>> + return ret;
>> + }
>> +
>> + if (id != RPR0521_MANUFACT_ID) {
>> + dev_err(&data->client->dev, "Wrong id, got %x expected %x\n",
>> + id, RPR0521_MANUFACT_ID);
>> + return -ENODEV;
>> + }
>> +
>> + /* set default measurement time - 100 ms for both ALS and PS */
>> + ret = regmap_update_bits(data->regmap, RPR0521_REG_MODE_CTRL,
>> + RPR0521_MODE_MEAS_TIME_MASK,
>> + RPR0521_DEFAULT_MEAS_TIME);
>> + if (ret) {
>> + pr_err("regmap_update bits returned %d\n", ret);
>> + return ret;
>> + }
>> +
>> + ret = rpr0521_als_enable(data, RPR0521_MODE_ALS_ENABLE);
>> + if (ret < 0)
>> + return ret;
>> + ret = rpr0521_pxs_enable(data, RPR0521_MODE_PXS_ENABLE);
>> + if (ret < 0)
>> + return ret;
>> +
>> + return 0;
>> +}
>> +
>> +static int rpr0521_poweroff(struct rpr0521_data *data)
>> +{
>> + int ret;
>> +
>> + ret = regmap_update_bits(data->regmap, RPR0521_REG_MODE_CTRL,
>> + RPR0521_MODE_ALS_MASK |
>> + RPR0521_MODE_PXS_MASK,
>> + 0x00);
>> + if (ret < 0)
>> + return ret;
>> +
>> + data->als_dev_en = false;
>> + data->pxs_dev_en = false;
>> +
>> + return 0;
>> +}
>> +
>> +static bool rpr0521_is_volatile_reg(struct device *dev, unsigned int reg)
>> +{
>> + switch (reg) {
>> + case RPR0521_REG_MODE_CTRL:
>> + case RPR0521_REG_ALS_CTRL:
>> + case RPR0521_REG_PXS_CTRL:
>> + return false;
>> + default:
>> + return true;
>> + }
>> +}
>> +
>> +static const struct regmap_config rpr0521_regmap_config = {
>> + .name = RPR0521_REGMAP_NAME,
>> +
>> + .reg_bits = 8,
>> + .val_bits = 8,
>> +
>> + .max_register = RPR0521_REG_ID,
>> + .cache_type = REGCACHE_RBTREE,
>> + .volatile_reg = rpr0521_is_volatile_reg,
>> +};
>> +
>> +static int rpr0521_probe(struct i2c_client *client,
>> + const struct i2c_device_id *id)
>> +{
>> + struct rpr0521_data *data;
>> + struct iio_dev *indio_dev;
>> + struct regmap *regmap;
>> + int ret;
>> +
>> + indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
>> + if (!indio_dev)
>> + return -ENOMEM;
>> +
>> + regmap = devm_regmap_init_i2c(client, &rpr0521_regmap_config);
>> + if (IS_ERR(regmap)) {
>> + dev_err(&client->dev, "Regmap init failed!\n");
>> + return PTR_ERR(regmap);
>> + }
>> +
>> + data = iio_priv(indio_dev);
>> + i2c_set_clientdata(client, indio_dev);
>> + data->client = client;
>> + data->regmap = regmap;
>> +
>> + mutex_init(&data->lock);
>> +
>> + indio_dev->dev.parent = &client->dev;
>> + indio_dev->info = &rpr0521_info;
>> + indio_dev->name = RPR0521_DRV_NAME;
>> + indio_dev->channels = rpr0521_channels;
>> + indio_dev->num_channels = ARRAY_SIZE(rpr0521_channels);
>> + indio_dev->modes = INDIO_DIRECT_MODE;
>> +
>> + ret = rpr0521_init(data);
>> + if (ret < 0) {
>> + dev_err(&client->dev, "rpr0521 chip init failed\n");
>> + return ret;
>> + }
>> + ret = iio_device_register(indio_dev);
>> + if (ret < 0)
>> + return ret;
>> +
>> + ret = pm_runtime_set_active(&client->dev);
>> + if (ret < 0)
>> + goto err_iio_unregister;
>> +
>> + pm_runtime_enable(&client->dev);
>> + pm_runtime_set_autosuspend_delay(&client->dev, RPR0521_SLEEP_DELAY_MS);
>> + pm_runtime_use_autosuspend(&client->dev);
>> +
>> + return 0;
>> +err_iio_unregister:
>> + iio_device_unregister(indio_dev);
>> + return ret;
>> +}
>> +
>> +static int rpr0521_remove(struct i2c_client *client)
>> +{
>> + struct iio_dev *indio_dev = i2c_get_clientdata(client);
>> +
>> + pm_runtime_disable(&client->dev);
>> + pm_runtime_set_suspended(&client->dev);
>> + pm_runtime_put_noidle(&client->dev);
>> +
>> + iio_device_unregister(indio_dev);
>> + rpr0521_poweroff(iio_priv(indio_dev));
>> +
>> + return 0;
>> +}
>> +
>> +#ifdef CONFIG_PM
>> +static int rpr0521_runtime_suspend(struct device *dev)
>> +{
>> + struct iio_dev *indio_dev = i2c_get_clientdata(to_i2c_client(dev));
>> + struct rpr0521_data *data = iio_priv(indio_dev);
>> + int ret;
>> +
>> + mutex_lock(&data->lock);
>> + ret = rpr0521_poweroff(data);
> //Disables the channels and sets *_dev_en = false.
>> + mutex_unlock(&data->lock);
>> +
>> + return ret;
>> +}
>> +
>> +static int rpr0521_runtime_resume(struct device *dev)
>> +{
>> + struct iio_dev *indio_dev = i2c_get_clientdata(to_i2c_client(dev));
>> + struct rpr0521_data *data = iio_priv(indio_dev);
>> + int ret;
>> +
>> + if (data->als_ps_need_en) {
>> + ret = rpr0521_als_enable(data, RPR0521_MODE_ALS_ENABLE);
>> + if (ret < 0)
>> + return ret;
>> + data->als_ps_need_en = false;
>> + }
>> +
>> + if (data->pxs_ps_need_en) {
>> + ret = rpr0521_pxs_enable(data, RPR0521_MODE_PXS_ENABLE);
>> + if (ret < 0)
>> + return ret;
>> + data->pxs_ps_need_en = false;
>> + }
>> +
>> + return 0;
>> +}
>> +#endif
>> +
>> +static const struct dev_pm_ops rpr0521_pm_ops = {
>> + SET_RUNTIME_PM_OPS(rpr0521_runtime_suspend,
>> + rpr0521_runtime_resume, NULL)
>> +};
>> +
>> +static const struct acpi_device_id rpr0521_acpi_match[] = {
>> + {"RPR0521", 0},
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(acpi, rpr0521_acpi_match);
>> +
>> +static const struct i2c_device_id rpr0521_id[] = {
>> + {"rpr0521", 0},
>> + { }
>> +};
>> +
>> +MODULE_DEVICE_TABLE(i2c, rpr0521_id);
>> +
>> +static struct i2c_driver rpr0521_driver = {
>> + .driver = {
>> + .name = RPR0521_DRV_NAME,
>> + .pm = &rpr0521_pm_ops,
>> + .acpi_match_table = ACPI_PTR(rpr0521_acpi_match),
>> + },
>> + .probe = rpr0521_probe,
>> + .remove = rpr0521_remove,
>> + .id_table = rpr0521_id,
>> +};
>> +
>> +module_i2c_driver(rpr0521_driver);
>> +
>> +MODULE_AUTHOR("Daniel Baluta <daniel.baluta@...el.com>");
>> +MODULE_DESCRIPTION("RPR0521 ROHM Ambient Light and Proximity Sensor driver");
>> +MODULE_LICENSE("GPL v2");
>>
>
> --
> 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