public inbox for linux-tegra@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] spi/tegra: fix compilation error in spi-tegra.c
@ 2011-10-13  9:01 Marc Dietrich
  2011-10-13 19:27 ` Grant Likely
  0 siblings, 1 reply; 2+ messages in thread
From: Marc Dietrich @ 2011-10-13  9:01 UTC (permalink / raw)
  To: Grant Likely
  Cc: Olof Johansson, Colin Cross, linux-tegra-u79uwXL29TY76Z2rM5mHXA

Add #include <linux/module.h> to spi-tegra.c to fix a compilation error
after the removal of module.h from device.h (in patch: "include: replace 
linux/module.h with "struct module" wherever possible").

Signed-off-by: Marc Dietrich <marvin24-Mmb7MZpHnFY@public.gmane.org>
Acked-by: Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>
---

Hi Grant,

this fixes the error below:

  CC      drivers/spi/spi-tegra.o
drivers/spi/spi-tegra.c:597: error: expected declaration specifiers or '...' before 
string constant
drivers/spi/spi-tegra.c:597: warning: data definition has no type or storage class
drivers/spi/spi-tegra.c:597: warning: type defaults to 'int' in declaration of 
'MODULE_ALIAS'
drivers/spi/spi-tegra.c:597: warning: function declaration isn't a prototype
drivers/spi/spi-tegra.c:604: warning: data definition has no type or storage class
drivers/spi/spi-tegra.c:604: warning: type defaults to 'int' in declaration of 
'MODULE_DEVICE_TABLE'
drivers/spi/spi-tegra.c:604: warning: parameter names (without types) in function 
declaration
drivers/spi/spi-tegra.c:612: error: 'THIS_MODULE' undeclared here (not in a function)
drivers/spi/spi-tegra.c:630: error: expected declaration specifiers or '...' before 
string constant
drivers/spi/spi-tegra.c:630: warning: data definition has no type or storage class
drivers/spi/spi-tegra.c:630: warning: type defaults to 'int' in declaration of 
'MODULE_LICENSE'
drivers/spi/spi-tegra.c:630: warning: function declaration isn't a prototype
make[2]: *** [drivers/spi/spi-tegra.o] Error 1

It is against linux-next (for 3.2). Please apply.

Thanks
			Marc

diff --git a/drivers/spi/spi-tegra.c b/drivers/spi/spi-tegra.c
index a5a6302..e8cd58f 100644
--- a/drivers/spi/spi-tegra.c
+++ b/drivers/spi/spi-tegra.c
@@ -18,6 +18,7 @@
  */
 
 #include <linux/kernel.h>
+#include <linux/module.h>
 #include <linux/init.h>
 #include <linux/err.h>
 #include <linux/platform_device.h>

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] spi/tegra: fix compilation error in spi-tegra.c
  2011-10-13  9:01 [PATCH] spi/tegra: fix compilation error in spi-tegra.c Marc Dietrich
@ 2011-10-13 19:27 ` Grant Likely
  0 siblings, 0 replies; 2+ messages in thread
From: Grant Likely @ 2011-10-13 19:27 UTC (permalink / raw)
  To: Marc Dietrich
  Cc: Olof Johansson, Colin Cross, linux-tegra-u79uwXL29TY76Z2rM5mHXA

On Thu, Oct 13, 2011 at 11:01:33AM +0200, Marc Dietrich wrote:
> Add #include <linux/module.h> to spi-tegra.c to fix a compilation error
> after the removal of module.h from device.h (in patch: "include: replace 
> linux/module.h with "struct module" wherever possible").
> 
> Signed-off-by: Marc Dietrich <marvin24-Mmb7MZpHnFY@public.gmane.org>
> Acked-by: Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>

Applied, thanks

g.

> ---
> 
> Hi Grant,
> 
> this fixes the error below:
> 
>   CC      drivers/spi/spi-tegra.o
> drivers/spi/spi-tegra.c:597: error: expected declaration specifiers or '...' before 
> string constant
> drivers/spi/spi-tegra.c:597: warning: data definition has no type or storage class
> drivers/spi/spi-tegra.c:597: warning: type defaults to 'int' in declaration of 
> 'MODULE_ALIAS'
> drivers/spi/spi-tegra.c:597: warning: function declaration isn't a prototype
> drivers/spi/spi-tegra.c:604: warning: data definition has no type or storage class
> drivers/spi/spi-tegra.c:604: warning: type defaults to 'int' in declaration of 
> 'MODULE_DEVICE_TABLE'
> drivers/spi/spi-tegra.c:604: warning: parameter names (without types) in function 
> declaration
> drivers/spi/spi-tegra.c:612: error: 'THIS_MODULE' undeclared here (not in a function)
> drivers/spi/spi-tegra.c:630: error: expected declaration specifiers or '...' before 
> string constant
> drivers/spi/spi-tegra.c:630: warning: data definition has no type or storage class
> drivers/spi/spi-tegra.c:630: warning: type defaults to 'int' in declaration of 
> 'MODULE_LICENSE'
> drivers/spi/spi-tegra.c:630: warning: function declaration isn't a prototype
> make[2]: *** [drivers/spi/spi-tegra.o] Error 1
> 
> It is against linux-next (for 3.2). Please apply.
> 
> Thanks
> 			Marc
> 
> diff --git a/drivers/spi/spi-tegra.c b/drivers/spi/spi-tegra.c
> index a5a6302..e8cd58f 100644
> --- a/drivers/spi/spi-tegra.c
> +++ b/drivers/spi/spi-tegra.c
> @@ -18,6 +18,7 @@
>   */
>  
>  #include <linux/kernel.h>
> +#include <linux/module.h>
>  #include <linux/init.h>
>  #include <linux/err.h>
>  #include <linux/platform_device.h>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-10-13 19:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-13  9:01 [PATCH] spi/tegra: fix compilation error in spi-tegra.c Marc Dietrich
2011-10-13 19:27 ` Grant Likely

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox