| .. | .. |
|---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-only |
|---|
| 1 | 2 | /* |
|---|
| 2 | 3 | * AD5592R Digital <-> Analog converters driver |
|---|
| 3 | 4 | * |
|---|
| 4 | 5 | * Copyright 2014-2016 Analog Devices Inc. |
|---|
| 5 | 6 | * Author: Paul Cercueil <paul.cercueil@analog.com> |
|---|
| 6 | | - * |
|---|
| 7 | | - * Licensed under the GPL-2. |
|---|
| 8 | 7 | */ |
|---|
| 9 | 8 | |
|---|
| 10 | 9 | #include <linux/bitops.h> |
|---|
| .. | .. |
|---|
| 16 | 15 | #include <linux/regulator/consumer.h> |
|---|
| 17 | 16 | #include <linux/gpio/consumer.h> |
|---|
| 18 | 17 | #include <linux/gpio/driver.h> |
|---|
| 19 | | -#include <linux/gpio.h> |
|---|
| 20 | 18 | #include <linux/property.h> |
|---|
| 21 | 19 | |
|---|
| 22 | 20 | #include <dt-bindings/iio/adi,ad5592r.h> |
|---|
| .. | .. |
|---|
| 158 | 156 | static int ad5592r_reset(struct ad5592r_state *st) |
|---|
| 159 | 157 | { |
|---|
| 160 | 158 | struct gpio_desc *gpio; |
|---|
| 161 | | - struct iio_dev *iio_dev = iio_priv_to_dev(st); |
|---|
| 162 | 159 | |
|---|
| 163 | 160 | gpio = devm_gpiod_get_optional(st->dev, "reset", GPIOD_OUT_LOW); |
|---|
| 164 | 161 | if (IS_ERR(gpio)) |
|---|
| .. | .. |
|---|
| 168 | 165 | udelay(1); |
|---|
| 169 | 166 | gpiod_set_value(gpio, 1); |
|---|
| 170 | 167 | } else { |
|---|
| 171 | | - mutex_lock(&iio_dev->mlock); |
|---|
| 168 | + mutex_lock(&st->lock); |
|---|
| 172 | 169 | /* Writing this magic value resets the device */ |
|---|
| 173 | 170 | st->ops->reg_write(st, AD5592R_REG_RESET, 0xdac); |
|---|
| 174 | | - mutex_unlock(&iio_dev->mlock); |
|---|
| 171 | + mutex_unlock(&st->lock); |
|---|
| 175 | 172 | } |
|---|
| 176 | 173 | |
|---|
| 177 | 174 | udelay(250); |
|---|
| .. | .. |
|---|
| 199 | 196 | const struct ad5592r_rw_ops *ops = st->ops; |
|---|
| 200 | 197 | int ret; |
|---|
| 201 | 198 | unsigned i; |
|---|
| 202 | | - struct iio_dev *iio_dev = iio_priv_to_dev(st); |
|---|
| 203 | 199 | u8 pulldown = 0, tristate = 0, dac = 0, adc = 0; |
|---|
| 204 | 200 | u16 read_back; |
|---|
| 205 | 201 | |
|---|
| .. | .. |
|---|
| 224 | 220 | break; |
|---|
| 225 | 221 | |
|---|
| 226 | 222 | case CH_MODE_UNUSED: |
|---|
| 227 | | - /* fall-through */ |
|---|
| 228 | 223 | default: |
|---|
| 229 | 224 | switch (st->channel_offstate[i]) { |
|---|
| 230 | 225 | case CH_OFFSTATE_OUT_TRISTATE: |
|---|
| .. | .. |
|---|
| 241 | 236 | break; |
|---|
| 242 | 237 | |
|---|
| 243 | 238 | case CH_OFFSTATE_PULLDOWN: |
|---|
| 244 | | - /* fall-through */ |
|---|
| 245 | 239 | default: |
|---|
| 246 | 240 | pulldown |= BIT(i); |
|---|
| 247 | 241 | break; |
|---|
| .. | .. |
|---|
| 249 | 243 | } |
|---|
| 250 | 244 | } |
|---|
| 251 | 245 | |
|---|
| 252 | | - mutex_lock(&iio_dev->mlock); |
|---|
| 246 | + mutex_lock(&st->lock); |
|---|
| 253 | 247 | |
|---|
| 254 | 248 | /* Pull down unused pins to GND */ |
|---|
| 255 | 249 | ret = ops->reg_write(st, AD5592R_REG_PULLDOWN, pulldown); |
|---|
| .. | .. |
|---|
| 287 | 281 | ret = -EIO; |
|---|
| 288 | 282 | |
|---|
| 289 | 283 | err_unlock: |
|---|
| 290 | | - mutex_unlock(&iio_dev->mlock); |
|---|
| 284 | + mutex_unlock(&st->lock); |
|---|
| 291 | 285 | return ret; |
|---|
| 292 | 286 | } |
|---|
| 293 | 287 | |
|---|
| .. | .. |
|---|
| 316 | 310 | if (!chan->output) |
|---|
| 317 | 311 | return -EINVAL; |
|---|
| 318 | 312 | |
|---|
| 319 | | - mutex_lock(&iio_dev->mlock); |
|---|
| 313 | + mutex_lock(&st->lock); |
|---|
| 320 | 314 | ret = st->ops->write_dac(st, chan->channel, val); |
|---|
| 321 | 315 | if (!ret) |
|---|
| 322 | 316 | st->cached_dac[chan->channel] = val; |
|---|
| 323 | | - mutex_unlock(&iio_dev->mlock); |
|---|
| 317 | + mutex_unlock(&st->lock); |
|---|
| 324 | 318 | return ret; |
|---|
| 325 | 319 | case IIO_CHAN_INFO_SCALE: |
|---|
| 326 | 320 | if (chan->type == IIO_VOLTAGE) { |
|---|
| .. | .. |
|---|
| 335 | 329 | else |
|---|
| 336 | 330 | return -EINVAL; |
|---|
| 337 | 331 | |
|---|
| 338 | | - mutex_lock(&iio_dev->mlock); |
|---|
| 332 | + mutex_lock(&st->lock); |
|---|
| 339 | 333 | |
|---|
| 340 | 334 | ret = st->ops->reg_read(st, AD5592R_REG_CTRL, |
|---|
| 341 | 335 | &st->cached_gp_ctrl); |
|---|
| 342 | 336 | if (ret < 0) { |
|---|
| 343 | | - mutex_unlock(&iio_dev->mlock); |
|---|
| 337 | + mutex_unlock(&st->lock); |
|---|
| 344 | 338 | return ret; |
|---|
| 345 | 339 | } |
|---|
| 346 | 340 | |
|---|
| .. | .. |
|---|
| 362 | 356 | |
|---|
| 363 | 357 | ret = st->ops->reg_write(st, AD5592R_REG_CTRL, |
|---|
| 364 | 358 | st->cached_gp_ctrl); |
|---|
| 365 | | - mutex_unlock(&iio_dev->mlock); |
|---|
| 359 | + mutex_unlock(&st->lock); |
|---|
| 366 | 360 | |
|---|
| 367 | 361 | return ret; |
|---|
| 368 | 362 | } |
|---|
| .. | .. |
|---|
| 380 | 374 | { |
|---|
| 381 | 375 | struct ad5592r_state *st = iio_priv(iio_dev); |
|---|
| 382 | 376 | u16 read_val; |
|---|
| 383 | | - int ret; |
|---|
| 377 | + int ret, mult; |
|---|
| 384 | 378 | |
|---|
| 385 | 379 | switch (m) { |
|---|
| 386 | 380 | case IIO_CHAN_INFO_RAW: |
|---|
| 387 | | - mutex_lock(&iio_dev->mlock); |
|---|
| 388 | | - |
|---|
| 389 | 381 | if (!chan->output) { |
|---|
| 382 | + mutex_lock(&st->lock); |
|---|
| 390 | 383 | ret = st->ops->read_adc(st, chan->channel, &read_val); |
|---|
| 384 | + mutex_unlock(&st->lock); |
|---|
| 391 | 385 | if (ret) |
|---|
| 392 | | - goto unlock; |
|---|
| 386 | + return ret; |
|---|
| 393 | 387 | |
|---|
| 394 | 388 | if ((read_val >> 12 & 0x7) != (chan->channel & 0x7)) { |
|---|
| 395 | 389 | dev_err(st->dev, "Error while reading channel %u\n", |
|---|
| 396 | 390 | chan->channel); |
|---|
| 397 | | - ret = -EIO; |
|---|
| 398 | | - goto unlock; |
|---|
| 391 | + return -EIO; |
|---|
| 399 | 392 | } |
|---|
| 400 | 393 | |
|---|
| 401 | 394 | read_val &= GENMASK(11, 0); |
|---|
| 402 | 395 | |
|---|
| 403 | 396 | } else { |
|---|
| 397 | + mutex_lock(&st->lock); |
|---|
| 404 | 398 | read_val = st->cached_dac[chan->channel]; |
|---|
| 399 | + mutex_unlock(&st->lock); |
|---|
| 405 | 400 | } |
|---|
| 406 | 401 | |
|---|
| 407 | 402 | dev_dbg(st->dev, "Channel %u read: 0x%04hX\n", |
|---|
| 408 | 403 | chan->channel, read_val); |
|---|
| 409 | 404 | |
|---|
| 410 | 405 | *val = (int) read_val; |
|---|
| 411 | | - ret = IIO_VAL_INT; |
|---|
| 412 | | - break; |
|---|
| 406 | + return IIO_VAL_INT; |
|---|
| 413 | 407 | case IIO_CHAN_INFO_SCALE: |
|---|
| 414 | 408 | *val = ad5592r_get_vref(st); |
|---|
| 415 | 409 | |
|---|
| .. | .. |
|---|
| 418 | 412 | *val = div_s64_rem(tmp, 1000000000LL, val2); |
|---|
| 419 | 413 | |
|---|
| 420 | 414 | return IIO_VAL_INT_PLUS_MICRO; |
|---|
| 421 | | - } else { |
|---|
| 422 | | - int mult; |
|---|
| 423 | | - |
|---|
| 424 | | - mutex_lock(&iio_dev->mlock); |
|---|
| 425 | | - |
|---|
| 426 | | - if (chan->output) |
|---|
| 427 | | - mult = !!(st->cached_gp_ctrl & |
|---|
| 428 | | - AD5592R_REG_CTRL_DAC_RANGE); |
|---|
| 429 | | - else |
|---|
| 430 | | - mult = !!(st->cached_gp_ctrl & |
|---|
| 431 | | - AD5592R_REG_CTRL_ADC_RANGE); |
|---|
| 432 | | - |
|---|
| 433 | | - *val *= ++mult; |
|---|
| 434 | | - |
|---|
| 435 | | - *val2 = chan->scan_type.realbits; |
|---|
| 436 | | - ret = IIO_VAL_FRACTIONAL_LOG2; |
|---|
| 437 | 415 | } |
|---|
| 438 | | - break; |
|---|
| 416 | + |
|---|
| 417 | + mutex_lock(&st->lock); |
|---|
| 418 | + |
|---|
| 419 | + if (chan->output) |
|---|
| 420 | + mult = !!(st->cached_gp_ctrl & |
|---|
| 421 | + AD5592R_REG_CTRL_DAC_RANGE); |
|---|
| 422 | + else |
|---|
| 423 | + mult = !!(st->cached_gp_ctrl & |
|---|
| 424 | + AD5592R_REG_CTRL_ADC_RANGE); |
|---|
| 425 | + |
|---|
| 426 | + mutex_unlock(&st->lock); |
|---|
| 427 | + |
|---|
| 428 | + *val *= ++mult; |
|---|
| 429 | + |
|---|
| 430 | + *val2 = chan->scan_type.realbits; |
|---|
| 431 | + |
|---|
| 432 | + return IIO_VAL_FRACTIONAL_LOG2; |
|---|
| 439 | 433 | case IIO_CHAN_INFO_OFFSET: |
|---|
| 440 | 434 | ret = ad5592r_get_vref(st); |
|---|
| 441 | 435 | |
|---|
| 442 | | - mutex_lock(&iio_dev->mlock); |
|---|
| 436 | + mutex_lock(&st->lock); |
|---|
| 443 | 437 | |
|---|
| 444 | 438 | if (st->cached_gp_ctrl & AD5592R_REG_CTRL_ADC_RANGE) |
|---|
| 445 | 439 | *val = (-34365 * 25) / ret; |
|---|
| 446 | 440 | else |
|---|
| 447 | 441 | *val = (-75365 * 25) / ret; |
|---|
| 448 | | - ret = IIO_VAL_INT; |
|---|
| 449 | | - break; |
|---|
| 442 | + |
|---|
| 443 | + mutex_unlock(&st->lock); |
|---|
| 444 | + |
|---|
| 445 | + return IIO_VAL_INT; |
|---|
| 450 | 446 | default: |
|---|
| 451 | 447 | return -EINVAL; |
|---|
| 452 | 448 | } |
|---|
| 453 | | - |
|---|
| 454 | | -unlock: |
|---|
| 455 | | - mutex_unlock(&iio_dev->mlock); |
|---|
| 456 | | - return ret; |
|---|
| 457 | 449 | } |
|---|
| 458 | 450 | |
|---|
| 459 | 451 | static int ad5592r_write_raw_get_fmt(struct iio_dev *indio_dev, |
|---|
| .. | .. |
|---|
| 488 | 480 | st->scale_avail[1][0], st->scale_avail[1][1]); |
|---|
| 489 | 481 | } |
|---|
| 490 | 482 | |
|---|
| 491 | | -static struct iio_chan_spec_ext_info ad5592r_ext_info[] = { |
|---|
| 483 | +static const struct iio_chan_spec_ext_info ad5592r_ext_info[] = { |
|---|
| 492 | 484 | { |
|---|
| 493 | 485 | .name = "scale_available", |
|---|
| 494 | 486 | .read = ad5592r_show_scale_available, |
|---|
| 495 | | - .shared = true, |
|---|
| 487 | + .shared = IIO_SHARED_BY_TYPE, |
|---|
| 496 | 488 | }, |
|---|
| 497 | 489 | {}, |
|---|
| 498 | 490 | }; |
|---|
| .. | .. |
|---|
| 512 | 504 | chan->ext_info = ad5592r_ext_info; |
|---|
| 513 | 505 | } |
|---|
| 514 | 506 | |
|---|
| 515 | | -static int ad5592r_alloc_channels(struct ad5592r_state *st) |
|---|
| 507 | +static int ad5592r_alloc_channels(struct iio_dev *iio_dev) |
|---|
| 516 | 508 | { |
|---|
| 509 | + struct ad5592r_state *st = iio_priv(iio_dev); |
|---|
| 517 | 510 | unsigned i, curr_channel = 0, |
|---|
| 518 | 511 | num_channels = st->num_channels; |
|---|
| 519 | | - struct iio_dev *iio_dev = iio_priv_to_dev(st); |
|---|
| 520 | 512 | struct iio_chan_spec *channels; |
|---|
| 521 | 513 | struct fwnode_handle *child; |
|---|
| 522 | 514 | u32 reg, tmp; |
|---|
| .. | .. |
|---|
| 531 | 523 | if (!ret) |
|---|
| 532 | 524 | st->channel_modes[reg] = tmp; |
|---|
| 533 | 525 | |
|---|
| 534 | | - fwnode_property_read_u32(child, "adi,off-state", &tmp); |
|---|
| 526 | + ret = fwnode_property_read_u32(child, "adi,off-state", &tmp); |
|---|
| 535 | 527 | if (!ret) |
|---|
| 536 | 528 | st->channel_offstate[reg] = tmp; |
|---|
| 537 | 529 | } |
|---|
| .. | .. |
|---|
| 622 | 614 | return ret; |
|---|
| 623 | 615 | } |
|---|
| 624 | 616 | |
|---|
| 625 | | - iio_dev->dev.parent = dev; |
|---|
| 626 | 617 | iio_dev->name = name; |
|---|
| 627 | 618 | iio_dev->info = &ad5592r_info; |
|---|
| 628 | 619 | iio_dev->modes = INDIO_DIRECT_MODE; |
|---|
| 620 | + |
|---|
| 621 | + mutex_init(&st->lock); |
|---|
| 629 | 622 | |
|---|
| 630 | 623 | ad5592r_init_scales(st, ad5592r_get_vref(st)); |
|---|
| 631 | 624 | |
|---|
| .. | .. |
|---|
| 638 | 631 | if (ret) |
|---|
| 639 | 632 | goto error_disable_reg; |
|---|
| 640 | 633 | |
|---|
| 641 | | - ret = ad5592r_alloc_channels(st); |
|---|
| 634 | + ret = ad5592r_alloc_channels(iio_dev); |
|---|
| 642 | 635 | if (ret) |
|---|
| 643 | 636 | goto error_disable_reg; |
|---|
| 644 | 637 | |
|---|