Commit 15770b1a authored by Andrei Emeltchenko's avatar Andrei Emeltchenko Committed by Gustavo F. Padovan

Bluetooth: convert force_active variable to flag in l2cap chan

force_active variable inside l2cap_chan is a logical one and can
be easily converted to flag
Signed-off-by: default avatarAndrei Emeltchenko <andrei.emeltchenko@intel.com>
Signed-off-by: default avatarGustavo F. Padovan <padovan@profusion.mobi>
parent ecf61bdb
...@@ -325,7 +325,6 @@ struct l2cap_chan { ...@@ -325,7 +325,6 @@ struct l2cap_chan {
__u8 sec_level; __u8 sec_level;
__u8 role_switch; __u8 role_switch;
__u8 force_active;
__u8 ident; __u8 ident;
...@@ -464,6 +463,7 @@ enum { ...@@ -464,6 +463,7 @@ enum {
/* Definitions for flags in l2cap_chan */ /* Definitions for flags in l2cap_chan */
enum { enum {
FLAG_FORCE_ACTIVE,
FLAG_FORCE_RELIABLE, FLAG_FORCE_RELIABLE,
FLAG_FLUSHABLE, FLAG_FLUSHABLE,
}; };
......
...@@ -605,7 +605,7 @@ static inline void l2cap_send_sframe(struct l2cap_chan *chan, u16 control) ...@@ -605,7 +605,7 @@ static inline void l2cap_send_sframe(struct l2cap_chan *chan, u16 control)
else else
flags = ACL_START; flags = ACL_START;
bt_cb(skb)->force_active = chan->force_active; bt_cb(skb)->force_active = test_bit(FLAG_FORCE_ACTIVE, &chan->flags);
hci_send_acl(chan->conn->hcon, skb, flags); hci_send_acl(chan->conn->hcon, skb, flags);
} }
...@@ -1259,7 +1259,7 @@ static void l2cap_do_send(struct l2cap_chan *chan, struct sk_buff *skb) ...@@ -1259,7 +1259,7 @@ static void l2cap_do_send(struct l2cap_chan *chan, struct sk_buff *skb)
else else
flags = ACL_START; flags = ACL_START;
bt_cb(skb)->force_active = chan->force_active; bt_cb(skb)->force_active = test_bit(FLAG_FORCE_ACTIVE, &chan->flags);
hci_send_acl(hcon, skb, flags); hci_send_acl(hcon, skb, flags);
} }
......
...@@ -459,7 +459,7 @@ static int l2cap_sock_getsockopt(struct socket *sock, int level, int optname, ch ...@@ -459,7 +459,7 @@ static int l2cap_sock_getsockopt(struct socket *sock, int level, int optname, ch
break; break;
} }
pwr.force_active = chan->force_active; pwr.force_active = test_bit(FLAG_FORCE_ACTIVE, &chan->flags);
len = min_t(unsigned int, len, sizeof(pwr)); len = min_t(unsigned int, len, sizeof(pwr));
if (copy_to_user(optval, (char *) &pwr, len)) if (copy_to_user(optval, (char *) &pwr, len))
...@@ -680,7 +680,11 @@ static int l2cap_sock_setsockopt(struct socket *sock, int level, int optname, ch ...@@ -680,7 +680,11 @@ static int l2cap_sock_setsockopt(struct socket *sock, int level, int optname, ch
err = -EFAULT; err = -EFAULT;
break; break;
} }
chan->force_active = pwr.force_active;
if (pwr.force_active)
set_bit(FLAG_FORCE_ACTIVE, &chan->flags);
else
clear_bit(FLAG_FORCE_ACTIVE, &chan->flags);
break; break;
default: default:
...@@ -939,7 +943,6 @@ static void l2cap_sock_init(struct sock *sk, struct sock *parent) ...@@ -939,7 +943,6 @@ static void l2cap_sock_init(struct sock *sk, struct sock *parent)
chan->sec_level = pchan->sec_level; chan->sec_level = pchan->sec_level;
chan->role_switch = pchan->role_switch; chan->role_switch = pchan->role_switch;
chan->flags = pchan->flags; chan->flags = pchan->flags;
chan->force_active = pchan->force_active;
} else { } else {
switch (sk->sk_type) { switch (sk->sk_type) {
...@@ -969,8 +972,7 @@ static void l2cap_sock_init(struct sock *sk, struct sock *parent) ...@@ -969,8 +972,7 @@ static void l2cap_sock_init(struct sock *sk, struct sock *parent)
chan->sec_level = BT_SECURITY_LOW; chan->sec_level = BT_SECURITY_LOW;
chan->role_switch = 0; chan->role_switch = 0;
chan->flags = 0; chan->flags = 0;
chan->force_active = BT_POWER_FORCE_ACTIVE_ON; set_bit(FLAG_FORCE_ACTIVE, &chan->flags);
} }
/* Default config options */ /* Default config options */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment