Commit cdee6a44 authored by Christian Eggers's avatar Christian Eggers Committed by Luiz Augusto von Dentz
Browse files

Bluetooth: hci_core: add missing braces when using macro parameters 



Macro parameters should always be put into braces when accessing it.

Fixes: 4fc9857a ("Bluetooth: hci_sync: Add check simultaneous roles support")
Signed-off-by: default avatarChristian Eggers <ceggers@arri.de>
Signed-off-by: default avatarLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
parent dfef8d87

include/net/bluetooth/hci_core.h

+11 −11
Original line number Diff line number Diff line
@@ -831,18 +831,18 @@ extern struct mutex hci_cb_list_lock; 


#define hci_dev_clear_volatile_flags(hdev)				\

	do {								\

		hci_dev_clear_flag(hdev, HCI_LE_SCAN);		\

		hci_dev_clear_flag(hdev, HCI_LE_ADV);		\

		hci_dev_clear_flag(hdev, HCI_LL_RPA_RESOLUTION);\

		hci_dev_clear_flag(hdev, HCI_PERIODIC_INQ);	\

		hci_dev_clear_flag(hdev, HCI_QUALITY_REPORT);	\

		hci_dev_clear_flag((hdev), HCI_LE_SCAN);		\

		hci_dev_clear_flag((hdev), HCI_LE_ADV);			\

		hci_dev_clear_flag((hdev), HCI_LL_RPA_RESOLUTION);	\

		hci_dev_clear_flag((hdev), HCI_PERIODIC_INQ);		\

		hci_dev_clear_flag((hdev), HCI_QUALITY_REPORT);		\

	} while (0)



#define hci_dev_le_state_simultaneous(hdev) \

	(!test_bit(HCI_QUIRK_BROKEN_LE_STATES, &hdev->quirks) && \

	 (hdev->le_states[4] & 0x08) &&	/* Central */ \

	 (hdev->le_states[4] & 0x40) &&	/* Peripheral */ \

	 (hdev->le_states[3] & 0x10))	/* Simultaneous */

	(!test_bit(HCI_QUIRK_BROKEN_LE_STATES, &(hdev)->quirks) && \

	 ((hdev)->le_states[4] & 0x08) &&	/* Central */ \

	 ((hdev)->le_states[4] & 0x40) &&	/* Peripheral */ \

	 ((hdev)->le_states[3] & 0x10))		/* Simultaneous */



/* ----- HCI interface to upper protocols ----- */

int l2cap_connect_ind(struct hci_dev *hdev, bdaddr_t *bdaddr);