Re: [PATCH] arm64: dts: qcom: ipq5424: Add reserved memory for TF-A

[Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>]

On 6/25/2025 10:20 AM, Kathiravan Thirumoorthy wrote:
>
> On 6/24/2025 6:57 PM, Konrad Dybcio wrote:
>> On 6/24/25 12:36 PM, Kathiravan Thirumoorthy wrote:
>>> From: Vignesh Viswanathan <quic_viswanat@quicinc.com>
>>>
>>> IPQ5424 supports both TZ and TF-A as secure software options and 
>>> various
>>> DDR sizes. In most cases, TF-A or TZ is loaded at the same memory
>>> location, but in the 256MB DDR configuration TF-A is loaded at a 
>>> different
>>> region.
>>>
>>> So, add the reserved memory node for TF-A and keep it disabled by 
>>> default.
>>> During bootup, U-Boot will detect which secure software is running and
>>> enable or disable the node accordingly.
>>>
>>> Signed-off-by: Vignesh Viswanathan <quic_viswanat@quicinc.com>
>>> Signed-off-by: Kathiravan Thirumoorthy 
>>> <kathiravan.thirumoorthy@oss.qualcomm.com>
>>> ---
>> Can the said u-boot simply dynamically add the reservation then?
>
> Yeah it could have been done in U-Boot itself but it wasn't. 256MB DDR 
> configuration solution is already shipped out and the stock U-Boot 
> enable this node to avoid the random issues.


Konrad, Do you have any further comments on this?


>
>>
>> Konrad