Re: [linux-pm] Re: [Resend][PATCH] PM: Fix dependencies ofCONFIG_SUSPEND and CONFIG_HIBERNATION (updated)

From: Johannes Berg
Date: Mon Aug 06 2007 - 05:28:20 EST


On Mon, 2007-08-06 at 11:07 +0200, Pavel Machek wrote:
> Hi!
>
> > +config SUSPEND_UP_POSSIBLE
> > + bool
> > + depends on (X86 && !X86_VOYAGER) || (PPC32 && PPC_MPC52xx) \
> > + || (PPC64 && (PPC_PSERIES || PPC_PMAC)) || ARM || BLACKFIN \
> > + || MIPS || SUPERH || FRV
> > + depends on !SMP
> > + default y
>
> I guess I'd rather left SUSPEND_UP_POSSIBLE to allways y (as it always
> was), and let architectures that can't handle it not return "mem"
> from list of valid states...

Yeah, that's the utterly broken interface we used to have. Until I fixed
it to have no valid states until architectures implement suspend_ops.
Still, I disagree, why bother with compiling code that can't ever be
used?

johannes

Attachment: signature.asc
Description: This is a digitally signed message part