| .. | .. |
|---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-or-later |
|---|
| 1 | 2 | /* |
|---|
| 2 | 3 | * Common library for ADIS16XXX devices |
|---|
| 3 | 4 | * |
|---|
| 4 | 5 | * Copyright 2012 Analog Devices Inc. |
|---|
| 5 | 6 | * Author: Lars-Peter Clausen <lars@metafoo.de> |
|---|
| 6 | | - * |
|---|
| 7 | | - * Licensed under the GPL-2 or later. |
|---|
| 8 | 7 | */ |
|---|
| 9 | 8 | |
|---|
| 10 | 9 | #include <linux/delay.h> |
|---|
| 10 | +#include <linux/gpio/consumer.h> |
|---|
| 11 | 11 | #include <linux/mutex.h> |
|---|
| 12 | 12 | #include <linux/device.h> |
|---|
| 13 | 13 | #include <linux/kernel.h> |
|---|
| .. | .. |
|---|
| 27 | 27 | #define ADIS_MSC_CTRL_DATA_RDY_DIO2 BIT(0) |
|---|
| 28 | 28 | #define ADIS_GLOB_CMD_SW_RESET BIT(7) |
|---|
| 29 | 29 | |
|---|
| 30 | | -int adis_write_reg(struct adis *adis, unsigned int reg, |
|---|
| 30 | +/** |
|---|
| 31 | + * __adis_write_reg() - write N bytes to register (unlocked version) |
|---|
| 32 | + * @adis: The adis device |
|---|
| 33 | + * @reg: The address of the lower of the two registers |
|---|
| 34 | + * @value: The value to write to device (up to 4 bytes) |
|---|
| 35 | + * @size: The size of the @value (in bytes) |
|---|
| 36 | + */ |
|---|
| 37 | +int __adis_write_reg(struct adis *adis, unsigned int reg, |
|---|
| 31 | 38 | unsigned int value, unsigned int size) |
|---|
| 32 | 39 | { |
|---|
| 33 | 40 | unsigned int page = reg / ADIS_PAGE_SIZE; |
|---|
| .. | .. |
|---|
| 39 | 46 | .bits_per_word = 8, |
|---|
| 40 | 47 | .len = 2, |
|---|
| 41 | 48 | .cs_change = 1, |
|---|
| 42 | | - .delay_usecs = adis->data->write_delay, |
|---|
| 49 | + .delay.value = adis->data->write_delay, |
|---|
| 50 | + .delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 51 | + .cs_change_delay.value = adis->data->cs_change_delay, |
|---|
| 52 | + .cs_change_delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 43 | 53 | }, { |
|---|
| 44 | 54 | .tx_buf = adis->tx + 2, |
|---|
| 45 | 55 | .bits_per_word = 8, |
|---|
| 46 | 56 | .len = 2, |
|---|
| 47 | 57 | .cs_change = 1, |
|---|
| 48 | | - .delay_usecs = adis->data->write_delay, |
|---|
| 58 | + .delay.value = adis->data->write_delay, |
|---|
| 59 | + .delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 60 | + .cs_change_delay.value = adis->data->cs_change_delay, |
|---|
| 61 | + .cs_change_delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 49 | 62 | }, { |
|---|
| 50 | 63 | .tx_buf = adis->tx + 4, |
|---|
| 51 | 64 | .bits_per_word = 8, |
|---|
| 52 | 65 | .len = 2, |
|---|
| 53 | 66 | .cs_change = 1, |
|---|
| 54 | | - .delay_usecs = adis->data->write_delay, |
|---|
| 67 | + .delay.value = adis->data->write_delay, |
|---|
| 68 | + .delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 69 | + .cs_change_delay.value = adis->data->cs_change_delay, |
|---|
| 70 | + .cs_change_delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 55 | 71 | }, { |
|---|
| 56 | 72 | .tx_buf = adis->tx + 6, |
|---|
| 57 | 73 | .bits_per_word = 8, |
|---|
| 58 | 74 | .len = 2, |
|---|
| 59 | | - .delay_usecs = adis->data->write_delay, |
|---|
| 75 | + .delay.value = adis->data->write_delay, |
|---|
| 76 | + .delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 60 | 77 | }, { |
|---|
| 61 | 78 | .tx_buf = adis->tx + 8, |
|---|
| 62 | 79 | .bits_per_word = 8, |
|---|
| 63 | 80 | .len = 2, |
|---|
| 64 | | - .delay_usecs = adis->data->write_delay, |
|---|
| 81 | + .delay.value = adis->data->write_delay, |
|---|
| 82 | + .delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 65 | 83 | }, |
|---|
| 66 | 84 | }; |
|---|
| 67 | | - |
|---|
| 68 | | - mutex_lock(&adis->txrx_lock); |
|---|
| 69 | 85 | |
|---|
| 70 | 86 | spi_message_init(&msg); |
|---|
| 71 | 87 | |
|---|
| .. | .. |
|---|
| 81 | 97 | adis->tx[9] = (value >> 24) & 0xff; |
|---|
| 82 | 98 | adis->tx[6] = ADIS_WRITE_REG(reg + 2); |
|---|
| 83 | 99 | adis->tx[7] = (value >> 16) & 0xff; |
|---|
| 84 | | - /* fall through */ |
|---|
| 100 | + fallthrough; |
|---|
| 85 | 101 | case 2: |
|---|
| 86 | 102 | adis->tx[4] = ADIS_WRITE_REG(reg + 1); |
|---|
| 87 | 103 | adis->tx[5] = (value >> 8) & 0xff; |
|---|
| 88 | | - /* fall through */ |
|---|
| 104 | + fallthrough; |
|---|
| 89 | 105 | case 1: |
|---|
| 90 | 106 | adis->tx[2] = ADIS_WRITE_REG(reg); |
|---|
| 91 | 107 | adis->tx[3] = value & 0xff; |
|---|
| 92 | 108 | break; |
|---|
| 93 | 109 | default: |
|---|
| 94 | | - ret = -EINVAL; |
|---|
| 95 | | - goto out_unlock; |
|---|
| 110 | + return -EINVAL; |
|---|
| 96 | 111 | } |
|---|
| 97 | 112 | |
|---|
| 98 | 113 | xfers[size].cs_change = 0; |
|---|
| .. | .. |
|---|
| 108 | 123 | adis->current_page = page; |
|---|
| 109 | 124 | } |
|---|
| 110 | 125 | |
|---|
| 111 | | -out_unlock: |
|---|
| 112 | | - mutex_unlock(&adis->txrx_lock); |
|---|
| 113 | | - |
|---|
| 114 | 126 | return ret; |
|---|
| 115 | 127 | } |
|---|
| 116 | | -EXPORT_SYMBOL_GPL(adis_write_reg); |
|---|
| 128 | +EXPORT_SYMBOL_GPL(__adis_write_reg); |
|---|
| 117 | 129 | |
|---|
| 118 | 130 | /** |
|---|
| 119 | | - * adis_read_reg() - read 2 bytes from a 16-bit register |
|---|
| 131 | + * __adis_read_reg() - read N bytes from register (unlocked version) |
|---|
| 120 | 132 | * @adis: The adis device |
|---|
| 121 | 133 | * @reg: The address of the lower of the two registers |
|---|
| 122 | 134 | * @val: The value read back from the device |
|---|
| 135 | + * @size: The size of the @val buffer |
|---|
| 123 | 136 | */ |
|---|
| 124 | | -int adis_read_reg(struct adis *adis, unsigned int reg, |
|---|
| 137 | +int __adis_read_reg(struct adis *adis, unsigned int reg, |
|---|
| 125 | 138 | unsigned int *val, unsigned int size) |
|---|
| 126 | 139 | { |
|---|
| 127 | 140 | unsigned int page = reg / ADIS_PAGE_SIZE; |
|---|
| .. | .. |
|---|
| 133 | 146 | .bits_per_word = 8, |
|---|
| 134 | 147 | .len = 2, |
|---|
| 135 | 148 | .cs_change = 1, |
|---|
| 136 | | - .delay_usecs = adis->data->write_delay, |
|---|
| 149 | + .delay.value = adis->data->write_delay, |
|---|
| 150 | + .delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 151 | + .cs_change_delay.value = adis->data->cs_change_delay, |
|---|
| 152 | + .cs_change_delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 137 | 153 | }, { |
|---|
| 138 | 154 | .tx_buf = adis->tx + 2, |
|---|
| 139 | 155 | .bits_per_word = 8, |
|---|
| 140 | 156 | .len = 2, |
|---|
| 141 | 157 | .cs_change = 1, |
|---|
| 142 | | - .delay_usecs = adis->data->read_delay, |
|---|
| 158 | + .delay.value = adis->data->read_delay, |
|---|
| 159 | + .delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 160 | + .cs_change_delay.value = adis->data->cs_change_delay, |
|---|
| 161 | + .cs_change_delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 143 | 162 | }, { |
|---|
| 144 | 163 | .tx_buf = adis->tx + 4, |
|---|
| 145 | 164 | .rx_buf = adis->rx, |
|---|
| 146 | 165 | .bits_per_word = 8, |
|---|
| 147 | 166 | .len = 2, |
|---|
| 148 | 167 | .cs_change = 1, |
|---|
| 149 | | - .delay_usecs = adis->data->read_delay, |
|---|
| 168 | + .delay.value = adis->data->read_delay, |
|---|
| 169 | + .delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 170 | + .cs_change_delay.value = adis->data->cs_change_delay, |
|---|
| 171 | + .cs_change_delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 150 | 172 | }, { |
|---|
| 151 | 173 | .rx_buf = adis->rx + 2, |
|---|
| 152 | 174 | .bits_per_word = 8, |
|---|
| 153 | 175 | .len = 2, |
|---|
| 154 | | - .delay_usecs = adis->data->read_delay, |
|---|
| 176 | + .delay.value = adis->data->read_delay, |
|---|
| 177 | + .delay.unit = SPI_DELAY_UNIT_USECS, |
|---|
| 155 | 178 | }, |
|---|
| 156 | 179 | }; |
|---|
| 157 | 180 | |
|---|
| 158 | | - mutex_lock(&adis->txrx_lock); |
|---|
| 159 | 181 | spi_message_init(&msg); |
|---|
| 160 | 182 | |
|---|
| 161 | 183 | if (adis->current_page != page) { |
|---|
| .. | .. |
|---|
| 169 | 191 | adis->tx[2] = ADIS_READ_REG(reg + 2); |
|---|
| 170 | 192 | adis->tx[3] = 0; |
|---|
| 171 | 193 | spi_message_add_tail(&xfers[1], &msg); |
|---|
| 172 | | - /* fall through */ |
|---|
| 194 | + fallthrough; |
|---|
| 173 | 195 | case 2: |
|---|
| 174 | 196 | adis->tx[4] = ADIS_READ_REG(reg); |
|---|
| 175 | 197 | adis->tx[5] = 0; |
|---|
| .. | .. |
|---|
| 177 | 199 | spi_message_add_tail(&xfers[3], &msg); |
|---|
| 178 | 200 | break; |
|---|
| 179 | 201 | default: |
|---|
| 180 | | - ret = -EINVAL; |
|---|
| 181 | | - goto out_unlock; |
|---|
| 202 | + return -EINVAL; |
|---|
| 182 | 203 | } |
|---|
| 183 | 204 | |
|---|
| 184 | 205 | ret = spi_sync(adis->spi, &msg); |
|---|
| 185 | 206 | if (ret) { |
|---|
| 186 | 207 | dev_err(&adis->spi->dev, "Failed to read register 0x%02X: %d\n", |
|---|
| 187 | 208 | reg, ret); |
|---|
| 188 | | - goto out_unlock; |
|---|
| 209 | + return ret; |
|---|
| 189 | 210 | } else { |
|---|
| 190 | 211 | adis->current_page = page; |
|---|
| 191 | 212 | } |
|---|
| .. | .. |
|---|
| 199 | 220 | break; |
|---|
| 200 | 221 | } |
|---|
| 201 | 222 | |
|---|
| 202 | | -out_unlock: |
|---|
| 203 | | - mutex_unlock(&adis->txrx_lock); |
|---|
| 204 | | - |
|---|
| 205 | 223 | return ret; |
|---|
| 206 | 224 | } |
|---|
| 207 | | -EXPORT_SYMBOL_GPL(adis_read_reg); |
|---|
| 225 | +EXPORT_SYMBOL_GPL(__adis_read_reg); |
|---|
| 226 | +/** |
|---|
| 227 | + * __adis_update_bits_base() - ADIS Update bits function - Unlocked version |
|---|
| 228 | + * @adis: The adis device |
|---|
| 229 | + * @reg: The address of the lower of the two registers |
|---|
| 230 | + * @mask: Bitmask to change |
|---|
| 231 | + * @val: Value to be written |
|---|
| 232 | + * @size: Size of the register to update |
|---|
| 233 | + * |
|---|
| 234 | + * Updates the desired bits of @reg in accordance with @mask and @val. |
|---|
| 235 | + */ |
|---|
| 236 | +int __adis_update_bits_base(struct adis *adis, unsigned int reg, const u32 mask, |
|---|
| 237 | + const u32 val, u8 size) |
|---|
| 238 | +{ |
|---|
| 239 | + int ret; |
|---|
| 240 | + u32 __val; |
|---|
| 241 | + |
|---|
| 242 | + ret = __adis_read_reg(adis, reg, &__val, size); |
|---|
| 243 | + if (ret) |
|---|
| 244 | + return ret; |
|---|
| 245 | + |
|---|
| 246 | + __val = (__val & ~mask) | (val & mask); |
|---|
| 247 | + |
|---|
| 248 | + return __adis_write_reg(adis, reg, __val, size); |
|---|
| 249 | +} |
|---|
| 250 | +EXPORT_SYMBOL_GPL(__adis_update_bits_base); |
|---|
| 208 | 251 | |
|---|
| 209 | 252 | #ifdef CONFIG_DEBUG_FS |
|---|
| 210 | 253 | |
|---|
| .. | .. |
|---|
| 218 | 261 | int ret; |
|---|
| 219 | 262 | |
|---|
| 220 | 263 | ret = adis_read_reg_16(adis, reg, &val16); |
|---|
| 221 | | - *readval = val16; |
|---|
| 264 | + if (ret == 0) |
|---|
| 265 | + *readval = val16; |
|---|
| 222 | 266 | |
|---|
| 223 | 267 | return ret; |
|---|
| 224 | 268 | } else { |
|---|
| .. | .. |
|---|
| 241 | 285 | int ret = 0; |
|---|
| 242 | 286 | uint16_t msc; |
|---|
| 243 | 287 | |
|---|
| 244 | | - if (adis->data->enable_irq) |
|---|
| 245 | | - return adis->data->enable_irq(adis, enable); |
|---|
| 288 | + mutex_lock(&adis->state_lock); |
|---|
| 246 | 289 | |
|---|
| 247 | | - ret = adis_read_reg_16(adis, adis->data->msc_ctrl_reg, &msc); |
|---|
| 290 | + if (adis->data->enable_irq) { |
|---|
| 291 | + ret = adis->data->enable_irq(adis, enable); |
|---|
| 292 | + goto out_unlock; |
|---|
| 293 | + } |
|---|
| 294 | + |
|---|
| 295 | + ret = __adis_read_reg_16(adis, adis->data->msc_ctrl_reg, &msc); |
|---|
| 248 | 296 | if (ret) |
|---|
| 249 | | - goto error_ret; |
|---|
| 297 | + goto out_unlock; |
|---|
| 250 | 298 | |
|---|
| 251 | 299 | msc |= ADIS_MSC_CTRL_DATA_RDY_POL_HIGH; |
|---|
| 252 | 300 | msc &= ~ADIS_MSC_CTRL_DATA_RDY_DIO2; |
|---|
| .. | .. |
|---|
| 255 | 303 | else |
|---|
| 256 | 304 | msc &= ~ADIS_MSC_CTRL_DATA_RDY_EN; |
|---|
| 257 | 305 | |
|---|
| 258 | | - ret = adis_write_reg_16(adis, adis->data->msc_ctrl_reg, msc); |
|---|
| 306 | + ret = __adis_write_reg_16(adis, adis->data->msc_ctrl_reg, msc); |
|---|
| 259 | 307 | |
|---|
| 260 | | -error_ret: |
|---|
| 308 | +out_unlock: |
|---|
| 309 | + mutex_unlock(&adis->state_lock); |
|---|
| 261 | 310 | return ret; |
|---|
| 262 | 311 | } |
|---|
| 263 | 312 | EXPORT_SYMBOL(adis_enable_irq); |
|---|
| 264 | 313 | |
|---|
| 265 | 314 | /** |
|---|
| 266 | | - * adis_check_status() - Check the device for error conditions |
|---|
| 315 | + * __adis_check_status() - Check the device for error conditions (unlocked) |
|---|
| 267 | 316 | * @adis: The adis device |
|---|
| 268 | 317 | * |
|---|
| 269 | 318 | * Returns 0 on success, a negative error code otherwise |
|---|
| 270 | 319 | */ |
|---|
| 271 | | -int adis_check_status(struct adis *adis) |
|---|
| 320 | +int __adis_check_status(struct adis *adis) |
|---|
| 272 | 321 | { |
|---|
| 273 | 322 | uint16_t status; |
|---|
| 274 | 323 | int ret; |
|---|
| 275 | 324 | int i; |
|---|
| 276 | 325 | |
|---|
| 277 | | - ret = adis_read_reg_16(adis, adis->data->diag_stat_reg, &status); |
|---|
| 278 | | - if (ret < 0) |
|---|
| 326 | + ret = __adis_read_reg_16(adis, adis->data->diag_stat_reg, &status); |
|---|
| 327 | + if (ret) |
|---|
| 279 | 328 | return ret; |
|---|
| 280 | 329 | |
|---|
| 281 | 330 | status &= adis->data->status_error_mask; |
|---|
| .. | .. |
|---|
| 292 | 341 | |
|---|
| 293 | 342 | return -EIO; |
|---|
| 294 | 343 | } |
|---|
| 295 | | -EXPORT_SYMBOL_GPL(adis_check_status); |
|---|
| 344 | +EXPORT_SYMBOL_GPL(__adis_check_status); |
|---|
| 296 | 345 | |
|---|
| 297 | 346 | /** |
|---|
| 298 | | - * adis_reset() - Reset the device |
|---|
| 347 | + * __adis_reset() - Reset the device (unlocked version) |
|---|
| 299 | 348 | * @adis: The adis device |
|---|
| 300 | 349 | * |
|---|
| 301 | 350 | * Returns 0 on success, a negative error code otherwise |
|---|
| 302 | 351 | */ |
|---|
| 303 | | -int adis_reset(struct adis *adis) |
|---|
| 352 | +int __adis_reset(struct adis *adis) |
|---|
| 304 | 353 | { |
|---|
| 305 | 354 | int ret; |
|---|
| 355 | + const struct adis_timeout *timeouts = adis->data->timeouts; |
|---|
| 306 | 356 | |
|---|
| 307 | | - ret = adis_write_reg_8(adis, adis->data->glob_cmd_reg, |
|---|
| 357 | + ret = __adis_write_reg_8(adis, adis->data->glob_cmd_reg, |
|---|
| 308 | 358 | ADIS_GLOB_CMD_SW_RESET); |
|---|
| 309 | | - if (ret) |
|---|
| 359 | + if (ret) { |
|---|
| 310 | 360 | dev_err(&adis->spi->dev, "Failed to reset device: %d\n", ret); |
|---|
| 361 | + return ret; |
|---|
| 362 | + } |
|---|
| 311 | 363 | |
|---|
| 312 | | - return ret; |
|---|
| 364 | + msleep(timeouts->sw_reset_ms); |
|---|
| 365 | + |
|---|
| 366 | + return 0; |
|---|
| 313 | 367 | } |
|---|
| 314 | | -EXPORT_SYMBOL_GPL(adis_reset); |
|---|
| 368 | +EXPORT_SYMBOL_GPL(__adis_reset); |
|---|
| 315 | 369 | |
|---|
| 316 | 370 | static int adis_self_test(struct adis *adis) |
|---|
| 317 | 371 | { |
|---|
| 318 | 372 | int ret; |
|---|
| 373 | + const struct adis_timeout *timeouts = adis->data->timeouts; |
|---|
| 319 | 374 | |
|---|
| 320 | | - ret = adis_write_reg_16(adis, adis->data->msc_ctrl_reg, |
|---|
| 321 | | - adis->data->self_test_mask); |
|---|
| 375 | + ret = __adis_write_reg_16(adis, adis->data->self_test_reg, |
|---|
| 376 | + adis->data->self_test_mask); |
|---|
| 322 | 377 | if (ret) { |
|---|
| 323 | 378 | dev_err(&adis->spi->dev, "Failed to initiate self test: %d\n", |
|---|
| 324 | 379 | ret); |
|---|
| 325 | 380 | return ret; |
|---|
| 326 | 381 | } |
|---|
| 327 | 382 | |
|---|
| 328 | | - msleep(adis->data->startup_delay); |
|---|
| 383 | + msleep(timeouts->self_test_ms); |
|---|
| 329 | 384 | |
|---|
| 330 | | - ret = adis_check_status(adis); |
|---|
| 385 | + ret = __adis_check_status(adis); |
|---|
| 331 | 386 | |
|---|
| 332 | 387 | if (adis->data->self_test_no_autoclear) |
|---|
| 333 | | - adis_write_reg_16(adis, adis->data->msc_ctrl_reg, 0x00); |
|---|
| 388 | + __adis_write_reg_16(adis, adis->data->self_test_reg, 0x00); |
|---|
| 334 | 389 | |
|---|
| 335 | 390 | return ret; |
|---|
| 336 | 391 | } |
|---|
| 337 | 392 | |
|---|
| 338 | 393 | /** |
|---|
| 339 | | - * adis_inital_startup() - Performs device self-test |
|---|
| 394 | + * __adis_initial_startup() - Device initial setup |
|---|
| 340 | 395 | * @adis: The adis device |
|---|
| 396 | + * |
|---|
| 397 | + * The function performs a HW reset via a reset pin that should be specified |
|---|
| 398 | + * via GPIOLIB. If no pin is configured a SW reset will be performed. |
|---|
| 399 | + * The RST pin for the ADIS devices should be configured as ACTIVE_LOW. |
|---|
| 400 | + * |
|---|
| 401 | + * After the self-test operation is performed, the function will also check |
|---|
| 402 | + * that the product ID is as expected. This assumes that drivers providing |
|---|
| 403 | + * 'prod_id_reg' will also provide the 'prod_id'. |
|---|
| 341 | 404 | * |
|---|
| 342 | 405 | * Returns 0 if the device is operational, a negative error code otherwise. |
|---|
| 343 | 406 | * |
|---|
| 344 | 407 | * This function should be called early on in the device initialization sequence |
|---|
| 345 | 408 | * to ensure that the device is in a sane and known state and that it is usable. |
|---|
| 346 | 409 | */ |
|---|
| 347 | | -int adis_initial_startup(struct adis *adis) |
|---|
| 410 | +int __adis_initial_startup(struct adis *adis) |
|---|
| 348 | 411 | { |
|---|
| 412 | + const struct adis_timeout *timeouts = adis->data->timeouts; |
|---|
| 413 | + struct gpio_desc *gpio; |
|---|
| 414 | + uint16_t prod_id; |
|---|
| 349 | 415 | int ret; |
|---|
| 350 | 416 | |
|---|
| 351 | | - ret = adis_self_test(adis); |
|---|
| 352 | | - if (ret) { |
|---|
| 353 | | - dev_err(&adis->spi->dev, "Self-test failed, trying reset.\n"); |
|---|
| 354 | | - adis_reset(adis); |
|---|
| 355 | | - msleep(adis->data->startup_delay); |
|---|
| 356 | | - ret = adis_self_test(adis); |
|---|
| 357 | | - if (ret) { |
|---|
| 358 | | - dev_err(&adis->spi->dev, "Second self-test failed, giving up.\n"); |
|---|
| 417 | + /* check if the device has rst pin low */ |
|---|
| 418 | + gpio = devm_gpiod_get_optional(&adis->spi->dev, "reset", GPIOD_OUT_HIGH); |
|---|
| 419 | + if (IS_ERR(gpio)) |
|---|
| 420 | + return PTR_ERR(gpio); |
|---|
| 421 | + |
|---|
| 422 | + if (gpio) { |
|---|
| 423 | + msleep(10); |
|---|
| 424 | + /* bring device out of reset */ |
|---|
| 425 | + gpiod_set_value_cansleep(gpio, 0); |
|---|
| 426 | + msleep(timeouts->reset_ms); |
|---|
| 427 | + } else { |
|---|
| 428 | + ret = __adis_reset(adis); |
|---|
| 429 | + if (ret) |
|---|
| 359 | 430 | return ret; |
|---|
| 360 | | - } |
|---|
| 361 | 431 | } |
|---|
| 432 | + |
|---|
| 433 | + ret = adis_self_test(adis); |
|---|
| 434 | + if (ret) |
|---|
| 435 | + return ret; |
|---|
| 436 | + |
|---|
| 437 | + adis_enable_irq(adis, false); |
|---|
| 438 | + |
|---|
| 439 | + if (!adis->data->prod_id_reg) |
|---|
| 440 | + return 0; |
|---|
| 441 | + |
|---|
| 442 | + ret = adis_read_reg_16(adis, adis->data->prod_id_reg, &prod_id); |
|---|
| 443 | + if (ret) |
|---|
| 444 | + return ret; |
|---|
| 445 | + |
|---|
| 446 | + if (prod_id != adis->data->prod_id) |
|---|
| 447 | + dev_warn(&adis->spi->dev, |
|---|
| 448 | + "Device ID(%u) and product ID(%u) do not match.\n", |
|---|
| 449 | + adis->data->prod_id, prod_id); |
|---|
| 362 | 450 | |
|---|
| 363 | 451 | return 0; |
|---|
| 364 | 452 | } |
|---|
| 365 | | -EXPORT_SYMBOL_GPL(adis_initial_startup); |
|---|
| 453 | +EXPORT_SYMBOL_GPL(__adis_initial_startup); |
|---|
| 366 | 454 | |
|---|
| 367 | 455 | /** |
|---|
| 368 | 456 | * adis_single_conversion() - Performs a single sample conversion |
|---|
| .. | .. |
|---|
| 386 | 474 | unsigned int uval; |
|---|
| 387 | 475 | int ret; |
|---|
| 388 | 476 | |
|---|
| 389 | | - mutex_lock(&indio_dev->mlock); |
|---|
| 477 | + mutex_lock(&adis->state_lock); |
|---|
| 390 | 478 | |
|---|
| 391 | | - ret = adis_read_reg(adis, chan->address, &uval, |
|---|
| 479 | + ret = __adis_read_reg(adis, chan->address, &uval, |
|---|
| 392 | 480 | chan->scan_type.storagebits / 8); |
|---|
| 393 | 481 | if (ret) |
|---|
| 394 | 482 | goto err_unlock; |
|---|
| 395 | 483 | |
|---|
| 396 | 484 | if (uval & error_mask) { |
|---|
| 397 | | - ret = adis_check_status(adis); |
|---|
| 485 | + ret = __adis_check_status(adis); |
|---|
| 398 | 486 | if (ret) |
|---|
| 399 | 487 | goto err_unlock; |
|---|
| 400 | 488 | } |
|---|
| .. | .. |
|---|
| 406 | 494 | |
|---|
| 407 | 495 | ret = IIO_VAL_INT; |
|---|
| 408 | 496 | err_unlock: |
|---|
| 409 | | - mutex_unlock(&indio_dev->mlock); |
|---|
| 497 | + mutex_unlock(&adis->state_lock); |
|---|
| 410 | 498 | return ret; |
|---|
| 411 | 499 | } |
|---|
| 412 | 500 | EXPORT_SYMBOL_GPL(adis_single_conversion); |
|---|
| .. | .. |
|---|
| 426 | 514 | int adis_init(struct adis *adis, struct iio_dev *indio_dev, |
|---|
| 427 | 515 | struct spi_device *spi, const struct adis_data *data) |
|---|
| 428 | 516 | { |
|---|
| 429 | | - mutex_init(&adis->txrx_lock); |
|---|
| 517 | + if (!data || !data->timeouts) { |
|---|
| 518 | + dev_err(&spi->dev, "No config data or timeouts not defined!\n"); |
|---|
| 519 | + return -EINVAL; |
|---|
| 520 | + } |
|---|
| 521 | + |
|---|
| 522 | + mutex_init(&adis->state_lock); |
|---|
| 430 | 523 | adis->spi = spi; |
|---|
| 431 | 524 | adis->data = data; |
|---|
| 432 | 525 | iio_device_set_drvdata(indio_dev, adis); |
|---|
| .. | .. |
|---|
| 439 | 532 | adis->current_page = 0; |
|---|
| 440 | 533 | } |
|---|
| 441 | 534 | |
|---|
| 442 | | - return adis_enable_irq(adis, false); |
|---|
| 535 | + return 0; |
|---|
| 443 | 536 | } |
|---|
| 444 | 537 | EXPORT_SYMBOL_GPL(adis_init); |
|---|
| 445 | 538 | |
|---|