Hi Andrei, On Friday 29 of August 2014 14:24:21 Andrei Emeltchenko wrote: > From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > > "!create_mdl(channel) < 0" is always false since it returns bool > --- > android/health.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/android/health.c b/android/health.c > index abf01b4..fb2a1e3 100644 > --- a/android/health.c > +++ b/android/health.c > @@ -1949,7 +1949,7 @@ static void bt_health_connect_channel(const void *buf, uint16_t len) > goto fail; > > /* create mdl if it does not exists */ > - if (!channel->mdl && !create_mdl(channel) < 0) > + if (!channel->mdl && !create_mdl(channel)) > goto fail; > > /* reconnect mdl if it exists */ Applied. Thanks. -- Best regards, Szymon Janc -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html