From: Krzysztof Kozlowski <krzk@kernel.org>
To: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com>, ath12k@lists.infradead.org
Cc: linux-wireless@vger.kernel.org, Kalle Valo <kvalo@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Jeff Johnson <jjohnson@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konradybcio@kernel.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-msm@vger.kernel.org
Subject: Re: [PATCH 2/2] arm64: dts: qcom: add wifi node for IPQ5332 based RDP441
Date: Tue, 28 Jan 2025 10:22:01 +0100 [thread overview]
Message-ID: <b3ff05c1-6424-4ace-a873-ddf1a0d3d07d@kernel.org> (raw)
In-Reply-To: <20250128091012.2574478-3-quic_rajkbhag@quicinc.com>
On 28/01/2025 10:10, Raj Kumar Bhagat wrote:
> + /* Default Profile
> + * +============+==============+=====================+
> + * | | | |
> + * | Region | Start Offset | Size |
> + * | | | |
> + * +------------+--------------+---------------------+
> + * | | | |
> + * | | | |
> + * | | | |
> + * | WLAN Q6 | 0x4A900000 | 43MB |
> + * | | | |
> + * | | | |
> + * +------------+--------------+---------------------+
> + * | M3 Dump | 0x4D400000 | 1MB |
> + * +============+==============+=====================+
> + * | |
> + * | |
> + * | |
> + * | Rest of memory for Linux |
> + * | |
> + * | |
> + * | |
> + * +=================================================+
> + */
> +
> + reserved-memory {
> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> +
> + q6_region: wcss@4a900000 {
> + reg = <0x0 0x4a900000 0x0 0x02b00000>;
> + no-map;
> + };
> +
> + m3_dump: m3-dump@4d400000 {
This fails with your wireless patchset.
> + reg = <0x0 0x4D400000 0x0 0x100000>;
> + no-map;
> + };
> + };
> };
>
> &blsp1_i2c1 {
> @@ -63,3 +104,16 @@ data-pins {
> };
> };
> };
> +
> +&wifi0 {
> + memory-region = <&q6_region>, <&m3_dump>;
> + memory-region-names = "q6-region", "m3-dump";
Binding said you have four items. I don't understand why this varies and
why this is 2 items instead.
Best regards,
Krzysztof
next prev parent reply other threads:[~2025-01-28 9:23 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-28 9:10 [PATCH 0/2] dt-bindings: net: wireless: add wifi node for IPQ5332 Raj Kumar Bhagat
2025-01-28 9:10 ` [PATCH 1/2] dt-bindings: net: wireless: describe the ath12k AHB module Raj Kumar Bhagat
2025-01-28 9:17 ` Krzysztof Kozlowski
2025-01-28 9:29 ` Raj Kumar Bhagat
2025-01-28 10:21 ` Krzysztof Kozlowski
2025-01-28 17:34 ` Raj Kumar Bhagat
2025-01-28 9:10 ` [PATCH 2/2] arm64: dts: qcom: add wifi node for IPQ5332 based RDP441 Raj Kumar Bhagat
2025-01-28 9:22 ` Krzysztof Kozlowski [this message]
2025-01-28 9:41 ` Raj Kumar Bhagat
2025-01-28 10:24 ` Krzysztof Kozlowski
2025-01-28 16:46 ` Raj Kumar Bhagat
2025-01-28 11:32 ` Konrad Dybcio
2025-01-28 17:00 ` Raj Kumar Bhagat
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b3ff05c1-6424-4ace-a873-ddf1a0d3d07d@kernel.org \
--to=krzk@kernel.org \
--cc=andersson@kernel.org \
--cc=ath12k@lists.infradead.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jjohnson@kernel.org \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=kvalo@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=quic_rajkbhag@quicinc.com \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.