Hi Gustavo, > > From: Johan Hedberg <johan.hedberg@xxxxxxxxx> > > > > This patch fixes sending an unnecessary HCI_LE_Host_Enable command if > > the command has already been sent as part of the default HCI init > > sequence. > > > > Signed-off-by: Johan Hedberg <johan.hedberg@xxxxxxxxx> > > --- > > net/bluetooth/mgmt.c | 10 ++++++++-- > > 1 file changed, 8 insertions(+), 2 deletions(-) > > > > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c > > index 02dc5f8..03dc176 100644 > > --- a/net/bluetooth/mgmt.c > > +++ b/net/bluetooth/mgmt.c > > @@ -2927,8 +2927,14 @@ int mgmt_powered(struct hci_dev *hdev, u8 powered) > > cp.le = 1; > > cp.simul = !!lmp_le_br_capable(hdev); > > > > - hci_send_cmd(hdev, HCI_OP_WRITE_LE_HOST_SUPPORTED, > > - sizeof(cp), &cp); > > + /* Check first if we already have the right > > + * host state (host features set) > > + */ > > + if (cp.le != !!lmp_host_le_capable(hdev) || > > + cp.simul != !!lmp_host_le_br_capable(hdev)) > > Shouldn't the !! be part of the macro itself? Looks we will be using !! always > with these macros. we could do that actually. However I prefer to do that with a separate patch. 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
