Hi David,
On Mon, Aug 09, 2010, David Scherba wrote:
> +static inline int is_bredr_hci_device_type(uint8_t type)
Could you just call this ignore_device() which combines the AMP and RAW
checks.
> +{
> + return (type >> 4) == HCI_BREDR;
> +}
Looks like the indentation is wrong inside the function (two spaces
instead of a tab).
> - if (hci_test_bit(HCI_RAW, &di.flags))
> + if (!is_bredr_hci_device_type(di.type) ||
> + hci_test_bit(HCI_RAW, &di.flags))
No spaces+tabs mixed indentation please. The second line should be
indented by at least two tabs more than the line above it (as many tabs
as you can as long as the line doesn't go beyond 79 characters). OTOH,
the second line goes away completely if you do the ignore_device
simplification as proposed above.
> - if (!hci_test_bit(HCI_RAW, &di.flags))
> + if (is_bredr_hci_device_type(di.type) &&
> + !hci_test_bit(HCI_RAW, &di.flags))
Same here.
> - if (hci_test_bit(HCI_RAW, &di.flags)) {
> + if (!is_bredr_hci_device_type(di.type) ||
> + hci_test_bit(HCI_RAW, &di.flags)) {
And here.
> - if (hci_test_bit(HCI_RAW, &di->flags))
> + if (!is_bredr_hci_device_type(di->type) ||
> + hci_test_bit(HCI_RAW, &di->flags))
And here.
> - if (hci_test_bit(HCI_RAW, &di->flags))
> + if (!is_bredr_hci_device_type(di->type) ||
> + hci_test_bit(HCI_RAW, &di->flags))
And here.
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
