Commit 6d923f8f authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

Merge tag 'iio-fixes-5.0a' of...

Merge tag 'iio-fixes-5.0a' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio into staging-linus

Jonathan writes:

First set of IIO fixes for the 5.0 cycle.

Been a busy month, so these are rather later than they should have been.

* atlas-ph-sensor:
  - Temperature scale didn't correspond to the ABI.
* axp288:
  - A few different fixes around the TS-pin handling.
* ti-ads8688
  - Not enough space in the buffer used to build the scan to allow for
    the timestamp.
* tools - iio_generic_buffer
  - Make num_loops signed so that we really are running for ever
    rather than just a long time when we specify -1.

* tag 'iio-fixes-5.0a' of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio:
  iio: ti-ads8688: Update buffer allocation for timestamps
  tools: iio: iio_generic_buffer: make num_loops signed
  iio: adc: axp288: Fix TS-pin handling
  iio: chemical: atlas-ph-sensor: correct IIO_TEMP values to millicelsius
parents 798badf8 f214ff52
...@@ -27,9 +27,18 @@ ...@@ -27,9 +27,18 @@
#include <linux/iio/machine.h> #include <linux/iio/machine.h>
#include <linux/iio/driver.h> #include <linux/iio/driver.h>
#define AXP288_ADC_EN_MASK 0xF1 /*
#define AXP288_ADC_TS_PIN_GPADC 0xF2 * This mask enables all ADCs except for the battery temp-sensor (TS), that is
#define AXP288_ADC_TS_PIN_ON 0xF3 * left as-is to avoid breaking charging on devices without a temp-sensor.
*/
#define AXP288_ADC_EN_MASK 0xF0
#define AXP288_ADC_TS_ENABLE 0x01
#define AXP288_ADC_TS_CURRENT_ON_OFF_MASK GENMASK(1, 0)
#define AXP288_ADC_TS_CURRENT_OFF (0 << 0)
#define AXP288_ADC_TS_CURRENT_ON_WHEN_CHARGING (1 << 0)
#define AXP288_ADC_TS_CURRENT_ON_ONDEMAND (2 << 0)
#define AXP288_ADC_TS_CURRENT_ON (3 << 0)
enum axp288_adc_id { enum axp288_adc_id {
AXP288_ADC_TS, AXP288_ADC_TS,
...@@ -44,6 +53,7 @@ enum axp288_adc_id { ...@@ -44,6 +53,7 @@ enum axp288_adc_id {
struct axp288_adc_info { struct axp288_adc_info {
int irq; int irq;
struct regmap *regmap; struct regmap *regmap;
bool ts_enabled;
}; };
static const struct iio_chan_spec axp288_adc_channels[] = { static const struct iio_chan_spec axp288_adc_channels[] = {
...@@ -115,21 +125,33 @@ static int axp288_adc_read_channel(int *val, unsigned long address, ...@@ -115,21 +125,33 @@ static int axp288_adc_read_channel(int *val, unsigned long address,
return IIO_VAL_INT; return IIO_VAL_INT;
} }
static int axp288_adc_set_ts(struct regmap *regmap, unsigned int mode, /*
unsigned long address) * The current-source used for the battery temp-sensor (TS) is shared
* with the GPADC. For proper fuel-gauge and charger operation the TS
* current-source needs to be permanently on. But to read the GPADC we
* need to temporary switch the TS current-source to ondemand, so that
* the GPADC can use it, otherwise we will always read an all 0 value.
*/
static int axp288_adc_set_ts(struct axp288_adc_info *info,
unsigned int mode, unsigned long address)
{ {
int ret; int ret;
/* channels other than GPADC do not need to switch TS pin */ /* No need to switch the current-source if the TS pin is disabled */
if (!info->ts_enabled)
return 0;
/* Channels other than GPADC do not need the current source */
if (address != AXP288_GP_ADC_H) if (address != AXP288_GP_ADC_H)
return 0; return 0;
ret = regmap_write(regmap, AXP288_ADC_TS_PIN_CTRL, mode); ret = regmap_update_bits(info->regmap, AXP288_ADC_TS_PIN_CTRL,
AXP288_ADC_TS_CURRENT_ON_OFF_MASK, mode);
if (ret) if (ret)
return ret; return ret;
/* When switching to the GPADC pin give things some time to settle */ /* When switching to the GPADC pin give things some time to settle */
if (mode == AXP288_ADC_TS_PIN_GPADC) if (mode == AXP288_ADC_TS_CURRENT_ON_ONDEMAND)
usleep_range(6000, 10000); usleep_range(6000, 10000);
return 0; return 0;
...@@ -145,14 +167,14 @@ static int axp288_adc_read_raw(struct iio_dev *indio_dev, ...@@ -145,14 +167,14 @@ static int axp288_adc_read_raw(struct iio_dev *indio_dev,
mutex_lock(&indio_dev->mlock); mutex_lock(&indio_dev->mlock);
switch (mask) { switch (mask) {
case IIO_CHAN_INFO_RAW: case IIO_CHAN_INFO_RAW:
if (axp288_adc_set_ts(info->regmap, AXP288_ADC_TS_PIN_GPADC, if (axp288_adc_set_ts(info, AXP288_ADC_TS_CURRENT_ON_ONDEMAND,
chan->address)) { chan->address)) {
dev_err(&indio_dev->dev, "GPADC mode\n"); dev_err(&indio_dev->dev, "GPADC mode\n");
ret = -EINVAL; ret = -EINVAL;
break; break;
} }
ret = axp288_adc_read_channel(val, chan->address, info->regmap); ret = axp288_adc_read_channel(val, chan->address, info->regmap);
if (axp288_adc_set_ts(info->regmap, AXP288_ADC_TS_PIN_ON, if (axp288_adc_set_ts(info, AXP288_ADC_TS_CURRENT_ON,
chan->address)) chan->address))
dev_err(&indio_dev->dev, "TS pin restore\n"); dev_err(&indio_dev->dev, "TS pin restore\n");
break; break;
...@@ -164,13 +186,35 @@ static int axp288_adc_read_raw(struct iio_dev *indio_dev, ...@@ -164,13 +186,35 @@ static int axp288_adc_read_raw(struct iio_dev *indio_dev,
return ret; return ret;
} }
static int axp288_adc_set_state(struct regmap *regmap) static int axp288_adc_initialize(struct axp288_adc_info *info)
{ {
/* ADC should be always enabled for internal FG to function */ int ret, adc_enable_val;
if (regmap_write(regmap, AXP288_ADC_TS_PIN_CTRL, AXP288_ADC_TS_PIN_ON))
return -EIO; /*
* Determine if the TS pin is enabled and set the TS current-source
* accordingly.
*/
ret = regmap_read(info->regmap, AXP20X_ADC_EN1, &adc_enable_val);
if (ret)
return ret;
if (adc_enable_val & AXP288_ADC_TS_ENABLE) {
info->ts_enabled = true;
ret = regmap_update_bits(info->regmap, AXP288_ADC_TS_PIN_CTRL,
AXP288_ADC_TS_CURRENT_ON_OFF_MASK,
AXP288_ADC_TS_CURRENT_ON);
} else {
info->ts_enabled = false;
ret = regmap_update_bits(info->regmap, AXP288_ADC_TS_PIN_CTRL,
AXP288_ADC_TS_CURRENT_ON_OFF_MASK,
AXP288_ADC_TS_CURRENT_OFF);
}
if (ret)
return ret;
return regmap_write(regmap, AXP20X_ADC_EN1, AXP288_ADC_EN_MASK); /* Turn on the ADC for all channels except TS, leave TS as is */
return regmap_update_bits(info->regmap, AXP20X_ADC_EN1,
AXP288_ADC_EN_MASK, AXP288_ADC_EN_MASK);
} }
static const struct iio_info axp288_adc_iio_info = { static const struct iio_info axp288_adc_iio_info = {
...@@ -200,7 +244,7 @@ static int axp288_adc_probe(struct platform_device *pdev) ...@@ -200,7 +244,7 @@ static int axp288_adc_probe(struct platform_device *pdev)
* Set ADC to enabled state at all time, including system suspend. * Set ADC to enabled state at all time, including system suspend.
* otherwise internal fuel gauge functionality may be affected. * otherwise internal fuel gauge functionality may be affected.
*/ */
ret = axp288_adc_set_state(axp20x->regmap); ret = axp288_adc_initialize(info);
if (ret) { if (ret) {
dev_err(&pdev->dev, "unable to enable ADC device\n"); dev_err(&pdev->dev, "unable to enable ADC device\n");
return ret; return ret;
......
...@@ -41,6 +41,7 @@ ...@@ -41,6 +41,7 @@
#define ADS8688_VREF_MV 4096 #define ADS8688_VREF_MV 4096
#define ADS8688_REALBITS 16 #define ADS8688_REALBITS 16
#define ADS8688_MAX_CHANNELS 8
/* /*
* enum ads8688_range - ADS8688 reference voltage range * enum ads8688_range - ADS8688 reference voltage range
...@@ -385,7 +386,7 @@ static irqreturn_t ads8688_trigger_handler(int irq, void *p) ...@@ -385,7 +386,7 @@ static irqreturn_t ads8688_trigger_handler(int irq, void *p)
{ {
struct iio_poll_func *pf = p; struct iio_poll_func *pf = p;
struct iio_dev *indio_dev = pf->indio_dev; struct iio_dev *indio_dev = pf->indio_dev;
u16 buffer[8]; u16 buffer[ADS8688_MAX_CHANNELS + sizeof(s64)/sizeof(u16)];
int i, j = 0; int i, j = 0;
for (i = 0; i < indio_dev->masklength; i++) { for (i = 0; i < indio_dev->masklength; i++) {
......
...@@ -444,9 +444,8 @@ static int atlas_read_raw(struct iio_dev *indio_dev, ...@@ -444,9 +444,8 @@ static int atlas_read_raw(struct iio_dev *indio_dev,
case IIO_CHAN_INFO_SCALE: case IIO_CHAN_INFO_SCALE:
switch (chan->type) { switch (chan->type) {
case IIO_TEMP: case IIO_TEMP:
*val = 1; /* 0.01 */ *val = 10;
*val2 = 100; return IIO_VAL_INT;
break;
case IIO_PH: case IIO_PH:
*val = 1; /* 0.001 */ *val = 1; /* 0.001 */
*val2 = 1000; *val2 = 1000;
...@@ -477,7 +476,7 @@ static int atlas_write_raw(struct iio_dev *indio_dev, ...@@ -477,7 +476,7 @@ static int atlas_write_raw(struct iio_dev *indio_dev,
int val, int val2, long mask) int val, int val2, long mask)
{ {
struct atlas_data *data = iio_priv(indio_dev); struct atlas_data *data = iio_priv(indio_dev);
__be32 reg = cpu_to_be32(val); __be32 reg = cpu_to_be32(val / 10);
if (val2 != 0 || val < 0 || val > 20000) if (val2 != 0 || val < 0 || val > 20000)
return -EINVAL; return -EINVAL;
......
...@@ -330,7 +330,7 @@ static const struct option longopts[] = { ...@@ -330,7 +330,7 @@ static const struct option longopts[] = {
int main(int argc, char **argv) int main(int argc, char **argv)
{ {
unsigned long long num_loops = 2; long long num_loops = 2;
unsigned long timedelay = 1000000; unsigned long timedelay = 1000000;
unsigned long buf_len = 128; unsigned long buf_len = 128;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment