Received-SPF: pass (google.com: best guess record for domain of linux-bluetooth-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Tue, 29 Oct 2019 22:32:19 +0200
From:   Pasi =?iso-8859-1?Q?K=E4rkk=E4inen?= <pasik@iki.fi>
To:     Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Cc:     "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>,
        Pali =?iso-8859-1?Q?Roh=E1r?= <pali.rohar@gmail.com>
Subject: Re: [PATCH BlueZ] a2dp: Remove experimental flag for remote
 MediaEndpoint
Message-ID: <20191029203218.GM28704@reaktio.net>
References: <20191018105339.1297-1-luiz.dentz@gmail.com>
 <CABBYNZJc_Z_w2tTOrWj8RhfOgtvx72dFn7hFE=HehcKKUBX-fw@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CABBYNZJc_Z_w2tTOrWj8RhfOgtvx72dFn7hFE=HehcKKUBX-fw@mail.gmail.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

Hi Luiz,

Now that A2DP codec switching related APIs are declared stable in bluez git,
I'd like to request bluez 5.52 release soon.

This is because feature freeze for PulseAudio 14.0 is scheduled for 2019-12-13,
so It'd be nice to have a bluez release before that.. Also to try to avoid 
distros shipping bluez 5.51, where MediaEndpoint API isn't stable yet..


Thanks a lot,

-- Pasi

On Sat, Oct 19, 2019 at 03:03:19PM +0300, Luiz Augusto von Dentz wrote:
> Hi,
> 
> On Fri, Oct 18, 2019 at 1:53 PM Luiz Augusto von Dentz
> <luiz.dentz@gmail.com> wrote:
> >
> > From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
> >
> > This makes the MediaEndpoint and stable API for remote endpoints which
> > aligns with RegisterApplication API which is already stable.
> > ---
> >  profiles/audio/a2dp.c | 17 +++++------------
> >  1 file changed, 5 insertions(+), 12 deletions(-)
> >
> > diff --git a/profiles/audio/a2dp.c b/profiles/audio/a2dp.c
> > index f98ec8505..e8262cdfe 100644
> > --- a/profiles/audio/a2dp.c
> > +++ b/profiles/audio/a2dp.c
> > @@ -1761,7 +1761,7 @@ static DBusMessage *set_configuration(DBusConnection *conn, DBusMessage *msg,
> >  }
> >
> >  static const GDBusMethodTable sep_methods[] = {
> > -       { GDBUS_EXPERIMENTAL_ASYNC_METHOD("SetConfiguration",
> > +       { GDBUS_ASYNC_METHOD("SetConfiguration",
> >                                         GDBUS_ARGS({ "endpoint", "o" },
> >                                                 { "properties", "a{sv}" } ),
> >                                         NULL, set_configuration) },
> > @@ -1837,14 +1837,10 @@ static gboolean get_device(const GDBusPropertyTable *property,
> >  }
> >
> >  static const GDBusPropertyTable sep_properties[] = {
> > -       { "UUID", "s", get_uuid, NULL, NULL,
> > -                                       G_DBUS_PROPERTY_FLAG_EXPERIMENTAL },
> > -       { "Codec", "y", get_codec, NULL, NULL,
> > -                                       G_DBUS_PROPERTY_FLAG_EXPERIMENTAL },
> > -       { "Capabilities", "ay", get_capabilities, NULL, NULL,
> > -                                       G_DBUS_PROPERTY_FLAG_EXPERIMENTAL },
> > -       { "Device", "o", get_device, NULL, NULL,
> > -                                       G_DBUS_PROPERTY_FLAG_EXPERIMENTAL },
> > +       { "UUID", "s", get_uuid, NULL, NULL },
> > +       { "Codec", "y", get_codec, NULL, NULL },
> > +       { "Capabilities", "ay", get_capabilities, NULL, NULL },
> > +       { "Device", "o", get_device, NULL, NULL },
> >         { }
> >  };
> >
> > @@ -1862,9 +1858,6 @@ static void register_remote_sep(void *data, void *user_data)
> >         sep->chan = chan;
> >         sep->sep = rsep;
> >
> > -       if (!(g_dbus_get_flags() & G_DBUS_FLAG_ENABLE_EXPERIMENTAL))
> > -               goto done;
> > -
> >         if (asprintf(&sep->path, "%s/sep%d",
> >                                 device_get_path(chan->device),
> >                                 avdtp_get_seid(rsep)) < 0) {
> > --
> > 2.21.0
> >
> 
> Pushed.
> 
> -- 
> Luiz Augusto von Dentz