NFC: Clearly separate NCI states from flags
authorIlan Elias <ilane@ti.com>
Wed, 18 Jan 2012 11:16:12 +0000 (13:16 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 24 Jan 2012 19:32:28 +0000 (14:32 -0500)
Make a clear separation between NCI states and flags.
This is required in order to support more NCI states (e.g.
for multiple targets support).

Signed-off-by: Ilan Elias <ilane@ti.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
include/net/nfc/nci_core.h
net/nfc/nci/core.c
net/nfc/nci/ntf.c
net/nfc/nci/rsp.c

index 9154663b606b9a3877c3b759e68137f6cf2eed32..b9c3f8de13ddd1ea772595873c1cd0b1f0e12a25 100644 (file)
 #include <net/nfc/nfc.h>
 #include <net/nfc/nci.h>
 
-/* NCI device state */
-enum {
+/* NCI device flags */
+enum nci_flag {
        NCI_INIT,
        NCI_UP,
-       NCI_DISCOVERY,
-       NCI_POLL_ACTIVE,
        NCI_DATA_EXCHANGE,
        NCI_DATA_EXCHANGE_TO,
 };
 
+/* NCI device states */
+enum nci_state {
+       NCI_IDLE,
+       NCI_DISCOVERY,
+       NCI_POLL_ACTIVE,
+};
+
 /* NCI timeouts */
 #define NCI_RESET_TIMEOUT                      5000
 #define NCI_INIT_TIMEOUT                       5000
@@ -70,6 +75,7 @@ struct nci_dev {
        int                     tx_headroom;
        int                     tx_tailroom;
 
+       atomic_t                state;
        unsigned long           flags;
 
        atomic_t                cmd_cnt;
index 815d28a0ed9d2eb86c40eeaf1867d6c9ff803b18..629b768459736f15f814d7fce81c30efa88fb149 100644 (file)
@@ -264,6 +264,7 @@ static int nci_open_device(struct nci_dev *ndev)
 
        if (!rc) {
                set_bit(NCI_UP, &ndev->flags);
+               atomic_set(&ndev->state, NCI_IDLE);
        } else {
                /* Init failed, cleanup */
                skb_queue_purge(&ndev->cmd_q);
@@ -360,7 +361,7 @@ static int nci_start_poll(struct nfc_dev *nfc_dev, __u32 protocols)
        struct nci_dev *ndev = nfc_get_drvdata(nfc_dev);
        int rc;
 
-       if (test_bit(NCI_DISCOVERY, &ndev->flags)) {
+       if (atomic_read(&ndev->state) == NCI_DISCOVERY) {
                pr_err("unable to start poll, since poll is already active\n");
                return -EBUSY;
        }
@@ -370,7 +371,7 @@ static int nci_start_poll(struct nfc_dev *nfc_dev, __u32 protocols)
                return -EBUSY;
        }
 
-       if (test_bit(NCI_POLL_ACTIVE, &ndev->flags)) {
+       if (atomic_read(&ndev->state) == NCI_POLL_ACTIVE) {
                pr_debug("target is active, implicitly deactivate...\n");
 
                rc = nci_request(ndev, nci_rf_deactivate_req, 0,
@@ -392,7 +393,7 @@ static void nci_stop_poll(struct nfc_dev *nfc_dev)
 {
        struct nci_dev *ndev = nfc_get_drvdata(nfc_dev);
 
-       if (!test_bit(NCI_DISCOVERY, &ndev->flags)) {
+       if (atomic_read(&ndev->state) != NCI_DISCOVERY) {
                pr_err("unable to stop poll, since poll is not active\n");
                return;
        }
@@ -408,7 +409,7 @@ static int nci_activate_target(struct nfc_dev *nfc_dev, __u32 target_idx,
 
        pr_debug("target_idx %d, protocol 0x%x\n", target_idx, protocol);
 
-       if (!test_bit(NCI_POLL_ACTIVE, &ndev->flags)) {
+       if (atomic_read(&ndev->state) != NCI_POLL_ACTIVE) {
                pr_err("there is no available target to activate\n");
                return -EINVAL;
        }
@@ -443,7 +444,7 @@ static void nci_deactivate_target(struct nfc_dev *nfc_dev, __u32 target_idx)
 
        ndev->target_active_prot = 0;
 
-       if (test_bit(NCI_POLL_ACTIVE, &ndev->flags)) {
+       if (atomic_read(&ndev->state) == NCI_POLL_ACTIVE) {
                nci_request(ndev, nci_rf_deactivate_req, 0,
                        msecs_to_jiffies(NCI_RF_DEACTIVATE_TIMEOUT));
        }
index a88be91e973f66a868a9b2382868cd7473a49392..8ec39464cea52b07f4f00ca11f97aabc9ec45c2a 100644 (file)
@@ -261,8 +261,7 @@ static void nci_rf_intf_activated_ntf_packet(struct nci_dev *ndev,
        __u8 *data = skb->data;
        int err = 0;
 
-       clear_bit(NCI_DISCOVERY, &ndev->flags);
-       set_bit(NCI_POLL_ACTIVE, &ndev->flags);
+       atomic_set(&ndev->state, NCI_POLL_ACTIVE);
 
        ntf.rf_discovery_id = *data++;
        ntf.rf_interface = *data++;
@@ -350,7 +349,7 @@ static void nci_rf_deactivate_ntf_packet(struct nci_dev *ndev,
 
        pr_debug("entry, type 0x%x, reason 0x%x\n", ntf->type, ntf->reason);
 
-       clear_bit(NCI_POLL_ACTIVE, &ndev->flags);
+       atomic_set(&ndev->state, NCI_IDLE);
        ndev->target_active_prot = 0;
 
        /* drop tx data queue */
index 3c73e92eb6257fa1c8f807ad4e8dec1989a0a3dd..cb8bce6899cf00267c7521c1eac2ca0d7c633e65 100644 (file)
@@ -137,7 +137,7 @@ static void nci_rf_disc_rsp_packet(struct nci_dev *ndev, struct sk_buff *skb)
        pr_debug("status 0x%x\n", status);
 
        if (status == NCI_STATUS_OK)
-               set_bit(NCI_DISCOVERY, &ndev->flags);
+               atomic_set(&ndev->state, NCI_DISCOVERY);
 
        nci_req_complete(ndev, status);
 }
@@ -149,12 +149,12 @@ static void nci_rf_deactivate_rsp_packet(struct nci_dev *ndev,
 
        pr_debug("status 0x%x\n", status);
 
-       clear_bit(NCI_DISCOVERY, &ndev->flags);
-
        /* If target was active, complete the request only in deactivate_ntf */
        if ((status != NCI_STATUS_OK) ||
-               (!test_bit(NCI_POLL_ACTIVE, &ndev->flags)))
+               (atomic_read(&ndev->state) != NCI_POLL_ACTIVE)) {
+               atomic_set(&ndev->state, NCI_IDLE);
                nci_req_complete(ndev, status);
+       }
 }
 
 void nci_rsp_packet(struct nci_dev *ndev, struct sk_buff *skb)