Subject: Re: [PATCH v2 3/3] dt-bindings: net: bluetooth: Add device tree bindings for QTI chip wcn6750

Hi Rob Herring,
On 2021-04-21 22:42, Rob Herring wrote:
> On Tue, Apr 20, 2021 at 09:56:50PM +0530, Venkata Lakshmi Narayana
> Gubba wrote:
>> This patch enables regulators and gpios for the Qualcomm Bluetooth
>> wcn6750
>> controller.
>>
>> Signed-off-by: Venkata Lakshmi Narayana Gubba
>> <[email protected]>
>> ---
>> .../bindings/net/qualcomm-bluetooth.yaml | 54
>> +++++++++++++++++++++-
>> 1 file changed, 53 insertions(+), 1 deletion(-)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/net/qualcomm-bluetooth.yaml
>> b/Documentation/devicetree/bindings/net/qualcomm-bluetooth.yaml
>> index 55cd995..1ceb02b 100644
>> --- a/Documentation/devicetree/bindings/net/qualcomm-bluetooth.yaml
>> +++ b/Documentation/devicetree/bindings/net/qualcomm-bluetooth.yaml
>> @@ -21,11 +21,17 @@ properties:
>> - qcom,wcn3990-bt
>> - qcom,wcn3991-bt
>> - qcom,wcn3998-bt
>> - - qcom,qca6390-bt
>> + - qcom,qca6390-bt
>> + - qcom,wcn6750-bt
>>
>> enable-gpios:
>> maxItems: 1
>> description: gpio specifier used to enable chip
>> +
>> + swctrl-gpios:
>> + maxItems: 1
>> + description: gpio specifier is used to find status
>> + of clock supply to SoC
>>
>> clocks:
>> maxItems: 1
>> @@ -43,6 +49,30 @@ properties:
>> vddch0-supply:
>> description: VDD_CH0 supply regulator handle
>>
>> + vddaon-supply:
>> + description: VDD_AON supply regulator handle
>> +
>> + vddbtcxmx-supply:
>> + description: VDD_BT_CXMX supply regualtor handle
>
> typo
>
[Venkata]:
I will update in next patchset.
>> +
>> + vddrfacmn-supply:
>> + description: VDD_RFA_CMN supply regulator handle
>> +
>> + vddrfa0p8-supply:
>> + description: VDD_RFA_0P8 suppply regulator handle
>> +
>> + vddrfa1p7-supply:
>> + description: VDD_RFA_1P7 supply regulator handle
>> +
>> + vddrfa1p2-supply:
>> + description: VDD_RFA_1P2 supply regulator handle
>> +
>> + vddrfa2p2-supply:
>> + description: VDD_RFA_2P2 supply regulator handle
>> +
>> + vddasd-supply:
>> + description: VDD_ASD supply regulator handle
>> +
>> max-speed:
>> description: see
>> Documentation/devicetree/bindings/serial/serial.yaml
>>
>> @@ -85,3 +115,25 @@ examples:
>> firmware-name = "crnv21.bin";
>> };
>> };
>> + - |
>> + uart {
>
> serial {
>
[Venkata]:
Sure, I will update in next patchset.
>> +
>> + bluetooth {
>> + compatible = "qcom,wcn6750-bt";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&bt_en_default>;
>> + enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
>> + swctrl-gpios = <&tlmm 86 GPIO_ACTIVE_HIGH>;
>> + vddio-supply = <&vreg_l19b_1p8>;
>> + vddaon-supply = <&vreg_s7b_0p9>;
>> + vddbtcxmx-supply = <&vreg_s7b_0p9>;
>> + vddrfacmn-supply = <&vreg_s7b_0p9>;
>> + vddrfa0p8-supply = <&vreg_s7b_0p9>;
>> + vddrfa1p7-supply = <&vreg_s1b_1p8>;
>> + vddrfa1p2-supply = <&vreg_s8b_1p2>;
>> + vddrfa2p2-supply = <&vreg_s1c_2p2>;
>> + vddasd-supply = <&vreg_l11c_2p8>;
>> + max-speed = <3200000>;
>> + firmware-name = "msnv11.bin";
>> + };
>> + };
>> --
>> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
>> member
>> of Code Aurora Forum, hosted by The Linux Foundation
>>

Regards,
Venkata.