2023-07-06 13:03:18

by Komal Bajaj

[permalink] [raw]
Subject: [PATCH v3 0/2] arm64: dts: qcom: qdu1000: Update reserved memory region

Update reserved memory region for QDU1000 and QRU1000 SoCs.

Changes in v3:
- Changes the node name as names should reflect the purpose of node.

Changes in v2:
- Changed the node name.
- Dropped unnecessary properties.

Komal Bajaj (2):
arm64: dts: qcom: qdu1000-idp: Update reserved memory region
arm64: dts: qcom: qru1000-idp: Update reserved memory region

arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 22 ++++++++++++++++++++++
arch/arm64/boot/dts/qcom/qru1000-idp.dts | 12 ++++++++++++
2 files changed, 34 insertions(+)

--
2.40.1



2023-07-06 13:10:18

by Komal Bajaj

[permalink] [raw]
Subject: [PATCH v3 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region

Add missing reserved regions as described in QDU1000 memory map.

Signed-off-by: Komal Bajaj <[email protected]>
---
arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
index 1d22f87fd238..8446eb438a34 100644
--- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
+++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
@@ -448,6 +448,28 @@ &qupv3_id_0 {
status = "okay";
};

+&reserved_memory {
+ ecc_meta_data_mem: ecc-meta-data@e0000000 {
+ reg = <0x0 0xe0000000 0x0 0x20000000>;
+ no-map;
+ };
+
+ harq_buffer_mem: harq-buffer@800000000 {
+ reg = <0x8 0x0 0x0 0x80000000>;
+ no-map;
+ };
+
+ tenx_sp_buffer_mem: tenx-sp-buffer@880000000 {
+ reg = <0x8 0x80000000 0x0 0x50000000>;
+ no-map;
+ };
+
+ fapi_buffer_mem: fapi-buffer@8d0000000 {
+ reg = <0x8 0xd0000000 0x0 0x20000000>;
+ no-map;
+ };
+};
+
&sdhc {
pinctrl-0 = <&sdc_on_state>;
pinctrl-1 = <&sdc_off_state>;
--
2.40.1


2023-07-06 13:11:26

by Komal Bajaj

[permalink] [raw]
Subject: [PATCH v3 2/2] arm64: dts: qcom: qru1000-idp: Update reserved memory region

Add missing reserved regions as described in QRU1000 memory map.

Signed-off-by: Komal Bajaj <[email protected]>
---
arch/arm64/boot/dts/qcom/qru1000-idp.dts | 12 ++++++++++++
1 file changed, 12 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/qru1000-idp.dts b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
index 2cc893ae4d10..377a4b414ad5 100644
--- a/arch/arm64/boot/dts/qcom/qru1000-idp.dts
+++ b/arch/arm64/boot/dts/qcom/qru1000-idp.dts
@@ -448,6 +448,18 @@ &qupv3_id_0 {
status = "okay";
};

+&reserved_memory {
+ ecc_meta_data_mem: ecc-meta-data@f0000000 {
+ reg = <0x0 0xf0000000 0x0 0x10000000>;
+ no-map;
+ };
+
+ tenx_sp_mem: tenx-sp-buffer@800000000 {
+ reg = <0x8 0x0 0x0 0x80000000>;
+ no-map;
+ };
+};
+
&uart7 {
status = "okay";
};
--
2.40.1


2023-07-15 14:15:12

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH v3 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region

On 6.07.2023 14:51, Komal Bajaj wrote:
> Add missing reserved regions as described in QDU1000 memory map.
>
> Signed-off-by: Komal Bajaj <[email protected]>
> ---
Are you sure you want to reserve 4352 MiB?

Konrad
> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 22 ++++++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
> index 1d22f87fd238..8446eb438a34 100644
> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
> @@ -448,6 +448,28 @@ &qupv3_id_0 {
> status = "okay";
> };
>
> +&reserved_memory {
> + ecc_meta_data_mem: ecc-meta-data@e0000000 {
> + reg = <0x0 0xe0000000 0x0 0x20000000>;
> + no-map;
> + };
> +
> + harq_buffer_mem: harq-buffer@800000000 {
> + reg = <0x8 0x0 0x0 0x80000000>;
> + no-map;
> + };
> +
> + tenx_sp_buffer_mem: tenx-sp-buffer@880000000 {
> + reg = <0x8 0x80000000 0x0 0x50000000>;
> + no-map;
> + };
> +
> + fapi_buffer_mem: fapi-buffer@8d0000000 {
> + reg = <0x8 0xd0000000 0x0 0x20000000>;
> + no-map;
> + };
> +};
> +
> &sdhc {
> pinctrl-0 = <&sdc_on_state>;
> pinctrl-1 = <&sdc_off_state>;
> --
> 2.40.1
>

2023-07-17 13:12:40

by Komal Bajaj

[permalink] [raw]
Subject: Re: [PATCH v3 1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region



On 7/15/2023 7:43 PM, Konrad Dybcio wrote:
> On 6.07.2023 14:51, Komal Bajaj wrote:
>> Add missing reserved regions as described in QDU1000 memory map.
>>
>> Signed-off-by: Komal Bajaj <[email protected]>
>> ---
> Are you sure you want to reserve 4352 MiB?

Yes, we want to reserve 4352 MiB of memory here, since this SoC enables
Open RAN solution, it requires such huge
memory for its modem usecase.

Thanks
Komal

>
> Konrad
>> arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 22 ++++++++++++++++++++++
>> 1 file changed, 22 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
>> index 1d22f87fd238..8446eb438a34 100644
>> --- a/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
>> +++ b/arch/arm64/boot/dts/qcom/qdu1000-idp.dts
>> @@ -448,6 +448,28 @@ &qupv3_id_0 {
>> status = "okay";
>> };
>>
>> +&reserved_memory {
>> + ecc_meta_data_mem: ecc-meta-data@e0000000 {
>> + reg = <0x0 0xe0000000 0x0 0x20000000>;
>> + no-map;
>> + };
>> +
>> + harq_buffer_mem: harq-buffer@800000000 {
>> + reg = <0x8 0x0 0x0 0x80000000>;
>> + no-map;
>> + };
>> +
>> + tenx_sp_buffer_mem: tenx-sp-buffer@880000000 {
>> + reg = <0x8 0x80000000 0x0 0x50000000>;
>> + no-map;
>> + };
>> +
>> + fapi_buffer_mem: fapi-buffer@8d0000000 {
>> + reg = <0x8 0xd0000000 0x0 0x20000000>;
>> + no-map;
>> + };
>> +};
>> +
>> &sdhc {
>> pinctrl-0 = <&sdc_on_state>;
>> pinctrl-1 = <&sdc_off_state>;
>> --
>> 2.40.1
>>


2023-07-22 05:36:10

by Bjorn Andersson

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] arm64: dts: qcom: qdu1000: Update reserved memory region


On Thu, 06 Jul 2023 18:21:27 +0530, Komal Bajaj wrote:
> Update reserved memory region for QDU1000 and QRU1000 SoCs.
>
> Changes in v3:
> - Changes the node name as names should reflect the purpose of node.
>
> Changes in v2:
> - Changed the node name.
> - Dropped unnecessary properties.
>
> [...]

Applied, thanks!

[1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region
commit: d52775a696bca996271fee0d49f138c69ceff93d
[2/2] arm64: dts: qcom: qru1000-idp: Update reserved memory region
commit: 3d5a9f6d5350a3a4b7f932939d9eb94a15988e0e

Best regards,
--
Bjorn Andersson <[email protected]>