Hi Lizardo, On Thu, Apr 11, 2013, Anderson Lizardo wrote: > On Thu, Apr 11, 2013 at 11:24 AM, Jefferson Delfes > <jefferson.delfes@xxxxxxxxxxxxx> wrote: > > diff --git a/src/shared/hciemu.c b/src/shared/hciemu.c > > index 60a4f47..8cd6548 100644 > > --- a/src/shared/hciemu.c > > +++ b/src/shared/hciemu.c > > @@ -192,20 +192,12 @@ static guint create_source_btdev(int fd, struct btdev *btdev) > > static bool create_vhci(struct hciemu *hciemu) > > { > > struct btdev *btdev; > > - uint8_t bdaddr[6]; > > - const char *str; > > - int fd, i; > > + int fd; > > > > btdev = btdev_create(hciemu->btdev_type, 0x00); > > if (!btdev) > > return false; > > > > - str = hciemu_get_address(hciemu); > > - > > - for (i = 5; i >= 0; i--, str += 3) > > - bdaddr[i] = strtol(str, NULL, 16); > > - > > - btdev_set_bdaddr(btdev, bdaddr); > > Looks like you can remove btdev_set_bdaddr() (in a patch after this > one) because it was the only user of this function. Yep. I already went ahead and pushed such a patch upstream. Johan -- 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