diff mbox series

[Bluez,v3,2/2] lib/mgmt: Adding Add Adv Patterns Monitor RSSI opcode

Message ID 20201216114231.Bluez.v3.2.I90bb4b25451e7f726fc2c8771cfe642b16312ce3@changeid
State Superseded
Headers show
Series [Bluez,v3,1/2] doc/mgmt-api: Add opcode for adding advertisement monitor with RSSI | expand

Commit Message

Archie Pusaka Dec. 16, 2020, 3:42 a.m. UTC
From: Archie Pusaka <apusaka@chromium.org>

The new op is to utilize RSSI in advertisement monitor

Reviewed-by: Manish Mandlik <mmandlik@chromium.org>
Reviewed-by: Miao-chen Chou <mcchou@chromium.org>
Reviewed-by: Yun-Hao Chung <howardchung@google.com>
---

Changes in v3:
* Flip the order of rssi data and pattern count

Changes in v2:
* Remove the trailing period on the commit title

 lib/mgmt.h | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

Comments

Archie Pusaka Jan. 6, 2021, 10:43 a.m. UTC | #1
Hi Bluez maintainers,

I just noticed that only patch 1/2 of the series is applied, and this
one has not.
Could you re-check this one please?
Or is it expected that I also submit the code which uses this API in
the same patch series?

Thanks,
Archie

On Wed, 16 Dec 2020 at 11:43, Archie Pusaka <apusaka@google.com> wrote:
>

> From: Archie Pusaka <apusaka@chromium.org>

>

> The new op is to utilize RSSI in advertisement monitor

>

> Reviewed-by: Manish Mandlik <mmandlik@chromium.org>

> Reviewed-by: Miao-chen Chou <mcchou@chromium.org>

> Reviewed-by: Yun-Hao Chung <howardchung@google.com>

> ---

>

> Changes in v3:

> * Flip the order of rssi data and pattern count

>

> Changes in v2:

> * Remove the trailing period on the commit title

>

>  lib/mgmt.h | 15 +++++++++++++++

>  1 file changed, 15 insertions(+)

>

> diff --git a/lib/mgmt.h b/lib/mgmt.h

> index f37f7e6540..76a03c9c24 100644

> --- a/lib/mgmt.h

> +++ b/lib/mgmt.h

> @@ -738,6 +738,21 @@ struct mgmt_rp_add_ext_adv_data {

>         uint8_t instance;

>  } __packed;

>

> +struct mgmt_adv_rssi_thresholds {

> +       int8_t   high_threshold;

> +       uint16_t high_threshold_timeout;

> +       int8_t   low_threshold;

> +       uint16_t low_threshold_timeout;

> +       uint8_t  sampling_period;

> +} __packed;

> +

> +#define MGMT_OP_ADD_ADV_PATTERNS_MONITOR_RSSI  0x0056

> +struct mgmt_cp_add_adv_patterns_monitor_rssi {

> +       struct mgmt_adv_rssi_thresholds rssi;

> +       uint8_t pattern_count;

> +       struct mgmt_adv_pattern patterns[0];

> +} __packed;

> +

>  #define MGMT_EV_CMD_COMPLETE           0x0001

>  struct mgmt_ev_cmd_complete {

>         uint16_t opcode;

> --

> 2.29.2.684.gfbc64c5ab5-goog

>
diff mbox series

Patch

diff --git a/lib/mgmt.h b/lib/mgmt.h
index f37f7e6540..76a03c9c24 100644
--- a/lib/mgmt.h
+++ b/lib/mgmt.h
@@ -738,6 +738,21 @@  struct mgmt_rp_add_ext_adv_data {
 	uint8_t	instance;
 } __packed;
 
+struct mgmt_adv_rssi_thresholds {
+	int8_t   high_threshold;
+	uint16_t high_threshold_timeout;
+	int8_t   low_threshold;
+	uint16_t low_threshold_timeout;
+	uint8_t  sampling_period;
+} __packed;
+
+#define MGMT_OP_ADD_ADV_PATTERNS_MONITOR_RSSI	0x0056
+struct mgmt_cp_add_adv_patterns_monitor_rssi {
+	struct mgmt_adv_rssi_thresholds rssi;
+	uint8_t pattern_count;
+	struct mgmt_adv_pattern patterns[0];
+} __packed;
+
 #define MGMT_EV_CMD_COMPLETE		0x0001
 struct mgmt_ev_cmd_complete {
 	uint16_t opcode;