On Tue, Feb 22, 2011 at 2:36 PM, Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> wrote: >> > Could we just make CONFIG_XEN_SAVE_RESTORE depend on CONFIG_HIBERANTION? >> > Like this: >> > >> > >> > diff --git a/arch/x86/xen/Kconfig b/arch/x86/xen/Kconfig >> > index 5b54892..838e20c 100644 >> > --- a/arch/x86/xen/Kconfig >> > +++ b/arch/x86/xen/Kconfig >> > @@ -38,7 +38,7 @@ config XEN_MAX_DOMAIN_MEMORY >> > >> > config XEN_SAVE_RESTORE >> > bool >> > - depends on XEN && PM >> > + depends on XEN && PM && HIBERNATION >> > default y >> > >> > config XEN_DEBUG_FS >> > >> > >> On that aspect, just noticed >> config PM_SLEEP >> bool >> depends on SUSPEND || HIBERNATION || XEN_SAVE_RESTORE >> default y >> we could get rid of XEN_SAVE_RESTORE as the suspend/resume code doesnt >> depend on PM_SLEEP > > OK. Do you want to respin patch #2 without the #ifdefs in the drivers/base/... > and provide a new patch (#4?) which will alter the Kconfig file appropriately? > > yep. will do. shriram _______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm