Commit d62f68c1 authored by Jonathan Cameron's avatar Jonathan Cameron

iio: proximity: sx9324: Use automated cleanup for locks and IIO mode claiming.

This simplifies error handling paths and generallly removes a bunch
of boilerplate.
Reviewed-by: default avatarNuno Sa <nuno.a@analog.com>
Link: https://lore.kernel.org/r/20240128150537.44592-8-jic23@kernel.orgSigned-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent fba730dd
...@@ -429,25 +429,16 @@ static int sx9324_read_raw(struct iio_dev *indio_dev, ...@@ -429,25 +429,16 @@ static int sx9324_read_raw(struct iio_dev *indio_dev,
int *val, int *val2, long mask) int *val, int *val2, long mask)
{ {
struct sx_common_data *data = iio_priv(indio_dev); struct sx_common_data *data = iio_priv(indio_dev);
int ret;
switch (mask) { switch (mask) {
case IIO_CHAN_INFO_RAW: case IIO_CHAN_INFO_RAW:
ret = iio_device_claim_direct_mode(indio_dev); iio_device_claim_direct_scoped(return -EBUSY, indio_dev)
if (ret) return sx_common_read_proximity(data, chan, val);
return ret; unreachable();
ret = sx_common_read_proximity(data, chan, val);
iio_device_release_direct_mode(indio_dev);
return ret;
case IIO_CHAN_INFO_HARDWAREGAIN: case IIO_CHAN_INFO_HARDWAREGAIN:
ret = iio_device_claim_direct_mode(indio_dev); iio_device_claim_direct_scoped(return -EBUSY, indio_dev)
if (ret) return sx9324_read_gain(data, chan, val);
return ret; unreachable();
ret = sx9324_read_gain(data, chan, val);
iio_device_release_direct_mode(indio_dev);
return ret;
case IIO_CHAN_INFO_SAMP_FREQ: case IIO_CHAN_INFO_SAMP_FREQ:
return sx9324_read_samp_freq(data, val, val2); return sx9324_read_samp_freq(data, val, val2);
default: default:
...@@ -484,7 +475,7 @@ static int sx9324_read_avail(struct iio_dev *indio_dev, ...@@ -484,7 +475,7 @@ static int sx9324_read_avail(struct iio_dev *indio_dev,
static int sx9324_set_samp_freq(struct sx_common_data *data, static int sx9324_set_samp_freq(struct sx_common_data *data,
int val, int val2) int val, int val2)
{ {
int i, ret; int i;
for (i = 0; i < ARRAY_SIZE(sx9324_samp_freq_table); i++) for (i = 0; i < ARRAY_SIZE(sx9324_samp_freq_table); i++)
if (val == sx9324_samp_freq_table[i].val && if (val == sx9324_samp_freq_table[i].val &&
...@@ -494,15 +485,11 @@ static int sx9324_set_samp_freq(struct sx_common_data *data, ...@@ -494,15 +485,11 @@ static int sx9324_set_samp_freq(struct sx_common_data *data,
if (i == ARRAY_SIZE(sx9324_samp_freq_table)) if (i == ARRAY_SIZE(sx9324_samp_freq_table))
return -EINVAL; return -EINVAL;
mutex_lock(&data->mutex); guard(mutex)(&data->mutex);
ret = regmap_update_bits(data->regmap,
SX9324_REG_GNRL_CTRL0,
SX9324_REG_GNRL_CTRL0_SCANPERIOD_MASK, i);
mutex_unlock(&data->mutex); return regmap_update_bits(data->regmap,
SX9324_REG_GNRL_CTRL0,
return ret; SX9324_REG_GNRL_CTRL0_SCANPERIOD_MASK, i);
} }
static int sx9324_read_thresh(struct sx_common_data *data, static int sx9324_read_thresh(struct sx_common_data *data,
...@@ -623,7 +610,6 @@ static int sx9324_write_thresh(struct sx_common_data *data, ...@@ -623,7 +610,6 @@ static int sx9324_write_thresh(struct sx_common_data *data,
const struct iio_chan_spec *chan, int _val) const struct iio_chan_spec *chan, int _val)
{ {
unsigned int reg, val = _val; unsigned int reg, val = _val;
int ret;
reg = SX9324_REG_PROX_CTRL6 + chan->channel / 2; reg = SX9324_REG_PROX_CTRL6 + chan->channel / 2;
...@@ -633,11 +619,9 @@ static int sx9324_write_thresh(struct sx_common_data *data, ...@@ -633,11 +619,9 @@ static int sx9324_write_thresh(struct sx_common_data *data,
if (val > 0xff) if (val > 0xff)
return -EINVAL; return -EINVAL;
mutex_lock(&data->mutex); guard(mutex)(&data->mutex);
ret = regmap_write(data->regmap, reg, val);
mutex_unlock(&data->mutex);
return ret; return regmap_write(data->regmap, reg, val);
} }
static int sx9324_write_hysteresis(struct sx_common_data *data, static int sx9324_write_hysteresis(struct sx_common_data *data,
...@@ -662,18 +646,15 @@ static int sx9324_write_hysteresis(struct sx_common_data *data, ...@@ -662,18 +646,15 @@ static int sx9324_write_hysteresis(struct sx_common_data *data,
return -EINVAL; return -EINVAL;
hyst = FIELD_PREP(SX9324_REG_PROX_CTRL5_HYST_MASK, hyst); hyst = FIELD_PREP(SX9324_REG_PROX_CTRL5_HYST_MASK, hyst);
mutex_lock(&data->mutex); guard(mutex)(&data->mutex);
ret = regmap_update_bits(data->regmap, SX9324_REG_PROX_CTRL5,
SX9324_REG_PROX_CTRL5_HYST_MASK, hyst);
mutex_unlock(&data->mutex);
return ret; return regmap_update_bits(data->regmap, SX9324_REG_PROX_CTRL5,
SX9324_REG_PROX_CTRL5_HYST_MASK, hyst);
} }
static int sx9324_write_far_debounce(struct sx_common_data *data, int _val) static int sx9324_write_far_debounce(struct sx_common_data *data, int _val)
{ {
unsigned int regval, val = _val; unsigned int regval, val = _val;
int ret;
if (val > 0) if (val > 0)
val = ilog2(val); val = ilog2(val);
...@@ -682,19 +663,16 @@ static int sx9324_write_far_debounce(struct sx_common_data *data, int _val) ...@@ -682,19 +663,16 @@ static int sx9324_write_far_debounce(struct sx_common_data *data, int _val)
regval = FIELD_PREP(SX9324_REG_PROX_CTRL5_FAR_DEBOUNCE_MASK, val); regval = FIELD_PREP(SX9324_REG_PROX_CTRL5_FAR_DEBOUNCE_MASK, val);
mutex_lock(&data->mutex); guard(mutex)(&data->mutex);
ret = regmap_update_bits(data->regmap, SX9324_REG_PROX_CTRL5,
SX9324_REG_PROX_CTRL5_FAR_DEBOUNCE_MASK,
regval);
mutex_unlock(&data->mutex);
return ret; return regmap_update_bits(data->regmap, SX9324_REG_PROX_CTRL5,
SX9324_REG_PROX_CTRL5_FAR_DEBOUNCE_MASK,
regval);
} }
static int sx9324_write_close_debounce(struct sx_common_data *data, int _val) static int sx9324_write_close_debounce(struct sx_common_data *data, int _val)
{ {
unsigned int regval, val = _val; unsigned int regval, val = _val;
int ret;
if (val > 0) if (val > 0)
val = ilog2(val); val = ilog2(val);
...@@ -703,13 +681,11 @@ static int sx9324_write_close_debounce(struct sx_common_data *data, int _val) ...@@ -703,13 +681,11 @@ static int sx9324_write_close_debounce(struct sx_common_data *data, int _val)
regval = FIELD_PREP(SX9324_REG_PROX_CTRL5_CLOSE_DEBOUNCE_MASK, val); regval = FIELD_PREP(SX9324_REG_PROX_CTRL5_CLOSE_DEBOUNCE_MASK, val);
mutex_lock(&data->mutex); guard(mutex)(&data->mutex);
ret = regmap_update_bits(data->regmap, SX9324_REG_PROX_CTRL5,
SX9324_REG_PROX_CTRL5_CLOSE_DEBOUNCE_MASK,
regval);
mutex_unlock(&data->mutex);
return ret; return regmap_update_bits(data->regmap, SX9324_REG_PROX_CTRL5,
SX9324_REG_PROX_CTRL5_CLOSE_DEBOUNCE_MASK,
regval);
} }
static int sx9324_write_event_val(struct iio_dev *indio_dev, static int sx9324_write_event_val(struct iio_dev *indio_dev,
...@@ -746,7 +722,6 @@ static int sx9324_write_gain(struct sx_common_data *data, ...@@ -746,7 +722,6 @@ static int sx9324_write_gain(struct sx_common_data *data,
const struct iio_chan_spec *chan, int val) const struct iio_chan_spec *chan, int val)
{ {
unsigned int gain, reg; unsigned int gain, reg;
int ret;
reg = SX9324_REG_PROX_CTRL0 + chan->channel / 2; reg = SX9324_REG_PROX_CTRL0 + chan->channel / 2;
...@@ -756,13 +731,11 @@ static int sx9324_write_gain(struct sx_common_data *data, ...@@ -756,13 +731,11 @@ static int sx9324_write_gain(struct sx_common_data *data,
gain = FIELD_PREP(SX9324_REG_PROX_CTRL0_GAIN_MASK, gain); gain = FIELD_PREP(SX9324_REG_PROX_CTRL0_GAIN_MASK, gain);
mutex_lock(&data->mutex); guard(mutex)(&data->mutex);
ret = regmap_update_bits(data->regmap, reg,
SX9324_REG_PROX_CTRL0_GAIN_MASK,
gain);
mutex_unlock(&data->mutex);
return ret; return regmap_update_bits(data->regmap, reg,
SX9324_REG_PROX_CTRL0_GAIN_MASK,
gain);
} }
static int sx9324_write_raw(struct iio_dev *indio_dev, static int sx9324_write_raw(struct iio_dev *indio_dev,
...@@ -1092,34 +1065,30 @@ static int sx9324_suspend(struct device *dev) ...@@ -1092,34 +1065,30 @@ static int sx9324_suspend(struct device *dev)
disable_irq_nosync(data->client->irq); disable_irq_nosync(data->client->irq);
mutex_lock(&data->mutex); guard(mutex)(&data->mutex);
ret = regmap_read(data->regmap, SX9324_REG_GNRL_CTRL1, &regval); ret = regmap_read(data->regmap, SX9324_REG_GNRL_CTRL1, &regval);
if (ret < 0)
return ret;
data->suspend_ctrl = data->suspend_ctrl =
FIELD_GET(SX9324_REG_GNRL_CTRL1_PHEN_MASK, regval); FIELD_GET(SX9324_REG_GNRL_CTRL1_PHEN_MASK, regval);
if (ret < 0)
goto out;
/* Disable all phases, send the device to sleep. */ /* Disable all phases, send the device to sleep. */
ret = regmap_write(data->regmap, SX9324_REG_GNRL_CTRL1, 0); return regmap_write(data->regmap, SX9324_REG_GNRL_CTRL1, 0);
out:
mutex_unlock(&data->mutex);
return ret;
} }
static int sx9324_resume(struct device *dev) static int sx9324_resume(struct device *dev)
{ {
struct sx_common_data *data = iio_priv(dev_get_drvdata(dev)); struct sx_common_data *data = iio_priv(dev_get_drvdata(dev));
int ret;
mutex_lock(&data->mutex); scoped_guard(mutex, &data->mutex) {
ret = regmap_write(data->regmap, SX9324_REG_GNRL_CTRL1, int ret = regmap_write(data->regmap, SX9324_REG_GNRL_CTRL1,
data->suspend_ctrl | SX9324_REG_GNRL_CTRL1_PAUSECTRL); data->suspend_ctrl |
mutex_unlock(&data->mutex); SX9324_REG_GNRL_CTRL1_PAUSECTRL);
if (ret) if (ret)
return ret; return ret;
}
enable_irq(data->client->irq); enable_irq(data->client->irq);
return 0; return 0;
......
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