On Thu, 28 Jul 2022 at 16:38, Mark Brown <broonie@xxxxxxxxxx> wrote:
>
> On Wed, Jul 27, 2022 at 11:09:01PM +0300, Dmitry Baryshkov wrote:
>
> > +} psci_fn_ids[] = {
> > + PSCI_ID_NATIVE(0_2, MIGRATE),
> > + PSCI_ID(0_2, MIGRATE_INFO_TYPE),
> > + PSCI_ID_NATIVE(0_2, MIGRATE_INFO_UP_CPU),
> > + PSCI_ID(1_0, CPU_FREEZE),
> > + PSCI_ID_NATIVE(1_0, CPU_DEFAULT_SUSPEND),
> > + PSCI_ID_NATIVE(1_0, NODE_HW_STATE),
> > + PSCI_ID_NATIVE(1_0, SYSTEM_SUSPEND),
> > + PSCI_ID(1_0, SET_SUSPEND_MODE),
> > + PSCI_ID_NATIVE(1_0, STAT_RESIDENCY),
> > + PSCI_ID_NATIVE(1_0, STAT_COUNT),
> > + PSCI_ID_NATIVE(1_1, SYSTEM_RESET2),
> > +};
>
> There's other functions like the MEM_PROTECT ones which we don't
> currently use but it might be interesting to enumerate...
Argh, missed the next page in PSCI spec. Will fix in v2.
>
> > #define PSCI_1_0_FN_PSCI_FEATURES PSCI_0_2_FN(10)
> > +#define PSCI_1_0_FN_CPU_FREEZE PSCI_0_2_FN(11)
> > +#define PSCI_1_0_FN_CPU_DEFAULT_SUSPEND PSCI_0_2_FN(12)
>
> ...we're already adding functions here.
>
> > +#define PSCI_1_0_FN_NODE_HW_STATE PSCI_0_2_FN(13)
>
> > +#define PSCI_1_0_FN_STAT_RESIDENCY PSCI_0_2_FN(16)
> > +#define PSCI_1_0_FN_STAT_COUNT PSCI_0_2_FN(17)
>
> Some of these state query things might be interesting to actually call
> and output results from at some point, doesn't seem like something that
> should be a blocker though.
I thought about it, but deferred for now.
--
With best wishes
Dmitry
