From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
To: Sonny Sasaka <sonnysasaka@chromium.org>
Cc: "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>
Subject: Re: [PATCH v2] client: Add battery command
Date: Wed, 1 Jul 2020 17:42:41 -0700 [thread overview]
Message-ID: <CABBYNZ+EY1VGQu0w3hvB1DfVFpxmPL6FP=JfCxmXckwp-Q+iKw@mail.gmail.com> (raw)
In-Reply-To: <CAOxioNkiPA01Ztw6ugCyF8H-96raKW9o8am1D+qV8veWvkrakg@mail.gmail.com>
Hi Sonny,
On Wed, Jul 1, 2020 at 12:23 PM Sonny Sasaka <sonnysasaka@chromium.org> wrote:
>
> Hi Luiz,
>
> I thought of doing that (combining with info command). But we foresee
> that the Battery API will get more complex (such as support of
> multiple batteries like Pixel Buds and Airpods) and deserve its own
> command. Furthermore, it's a different API (org.bluez.Device1 vs
> org.bluez.Battery1) and its inner working uses a specific profile (BAS
> over GATT), so I think that it deserves its own command. But anyway I
> sent another patch that combines this into the "info" command and if
> you think this is more appropriate we can combine it for now and
> separate it in the future when needed.
It should be a problem to combine different interfaces, and if the
Battery1 start to have more properties we just list them as well, our
D-Bus client will have them cached anything since we do get the
properties of all interface due to use of ObjectManager.
>
> On Wed, Jul 1, 2020 at 10:56 AM Luiz Augusto von Dentz
> <luiz.dentz@gmail.com> wrote:
> >
> > Hi Sonny,
> >
> > On Tue, Jun 30, 2020 at 1:48 PM Sonny Sasaka <sonnysasaka@chromium.org> wrote:
> > >
> > > This adds the "battery" command to show battery information of a peer
> > > device based on org.bluez.Battery1 API. Example usage:
> > >
> > > [bluetooth]# battery XX:XX:XX:XX:XX:XX
> > > Percentage: 100%
> >
> > It might be better to put the battery level under info command.
> >
> > > ---
> > > client/main.c | 62 +++++++++++++++++++++++++++++++++++++++++++++++++++
> > > 1 file changed, 62 insertions(+)
> > >
> > > diff --git a/client/main.c b/client/main.c
> > > index 422da5593..8c1ed00fb 100644
> > > --- a/client/main.c
> > > +++ b/client/main.c
> > > @@ -65,6 +65,7 @@ static struct adapter *default_ctrl;
> > > static GDBusProxy *default_dev;
> > > static GDBusProxy *default_attr;
> > > static GList *ctrl_list;
> > > +static GList *battery_proxies;
> > >
> > > static const char *agent_arguments[] = {
> > > "on",
> > > @@ -107,7 +108,9 @@ static void disconnect_handler(DBusConnection *connection, void *user_data)
> > > bt_shell_set_prompt(PROMPT_OFF);
> > >
> > > g_list_free_full(ctrl_list, proxy_leak);
> > > + g_list_free_full(battery_proxies, proxy_leak);
> > > ctrl_list = NULL;
> > > + battery_proxies = NULL;
> > >
> > > default_ctrl = NULL;
> > > }
> > > @@ -445,6 +448,16 @@ done:
> > > g_free(desc);
> > > }
> > >
> > > +static void battery_added(GDBusProxy *proxy)
> > > +{
> > > + battery_proxies = g_list_append(battery_proxies, proxy);
> > > +}
> > > +
> > > +static void battery_removed(GDBusProxy *proxy)
> > > +{
> > > + battery_proxies = g_list_remove(battery_proxies, proxy);
> > > +}
> > > +
> > > static void device_added(GDBusProxy *proxy)
> > > {
> > > DBusMessageIter iter;
> > > @@ -539,6 +552,8 @@ static void proxy_added(GDBusProxy *proxy, void *user_data)
> > > gatt_add_manager(proxy);
> > > } else if (!strcmp(interface, "org.bluez.LEAdvertisingManager1")) {
> > > ad_manager_added(proxy);
> > > + } else if (!strcmp(interface, "org.bluez.Battery1")) {
> > > + battery_added(proxy);
> > > }
> > > }
> > >
> > > @@ -630,6 +645,8 @@ static void proxy_removed(GDBusProxy *proxy, void *user_data)
> > > gatt_remove_manager(proxy);
> > > } else if (!strcmp(interface, "org.bluez.LEAdvertisingManager1")) {
> > > ad_unregister(dbus_conn, NULL);
> > > + } else if (!strcmp(interface, "org.bluez.Battery1")) {
> > > + battery_removed(proxy);
> > > }
> > > }
> > >
> > > @@ -763,6 +780,20 @@ static struct adapter *find_ctrl_by_address(GList *source, const char *address)
> > > return NULL;
> > > }
> > >
> > > +static GDBusProxy *find_battery_by_path(GList *source, const char *path)
> > > +{
> > > + GList *list;
> > > +
> > > + for (list = g_list_first(source); list; list = g_list_next(list)) {
> > > + GDBusProxy *proxy = list->data;
> > > +
> > > + if (strcmp(g_dbus_proxy_get_path(proxy), path) == 0)
> > > + return proxy;
> > > + }
> > > +
> > > + return NULL;
> > > +}
> > > +
> > > static GDBusProxy *find_proxy_by_address(GList *source, const char *address)
> > > {
> > > GList *list;
> > > @@ -1650,6 +1681,35 @@ static void cmd_info(int argc, char *argv[])
> > > return bt_shell_noninteractive_quit(EXIT_SUCCESS);
> > > }
> > >
> > > +static void cmd_battery(int argc, char *argv[])
> > > +{
> > > + DBusMessageIter iter;
> > > + GDBusProxy *device_proxy;
> > > + GDBusProxy *battery_proxy;
> > > + unsigned char percentage;
> > > +
> > > + device_proxy = find_device(argc, argv);
> > > + if (!device_proxy)
> > > + return bt_shell_noninteractive_quit(EXIT_FAILURE);
> > > +
> > > + battery_proxy = find_battery_by_path(battery_proxies,
> > > + g_dbus_proxy_get_path(device_proxy));
> > > + if (!battery_proxy) {
> > > + bt_shell_printf("Device %s does not have battery information\n",
> > > + argv[1]);
> > > + return bt_shell_noninteractive_quit(EXIT_FAILURE);
> > > + }
> > > +
> > > + if (g_dbus_proxy_get_property(battery_proxy, "Percentage", &iter) ==
> > > + FALSE)
> > > + return bt_shell_noninteractive_quit(EXIT_FAILURE);
> > > +
> > > + dbus_message_iter_get_basic(&iter, &percentage);
> > > + bt_shell_printf("Percentage: %d%%\n", percentage);
> > > +
> > > + return bt_shell_noninteractive_quit(EXIT_SUCCESS);
> > > +}
> > > +
> > > static void pair_reply(DBusMessage *message, void *user_data)
> > > {
> > > DBusError error;
> > > @@ -2785,6 +2845,8 @@ static const struct bt_shell_menu main_menu = {
> > > dev_generator },
> > > { "disconnect", "[dev]", cmd_disconn, "Disconnect device",
> > > dev_generator },
> > > + { "battery", "[dev]", cmd_battery, "Show device battery",
> > > + dev_generator },
> > > { } },
> > > };
> > >
> > > --
> > > 2.26.2
> > >
> >
> >
> > --
> > Luiz Augusto von Dentz
--
Luiz Augusto von Dentz
prev parent reply other threads:[~2020-07-02 0:42 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-30 18:46 [PATCH v2] client: Add battery command Sonny Sasaka
2020-07-01 17:56 ` Luiz Augusto von Dentz
2020-07-01 19:23 ` Sonny Sasaka
2020-07-02 0:42 ` Luiz Augusto von Dentz [this message]
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='CABBYNZ+EY1VGQu0w3hvB1DfVFpxmPL6FP=JfCxmXckwp-Q+iKw@mail.gmail.com' \
--to=luiz.dentz@gmail.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=sonnysasaka@chromium.org \
/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;
as well as URLs for NNTP newsgroup(s).