diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2017-03-07 11:25:42 +0100 |
|---|---|---|
| committer | Samuel Ortiz <sameo@linux.intel.com> | 2017-04-05 10:07:35 +0200 |
| commit | bacf2a6a05726e3f10aa94ef7a20feba62ee5579 (patch) | |
| tree | db5584b70e8cd7ac79cfecb82cf80d5ce5b4d300 /drivers/nfc/st21nfca | |
| parent | NFC: pn544: Get rid of code duplication in ->probe() (diff) | |
| download | linux-bacf2a6a05726e3f10aa94ef7a20feba62ee5579.tar.xz linux-bacf2a6a05726e3f10aa94ef7a20feba62ee5579.zip | |
NFC: st21nfca: Fix obvious typo when check error code
We return -ENODEV if ACPI provides a GPIO resource. Looks really wrong.
If it has even been tested?
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
Diffstat (limited to 'drivers/nfc/st21nfca')
| -rw-r--r-- | drivers/nfc/st21nfca/i2c.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/nfc/st21nfca/i2c.c b/drivers/nfc/st21nfca/i2c.c index d16f58ac09bc..c4f0d048359b 100644 --- a/drivers/nfc/st21nfca/i2c.c +++ b/drivers/nfc/st21nfca/i2c.c @@ -513,9 +513,9 @@ static int st21nfca_hci_i2c_acpi_request_resources(struct i2c_client *client) /* Get EN GPIO from ACPI */ gpiod_ena = devm_gpiod_get_index(dev, ST21NFCA_GPIO_NAME_EN, 1, GPIOD_OUT_LOW); - if (!IS_ERR(gpiod_ena)) { + if (IS_ERR(gpiod_ena)) { nfc_err(dev, "Unable to get ENABLE GPIO\n"); - return -ENODEV; + return PTR_ERR(gpiod_ena); } phy->gpio_ena = desc_to_gpio(gpiod_ena); |