From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1g4m9M-0002tW-5X for ath10k@lists.infradead.org; Tue, 25 Sep 2018 12:09:11 +0000 MIME-Version: 1.0 Date: Tue, 25 Sep 2018 17:38:35 +0530 From: Govind Singh Subject: Re: [PATCH 1/4] dt: bindings: add missing dt properties for WCN3990 wifi node In-Reply-To: <5baa1b0a.1c69fb81.c0b19.4038@mx.google.com> References: <20180921090907.25937-1-govinds@codeaurora.org> <20180921090907.25937-2-govinds@codeaurora.org> <5baa1b0a.1c69fb81.c0b19.4038@mx.google.com> Message-ID: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Rob Herring Cc: andy.gross@linaro.org, devicetree@vger.kernel.org, briannorris@chromium.org, linux-wireless@vger.kernel.org, ath10k@lists.infradead.org Hi Rob, On 2018-09-25 04:21, Rob Herring wrote: > On Fri, Sep 21, 2018 at 02:39:04PM +0530, Govind Singh wrote: >> Add missing optional properties in WCN3990 wifi node. >> >> Signed-off-by: Govind Singh >> --- >> .../bindings/net/wireless/qcom,ath10k.txt | 22 >> +++++++++++++------ >> 1 file changed, 15 insertions(+), 7 deletions(-) >> >> diff --git >> a/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt >> b/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt >> index 7fd4e8ce4149..22dea7d83ee8 100644 >> --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt >> +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt >> @@ -37,12 +37,14 @@ Optional properties: >> - clocks: List of clock specifiers, must contain an entry for each >> required >> entry in clock-names. >> - clock-names: Should contain the clock names "wifi_wcss_cmd", >> "wifi_wcss_ref", >> - "wifi_wcss_rtc". >> -- interrupts: List of interrupt lines. Must contain an entry >> - for each entry in the interrupt-names property. >> + "wifi_wcss_rtc", "cxo_ref_clk_pin", >> "smmu_aggre2_noc_clk". >> +- interrupts: List of interrupt lines. Must contain interrupt-names >> property >> + per entry for "qcom,ath10k", "qcom,ipq4019-wifi" compatible >> + targets. > > It would be clearer to list how many interrupts for each set of > compatibles. > Sure, I will address this in next revision. >> - interrupt-names: Must include the entries for MSI interrupt >> names ("msi0" to "msi15") and legacy interrupt >> - name ("legacy"), >> + name ("legacy") for "qcom,ath10k", "qcom,ipq4019-wifi" >> + compatible targets. > > So no legacy interrupt for wcn3990? > Yes, no legacy interrupt for wcn3990. >> - qcom,msi_addr: MSI interrupt address. >> - qcom,msi_base: Base value to add before writing MSI data into >> MSI address register. >> @@ -55,7 +57,8 @@ Optional properties: >> - qcom,ath10k-pre-calibration-data : pre calibration data as an >> array, >> the length can vary between hw versions. >> - -supply: handle to the regulator device tree node >> - optional "supply-name" is "vdd-0.8-cx-mx". >> + optional "supply-name" are "vdd-0.8-cx-mx", >> + "vdd-1.8-xo", "vdd-1.3-rfa" and "vdd-3.3-ch0". >> >> Example (to supply the calibration data alone): >> >> @@ -133,8 +136,10 @@ wifi@18000000 { >> compatible = "qcom,wcn3990-wifi"; >> reg = <0x18800000 0x800000>; >> reg-names = "membase"; >> - clocks = <&clock_gcc clk_aggre2_noc_clk>; >> - clock-names = "smmu_aggre2_noc_clk" >> + clocks = <&clock_gcc clk_aggre2_noc_clk>, >> + <&clock_gcc clk_rf_clk2_pin>; >> + clock-names = "smmu_aggre2_noc_clk", >> + "cxo_ref_clk_pin"; > > The binding now says there should be 5 clocks... > Only 2 clocks are applicable for wcn3990. I will list no of clocks for each set of compatible for more clarity. >> interrupts = >> <0 130 0 /* CE0 */ >, >> <0 131 0 /* CE1 */ >, >> @@ -149,4 +154,7 @@ wifi@18000000 { >> <0 140 0 /* CE10 */ >, >> <0 141 0 /* CE11 */ >; >> vdd-0.8-cx-mx-supply = <&pm8998_l5>; >> + vdd-1.8-xo-supply = <&vreg_l7a_1p8>; >> + vdd-1.3-rfa-supply = <&vreg_l17a_1p3>; >> + vdd-3.3-ch0-supply = <&vreg_l25a_3p3>; >> }; >> -- >> 2.17.0 >> Thanks, Govind _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k