2008-07-11 15:28:35

by Philipp Zabel

Subject: [PATCH] regulator: include suspend.h in machine.h for suspend_state_t

linux/regulator/machine.h shouldn't depend on files including this
header to have included linux/suspend.h before.

Signed-off-by: Philipp Zabel <[email protected]>
---
include/linux/regulator/machine.h | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/linux/regulator/machine.h b/include/linux/regulator/machine.h
index 60e6873..bed2b5e 100644
--- a/include/linux/regulator/machine.h
+++ b/include/linux/regulator/machine.h
@@ -16,6 +16,7 @@
#define __LINUX_REGULATOR_MACHINE_H_

#include <linux/regulator/consumer.h>
+#include <linux/suspend.h>

struct regulator;

--
1.5.6.2

2008-07-12 10:07:09

by Liam Girdwood

Subject: Re: [PATCH] regulator: include suspend.h in machine.h for suspend_state_t

On Fri, 2008-07-11 at 17:28 +0200, Philipp Zabel wrote:
> linux/regulator/machine.h shouldn't depend on files including this
> header to have included linux/suspend.h before.
>
> Signed-off-by: Philipp Zabel <[email protected]>
> ---
> include/linux/regulator/machine.h | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/include/linux/regulator/machine.h b/include/linux/regulator/machine.h
> index 60e6873..bed2b5e 100644
> --- a/include/linux/regulator/machine.h
> +++ b/include/linux/regulator/machine.h
> @@ -16,6 +16,7 @@
> #define __LINUX_REGULATOR_MACHINE_H_
>
> #include <linux/regulator/consumer.h>
> +#include <linux/suspend.h>
>
> struct regulator;
>

Applied.

Thanks

Liam