From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Cong Wang <amwang@redhat.com>
Cc: Len Brown <len.brown@intel.com>,
Philip Rakity <prakity@marvell.com>,
Lucas De Marchi <lucas.demarchi@profusion.mobi>,
netdev@vger.kernel.org, linux-mmc@vger.kernel.org,
linux-kernel@vger.kernel.org,
Josh Triplett <josh@joshtriplett.org>,
linux-pm@lists.linux-foundation.org,
"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
akpm@linux-foundation.org, Chris Ball <cjb@laptop.org>,
"David S. Miller" <davem@davemloft.net>
Subject: Re: [PATCH 4/5] pm: move pm notifiers into suspend.h
Date: Thu, 23 Jun 2011 14:04:11 +0200 [thread overview]
Message-ID: <201106231404.12311.rjw__18829.2399318472$1308830961$gmane$org@sisk.pl> (raw)
In-Reply-To: <4E02C93A.7010601@redhat.com>
On Thursday, June 23, 2011, Cong Wang wrote:
> 于 2011年06月23日 03:49, Rafael J. Wysocki 写道:
> >> +#ifdef CONFIG_PM
> >> > +#include<linux/suspend.h>
> >> > +#endif
> > I don't think the #ifdef in necessary. Any dependencies on CONFIG_PM
> > (or CONFIG_SUSPEND etc.) should be taken care of inside of suspend.h.
> > This file should be fixed if they aren't.
> >
>
> Ok, please check the updated version below.
That's fine by me. Please feel free to add my ACK to the patch.
Thanks,
Rafael
_______________________________________________
linux-pm mailing list
linux-pm@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/linux-pm
next prev parent reply other threads:[~2011-06-23 12:04 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-22 6:35 [PATCH 0/5] notifiers: split notifier.h into subsystem headers Amerigo Wang
2011-06-22 6:35 ` [PATCH 1/5] cpu: move cpu notifiers into cpu.h Amerigo Wang
2011-06-22 6:35 ` [PATCH 2/5] net: move netdevice notifiers into netdevice.h Amerigo Wang
2011-06-22 6:35 ` [PATCH 3/5] sys: move reboot notifiers into reboot.h Amerigo Wang
2011-06-22 6:35 ` [PATCH 4/5] pm: move pm notifiers into suspend.h Amerigo Wang
2011-06-22 6:35 ` Amerigo Wang
2011-06-22 19:49 ` Rafael J. Wysocki
2011-06-22 19:49 ` Rafael J. Wysocki
2011-06-23 5:03 ` Cong Wang
2011-06-23 5:03 ` Cong Wang
2011-06-23 12:04 ` Rafael J. Wysocki
2011-06-23 12:04 ` Rafael J. Wysocki [this message]
2011-06-23 20:24 ` Pavel Machek
2011-06-23 20:24 ` Pavel Machek
2011-06-22 6:35 ` [PATCH 5/5] vt: move vt notifiers into vt.h Amerigo Wang
2011-06-22 6:42 ` [PATCH 0/5] notifiers: split notifier.h into subsystem headers David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='201106231404.12311.rjw__18829.2399318472$1308830961$gmane$org@sisk.pl' \
--to=rjw@sisk.pl \
--cc=akpm@linux-foundation.org \
--cc=amwang@redhat.com \
--cc=cjb@laptop.org \
--cc=davem@davemloft.net \
--cc=josh@joshtriplett.org \
--cc=len.brown@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-pm@lists.linux-foundation.org \
--cc=lucas.demarchi@profusion.mobi \
--cc=netdev@vger.kernel.org \
--cc=paulmck@linux.vnet.ibm.com \
--cc=prakity@marvell.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.