[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1394653850.3915.26.camel@joe-AO722>
Date: Wed, 12 Mar 2014 12:50:50 -0700
From: Joe Perches <joe@...ches.com>
To: Ruchi Kandoi <kandoiruchi@...gle.com>
Cc: linux-kernel@...r.kernel.org, linux-pm@...r.kernel.org,
rjw@...ysocki.net, ghackmann@...gle.com, john.stultz@...aro.org,
toddpoynor@...gle.com
Subject: Re: [PATCH v3] power: add an API to log wakeup reasons
On Wed, 2014-03-12 at 12:46 -0700, Ruchi Kandoi wrote:
> For power management diagnostic purposes, it is often useful to know
> what interrupts are frequently waking the system from low power
> suspend mode, especially on battery-powered consumer electronics
> devices that are expected to spend much of their time in low-power
> suspend while not in active use. For example, reduced battery life on
> a mobile phone may be caused in part by frequent wakeups by broadcast
> traffic on a busy wireless LAN even while the screen is off and the
> phone not in active use.
trivial note:
> diff --git a/kernel/power/wakeup_reason.c b/kernel/power/wakeup_reason.c
[]
> +#define pr_fmt(fmt) "wakeup_reason:" fmt
It's almost universal in the kernel to have a space after the :
in the prefix
#define pr_fmt(fmt) "wakeup_reason: " fmt
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists