Re: [PATCH 01/15] android/hal-msg: Add HDP app registration struct

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 03/12/2014 05:39 PM, Luiz Augusto von Dentz wrote:
Hi Ravi,

On Wed, Mar 12, 2014 at 5:10 PM, Ravi kumar Veeramally
<ravikumar.veeramally@xxxxxxxxxxxxxxx> wrote:
---
  android/hal-msg.h | 21 +++++++++++++++++++++
  1 file changed, 21 insertions(+)

diff --git a/android/hal-msg.h b/android/hal-msg.h
index 0abbbe6..b1942de 100644
--- a/android/hal-msg.h
+++ b/android/hal-msg.h
@@ -383,6 +383,27 @@ struct hal_cmd_pan_disconnect {
         uint8_t bdaddr[6];
  } __attribute__((packed));

+#define HAL_OP_HEALTH_REG_APP          0x01
+struct hal_cmd_health_reg_app {
+       uint8_t app_name[255];
+       uint8_t provider_name[255];
+       uint8_t service_name[255];
+       uint8_t service_descr[200];
+       uint8_t num_of_mdep;
+
+       struct {
+               uint8_t role;
+               uint8_t data_type;
+               uint8_t channel_type;
+               uint8_t descr[255];
+       } mdep_cfg[20];
+} __attribute__((packed));
+
+struct hal_rsp_health_reg_app {
+       uint16_t app_id;
+} __attribute__((packed));
This most likely will not fit in our MTU which currently is 1024 bytes
and given that you have 20 cfgs this is going to be huge. We should
probably define a hal_string e.g:

struct hal_string {
     uint8_t len;
     uint8_t data[0];
};
  Ok, I will update it.
Also we do not need to send the 20 cfgs every time so that should
probably be set to 0 and we write only num_of_mdep, if that still
doesn't fit in our MTU perhaps we should consider a bigger size.

  Ok.

 Thanks,
 Ravi.
--
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




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux