Hi Marcel, > On Wed, Mar 25, 2015 at 9:21 AM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote: > Hi Arman, > >> This patch adds macro definitions for possible advertising instance >> flags that can be passed to the "Add Advertising" command. >> >> Signed-off-by: Arman Uguray <armansito@xxxxxxxxxxxx> >> --- >> include/net/bluetooth/mgmt.h | 8 ++++++++ >> 1 file changed, 8 insertions(+) >> >> diff --git a/include/net/bluetooth/mgmt.h b/include/net/bluetooth/mgmt.h >> index 68abd4b..fc50cee 100644 >> --- a/include/net/bluetooth/mgmt.h >> +++ b/include/net/bluetooth/mgmt.h >> @@ -554,6 +554,14 @@ struct mgmt_rp_add_advertising { >> __u8 instance; >> } __packed; >> >> +#define MGMT_ADV_FLAG_CONNECTABLE (1 << 0) >> +#define MGMT_ADV_FLAG_DISCOV (1 << 1) >> +#define MGMT_ADV_FLAG_LIMITED_DISCOV (1 << 2) >> +#define MGMT_ADV_FLAG_MANAGED_FLAGS (1 << 3) >> +#define MGMT_ADV_FLAG_TX_POWER (1 << 4) >> +#define MGMT_ADV_FLAG_APPEARANCE (1 << 5) >> +#define MGMT_ADV_FLAG_LOCAL_NAME (1 << 6) >> + > > any reason to not use BIT(0) etc. here. > No reason at all, fixed for next patch set. > Regards > > Marcel > Arman -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html