public inbox for linux-bluetooth@vger.kernel.org
 help / color / mirror / Atom feed
From: Fabien Chevalier <fchevalier@silicom.fr>
To: Brad Midgley <bmidgley@xmission.com>,
	Johan Hedberg <johan.hedberg@nokia.com>,
	Luiz Augusto von Dentz <luiz.dentz@gmail.com>,
	thiagossantos@gmail.com, Marcel Holtmann <marcel@holtmann.org>
Cc: BlueZ development <bluez-devel@lists.sourceforge.net>
Subject: [PATCH] newapi patch v1
Date: Fri, 19 Oct 2007 19:26:56 +0200	[thread overview]
Message-ID: <4718E8E0.5050608@silicom.fr> (raw)

[-- Attachment #1: Type: text/plain, Size: 1380 bytes --]


All, here is the first version of the patch that moves to the newapi
usage... yeheee !!
As nobody suggested a new name, i've just chose one, which is
"btaudioservice.h" :-). Of course this can be changed if somebody
prefers a better name :-)

This patch does not bring any new feature by itself, it's goal is only
to get rid from the ipc.h stuff. However is is already quite big, but i
really can't make it smaller without breaking stuff that is known to
work today :-(

I did some basic testing of the patch, which is:
   - play a file using a2dp, seek during playing (audacious)
   - play a file using a2dp, seek during playing (rhythmnbox)
   - play a file using SCO, seek during playing (audacious)
   - record and play file using aplay & arecord.

I didn't notice any regression, but i don't want to fool anybody : such
a big patch will inevitably bring its set of regressions, it's just that
i haven't found them yet :-)

Due to the fairly big nature of the patch, i would suggest to wait for
3.21 before moving to the new api, this will give us the usual time
between two bluez releases to :
   - fix regressions introduced (my job :-) )
   - upgrading the gstreamer plugin to use the new api ( i'd prefer
thiagoss to work on this one (with my support of course), as i've never
put my nose on this part of the code )

As usual, comments are welcome. ;-)

Cheers,

Fabien


[-- Attachment #2: fchevalier.vcf --]
[-- Type: text/x-vcard, Size: 253 bytes --]

begin:vcard
fn:Fabien CHEVALIER
n:CHEVALIER;Fabien
org:SILICOM
adr:;;4 rue de Jouanet; RENNES ATALANTE;;35700;FRANCE
email;internet:fchevalier@silicom.fr
title:Software & Studies Engineer
tel;work:+33 (0) 2 99 84 17 17
version:2.1
end:vcard


             reply	other threads:[~2007-10-19 17:26 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-19 17:26 Fabien Chevalier [this message]
2007-10-19 17:30 ` [Bluez-devel] [PATCH] newapi patch v1 Fabien Chevalier
2007-10-19 17:30 ` Fabien Chevalier
2007-10-19 23:35 ` Brad Midgley
2007-10-23 18:01   ` Fabien Chevalier
2007-10-23 18:28     ` Marcel Holtmann
2007-10-24 12:12       ` Fabien Chevalier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4718E8E0.5050608@silicom.fr \
    --to=fchevalier@silicom.fr \
    --cc=bluez-devel@lists.sourceforge.net \
    --cc=bmidgley@xmission.com \
    --cc=johan.hedberg@nokia.com \
    --cc=luiz.dentz@gmail.com \
    --cc=marcel@holtmann.org \
    --cc=thiagossantos@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox