Re: [PATCH 0/3] riscv: sbi: Switch to the sys-off handler API
From: Conor Dooley
Date: Tue Jan 03 2023 - 05:55:07 EST
On Wed, Dec 28, 2022 at 06:30:11PM +0000, Conor Dooley wrote:
> Hey Samuel,
>
> On Wed, Dec 28, 2022 at 10:19:12AM -0600, Samuel Holland wrote:
> > I want to convert the axp20x PMIC poweroff handler to use the sys-off
> > API, so it can be used as a fallback for if the SBI poweroff handler
> > is unavailable. But the SBI poweroff handler still uses pm_power_off, so
> > done alone, this would cause the axp20x callback to be called first,
> > before the SBI poweroff handler has a chance to run.
> >
> > In order to prevent this change in behavior, the SBI poweroff handler
> > needs to be converted to the sys-off API first, at a higher priority.
> >
> > This series performs the conversion, after accounting for the fact that
> > the SBI poweroff handler is registered quite early during boot.
> >
> > The first patch is a dependency for both this series and the PSCI
> > series[1], so I would like to get at least patch 1 merged soon.
> >
> > [1]: https://lore.kernel.org/lkml/20221105214841.7828-1-samuel@xxxxxxxxxxxx/
> >
> >
> > Samuel Holland (3):
> > kernel/reboot: Use the static sys-off handler for any priority
> > riscv: sbi: Share the code for unsupported extension warnings
> > riscv: sbi: Switch to the sys-off handler API
>
> Not what other stuff has reboot support, so I gave it a whirl on
I happened to see this today and could barely understand what I wrote.
s/Not what/Not sure what"
> PolarFire SoC & it seemed to work as expected:
> Tested-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
>
> Thanks,
> Conor.
Attachment:
signature.asc
Description: PGP signature