NFC: pn533: reset poll modulation list before calling targets_found
authorMichael Thalmeier <michael.thalmeier@hale.at>
Thu, 21 Apr 2016 14:43:52 +0000 (16:43 +0200)
committerSamuel Ortiz <sameo@linux.intel.com>
Mon, 9 May 2016 22:01:16 +0000 (00:01 +0200)
We need to reset the poll modulation list before calling
nfc_targets_found because otherwise userspace could run
before the modulation list is cleared and then get a "Cannot
activate target while polling" error upon calling activate_target.

Signed-off-by: Michael Thalmeier <michael.thalmeier@hale.at>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
drivers/nfc/pn533/pn533.c

index d82eecd8daadff2c95d4f54eb6d2083c17770999..745181ea693b044e3ddb5a0a27f79475188e8add 100644 (file)
@@ -865,6 +865,7 @@ static int pn533_target_found_type_b(struct nfc_target *nfc_tgt, u8 *tgt_data,
        return 0;
 }
 
+static void pn533_poll_reset_mod_list(struct pn533 *dev);
 static int pn533_target_found(struct pn533 *dev, u8 tg, u8 *tgdata,
                              int tgdata_len)
 {
@@ -914,6 +915,7 @@ static int pn533_target_found(struct pn533 *dev, u8 tg, u8 *tgdata,
 
        dev->tgt_available_prots = nfc_tgt.supported_protocols;
 
+       pn533_poll_reset_mod_list(dev);
        nfc_targets_found(dev->nfc_dev, &nfc_tgt, 1);
 
        return 0;
@@ -980,10 +982,8 @@ static int pn533_start_poll_complete(struct pn533 *dev, struct sk_buff *resp)
                rc = pn533_target_found(dev, tg, tgdata, tgdata_len);
 
                /* We must stop the poll after a valid target found */
-               if (rc == 0) {
-                       pn533_poll_reset_mod_list(dev);
+               if (rc == 0)
                        return 0;
-               }
        }
 
        return -EAGAIN;