NFC: Only warn on SE discovery error
authorSamuel Ortiz <sameo@linux.intel.com>
Sun, 22 Dec 2013 00:00:20 +0000 (01:00 +0100)
committerSamuel Ortiz <sameo@linux.intel.com>
Sat, 4 Jan 2014 02:32:27 +0000 (03:32 +0100)
SE discovery errors are currently overwriting the dev_up() return error.
This is wrong for many reasons:

- We don't want to report an error if we actually brought the device up
  but it failed to discover SEs. By doing so we pretend we don't have an
  NFC functional device even we do. The only thing we could not do was
  checking for SEs availability. This is the false negative case.

- In some cases the actual device power up failed but the SE discovery
  succeeded. Userspace then believes the device is up while it's not.
  This is the false positive case.

Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
net/nfc/core.c

index 02ab34132157066d9a0def95276b7abe3f8d7f71..c1903f439aacb1d0e0ce8deac8e15ad54c3536b6 100644 (file)
@@ -133,11 +133,8 @@ int nfc_dev_up(struct nfc_dev *dev)
                dev->dev_up = true;
 
        /* We have to enable the device before discovering SEs */
-       if (dev->ops->discover_se) {
-               rc = dev->ops->discover_se(dev);
-               if (rc)
-                       pr_warn("SE discovery failed\n");
-       }
+       if (dev->ops->discover_se && dev->ops->discover_se(dev))
+               pr_err("SE discovery failed\n");
 
 error:
        device_unlock(&dev->dev);