Skip to contentSkip to conversation

Bluetooth:hci_qca: Changed unsigned long to bool

PATCH In review Mutharaju, Prasanna (P.) 2 participants 2 messages Sep 28, 2015

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 Bluetooth:hci_qca: Changed unsigned long to bool

From: Prasanna Karthik <[email protected]>

'retransmit' being set in HCI_IBS_TX_WAKING case, using bool would be efficient. Initialize local bool to false.

---
 drivers/bluetooth/hci_qca.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/bluetooth/hci_qca.c +0 -0
index a8aa077..6fbd7d8 100644
@@ -347,7 +347,7 @@ static void hci_ibs_wake_retrans_timeout(unsigned long =

arg)
struct hci_uart *hu =3D (struct hci_uart *)arg;
struct qca_data *qca =3D hu->priv;
unsigned long flags, retrans_delay;
- unsigned long retransmit =3D 0;
+ bool retransmit =3D false;
=20
BT_DBG("hu %p wake retransmit timeout in %d state",
hu, qca->tx_ibs_state);
@@ -358,7 +358,7 @@ static void hci_ibs_wake_retrans_timeout(unsigned long = arg)
switch (qca->tx_ibs_state) {
case HCI_IBS_TX_WAKING:
/* No WAKE_ACK, retransmit WAKE */
- retransmit =3D 1;
+ retransmit =3D true;
if (send_hci_ibs_cmd(HCI_IBS_WAKE_IND, hu) < 0) {
BT_ERR("Failed to acknowledge device wake up"); break;
--=20
1.9.1

maintainer
#1 lore raw

Hi Prasanna,

Quoted 7 lines

'retransmit' being set in HCI_IBS_TX_WAKING case, using bool would be efficient. Initialize local bool to false.

---
drivers/bluetooth/hci_qca.c | 4 ++--

1 file changed, 2 insertions(+), 2 deletions(-)

patch has been applied to bluetooth-next tree.

Regards

Marcel