| .. | .. |
|---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-only |
|---|
| 1 | 2 | /* |
|---|
| 2 | 3 | * Simple driver for Texas Instruments LM3642 LED Flash driver chip |
|---|
| 3 | 4 | * Copyright (C) 2012 Texas Instruments |
|---|
| 4 | | -* |
|---|
| 5 | | -* This program is free software; you can redistribute it and/or modify |
|---|
| 6 | | -* it under the terms of the GNU General Public License version 2 as |
|---|
| 7 | | -* published by the Free Software Foundation. |
|---|
| 8 | | -* |
|---|
| 9 | 5 | */ |
|---|
| 10 | 6 | #include <linux/module.h> |
|---|
| 11 | 7 | #include <linux/delay.h> |
|---|
| .. | .. |
|---|
| 110 | 106 | ret = regmap_read(chip->regmap, REG_FLAG, &chip->last_flag); |
|---|
| 111 | 107 | if (ret < 0) { |
|---|
| 112 | 108 | dev_err(chip->dev, "Failed to read REG_FLAG Register\n"); |
|---|
| 113 | | - goto out; |
|---|
| 109 | + return ret; |
|---|
| 114 | 110 | } |
|---|
| 115 | 111 | |
|---|
| 116 | 112 | if (chip->last_flag) |
|---|
| .. | .. |
|---|
| 150 | 146 | break; |
|---|
| 151 | 147 | |
|---|
| 152 | 148 | default: |
|---|
| 153 | | - return ret; |
|---|
| 149 | + return -EINVAL; |
|---|
| 154 | 150 | } |
|---|
| 155 | 151 | if (ret < 0) { |
|---|
| 156 | 152 | dev_err(chip->dev, "Failed to write REG_I_CTRL Register\n"); |
|---|
| 157 | | - goto out; |
|---|
| 153 | + return ret; |
|---|
| 158 | 154 | } |
|---|
| 159 | 155 | |
|---|
| 160 | 156 | if (chip->tx_pin) |
|---|
| .. | .. |
|---|
| 163 | 159 | ret = regmap_update_bits(chip->regmap, REG_ENABLE, |
|---|
| 164 | 160 | MODE_BITS_MASK << MODE_BITS_SHIFT, |
|---|
| 165 | 161 | opmode << MODE_BITS_SHIFT); |
|---|
| 166 | | -out: |
|---|
| 167 | 162 | return ret; |
|---|
| 168 | 163 | } |
|---|
| 169 | 164 | |
|---|
| 170 | 165 | /* torch */ |
|---|
| 171 | 166 | |
|---|
| 172 | | -/* torch pin config for lm3642*/ |
|---|
| 167 | +/* torch pin config for lm3642 */ |
|---|
| 173 | 168 | static ssize_t lm3642_torch_pin_store(struct device *dev, |
|---|
| 174 | 169 | struct device_attribute *attr, |
|---|
| 175 | 170 | const char *buf, size_t size) |
|---|
| .. | .. |
|---|
| 182 | 177 | |
|---|
| 183 | 178 | ret = kstrtouint(buf, 10, &state); |
|---|
| 184 | 179 | if (ret) |
|---|
| 185 | | - goto out_strtoint; |
|---|
| 180 | + return ret; |
|---|
| 186 | 181 | if (state != 0) |
|---|
| 187 | 182 | state = 0x01 << TORCH_PIN_EN_SHIFT; |
|---|
| 188 | 183 | |
|---|
| .. | .. |
|---|
| 190 | 185 | ret = regmap_update_bits(chip->regmap, REG_ENABLE, |
|---|
| 191 | 186 | TORCH_PIN_EN_MASK << TORCH_PIN_EN_SHIFT, |
|---|
| 192 | 187 | state); |
|---|
| 193 | | - if (ret < 0) |
|---|
| 194 | | - goto out; |
|---|
| 188 | + if (ret < 0) { |
|---|
| 189 | + dev_err(chip->dev, "%s:i2c access fail to register\n", __func__); |
|---|
| 190 | + return ret; |
|---|
| 191 | + } |
|---|
| 195 | 192 | |
|---|
| 196 | 193 | return size; |
|---|
| 197 | | -out: |
|---|
| 198 | | - dev_err(chip->dev, "%s:i2c access fail to register\n", __func__); |
|---|
| 199 | | - return ret; |
|---|
| 200 | | -out_strtoint: |
|---|
| 201 | | - dev_err(chip->dev, "%s: fail to change str to int\n", __func__); |
|---|
| 202 | | - return ret; |
|---|
| 203 | 194 | } |
|---|
| 204 | 195 | |
|---|
| 205 | 196 | static DEVICE_ATTR(torch_pin, S_IWUSR, NULL, lm3642_torch_pin_store); |
|---|
| .. | .. |
|---|
| 233 | 224 | |
|---|
| 234 | 225 | ret = kstrtouint(buf, 10, &state); |
|---|
| 235 | 226 | if (ret) |
|---|
| 236 | | - goto out_strtoint; |
|---|
| 227 | + return ret; |
|---|
| 237 | 228 | if (state != 0) |
|---|
| 238 | 229 | state = 0x01 << STROBE_PIN_EN_SHIFT; |
|---|
| 239 | 230 | |
|---|
| .. | .. |
|---|
| 241 | 232 | ret = regmap_update_bits(chip->regmap, REG_ENABLE, |
|---|
| 242 | 233 | STROBE_PIN_EN_MASK << STROBE_PIN_EN_SHIFT, |
|---|
| 243 | 234 | state); |
|---|
| 244 | | - if (ret < 0) |
|---|
| 245 | | - goto out; |
|---|
| 235 | + if (ret < 0) { |
|---|
| 236 | + dev_err(chip->dev, "%s:i2c access fail to register\n", __func__); |
|---|
| 237 | + return ret; |
|---|
| 238 | + } |
|---|
| 246 | 239 | |
|---|
| 247 | 240 | return size; |
|---|
| 248 | | -out: |
|---|
| 249 | | - dev_err(chip->dev, "%s:i2c access fail to register\n", __func__); |
|---|
| 250 | | - return ret; |
|---|
| 251 | | -out_strtoint: |
|---|
| 252 | | - dev_err(chip->dev, "%s: fail to change str to int\n", __func__); |
|---|
| 253 | | - return ret; |
|---|
| 254 | 241 | } |
|---|
| 255 | 242 | |
|---|
| 256 | 243 | static DEVICE_ATTR(strobe_pin, S_IWUSR, NULL, lm3642_strobe_pin_store); |
|---|
| .. | .. |
|---|
| 353 | 340 | chip->cdev_flash.brightness_set_blocking = lm3642_strobe_brightness_set; |
|---|
| 354 | 341 | chip->cdev_flash.default_trigger = "flash"; |
|---|
| 355 | 342 | chip->cdev_flash.groups = lm3642_flash_groups, |
|---|
| 356 | | - err = led_classdev_register((struct device *) |
|---|
| 357 | | - &client->dev, &chip->cdev_flash); |
|---|
| 343 | + err = led_classdev_register(&client->dev, &chip->cdev_flash); |
|---|
| 358 | 344 | if (err < 0) { |
|---|
| 359 | 345 | dev_err(chip->dev, "failed to register flash\n"); |
|---|
| 360 | 346 | goto err_out; |
|---|
| .. | .. |
|---|
| 366 | 352 | chip->cdev_torch.brightness_set_blocking = lm3642_torch_brightness_set; |
|---|
| 367 | 353 | chip->cdev_torch.default_trigger = "torch"; |
|---|
| 368 | 354 | chip->cdev_torch.groups = lm3642_torch_groups, |
|---|
| 369 | | - err = led_classdev_register((struct device *) |
|---|
| 370 | | - &client->dev, &chip->cdev_torch); |
|---|
| 355 | + err = led_classdev_register(&client->dev, &chip->cdev_torch); |
|---|
| 371 | 356 | if (err < 0) { |
|---|
| 372 | 357 | dev_err(chip->dev, "failed to register torch\n"); |
|---|
| 373 | 358 | goto err_create_torch_file; |
|---|
| .. | .. |
|---|
| 378 | 363 | chip->cdev_indicator.max_brightness = 8; |
|---|
| 379 | 364 | chip->cdev_indicator.brightness_set_blocking = |
|---|
| 380 | 365 | lm3642_indicator_brightness_set; |
|---|
| 381 | | - err = led_classdev_register((struct device *) |
|---|
| 382 | | - &client->dev, &chip->cdev_indicator); |
|---|
| 366 | + err = led_classdev_register(&client->dev, &chip->cdev_indicator); |
|---|
| 383 | 367 | if (err < 0) { |
|---|
| 384 | 368 | dev_err(chip->dev, "failed to register indicator\n"); |
|---|
| 385 | 369 | goto err_create_indicator_file; |
|---|