Re: [PATCH v6 5/5] arm64: dts: qcom: ipq5424: add support to get watchdog bootstatus from IMEM

From: Mukesh Ojha

Date: Tue Feb 03 2026 - 01:15:05 EST


On Tue, Feb 03, 2026 at 10:05:26AM +0530, Kathiravan Thirumoorthy wrote:
>
> On 2/2/2026 7:14 PM, Mukesh Ojha wrote:
> > On Sat, Jan 31, 2026 at 10:18:29AM +0200, Dmitry Baryshkov wrote:
> > > On Fri, Jan 30, 2026 at 04:14:34PM +0530, Kathiravan Thirumoorthy wrote:
> > > > Add the "sram" property to the watchdog device node to enable
> > > > retrieval of the system restart reason from IMEM, populated by XBL.
> > > > Parse this information in the watchdog driver and update the bootstatus
> > > > sysFS if the restart was triggered by a watchdog timeout.
> > > >
> > > > Signed-off-by: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@xxxxxxxxxxxxxxxx>
> > > > ---
> > > > Changes in v6:
> > > > - Update the 'sram' property to point to the SRAM region
> > > > Changes in v5:
> > > > - Rename the property 'qcom,imem' to 'sram'
> > > > Changes in v4:
> > > > - New patch
> > > > ---
> > > > arch/arm64/boot/dts/qcom/ipq5424.dtsi | 5 +++++
> > > > 1 file changed, 5 insertions(+)
> > > >
> > > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> >
> > I have a few more cookies (stored in a fixed IMEM location supported
> > downstream) that I want to add, and they are available on all Qualcomm
> > mobile SoCs. Should it be added under SMEM now?
>
>
> Mukesh, you mean SMEM or SRAM? Do we have the consumers in upstream for all
> those cookies?

Forgive me,. yeah I meant SRAM.

>
>
> >
> >
> > >
> > > --
> > > With best wishes
> > > Dmitry

--
-Mukesh Ojha