diff options
author | Johan Hedberg <johan.hedberg@intel.com> | 2014-09-01 02:45:03 -0400 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2014-09-08 13:07:55 -0400 |
commit | 434714dc02b286d3f21179c651a6f1a84e199eb7 (patch) | |
tree | b8869a407a8023070a209a6775ca7c312e24944b /net/bluetooth | |
parent | 839035a7b3acd17a6f739b1fb50298e9499d3fa4 (diff) |
Bluetooth: Add clarifying comment for LE CoC result value
The "pending" L2CAP response value is not defined for LE CoC. This patch
adds a clarifying comment to the code so that the reader will not think
there is a bug in trying to use this value for LE CoC.
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'net/bluetooth')
-rw-r--r-- | net/bluetooth/l2cap_core.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c index 2d9a2b58d2c8..ab405f0e53cb 100644 --- a/net/bluetooth/l2cap_core.c +++ b/net/bluetooth/l2cap_core.c | |||
@@ -5413,6 +5413,11 @@ static int l2cap_le_connect_req(struct l2cap_conn *conn, | |||
5413 | 5413 | ||
5414 | if (test_bit(FLAG_DEFER_SETUP, &chan->flags)) { | 5414 | if (test_bit(FLAG_DEFER_SETUP, &chan->flags)) { |
5415 | l2cap_state_change(chan, BT_CONNECT2); | 5415 | l2cap_state_change(chan, BT_CONNECT2); |
5416 | /* The following result value is actually not defined | ||
5417 | * for LE CoC but we use it to let the function know | ||
5418 | * that it should bail out after doing its cleanup | ||
5419 | * instead of sending a response. | ||
5420 | */ | ||
5416 | result = L2CAP_CR_PEND; | 5421 | result = L2CAP_CR_PEND; |
5417 | chan->ops->defer(chan); | 5422 | chan->ops->defer(chan); |
5418 | } else { | 5423 | } else { |