[v3,1/2] arm64: dts: qcom: qdu1000-idp: Update reserved memory region

Message ID 20230706125129.20969-2-quic_kbajaj@quicinc.com
State New
Headers
Series arm64: dts: qcom: qdu1000: Update reserved memory region |

Commit Message

Komal Bajaj July 6, 2023, 12:51 p.m. UTC
  Add missing reserved regions as described in QDU1000 memory map.

Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com>
---
 arch/arm64/boot/dts/qcom/qdu1000-idp.dts | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

--
2.40.1
  

Comments

Konrad Dybcio July 15, 2023, 2:13 p.m. UTC | #1
On 6.07.2023 14:51, Komal Bajaj wrote:
> Add missing reserved regions as described in QDU1000 memory map.
> 
> Signed-off-by: Komal Bajaj <quic_kbajaj@quicinc.com>
> ---
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
>
  
Komal Bajaj July 17, 2023, 1:02 p.m. UTC | #2
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 <quic_kbajaj@quicinc.com>
>> ---
> 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
>>
  

Patch

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>;