When closing the device some data (proprietary commands) might be sent.
The core state machine needs to be set for correct command execution.

Signed-off-by: Christophe Ricard <[email protected]>
---
 net/nfc/nci/core.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/net/nfc/nci/core.c b/net/nfc/nci/core.c
index 19caec8..f10da2f 100644
--- a/net/nfc/nci/core.c
+++ b/net/nfc/nci/core.c
@@ -441,6 +441,12 @@ static int nci_close_device(struct nci_dev *ndev)
        set_bit(NCI_INIT, &ndev->flags);
        __nci_request(ndev, nci_reset_req, 0,
                      msecs_to_jiffies(NCI_RESET_TIMEOUT));
+
+       /* After this point our queues are empty
+        * and no works are scheduled.
+        */
+       ndev->ops->close(ndev);
+
        clear_bit(NCI_INIT, &ndev->flags);
 
        del_timer_sync(&ndev->cmd_timer);
@@ -448,10 +454,6 @@ static int nci_close_device(struct nci_dev *ndev)
        /* Flush cmd wq */
        flush_workqueue(ndev->cmd_wq);
 
-       /* After this point our queues are empty
-        * and no works are scheduled. */
-       ndev->ops->close(ndev);
-
        /* Clear flags */
        ndev->flags = 0;
 
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to