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

[Kathiravan Thirumoorthy]

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@xxxxxxxxxxx>
> > >
> > > 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@xxxxxxxxxxx>
> > > Signed-off-by: Kathiravan Thirumoorthy 
> > > <kathiravan.thirumoorthy@xxxxxxxxxxxxxxxx>
> > > ---
> > 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