Re: [PATCH 1/6] android/hal-ipc-api: Add MCE HAL

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

 



Hi Grzegorz,

On Monday 22 of September 2014 13:01:43 Grzegorz Kolodziejczyk wrote:
> ---
>  android/hal-ipc-api.txt | 51 +++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 51 insertions(+)
> 
> diff --git a/android/hal-ipc-api.txt b/android/hal-ipc-api.txt
> index f6e1680..38f75e6 100644
> --- a/android/hal-ipc-api.txt
> +++ b/android/hal-ipc-api.txt
> @@ -2396,3 +2396,54 @@ Notifications:
>  	Opcode 0x95 - Ring Indication notification
>  
>  		Notification parameters: <none>
> +
> +
> +Bluetooth MCE HAL (ID 11)
> +=========================

I'd prefer if we stick to term 'map client' instead of MCE.

> +
> +Android HAL name: "map_client" (BT_PROFILE_MAP_CLIENT_ID)
> +
> +Structures:

Since this is used only in one place I'd defined it there (see below).

> +
> +	MAS instance: Instance ID (1 octet)
> +	              Channel (1 octet)
> +	              Message type (1 octet)

This should be named 'Message types'.

> +	              Name (string)
> +
> +	Valid Message type: 0x01 = EMAIL
> +	                    0x02 = SMS_GSM
> +	                    0x04 = SMS_CDMA
> +	                    0x08 = MMS

I think we should not define that in IPC since this is not defined in HAL.
Especially that this should be bitmask, not values. Just pass whatever we
get from remote SDP.

> +
> +Commands and responses:
> +
> +	Opcode 0x00 - Error response
> +
> +		Response parameters: Status (1 octet)
> +
> +		Valid status values: 0x01 = Fail
> +		                     0x02 = Not ready
> +		                     0x03 = No memory
> +		                     0x04 = Busy
> +		                     0x05 = Done (already completed)
> +		                     0x06 = Unsupported
> +		                     0x07 = Parameter invalid
> +		                     0x08 = Unhandled
> +		                     0x09 = Authentication failure
> +		                     0x0a = Remote device down
> +
> +	Opcode 0x01 - Get Remote MAS Instances
> +
> +		Command parameters: Remote address (6 octets)
> +		Response parameters: <none>
> +
> +		In case of an error, the error response will be returned.
> +
> +Notifications:
> +
> +	Opcode 0x81 - Remote MAS Instances notification
> +
> +		Notification parameters: Status (1 octet)
> +		                         Remote address (6 octets)
> +		                         Number of instances (1 octet)
> +		                         MAS instance (variable)

Number of instances (1 octet)
Instance ID #(1 octet)
Channel #(1 octet)
Message type #(1 octet)
Name # (string)

-- 
Best regards, 
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