Re: [RFC 10/11] android/tester: Execute hh report cbacks in main loop

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

 



Hi Anderson,

On 02/19/2014 12:50 PM, Anderson Lizardo wrote:
Hi Jakub,

On Wed, Feb 19, 2014 at 6:25 AM, Jakub Tyszkowski
<jakub.tyszkowski@xxxxxxxxx> wrote:
Execute HIDHost generic get_report_cb in tester's main loop.
---
  android/android-tester.c | 25 +++++++++++++++++++++----
  1 file changed, 21 insertions(+), 4 deletions(-)

diff --git a/android/android-tester.c b/android/android-tester.c
index 1e1d9aa..2bf897d 100644
--- a/android/android-tester.c
+++ b/android/android-tester.c
@@ -129,6 +129,7 @@ struct generic_cb_data {

         bthh_hid_info_t hid_info;
         bthh_protocol_mode_t mode;
+       uint8_t report;
  };

  static char exec_dir[PATH_MAX + 1];
@@ -3311,17 +3312,33 @@ static void hidhost_protocol_mode_cb(bt_bdaddr_t *bd_addr,
         g_idle_add(hidhost_protocol_mode, cb_data);
  }

-static void hidhost_get_report_cb(bt_bdaddr_t *bd_addr, bthh_status_t status,
-                                               uint8_t *report, int size)
+static gboolean hidhost_get_report(gpointer user_data)
  {
         struct test_data *data = tester_get_data();
         const struct hidhost_generic_data *test = data->test_data;
+       struct generic_cb_data *cb_data = user_data;

         data->cb_count++;

         if (test && test->expected_hal_cb.get_report_cb)
-               test->expected_hal_cb.get_report_cb(bd_addr, status, report,
-                                                                       size);
+               test->expected_hal_cb.get_report_cb(&cb_data->bdaddr,
+                       cb_data->status, &cb_data->report, cb_data->num);
+
+       g_free(cb_data);
+       return FALSE;
+}
+
+static void hidhost_get_report_cb(bt_bdaddr_t *bd_addr, bthh_status_t status,
+                                               uint8_t *report, int size)
+{
+       struct generic_cb_data *cb_data = g_new0(struct generic_cb_data, 1);
+
+       cb_data->bdaddr = *bd_addr;
+       cb_data->status = status;
+       cb_data->report = *report;
+       cb_data->num = size;

Shouldn't cb_data->report be an array allocated with g_memdup()?
Yes it should. Sorry about that.


+
+       g_idle_add(hidhost_get_report, cb_data);
  }

Best Regards,

Best Regards,

Jakub Tyszkowski
--
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