From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Andersson Subject: Re: [PATCH] arm64: dts: fix regulator property name for wlan pcie endpoint Date: Fri, 25 May 2018 12:32:19 -0700 Message-ID: <20180525193219.GY2259@tuxbook-pro> References: <20180522195758.24607-1-niklas.cassel@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180522195758.24607-1-niklas.cassel@linaro.org> Sender: linux-kernel-owner@vger.kernel.org To: Niklas Cassel Cc: Andy Gross , David Brown , Rob Herring , Mark Rutland , Catalin Marinas , Will Deacon , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org 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 Reviewed-by: Bjorn Andersson 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