Hi Johan, On Wed, Jun 19, 2013 at 5:57 AM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: >> +static void le_set_adv_enable_complete(struct btdev *btdev) >> +{ >> + int i; >> + >> + for (i = 0; i < MAX_BTDEV_ENTRIES; i++) { >> + if (!btdev_list[i] || btdev_list[i] == btdev) >> + continue; >> + >> + if (!btdev_list[i]->le_scan_enable) >> + continue; >> + } >> +} > > Same thing as with my earlier comment. This function doesn't seem to do > anything useful and should probably be merged with some later patch that > actually does something (assuming such a patch exists). Same here. This function is used in a later patch. I will merge it too. BR, -- Bruna Moreira Instituto Nokia de Tecnologia (INdT) Manaus - Brazil -- 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