NFC: Small nfc_hci_create_pipe refactoring
Check for error and return if any. This makes it easier to see what is a 'positive' function flow. Signed-off-by: Szymon Janc <szymon.janc@tieto.com> Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
This commit is contained in:
parent
36b05114d1
commit
80e4232ecb
|
@ -257,16 +257,16 @@ static u8 nfc_hci_create_pipe(struct nfc_hci_dev *hdev, u8 dest_host,
|
||||||
*result = nfc_hci_execute_cmd(hdev, NFC_HCI_ADMIN_PIPE,
|
*result = nfc_hci_execute_cmd(hdev, NFC_HCI_ADMIN_PIPE,
|
||||||
NFC_HCI_ADM_CREATE_PIPE,
|
NFC_HCI_ADM_CREATE_PIPE,
|
||||||
(u8 *) ¶ms, sizeof(params), &skb);
|
(u8 *) ¶ms, sizeof(params), &skb);
|
||||||
if (*result == 0) {
|
if (*result < 0)
|
||||||
resp = (struct hci_create_pipe_resp *)skb->data;
|
|
||||||
pipe = resp->pipe;
|
|
||||||
kfree_skb(skb);
|
|
||||||
|
|
||||||
pr_debug("pipe created=%d\n", pipe);
|
|
||||||
|
|
||||||
return pipe;
|
|
||||||
} else
|
|
||||||
return NFC_HCI_INVALID_PIPE;
|
return NFC_HCI_INVALID_PIPE;
|
||||||
|
|
||||||
|
resp = (struct hci_create_pipe_resp *)skb->data;
|
||||||
|
pipe = resp->pipe;
|
||||||
|
kfree_skb(skb);
|
||||||
|
|
||||||
|
pr_debug("pipe created=%d\n", pipe);
|
||||||
|
|
||||||
|
return pipe;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int nfc_hci_delete_pipe(struct nfc_hci_dev *hdev, u8 pipe)
|
static int nfc_hci_delete_pipe(struct nfc_hci_dev *hdev, u8 pipe)
|
||||||
|
|
Loading…
Reference in a new issue