RE: [PATCH 2/2] xilinx: firmware: Add sysfs to get last reset reason

From: Jain, Ronak
Date: Wed Nov 29 2023 - 06:53:27 EST


Hi Arnd,

> -----Original Message-----
> From: Arnd Bergmann <arnd@xxxxxxxx>
> Sent: Tuesday, November 21, 2023 6:01 PM
> To: Jain, Ronak <ronak.jain@xxxxxxx>; Simek, Michal
> <michal.simek@xxxxxxx>; Greg Kroah-Hartman
> <gregkh@xxxxxxxxxxxxxxxxxxx>; Linus Walleij <linus.walleij@xxxxxxxxxx>;
> Potthuri, Sai Krishna <sai.krishna.potthuri@xxxxxxx>; Shah, Tanmay
> <tanmay.shah@xxxxxxx>; Levinsky, Ben <ben.levinsky@xxxxxxx>; Manne,
> Nava kishore <nava.kishore.manne@xxxxxxx>; Rob Herring
> <robh@xxxxxxxxxx>; dhaval.r.shah@xxxxxxx; Marek Vasut
> <marex@xxxxxxx>; Roman Gushchin <roman.gushchin@xxxxxxxxx>; Mathieu
> Poirier <mathieu.poirier@xxxxxxxxxx>; Datta, Shubhrajyoti
> <shubhrajyoti.datta@xxxxxxx>
> Cc: linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 2/2] xilinx: firmware: Add sysfs to get last reset reason
>
> On Tue, Nov 21, 2023, at 12:20, Ronak Jain wrote:
> > Add sysfs interface to get the last reset reason of the system from
> > a user space.
> >
> > When the system reset happens whether intentionally or due to some
> > errors, the firmware used to store the last reset reason and the same
> > be queried by using the IOCTL API or the sysfs entry to know the
> > reason behind the system reset and accordingly, the error can be
> > diagnostic.
> >
> > Signed-off-by: Ronak Jain <ronak.jain@xxxxxxx>
>
> I'm fairly sure we already have other interfaces to access
> the reset reason, please see if you can either reuse one
> of the existing ones, or come up with a generic one that
> can be used by others as well.
Could you please provide more information regarding the interface you're talking about?

Thanks,
Ronak
>
> Arnd