* [PATCH] ARM: mxs: pm: Include "pm.h"
@ 2013-07-16 13:32 Fabio Estevam
2013-07-17 3:53 ` Shawn Guo
0 siblings, 1 reply; 4+ messages in thread
From: Fabio Estevam @ 2013-07-16 13:32 UTC (permalink / raw)
To: linux-arm-kernel
Fix the following sparse warning:
arch/arm/mach-mxs/pm.c:37:13: warning: symbol 'mxs_pm_init' was not declared. Should it be static?
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
arch/arm/mach-mxs/pm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-mxs/pm.c b/arch/arm/mach-mxs/pm.c
index b2494d2..0170e99 100644
--- a/arch/arm/mach-mxs/pm.c
+++ b/arch/arm/mach-mxs/pm.c
@@ -15,6 +15,7 @@
#include <linux/kernel.h>
#include <linux/suspend.h>
#include <linux/io.h>
+#include "pm.h"
static int mxs_suspend_enter(suspend_state_t state)
{
--
1.8.1.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] ARM: mxs: pm: Include "pm.h"
2013-07-16 13:32 Fabio Estevam
@ 2013-07-17 3:53 ` Shawn Guo
0 siblings, 0 replies; 4+ messages in thread
From: Shawn Guo @ 2013-07-17 3:53 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jul 16, 2013 at 10:32:39AM -0300, Fabio Estevam wrote:
> Fix the following sparse warning:
>
> arch/arm/mach-mxs/pm.c:37:13: warning: symbol 'mxs_pm_init' was not declared. Should it be static?
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] ARM: mxs: pm: Include "pm.h"
@ 2013-09-02 2:11 Fabio Estevam
2013-09-03 0:54 ` Shawn Guo
0 siblings, 1 reply; 4+ messages in thread
From: Fabio Estevam @ 2013-09-02 2:11 UTC (permalink / raw)
To: linux-arm-kernel
From: Fabio Estevam <fabio.estevam@freescale.com>
Include "pm.h" in order to fix the following sparse warning:
arch/arm/mach-mxs/pm.c:37:13: warning: symbol 'mxs_pm_init' was not declared. Should it be static?
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
arch/arm/mach-mxs/pm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-mxs/pm.c b/arch/arm/mach-mxs/pm.c
index b2494d2..0170e99 100644
--- a/arch/arm/mach-mxs/pm.c
+++ b/arch/arm/mach-mxs/pm.c
@@ -15,6 +15,7 @@
#include <linux/kernel.h>
#include <linux/suspend.h>
#include <linux/io.h>
+#include "pm.h"
static int mxs_suspend_enter(suspend_state_t state)
{
--
1.8.1.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] ARM: mxs: pm: Include "pm.h"
2013-09-02 2:11 [PATCH] ARM: mxs: pm: Include "pm.h" Fabio Estevam
@ 2013-09-03 0:54 ` Shawn Guo
0 siblings, 0 replies; 4+ messages in thread
From: Shawn Guo @ 2013-09-03 0:54 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Sep 01, 2013 at 11:11:08PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Include "pm.h" in order to fix the following sparse warning:
>
> arch/arm/mach-mxs/pm.c:37:13: warning: symbol 'mxs_pm_init' was not declared. Should it be static?
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Why do you resend a patch that I already applied [1]?
Shawn
[1] http://www.spinics.net/lists/arm-kernel/msg259101.html
> ---
> arch/arm/mach-mxs/pm.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/mach-mxs/pm.c b/arch/arm/mach-mxs/pm.c
> index b2494d2..0170e99 100644
> --- a/arch/arm/mach-mxs/pm.c
> +++ b/arch/arm/mach-mxs/pm.c
> @@ -15,6 +15,7 @@
> #include <linux/kernel.h>
> #include <linux/suspend.h>
> #include <linux/io.h>
> +#include "pm.h"
>
> static int mxs_suspend_enter(suspend_state_t state)
> {
> --
> 1.8.1.2
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-09-03 0:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-02 2:11 [PATCH] ARM: mxs: pm: Include "pm.h" Fabio Estevam
2013-09-03 0:54 ` Shawn Guo
-- strict thread matches above, loose matches on Subject: below --
2013-07-16 13:32 Fabio Estevam
2013-07-17 3:53 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).