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

[gubbaven]

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 
> > <gubbaven@xxxxxxxxxxxxxx>
> > ---
> >  .../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.