| Message ID | 20180522195758.24607-1-niklas.cassel@linaro.org |
|---|---|
| State | Accepted |
| Commit | da34314f72c6a2ce53753fb63012080164d0f953 |
| Headers | show |
| Series | arm64: dts: fix regulator property name for wlan pcie endpoint | expand |
On Tue 22 May 12:57 PDT 2018, Niklas Cassel wrote: > The property name vddpe-supply is not included in > Documentation/devicetree/bindings/pci/qcom,pcie.txt > nor in the pcie-qcom PCIe Root Complex driver. > > This property name was used in an initial patchset for pcie-qcom, > but was renamed in a later revision. > > Therefore, the regulator is currently never enabled, leaving us with > unoperational wlan. > > Fix this by using the correct regulator property name, so that wlan > comes up correctly. > > Fixes: 1c8ca74a2ea1 ("arm64: dts: apq8096-db820c: Enable wlan and bt en pins") > Signed-off-by: Niklas Cassel <niklas.cassel@linaro.org> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Regards, Bjorn > --- > Bluetooth needs a similar patch, but since bluetooth needs some more > work, submit this right now, so we at least have working wlan. > > arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi > index 818bf0efd501..804268f54f37 100644 > --- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi > +++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi > @@ -203,7 +203,7 @@ > pcie@600000 { > status = "okay"; > perst-gpio = <&msmgpio 35 GPIO_ACTIVE_LOW>; > - vddpe-supply = <&wlan_en>; > + vddpe-3v3-supply = <&wlan_en>; > vddpe1-supply = <&bt_en>; > }; > > -- > 2.17.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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
diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi index 818bf0efd501..804268f54f37 100644 --- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi +++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi @@ -203,7 +203,7 @@ pcie@600000 { status = "okay"; perst-gpio = <&msmgpio 35 GPIO_ACTIVE_LOW>; - vddpe-supply = <&wlan_en>; + vddpe-3v3-supply = <&wlan_en>; vddpe1-supply = <&bt_en>; };
The property name vddpe-supply is not included in Documentation/devicetree/bindings/pci/qcom,pcie.txt nor in the pcie-qcom PCIe Root Complex driver. This property name was used in an initial patchset for pcie-qcom, but was renamed in a later revision. Therefore, the regulator is currently never enabled, leaving us with unoperational wlan. Fix this by using the correct regulator property name, so that wlan comes up correctly. Fixes: 1c8ca74a2ea1 ("arm64: dts: apq8096-db820c: Enable wlan and bt en pins") Signed-off-by: Niklas Cassel <niklas.cassel@linaro.org> --- Bluetooth needs a similar patch, but since bluetooth needs some more work, submit this right now, so we at least have working wlan. arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.17.0 -- 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