NFC: st21nfcb: few code clean up
authorChristophe Ricard <christophe.ricard@gmail.com>
Sun, 25 May 2014 20:46:58 +0000 (22:46 +0200)
committerSamuel Ortiz <sameo@linux.intel.com>
Tue, 22 Jul 2014 22:49:37 +0000 (00:49 +0200)
Signed-off-by: Christophe Ricard <christophe-h.ricard@st.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
drivers/nfc/st21nfcb/i2c.c
include/linux/platform_data/st21nfcb.h

index 0f690baaef7a4aa6c9ca7670a896e22be7d1b6bc..8af880ead5db3f1b558aaa1ff28ac38c1b1e078c 100644 (file)
@@ -164,11 +164,11 @@ static int st21nfcb_nci_i2c_read(struct st21nfcb_i2c_phy *phy,
        u8 buf[ST21NFCB_NCI_I2C_MAX_SIZE];
        struct i2c_client *client = phy->i2c_dev;
 
-       r = i2c_master_recv(client, buf, 4);
+       r = i2c_master_recv(client, buf, ST21NFCB_NCI_I2C_MIN_SIZE);
        if (r == -EREMOTEIO) {  /* Retry, chip was in standby */
                usleep_range(1000, 4000);
-               r = i2c_master_recv(client, buf, 4);
-       } else if (r != 4) {
+               r = i2c_master_recv(client, buf, ST21NFCB_NCI_I2C_MIN_SIZE);
+       } else if (r != ST21NFCB_NCI_I2C_MIN_SIZE) {
                nfc_err(&client->dev, "cannot read ndlc & nci header\n");
                return -EREMOTEIO;
        }
@@ -179,13 +179,13 @@ static int st21nfcb_nci_i2c_read(struct st21nfcb_i2c_phy *phy,
                return -EBADMSG;
        }
 
-       *skb = alloc_skb(4 + len, GFP_KERNEL);
+       *skb = alloc_skb(ST21NFCB_NCI_I2C_MIN_SIZE + len, GFP_KERNEL);
        if (*skb == NULL)
                return -ENOMEM;
 
-       skb_reserve(*skb, 4);
-       skb_put(*skb, 4);
-       memcpy((*skb)->data, buf, 4);
+       skb_reserve(*skb, ST21NFCB_NCI_I2C_MIN_SIZE);
+       skb_put(*skb, ST21NFCB_NCI_I2C_MIN_SIZE);
+       memcpy((*skb)->data, buf, ST21NFCB_NCI_I2C_MIN_SIZE);
 
        if (!len)
                return 0;
@@ -197,7 +197,7 @@ static int st21nfcb_nci_i2c_read(struct st21nfcb_i2c_phy *phy,
        }
 
        skb_put(*skb, len);
-       memcpy((*skb)->data + 4, buf, len);
+       memcpy((*skb)->data + ST21NFCB_NCI_I2C_MIN_SIZE, buf, len);
 
        I2C_DUMP_SKB("i2c frame read", *skb);
 
index 2a7b769c714dfa3858f085c85dbbbd64374d44a5..2d11f1f5efaba8884b9e2c7a07ad7e26732b9d48 100644 (file)
@@ -16,8 +16,8 @@
  * along with this program; if not, see <http://www.gnu.org/licenses/>.
  */
 
-#ifndef _ST21NFCA_HCI_H_
-#define _ST21NFCA_HCI_H_
+#ifndef _ST21NFCB_NCI_H_
+#define _ST21NFCB_NCI_H_
 
 #include <linux/i2c.h>