[RFC PATCH 1/1] riscv: sbi: Introduce system suspend support
Andrew Jones
ajones at ventanamicro.com
Tue Jan 10 07:52:32 PST 2023
On Tue, Jan 10, 2023 at 02:55:20PM +0000, Leyfoon Tan wrote:
>
> >
> > source "kernel/power/Kconfig"
> > diff --git a/arch/riscv/include/asm/sbi.h b/arch/riscv/include/asm/sbi.h index
> > 4ca7fbacff42..9834ba4ce3e4 100644
> > --- a/arch/riscv/include/asm/sbi.h
> > +++ b/arch/riscv/include/asm/sbi.h
> > @@ -29,6 +29,7 @@ enum sbi_ext_id {
> > SBI_EXT_RFENCE = 0x52464E43,
> > SBI_EXT_HSM = 0x48534D,
> > SBI_EXT_SRST = 0x53525354,
> > + SBI_EXT_SUSP = 0x53555350,
> > SBI_EXT_PMU = 0x504D55,
> >
> > /* Experimentals extensions must lie within this range */ @@ -113,6
> > +114,14 @@ enum sbi_srst_reset_reason {
> > SBI_SRST_RESET_REASON_SYS_FAILURE,
> > };
> >
> > +enum sbi_ext_susp_fid {
> > + SBI_EXT_SUSP_SUSPEND = 0,
>
> Macro name with "*_SYSTEM_SUSPEND" is better? Avoid confusing with CPU suspend.
The _SUSP_ part should cover that, but I'm OK with adding SYSTEM as well.
>
> > +};
> > +
> > +enum sbi_ext_susp_sleep_type {
> > + SBI_SUSP_SLEEP_TYPE_SUSPEND = 0,
> This should be SBI_SUSP_SLEEP_TYPE_SUSPEND_TO_RAM?
Yes, that would be better. I'll change that for the next version.
Thanks,
drew
More information about the linux-riscv
mailing list