Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755935Ab2BLVd6 (ORCPT ); Sun, 12 Feb 2012 16:33:58 -0500 Received: from ogre.sisk.pl ([217.79.144.158]:48763 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754696Ab2BLVd5 (ORCPT ); Sun, 12 Feb 2012 16:33:57 -0500 From: "Rafael J. Wysocki" To: Jean Pihet Subject: Re: [PATCH 1/2] PM / QoS: unconditionally build the feature Date: Sun, 12 Feb 2012 22:37:46 +0100 User-Agent: KMail/1.13.6 (Linux/3.3.0-rc3+; KDE/4.6.0; x86_64; ; ) Cc: Linux PM mailing list , linux-kernel@vger.kernel.org, Mark Gross , Venkatesh Pallipadi , Jean Pihet References: <1328603646-2714-1-git-send-email-j-pihet@ti.com> <1328603646-2714-2-git-send-email-j-pihet@ti.com> In-Reply-To: <1328603646-2714-2-git-send-email-j-pihet@ti.com> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-2" Content-Transfer-Encoding: 7bit Message-Id: <201202122237.46237.rjw@sisk.pl> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3731 Lines: 97 On Tuesday, February 07, 2012, Jean Pihet wrote: > The PM QoS feature depends on CONFIG_PM which depends on > PM_SLEEP || PM_RUNTIME. This breaks CPU C-states with kernels > not having these CONFIGs. > > This patch allows the feature in all cases. Since the Venki's patch has been merged in the meantime, can you please rebase this on top of the current Linus' tree? Rafael > Reported-by: Venkatesh Pallipadi > Signed-off-by: Jean Pihet > Cc: Rafael J. Wysocki > Cc: Mark Gross > --- > include/linux/pm_qos.h | 29 +---------------------------- > kernel/power/Makefile | 3 ++- > 2 files changed, 3 insertions(+), 29 deletions(-) > > diff --git a/include/linux/pm_qos.h b/include/linux/pm_qos.h > index e5bbcba..105be69 100644 > --- a/include/linux/pm_qos.h > +++ b/include/linux/pm_qos.h > @@ -63,7 +63,6 @@ static inline int dev_pm_qos_request_active(struct dev_pm_qos_request *req) > return req->dev != 0; > } > > -#ifdef CONFIG_PM > int pm_qos_update_target(struct pm_qos_constraints *c, struct plist_node *node, > enum pm_qos_req_action action, int value); > void pm_qos_add_request(struct pm_qos_request *req, int pm_qos_class, > @@ -78,6 +77,7 @@ int pm_qos_remove_notifier(int pm_qos_class, struct notifier_block *notifier); > int pm_qos_request_active(struct pm_qos_request *req); > s32 pm_qos_read_value(struct pm_qos_constraints *c); > > +#ifdef CONFIG_PM > s32 __dev_pm_qos_read_value(struct device *dev); > s32 dev_pm_qos_read_value(struct device *dev); > int dev_pm_qos_add_request(struct device *dev, struct dev_pm_qos_request *req, > @@ -95,33 +95,6 @@ void dev_pm_qos_constraints_destroy(struct device *dev); > int dev_pm_qos_add_ancestor_request(struct device *dev, > struct dev_pm_qos_request *req, s32 value); > #else > -static inline int pm_qos_update_target(struct pm_qos_constraints *c, > - struct plist_node *node, > - enum pm_qos_req_action action, > - int value) > - { return 0; } > -static inline void pm_qos_add_request(struct pm_qos_request *req, > - int pm_qos_class, s32 value) > - { return; } > -static inline void pm_qos_update_request(struct pm_qos_request *req, > - s32 new_value) > - { return; } > -static inline void pm_qos_remove_request(struct pm_qos_request *req) > - { return; } > - > -static inline int pm_qos_request(int pm_qos_class) > - { return 0; } > -static inline int pm_qos_add_notifier(int pm_qos_class, > - struct notifier_block *notifier) > - { return 0; } > -static inline int pm_qos_remove_notifier(int pm_qos_class, > - struct notifier_block *notifier) > - { return 0; } > -static inline int pm_qos_request_active(struct pm_qos_request *req) > - { return 0; } > -static inline s32 pm_qos_read_value(struct pm_qos_constraints *c) > - { return 0; } > - > static inline s32 __dev_pm_qos_read_value(struct device *dev) > { return 0; } > static inline s32 dev_pm_qos_read_value(struct device *dev) > diff --git a/kernel/power/Makefile b/kernel/power/Makefile > index 07e0e28..66d808e 100644 > --- a/kernel/power/Makefile > +++ b/kernel/power/Makefile > @@ -1,7 +1,8 @@ > > ccflags-$(CONFIG_PM_DEBUG) := -DDEBUG > > -obj-$(CONFIG_PM) += main.o qos.o > +obj-y += qos.o > +obj-$(CONFIG_PM) += main.o > obj-$(CONFIG_VT_CONSOLE_SLEEP) += console.o > obj-$(CONFIG_FREEZER) += process.o > obj-$(CONFIG_SUSPEND) += suspend.o > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/