diff options
| author |   Christophe Ricard <christophe.ricard@gmail.com> | 2014-08-11 00:04:53 +0200 |
|---|---|---|
| committer |   Samuel Ortiz <sameo@linux.intel.com> | 2014-09-08 00:04:22 +0200 |
| commit | 32b41d8f7587b6e8a2cbb3a15f83c3a9687f1f17 (patch) | |
| tree | 1b2987968a9970a981bc586e0a5dc8fb9a1b17aa | |
| parent | NFC: st21nfca: Change nfcid3 generation (diff) | |
| download | linux-dev-32b41d8f7587b6e8a2cbb3a15f83c3a9687f1f17.tar.xz linux-dev-32b41d8f7587b6e8a2cbb3a15f83c3a9687f1f17.zip | |
NFC: st21nfca: Remove useless err == 0 condition
On top of st21nfca_im_recv_atr_res_cb and st21nfca_im_recv_dep_res_cb a
condition if (err != 0) is present. Because err is never modified in the
code err will always be 0 at the end.
The condition can be removed.
Signed-off-by: Christophe Ricard <christophe-h.ricard@st.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
| -rw-r--r-- | drivers/nfc/st21nfca/st21nfca_dep.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/nfc/st21nfca/st21nfca_dep.c b/drivers/nfc/st21nfca/st21nfca_dep.c index cd942c5adc53..979c96986b34 100644 --- a/drivers/nfc/st21nfca/st21nfca_dep.c +++ b/drivers/nfc/st21nfca/st21nfca_dep.c @@ -485,8 +485,7 @@ static void st21nfca_im_recv_atr_res_cb(void *context, struct sk_buff *skb, ST21NFCA_PP2LRI(atr_res->ppi)); break; default: - if (err == 0) - kfree_skb(skb); + kfree_skb(skb); break; } } @@ -616,8 +615,7 @@ static void st21nfca_im_recv_dep_res_cb(void *context, struct sk_buff *skb, } exit: - if (err == 0) - kfree_skb(skb); + kfree_skb(skb); } int st21nfca_im_send_dep_req(struct nfc_hci_dev *hdev, struct sk_buff *skb) |