Re: [PATCH] arm64: dts: qcom: sdm845-db845c: Mark cont splash memory region as reserved

[Bryan O'Donoghue]

On 10/04/2023 09:54, Amit Pundir wrote:
> On Thu, 9 Feb 2023 at 16:33, Bryan O'Donoghue
> <bryan.odonoghue@xxxxxxxxxx> wrote:
> > On 09/02/2023 09:05, Amit Pundir wrote:
> > > Hi, So what is the verdict on this patch?
> > >
> > > I submitted this fix to make sure UFS don't map and crash on it, which
> > > I have seen happening occassionaly on db845c and Caleb reported
> > > similar issues on his sdm845 device iirc. I should have probably put
> > > that in my commit message as well.
> > >
> > > Regards,
> > > Amit Pundir
> >
> > So the memory _is_ being used by ... continuous splash on an Android
> > image, i.e. your Android ? limited to Android - image continues on with
> > the splash but other blocks erroneously reuse the memory then, UFS as an
> > example ?
>
> Hi Bryan,
>
> Yes UFS (reported only on v5.10) tries to map this reserved memory and
> system crash and reboot. Plan is to backport this patch to v5.10.y.
>
> Regards,
> Amit Pundir
>
> > ---
> > bod

Personally I'm fine with this patch on the proviso we somehow associate 
it the memory MDP - even if its just a comment in the dts with the MDP.

i.e. if we run headless we want to be able to use that RAM for something.

A dts comment would do

---
bod