diff options
| author | Christophe Ricard <christophe.ricard@gmail.com> | 2014-12-02 21:27:47 +0100 |
|---|---|---|
| committer | Samuel Ortiz <sameo@linux.intel.com> | 2014-12-02 22:46:47 +0100 |
| commit | 98ff416f97c53b727d6a52f4d2b29bdf4775ac69 (patch) | |
| tree | 3ac6186c350a9868e2af63cf2877a2347cbac67e | |
| parent | aceb32d05aaf559758a29ffdefdda025a131c978 (diff) | |
| download | op-kernel-dev-98ff416f97c53b727d6a52f4d2b29bdf4775ac69.zip op-kernel-dev-98ff416f97c53b727d6a52f4d2b29bdf4775ac69.tar.gz | |
NFC: nci: Add status byte management in case of error.
The nci status byte was ignored. In case of tag reading for example,
if the tag is removed from the antenna there is no way for the upper
layers (aka: stack) to get inform about such event.
Signed-off-by: Christophe Ricard <christophe-h.ricard@st.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
| -rw-r--r-- | net/nfc/nci/data.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/net/nfc/nci/data.c b/net/nfc/nci/data.c index 91f5d55..a2de2a8 100644 --- a/net/nfc/nci/data.c +++ b/net/nfc/nci/data.c @@ -185,11 +185,16 @@ exit: static void nci_add_rx_data_frag(struct nci_dev *ndev, struct sk_buff *skb, - __u8 pbf) + __u8 pbf, __u8 status) { int reassembly_len; int err = 0; + if (status) { + err = status; + goto exit; + } + if (ndev->rx_data_reassembly) { reassembly_len = ndev->rx_data_reassembly->len; @@ -241,6 +246,7 @@ exit: void nci_rx_data_packet(struct nci_dev *ndev, struct sk_buff *skb) { __u8 pbf = nci_pbf(skb->data); + __u8 status = 0; pr_debug("len %d\n", skb->len); @@ -258,8 +264,9 @@ void nci_rx_data_packet(struct nci_dev *ndev, struct sk_buff *skb) ndev->target_active_prot == NFC_PROTO_ISO15693) { /* frame I/F => remove the status byte */ pr_debug("frame I/F => remove the status byte\n"); + status = skb->data[skb->len - 1]; skb_trim(skb, (skb->len - 1)); } - nci_add_rx_data_frag(ndev, skb, pbf); + nci_add_rx_data_frag(ndev, skb, pbf, nci_to_errno(status)); } |
