Subject: [PATCHv2 07/12] arm64: psci: kill psci_power_state

From: [email-masked] (Mark Rutland)
Date: Mon, 18 May 2015 19:11:57 +0100

On Mon, May 18, 2015 at 06:36:55PM +0100, Catalin Marinas wrote:
> On Mon, May 18, 2015 at 11:38:18AM +0100, Mark Rutland wrote:
> > A PSCI 1.0 implementation may choose to use the new extended StateID
> > format, the presence of which may be queried via the PSCI_FEATURES call.
> > The layout of this new StateID format is incompatible with the existing
> > format, and so to handle both we must abstract attempts to parse the
> > fields.
> > 
> > In preparation for PSCI 1.0 support, this patch introduces
> > psci_power_state_loses_context and psci_power_state_is_valid functions
> > to query information from a PSCI power state, which is no longer
> > decomposed (and hence the pack/unpack functions are removed). As it is
> > no longer decomposed, it is now passed round as an opaque u32 token.
> > 
> > Signed-off-by: Mark Rutland <[email-masked]>
> > Acked-by: Lorenzo Pieralisi <[email-masked]>
> > Cc: Catalin Marinas <[email-masked]>
> > Cc: Will Deacon <[email-masked]>
> > ---
> >  arch/arm64/kernel/psci.c | 89 ++++++++++++++++++++----------------------------
> >  1 file changed, 37 insertions(+), 52 deletions(-)
> > 
> > diff --git a/arch/arm64/kernel/psci.c b/arch/arm64/kernel/psci.c
> > index 3ef677e..4ef9472 100644
> > --- a/arch/arm64/kernel/psci.c
> > +++ b/arch/arm64/kernel/psci.c
> > @@ -37,11 +37,19 @@
> >  #define PSCI_POWER_STATE_TYPE_STANDBY		0
> >  #define PSCI_POWER_STATE_TYPE_POWER_DOWN	1
> >  
> > -struct psci_power_state {
> > -	u16	id;
> > -	u8	type;
> > -	u8	affinity_level;
> > -};
> > +static bool psci_power_state_loses_context(u32 state)
> > +{
> > +	return !!(state & PSCI_0_2_POWER_STATE_TYPE_MASK);
> > +}
> 
> Nitpick: I don't think you need the !! here since the default conversion
> is to bool anyway.

True. I'll drop that, assuming Lorenzo is happy to so.

> Acked-by: Catalin Marinas <[email-masked]>

Thanks,
Mark.