Return-Path: <cyrus@holtmann.org>
From: =?iso-8859-1?q?Jos=E9_Antonio_Santos_Cadenas?=
	<santoscadenas@gmail.com>
To: "Gustavo F. Padovan" <gustavo@padovan.org>
Subject: Re: Proposed API for HDP
Date: Thu, 8 Jul 2010 20:36:00 +0200
Cc: linux-bluetooth@vger.kernel.org
References: <201007081912.31407.santoscadenas@gmail.com> <20100708175430.GB26759@vigoh>
In-Reply-To: <20100708175430.GB26759@vigoh>
MIME-Version: 1.0
Content-Type: Text/Plain;
  charset="iso-8859-1"
Message-Id: <201007082036.00289.santoscadenas@gmail.com>
Sender: linux-bluetooth-owner@vger.kernel.org
List-ID: <linux-bluetooth.vger.kernel.org>

El Thursday 08 July 2010 19:54:30 Gustavo F. Padovan escribi?:
> Hi Jos?,
> 
> * Jos? Antonio Santos Cadenas <santoscadenas@gmail.com> [2010-07-08
> 19:12:31 +0200]:
> 
> <snip>
> 
> > Service		org.bluez
> > Interface	org.bluez.HealthDeviceApplication
> > Object path	[variable
> > prefix]/{hci0,hci1,...}/dev_XX_XX_XX_XX_XX_XX/hdp_YYYY
> > 
> > Methods:
> > 	array GetProperties()
> > 	
> > 		Gets the information of the remote application published on its
> > 		SDP record. The returned data format is as follows:
> > 		
> > 		{
> > 		
> > 			"end_points": [
> > 			
> > 				"mdepid": uint8,
> > 				"role"  : "source" or "sink" ,
> > 				"specs" : [{
> > 				
> > 					"dtype"       : uint16,
> > 					"description" : string, (optional)
> > 					}]
> > 				
> > 				]
> > 		
> > 		}
> > 	
> > 	object Connect(path local_application_id)
> > 	
> > 		Connects the local application with the remote application.
> > 		
> > 		Only the bus client that created the local session will be able
> > 		to create connections using it.
> > 		
> > 		If the Device is already connected with an other application an
> > 		org.bluez.Error.AlreadyConnected error will be received.
> > 		
> > 		Possible errors: org.bluez.Error.InvalidArguments
> > 		
> > 				org.bluez.Error.AlreadyConnected
> > 				org.bluez.Error.HealthError
> > 	
> > 	void Disconnect()
> > 	
> > 		Disconnect from the remote application the state will also be
> > 		deleted. And no future reconnections will be possible. For
> > 		keeping the state the method Pause of the health link should be
> > 		used.
> > 		
> > 		Possible errors: org.bluez.Error.InvalidArguments
> > 		
> > 				org.bluez.Error.NotFound
> > 				org.bluez.Error.HealthError
> > 	
> > 	boolean Echo(array{byte})
> > 	
> > 		Sends an echo petition to the remote intance. Returns True if
> > 		response matches with the buffer sent. If some error is detected
> > 		False value is returned and the associated MCL is closed.
> > 	
> > 	path OpenDataChannel(byte mdepid, string conf)
> > 	
> > 		Creates a new data channel with the indicated config to the
> > 		remote MCAP Data End Point (MDEP).
> > 		The configuration should indicate the channel quality of
> > 		service using one of this values "reliable", "streaming", "any".
> > 		
> > 		Returns the data channel path.
> > 		
> > 		Possible errors: org.bluez.Error.InvalidArguments
> > 		
> > 				org.bluez.Error.HealthError
> > 	
> > 	void ReconnectDataChannel(path data_channel)
> > 	
> > 		Reconnects a previously created data channel indicated by its
> > 		path.
> > 		
> > 		Possible errors: org.bluez.Error.InvalidArguments
> > 		
> > 				org.bluez.Error.HealthError
> > 				org.bluez.Error.NotFound
> 
> So are we really going to expose the temporally disconnect and
> reconnection to the applicantion level? The application should not
> care if the data channel is there or not.

We changed our mind about this issue because there are cases were an implicit 
reconnection will not be possible (when the remote do not publish an SDP 
record) so if the application is concerned about this issues could try to 
avoid this kind of failure waiting for the remote to reconnect the data 
channel.

> 
> > 	int GetDataChannelFileDescriptor(path data_channel)
> > 	
> > 		Gets a file descriptor where data can be read or written.
> > 		
> > 		Possible errors: org.bluez.Error.InvalidArguments
> > 		
> > 				org.bluez.Error.NotFound
> > 				org.bluez.Error.HealthError
> 
> What about the fd-passing feature of D-Bus here?

There is an error here, the return value is not an integer is a file 
descriptor.

Thanks for the comments.

Regards.

Jose.