From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <1508832299.28409.13.camel@hadess.net> Subject: Re: [PATCH v2 3/3] profiles/battery: Remove unused bas.[ch] From: Bastien Nocera To: Luiz Augusto von Dentz Cc: "linux-bluetooth@vger.kernel.org" Date: Tue, 24 Oct 2017 10:04:59 +0200 In-Reply-To: References: <20171023154814.20607-1-hadess@hadess.net> <20171023154814.20607-3-hadess@hadess.net> <1508796837.28409.5.camel@hadess.net> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: On Tue, 2017-10-24 at 10:40 +0300, Luiz Augusto von Dentz wrote: > Hi Bastien, > > On Tue, Oct 24, 2017 at 1:13 AM, Bastien Nocera > wrote: > > On Mon, 2017-10-23 at 23:39 +0300, Luiz Augusto von Dentz wrote: > > > Hi Bastian, > > > > > > On Mon, Oct 23, 2017 at 6:48 PM, Bastien Nocera > > t> > > > wrote: > > > > The Battery Service characteristics are not children of the HoG > > > > characteristics, so there's nothing to consume in the input > > > > profile. > > > > Remove that code which duplicated most of the battery profile > > > > plugin. > > > > > > Id rather not do this, we actually do intend the maintain a small > > > helper library that can be unit tested, what bt_bas lacks is just > > > proper support of bt_gatt_client nothing else. Also for the > > > record > > > this would probably break BfA HoG qualification which requires > > > battery > > > service to be supported as well, Android code does not have any > > > other > > > bas implementation. > > > > That's fine, we can leave this file around. What about the other > > patches? > > Id move the code around bt_gatt_client to bt_bas and let it handle > the > internal details, the long term plan is to get rid of GAttrib so > perhaps we should start with bas. Right. That won't be me though. bas.c is a stub which isn't even capable of getting a value from the device. If somebody wants to split off the battery.c code into a newer bas.c, I'm fine with that, but I'm not porting that code, _then_ implementing its logic when I've already done that in battery.c. Cheers