mbox series

[v3,0/3] Bluetooth: hci_qca: Add serdev support

Message ID 20180313183807.6701-1-thierry.escande@linaro.org
Headers show
Series Bluetooth: hci_qca: Add serdev support | expand

Message

Thierry Escande March 13, 2018, 6:38 p.m. UTC 
Hi,

This patchset enables the Qualcomm BT controller QCA6174 node in the
device tree of the db820c board. This allows the bluetooth chipset to
be probed and registered against the hci layer by using the serdev
framework.

This patchset also contains the documentation for the compatible
string "qcom,qca6174-bt" related to this chipset.

v3:
- Address comments for patch #3 (details in patch)

v2:
- Fix author email

Thierry Escande (3):
  arm64: dts: apq8096-db820c: enable bluetooth node
  dt-bindings: net: bluetooth: Add qualcomm-bluetooth
  Bluetooth: hci_qca: Add serdev support

 .../devicetree/bindings/net/qualcomm-bluetooth.txt |  34 +++++++
 arch/arm64/boot/dts/qcom/apq8096-db820c-pins.dtsi  |  14 +++
 .../boot/dts/qcom/apq8096-db820c-pmic-pins.dtsi    |  17 ++++
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi       |  32 ++++++
 arch/arm64/boot/dts/qcom/msm8996.dtsi              |  10 ++
 drivers/bluetooth/Kconfig                          |   2 +-
 drivers/bluetooth/hci_qca.c                        | 109 ++++++++++++++++++++-
 7 files changed, 215 insertions(+), 3 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/net/qualcomm-bluetooth.txt

Comments

Andy Shevchenko March 13, 2018, 8:57 p.m. UTC | #1 
On Tue, Mar 13, 2018 at 8:38 PM, Thierry Escande
<thierry.escande@linaro.org> wrote:
> Add support for Qualcomm serial slave devices. Probe the serial device,
> retrieve its maximum speed and register a new hci uart device.

>  config BT_HCIUART_QCA
>         bool "Qualcomm Atheros protocol support"
> -       depends on BT_HCIUART
> +       depends on BT_HCIUART_SERDEV

Consider to address Loic's comments.

> +#include <linux/of.h>

This is still here. Why?
Thierry Escande March 14, 2018, 8:48 a.m. UTC | #2 
On 13/03/2018 21:57, Andy Shevchenko wrote:
> On Tue, Mar 13, 2018 at 8:38 PM, Thierry Escande
> <thierry.escande@linaro.org> wrote:
>> Add support for Qualcomm serial slave devices. Probe the serial device,
>> retrieve its maximum speed and register a new hci uart device.
> 
>>   config BT_HCIUART_QCA
>>          bool "Qualcomm Atheros protocol support"
>> -       depends on BT_HCIUART
>> +       depends on BT_HCIUART_SERDEV
> 
> Consider to address Loic's comments.
> 
>> +#include <linux/of.h>
> 
> This is still here. Why?

I forgot that one. This was for struct of_device_id. I'll replace it 
with mod_devicetable.h.

Regards,
Thierry

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html