* [PATCH v3 3/6] dt: bindings: add bindings for msa memory region
@ 2018-07-06 9:00 Govind Singh
2018-07-10 23:44 ` Brian Norris
2018-07-11 15:27 ` Rob Herring
0 siblings, 2 replies; 4+ messages in thread
From: Govind Singh @ 2018-07-06 9:00 UTC (permalink / raw)
To: devicetree, bjorn.andersson, david.brown, andy.gross,
niklas.cassel, ath10k
Cc: linux-wireless, Govind Singh
Add device tree binding documentation details of msa
memory region for ath10k qmi client for SDM845/APQ8098
SoC into "qcom,ath10k.txt".
Signed-off-by: Govind Singh <govinds@codeaurora.org>
---
.../bindings/net/wireless/qcom,ath10k.txt | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt b/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
index 7fd4e8ce4149..87489054aea5 100644
--- a/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
+++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
@@ -57,6 +57,16 @@ Optional properties:
- <supply-name>-supply: handle to the regulator device tree node
optional "supply-name" is "vdd-0.8-cx-mx".
+= SUBNODES:
+wcn3990-wifi node may contain one subnode, named "msa" representing
+the fixed msa memory regions used by the wifi firmware running in Q6.
+This sub-node must contain:
+
+- memory-region:
+ Usage: required
+ Value type: <phandle>
+ Definition: reference to the reserved-memory for the region
+
Example (to supply the calibration data alone):
In this example, the node is defined as child node of the PCI controller.
@@ -149,4 +159,7 @@ wifi@18000000 {
<0 140 0 /* CE10 */ >,
<0 141 0 /* CE11 */ >;
vdd-0.8-cx-mx-supply = <&pm8998_l5>;
+ msa {
+ memory-region = <&wifi_msa_mem>;
+ };
};
--
2.17.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v3 3/6] dt: bindings: add bindings for msa memory region
2018-07-06 9:00 [PATCH v3 3/6] dt: bindings: add bindings for msa memory region Govind Singh
@ 2018-07-10 23:44 ` Brian Norris
2018-07-11 15:27 ` Rob Herring
1 sibling, 0 replies; 4+ messages in thread
From: Brian Norris @ 2018-07-10 23:44 UTC (permalink / raw)
To: Govind Singh
Cc: devicetree, bjorn.andersson, david.brown, andy.gross,
niklas.cassel, ath10k, linux-wireless
On Fri, Jul 06, 2018 at 02:30:43PM +0530, Govind Singh wrote:
[snip]
You still don't quite have the threading right on this series. All the
other patches seem OK -- they have this in the header, which links them
to the cover letter:
In-Reply-To: <20180706085622.27945-1-govinds@codeaurora.org>
References: <20180706085622.27945-1-govinds@codeaurora.org>
But you seem to have missed that on this one patch for some reason.
Brian
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3 3/6] dt: bindings: add bindings for msa memory region
2018-07-06 9:00 [PATCH v3 3/6] dt: bindings: add bindings for msa memory region Govind Singh
2018-07-10 23:44 ` Brian Norris
@ 2018-07-11 15:27 ` Rob Herring
2018-07-23 12:36 ` Govind Singh
1 sibling, 1 reply; 4+ messages in thread
From: Rob Herring @ 2018-07-11 15:27 UTC (permalink / raw)
To: Govind Singh
Cc: devicetree, bjorn.andersson, david.brown, andy.gross,
niklas.cassel, ath10k, linux-wireless
On Fri, Jul 06, 2018 at 02:30:43PM +0530, Govind Singh wrote:
> Add device tree binding documentation details of msa
> memory region for ath10k qmi client for SDM845/APQ8098
> SoC into "qcom,ath10k.txt".
>
> Signed-off-by: Govind Singh <govinds@codeaurora.org>
> ---
> .../bindings/net/wireless/qcom,ath10k.txt | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt b/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
> index 7fd4e8ce4149..87489054aea5 100644
> --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
> +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
> @@ -57,6 +57,16 @@ Optional properties:
> - <supply-name>-supply: handle to the regulator device tree node
> optional "supply-name" is "vdd-0.8-cx-mx".
>
> += SUBNODES:
> +wcn3990-wifi node may contain one subnode, named "msa" representing
> +the fixed msa memory regions used by the wifi firmware running in Q6.
> +This sub-node must contain:
Why do you need a sub-node? Just add memory-region property.
> +
> +- memory-region:
> + Usage: required
> + Value type: <phandle>
> + Definition: reference to the reserved-memory for the region
> +
> Example (to supply the calibration data alone):
>
> In this example, the node is defined as child node of the PCI controller.
> @@ -149,4 +159,7 @@ wifi@18000000 {
> <0 140 0 /* CE10 */ >,
> <0 141 0 /* CE11 */ >;
> vdd-0.8-cx-mx-supply = <&pm8998_l5>;
> + msa {
> + memory-region = <&wifi_msa_mem>;
> + };
> };
> --
> 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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3 3/6] dt: bindings: add bindings for msa memory region
2018-07-11 15:27 ` Rob Herring
@ 2018-07-23 12:36 ` Govind Singh
0 siblings, 0 replies; 4+ messages in thread
From: Govind Singh @ 2018-07-23 12:36 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, bjorn.andersson, david.brown, andy.gross,
niklas.cassel, ath10k, linux-wireless
Hi Rob,
On 2018-07-11 20:57, Rob Herring wrote:
> On Fri, Jul 06, 2018 at 02:30:43PM +0530, Govind Singh wrote:
>> Add device tree binding documentation details of msa
>> memory region for ath10k qmi client for SDM845/APQ8098
>> SoC into "qcom,ath10k.txt".
>>
>> Signed-off-by: Govind Singh <govinds@codeaurora.org>
>> ---
>> .../bindings/net/wireless/qcom,ath10k.txt | 13
>> +++++++++++++
>> 1 file changed, 13 insertions(+)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
>> b/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
>> index 7fd4e8ce4149..87489054aea5 100644
>> --- a/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
>> +++ b/Documentation/devicetree/bindings/net/wireless/qcom,ath10k.txt
>> @@ -57,6 +57,16 @@ Optional properties:
>> - <supply-name>-supply: handle to the regulator device tree node
>> optional "supply-name" is "vdd-0.8-cx-mx".
>>
>> += SUBNODES:
>> +wcn3990-wifi node may contain one subnode, named "msa" representing
>> +the fixed msa memory regions used by the wifi firmware running in Q6.
>> +This sub-node must contain:
>
> Why do you need a sub-node? Just add memory-region property.
>
Fixed in v4 version.
Thanks,
Govind
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-07-23 13:38 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-06 9:00 [PATCH v3 3/6] dt: bindings: add bindings for msa memory region Govind Singh
2018-07-10 23:44 ` Brian Norris
2018-07-11 15:27 ` Rob Herring
2018-07-23 12:36 ` Govind Singh
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).