[PATCH v3 01/10] dt-bindings: net: bluetooth: qualcomm: Fix WCN6855 regulator names

Konrad Dybcio posted 10 patches 1 month, 1 week ago
[PATCH v3 01/10] dt-bindings: net: bluetooth: qualcomm: Fix WCN6855 regulator names
Posted by Konrad Dybcio 1 month, 1 week ago
From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>

Commit 5f4f954bba12 ("dt-bindings: bluetooth: bring the HW description
closer to reality for wcn6855") changed the vddrfa1p7-supply to 1p8
for whatever reason.

The schematics footprint for this chip definitely says 7 on the input
leg and the driver still expects 1p7. Bring it back.

Fixes: 5f4f954bba12 ("dt-bindings: bluetooth: bring the HW description closer to reality for wcn6855")
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Reviewed-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
Reviewed-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
Acked-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
---
 .../devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml         | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml b/Documentation/devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml
index 45630067d3c8..0beda26ae8bb 100644
--- a/Documentation/devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml
+++ b/Documentation/devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml
@@ -50,9 +50,6 @@ properties:
     description: VDD_RFA_1P7 supply regulator handle
     deprecated: true
 
-  vddrfa1p8-supply:
-    description: VDD_RFA_1P8 supply regulator handle
-
   vddrfacmn-supply:
     description: VDD_RFA_CMN supply regulator handle
 
@@ -68,7 +65,7 @@ required:
   - vddbtcmx-supply
   - vddrfa0p8-supply
   - vddrfa1p2-supply
-  - vddrfa1p8-supply
+  - vddrfa1p7-supply
   - vddrfacmn-supply
   - vddwlcx-supply
   - vddwlmx-supply
@@ -91,7 +88,7 @@ examples:
             vddbtcmx-supply = <&vreg_pmu_btcmx_0p8>;
             vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
             vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
-            vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
+            vddrfa1p7-supply = <&vreg_pmu_rfa_1p7>;
             vddrfacmn-supply = <&vreg_pmu_rfa_cmn_0p8>;
             vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
             vddwlmx-supply = <&vreg_pmu_wlmx_0p8>;

-- 
2.53.0
Re: [PATCH v3 01/10] dt-bindings: net: bluetooth: qualcomm: Fix WCN6855 regulator names
Posted by Bjorn Andersson 2 weeks ago
On Wed, Feb 25, 2026 at 01:23:21PM +0100, Konrad Dybcio wrote:
> From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
> 
> Commit 5f4f954bba12 ("dt-bindings: bluetooth: bring the HW description
> closer to reality for wcn6855") changed the vddrfa1p7-supply to 1p8
> for whatever reason.
> 
> The schematics footprint for this chip definitely says 7 on the input
> leg and the driver still expects 1p7. Bring it back.
> 

@Marcel, can you merge this binding, or would you prefer that I do it
together with the dts changes?

Regards,
Bjorn

> Fixes: 5f4f954bba12 ("dt-bindings: bluetooth: bring the HW description closer to reality for wcn6855")
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> Reviewed-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> Reviewed-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
> Acked-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
> ---
>  .../devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml         | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml b/Documentation/devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml
> index 45630067d3c8..0beda26ae8bb 100644
> --- a/Documentation/devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml
> +++ b/Documentation/devicetree/bindings/net/bluetooth/qcom,wcn6855-bt.yaml
> @@ -50,9 +50,6 @@ properties:
>      description: VDD_RFA_1P7 supply regulator handle
>      deprecated: true
>  
> -  vddrfa1p8-supply:
> -    description: VDD_RFA_1P8 supply regulator handle
> -
>    vddrfacmn-supply:
>      description: VDD_RFA_CMN supply regulator handle
>  
> @@ -68,7 +65,7 @@ required:
>    - vddbtcmx-supply
>    - vddrfa0p8-supply
>    - vddrfa1p2-supply
> -  - vddrfa1p8-supply
> +  - vddrfa1p7-supply
>    - vddrfacmn-supply
>    - vddwlcx-supply
>    - vddwlmx-supply
> @@ -91,7 +88,7 @@ examples:
>              vddbtcmx-supply = <&vreg_pmu_btcmx_0p8>;
>              vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
>              vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
> -            vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
> +            vddrfa1p7-supply = <&vreg_pmu_rfa_1p7>;
>              vddrfacmn-supply = <&vreg_pmu_rfa_cmn_0p8>;
>              vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
>              vddwlmx-supply = <&vreg_pmu_wlmx_0p8>;
> 
> -- 
> 2.53.0
>