Hi Andre, > Move debug macros so they provide us more meaningful information > during debugging analysis. > > Signed-off-by: Andre Guedes <andre.guedes@xxxxxxxxxxxxx> > --- > net/bluetooth/hci_core.c | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c > index ef0423e..8487483 100644 > --- a/net/bluetooth/hci_core.c > +++ b/net/bluetooth/hci_core.c > @@ -2594,11 +2594,11 @@ int hci_do_inquiry(struct hci_dev *hdev, u8 length) > u8 lap[3] = { 0x33, 0x8b, 0x9e }; > struct hci_cp_inquiry cp; > > - BT_DBG("%s", hdev->name); > - > if (test_bit(HCI_INQUIRY, &hdev->flags)) > return -EINPROGRESS; > > + BT_DBG("%s", hdev->name); > + I actually do not like this change much. Can we just add the flags into the debug print instead. Regards Marcel -- 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