Hi Jaganath,
> 1M is mandatory to be supported by LE controllers and the same
> would be set in power on. This patch defines hdev flags for
> LE PHYs and set 1M to default.
>
> Signed-off-by: Jaganath Kanakkassery <jaganathx.kanakkassery@xxxxxxxxx>
> ---
> include/net/bluetooth/hci.h | 7 +++++++
> net/bluetooth/hci_core.c | 4 ++++
> 2 files changed, 11 insertions(+)
>
> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
> index 73e48be..1b5d79b 100644
> --- a/include/net/bluetooth/hci.h
> +++ b/include/net/bluetooth/hci.h
> @@ -261,6 +261,13 @@ enum {
> HCI_FORCE_BREDR_SMP,
> HCI_FORCE_STATIC_ADDR,
>
> + HCI_LE_PHY_1M_TX,
> + HCI_LE_PHY_1M_RX,
> + HCI_LE_PHY_2M_TX,
> + HCI_LE_PHY_2M_RX,
> + HCI_LE_PHY_CODED_TX,
> + HCI_LE_PHY_CODED_RX,
> +
> __HCI_NUM_FLAGS,
> };
so first of all, these are not yet defined as of patch 05/18 and I am not sure that using flags is the best choice here. You need to provide some reasoning why you did it this way.
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
