Re: android/handsfree.c | Possible Fix | Review

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

 



Hi Prabhu,

On Wednesday 01 of April 2015 18:40:52 Prabhu Chawandi wrote:
> Hi,
> 
>   When we were testing handsfree we were seeing command is not being
> identified and when looked size was  sent zero when we set it as below
> it started working.
> 
> static const struct ipc_handler sco_handlers[] = {
>         /* SCO_OP_GET_FD */
> -        { bt_sco_get_fd, false, 0 }
> };
> 
> 
> static const struct ipc_handler sco_handlers[] = {
>         /* SCO_OP_GET_FD */
>  +       { bt_sco_get_fd, false, sizeof(struct sco_cmd_get_fd) }
> };

Yes, the fix is correct. Could you send a proper patch for it?
 
> one request -  have you documented anywhere the sequence of commands
> to test each profiles supported till now. if we get the the proper
> sequences we can validate much faster stabilising the code. Else we
> need to spend time in identifying the sequences.
> 
> So, if you have documented anywhere request you to share,

Those sequences are defined by android code (framework or multimedia) and are 
not really documented. You could find some suggestion if you look at our PTS 
testing documentation (android/pts-*.txt) where we do use haltest for tests. 

Note that if possible we prefer to use Android Framework for validation 
instead of haltest (which is mostly development tool).

-- 
BR
Szymon Janc
--
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