Re: [PATCH v3] power: add an API to log wakeup reasons

From: Joe Perches
Date: Wed Mar 12 2014 - 15:51:08 EST


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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/