Hi Johan, On Tue, Apr 30, 2013 at 5:36 AM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > Hi Andre, > > On Tue, Apr 30, 2013, Andre Guedes wrote: >> Since all mgmt stop discovery command complete events are now handled >> in stop_discovery_complete callback in mgmt.c, we can remove this >> handling from hci_event.c. >> >> Signed-off-by: Andre Guedes <andre.guedes@xxxxxxxxxxxxx> >> --- >> net/bluetooth/hci_event.c | 16 ++-------------- >> 1 file changed, 2 insertions(+), 14 deletions(-) > > Acked-by: Johan Hedberg <johan.hedberg@xxxxxxxxx> > > However, looks like you're missing a patch to remove > mgmt_stop_discovery_failed() from hci_core.h (since it seems it's now > only needed inside mgmt.c). Yes, you're right. I'll make mgmt_stop_disovery_failed static and remove it from hci_core.h. I'm going to do all these fixes and send a v3 patch set. Thanks, Andre -- 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