* [PATCH] OMAP2+: Fix Compilation break on current mainline
@ 2011-11-07 12:38 Govindraj.R
2011-11-07 12:59 ` Shilimkar, Santosh
0 siblings, 1 reply; 10+ messages in thread
From: Govindraj.R @ 2011-11-07 12:38 UTC (permalink / raw)
To: linux-omap; +Cc: Tony Lindgren, Govindraj.R
I see below errors and warnings on latest mainline commit
b32fc0a0629bf5894b35f33554c118aacfd0d1e2 with omap2plus_defconfig.
Patch to fix the same.
arch/arm/plat-omap/dmtimer.c:184: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:184: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:184: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:215: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:215: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:215: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:228: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:228: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:228: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:234: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:234: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:234: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:240: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:240: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:240: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:248: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:248: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:248: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:294: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:294: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:294: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:302: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:302: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:302: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:316: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:316: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:316: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:344: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:344: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:344: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:361: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:361: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:361: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:380: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:380: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:380: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:406: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:406: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:406: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:443: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:443: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:443: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:468: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:468: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:468: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:494: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:494: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:494: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:517: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:517: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:517: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:534: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:534: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:534: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:549: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:549: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:549: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:561: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:561: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:561: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:572: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:572: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:572: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:587: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:587: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:587: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:604: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:604: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/arm/plat-omap/dmtimer.c:604: warning: parameter names (without types) in function declaration
arch/arm/plat-omap/dmtimer.c:746: error: expected declaration specifiers or '...' before string constant
arch/arm/plat-omap/dmtimer.c:746: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:746: warning: type defaults to 'int' in declaration of 'MODULE_DESCRIPTION'
arch/arm/plat-omap/dmtimer.c:746: warning: function declaration isn't a prototype
arch/arm/plat-omap/dmtimer.c:747: error: expected declaration specifiers or '...' before string constant
arch/arm/plat-omap/dmtimer.c:747: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:747: warning: type defaults to 'int' in declaration of 'MODULE_LICENSE'
arch/arm/plat-omap/dmtimer.c:747: warning: function declaration isn't a prototype
arch/arm/plat-omap/dmtimer.c:748: error: expected declaration specifiers or '...' before string constant
arch/arm/plat-omap/dmtimer.c:748: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:748: warning: type defaults to 'int' in declaration of 'MODULE_ALIAS'
arch/arm/plat-omap/dmtimer.c:748: warning: function declaration isn't a prototype
arch/arm/plat-omap/dmtimer.c:749: error: expected declaration specifiers or '...' before string constant
arch/arm/plat-omap/dmtimer.c:749: warning: data definition has no type or storage class
arch/arm/plat-omap/dmtimer.c:749: warning: type defaults to 'int' in declaration of 'MODULE_AUTHOR'
arch/arm/plat-omap/dmtimer.c:749: warning: function declaration isn't a prototype
make[1]: *** [arch/arm/plat-omap/dmtimer.o] Error 1
make: *** [arch/arm/plat-omap] Error 2
make: *** Waiting for unfinished jobs....
arch/arm/mach-omap2/omap_l3_noc.c:240: error: expected ',' or ';' before 'MODULE_DEVICE_TABLE'
arch/arm/mach-omap2/omap_l3_noc.c:250: error: 'THIS_MODULE' undeclared here (not in a function)
make[1]: *** [arch/arm/mach-omap2/omap_l3_noc.o] Error 1
make: *** [arch/arm/mach-omap2] Error 2
Signed-off-by: Govindraj.R <govindraj.raja@ti.com>
---
Sorry for the noise if this is already fixed.
arch/arm/mach-omap2/omap_l3_noc.c | 3 ++-
arch/arm/plat-omap/dmtimer.c | 1 +
2 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/omap_l3_noc.c b/arch/arm/mach-omap2/omap_l3_noc.c
index c8b1bef..eacbe55 100644
--- a/arch/arm/mach-omap2/omap_l3_noc.c
+++ b/arch/arm/mach-omap2/omap_l3_noc.c
@@ -26,6 +26,7 @@
#include <linux/interrupt.h>
#include <linux/kernel.h>
#include <linux/slab.h>
+#include <linux/module.h>
#include "omap_l3_noc.h"
@@ -236,7 +237,7 @@ static int __devexit omap4_l3_remove(struct platform_device *pdev)
static const struct of_device_id l3_noc_match[] = {
{.compatible = "ti,omap4-l3-noc", },
{},
-}
+};
MODULE_DEVICE_TABLE(of, l3_noc_match);
#else
#define l3_noc_match NULL
diff --git a/arch/arm/plat-omap/dmtimer.c b/arch/arm/plat-omap/dmtimer.c
index 2def4e1..b9acaca 100644
--- a/arch/arm/plat-omap/dmtimer.c
+++ b/arch/arm/plat-omap/dmtimer.c
@@ -39,6 +39,7 @@
#include <linux/slab.h>
#include <linux/err.h>
#include <linux/pm_runtime.h>
+#include <linux/module.h>
#include <plat/dmtimer.h>
--
1.7.4.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP2+: Fix Compilation break on current mainline
2011-11-07 12:38 [PATCH] OMAP2+: Fix Compilation break on current mainline Govindraj.R
@ 2011-11-07 12:59 ` Shilimkar, Santosh
2011-11-07 14:02 ` Cousson, Benoit
0 siblings, 1 reply; 10+ messages in thread
From: Shilimkar, Santosh @ 2011-11-07 12:59 UTC (permalink / raw)
To: Govindraj.R; +Cc: linux-omap, Tony Lindgren
On Mon, Nov 7, 2011 at 6:08 PM, Govindraj.R <govindraj.raja@ti.com> wrote:
> I see below errors and warnings on latest mainline commit
> b32fc0a0629bf5894b35f33554c118aacfd0d1e2 with omap2plus_defconfig.
>
> Patch to fix the same.
>
Patches are already posted and applied by Tony on linux-omap master.
http://www.spinics.net/lists/linux-omap/msg59848.html
http://www.spinics.net/lists/linux-omap/msg59847.html
Regards
Santosh
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP2+: Fix Compilation break on current mainline
2011-11-07 12:59 ` Shilimkar, Santosh
@ 2011-11-07 14:02 ` Cousson, Benoit
2011-11-08 6:28 ` [PATCH] OMAP2+: Fix Compilation error " Govindraj.R
0 siblings, 1 reply; 10+ messages in thread
From: Cousson, Benoit @ 2011-11-07 14:02 UTC (permalink / raw)
To: Shilimkar, Santosh, Govindraj.R; +Cc: linux-omap, Tony Lindgren
On 11/7/2011 1:59 PM, Shilimkar, Santosh wrote:
> On Mon, Nov 7, 2011 at 6:08 PM, Govindraj.R<govindraj.raja@ti.com> wrote:
>> I see below errors and warnings on latest mainline commit
>> b32fc0a0629bf5894b35f33554c118aacfd0d1e2 with omap2plus_defconfig.
>>
>> Patch to fix the same.
>>
> Patches are already posted and applied by Tony on linux-omap master.
Except the missing semicolon... This is a similar issue that Ohad
reported for hwspinlock.
It will build fine until you will try to build the omap_l3_noc as a module.
Govind,
You can maybe repost a fix for that one only.
Regards,
Benoit
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] OMAP2+: Fix Compilation error on current mainline
2011-11-07 14:02 ` Cousson, Benoit
@ 2011-11-08 6:28 ` Govindraj.R
2011-11-08 6:33 ` Shilimkar, Santosh
2011-11-08 10:46 ` Cousson, Benoit
0 siblings, 2 replies; 10+ messages in thread
From: Govindraj.R @ 2011-11-08 6:28 UTC (permalink / raw)
To: linux-omap; +Cc: Tony Lindgren, Benoit Cousson, Santosh Shilimkar, Govindraj.R
Patch to fix below compilation error on latest mainline commit
b32fc0a0629bf5894b35f33554c118aacfd0d1e2 with omap2plus_defconfig.
arch/arm/mach-omap2/omap_l3_noc.c:250: error: 'THIS_MODULE' undeclared here (not in a function)
make[1]: *** [arch/arm/mach-omap2/omap_l3_noc.o] Error 1
Signed-off-by: Govindraj.R <govindraj.raja@ti.com>
---
arch/arm/mach-omap2/omap_l3_noc.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-omap2/omap_l3_noc.c b/arch/arm/mach-omap2/omap_l3_noc.c
index c8b1bef..c2cc196 100644
--- a/arch/arm/mach-omap2/omap_l3_noc.c
+++ b/arch/arm/mach-omap2/omap_l3_noc.c
@@ -26,6 +26,7 @@
#include <linux/interrupt.h>
#include <linux/kernel.h>
#include <linux/slab.h>
+#include <linux/module.h>
#include "omap_l3_noc.h"
--
1.7.4.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP2+: Fix Compilation error on current mainline
2011-11-08 6:28 ` [PATCH] OMAP2+: Fix Compilation error " Govindraj.R
@ 2011-11-08 6:33 ` Shilimkar, Santosh
2011-11-08 10:46 ` Cousson, Benoit
1 sibling, 0 replies; 10+ messages in thread
From: Shilimkar, Santosh @ 2011-11-08 6:33 UTC (permalink / raw)
To: Govindraj.R; +Cc: linux-omap, Tony Lindgren, Benoit Cousson
On Tue, Nov 8, 2011 at 11:58 AM, Govindraj.R <govindraj.raja@ti.com> wrote:
> Patch to fix below compilation error on latest mainline commit
> b32fc0a0629bf5894b35f33554c118aacfd0d1e2 with omap2plus_defconfig.
>
> arch/arm/mach-omap2/omap_l3_noc.c:250: error: 'THIS_MODULE' undeclared here (not in a function)
> make[1]: *** [arch/arm/mach-omap2/omap_l3_noc.o] Error 1
>
> Signed-off-by: Govindraj.R <govindraj.raja@ti.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP2+: Fix Compilation error on current mainline
2011-11-08 6:28 ` [PATCH] OMAP2+: Fix Compilation error " Govindraj.R
2011-11-08 6:33 ` Shilimkar, Santosh
@ 2011-11-08 10:46 ` Cousson, Benoit
2011-11-08 10:54 ` Santosh Shilimkar
1 sibling, 1 reply; 10+ messages in thread
From: Cousson, Benoit @ 2011-11-08 10:46 UTC (permalink / raw)
To: Govindraj.R; +Cc: linux-omap, Tony Lindgren, Santosh Shilimkar
Hi Govind,
On 11/8/2011 7:28 AM, Govindraj.R wrote:
> Patch to fix below compilation error on latest mainline commit
> b32fc0a0629bf5894b35f33554c118aacfd0d1e2 with omap2plus_defconfig.
>
> arch/arm/mach-omap2/omap_l3_noc.c:250: error: 'THIS_MODULE' undeclared here (not in a function)
> make[1]: *** [arch/arm/mach-omap2/omap_l3_noc.o] Error 1
>
> Signed-off-by: Govindraj.R<govindraj.raja@ti.com>
> ---
> arch/arm/mach-omap2/omap_l3_noc.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/omap_l3_noc.c b/arch/arm/mach-omap2/omap_l3_noc.c
> index c8b1bef..c2cc196 100644
> --- a/arch/arm/mach-omap2/omap_l3_noc.c
> +++ b/arch/arm/mach-omap2/omap_l3_noc.c
> @@ -26,6 +26,7 @@
> #include<linux/interrupt.h>
> #include<linux/kernel.h>
> #include<linux/slab.h>
> +#include<linux/module.h>
That one is already fixed by Axel Lin, and already pulled by Tony.
http://www.spinics.net/lists/linux-omap/msg59847.html
This is the missing semi-colon that still need to be fixed.
Regards,
Benoit
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP2+: Fix Compilation error on current mainline
2011-11-08 10:46 ` Cousson, Benoit
@ 2011-11-08 10:54 ` Santosh Shilimkar
2011-11-08 11:59 ` Govindraj
0 siblings, 1 reply; 10+ messages in thread
From: Santosh Shilimkar @ 2011-11-08 10:54 UTC (permalink / raw)
To: Cousson, Benoit; +Cc: Govindraj.R, linux-omap, Tony Lindgren
On Tuesday 08 November 2011 04:16 PM, Cousson, Benoit wrote:
> Hi Govind,
>
> On 11/8/2011 7:28 AM, Govindraj.R wrote:
>> Patch to fix below compilation error on latest mainline commit
>> b32fc0a0629bf5894b35f33554c118aacfd0d1e2 with omap2plus_defconfig.
>>
>> arch/arm/mach-omap2/omap_l3_noc.c:250: error: 'THIS_MODULE' undeclared
>> here (not in a function)
>> make[1]: *** [arch/arm/mach-omap2/omap_l3_noc.o] Error 1
>>
>> Signed-off-by: Govindraj.R<govindraj.raja@ti.com>
>> ---
>> arch/arm/mach-omap2/omap_l3_noc.c | 1 +
>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/omap_l3_noc.c
>> b/arch/arm/mach-omap2/omap_l3_noc.c
>> index c8b1bef..c2cc196 100644
>> --- a/arch/arm/mach-omap2/omap_l3_noc.c
>> +++ b/arch/arm/mach-omap2/omap_l3_noc.c
>> @@ -26,6 +26,7 @@
>> #include<linux/interrupt.h>
>> #include<linux/kernel.h>
>> #include<linux/slab.h>
>> +#include<linux/module.h>
>
> That one is already fixed by Axel Lin, and already pulled by Tony.
>
> http://www.spinics.net/lists/linux-omap/msg59847.html
>
> This is the missing semi-colon that still need to be fixed.
>
Opps. I missed that one while acking the patch.
Indeed the semi-colon one was agreed to be the left over
in the last patch discussion.
Regards
Santosh
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP2+: Fix Compilation error on current mainline
2011-11-08 10:54 ` Santosh Shilimkar
@ 2011-11-08 11:59 ` Govindraj
2011-11-08 15:25 ` Cousson, Benoit
0 siblings, 1 reply; 10+ messages in thread
From: Govindraj @ 2011-11-08 11:59 UTC (permalink / raw)
To: Santosh Shilimkar; +Cc: Cousson, Benoit, Govindraj.R, linux-omap, Tony Lindgren
On Tue, Nov 8, 2011 at 4:24 PM, Santosh Shilimkar
<santosh.shilimkar@ti.com> wrote:
> On Tuesday 08 November 2011 04:16 PM, Cousson, Benoit wrote:
>> Hi Govind,
>>
>> On 11/8/2011 7:28 AM, Govindraj.R wrote:
>>> Patch to fix below compilation error on latest mainline commit
>>> b32fc0a0629bf5894b35f33554c118aacfd0d1e2 with omap2plus_defconfig.
>>>
>>> arch/arm/mach-omap2/omap_l3_noc.c:250: error: 'THIS_MODULE' undeclared
>>> here (not in a function)
>>> make[1]: *** [arch/arm/mach-omap2/omap_l3_noc.o] Error 1
>>>
>>> Signed-off-by: Govindraj.R<govindraj.raja@ti.com>
>>> ---
>>> arch/arm/mach-omap2/omap_l3_noc.c | 1 +
>>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-omap2/omap_l3_noc.c
>>> b/arch/arm/mach-omap2/omap_l3_noc.c
>>> index c8b1bef..c2cc196 100644
>>> --- a/arch/arm/mach-omap2/omap_l3_noc.c
>>> +++ b/arch/arm/mach-omap2/omap_l3_noc.c
>>> @@ -26,6 +26,7 @@
>>> #include<linux/interrupt.h>
>>> #include<linux/kernel.h>
>>> #include<linux/slab.h>
>>> +#include<linux/module.h>
>>
>> That one is already fixed by Axel Lin, and already pulled by Tony.
>>
>> http://www.spinics.net/lists/linux-omap/msg59847.html
>>
>> This is the missing semi-colon that still need to be fixed.
>>
> Opps. I missed that one while acking the patch.
> Indeed the semi-colon one was agreed to be the left over
> in the last patch discussion.
Sorry, my bad sent the wrong patch here's the right one.[1]
--
Thanks,
Govindraj.R
[1]:
From 9d244c0dea4de10ab9052fc1fe7bde8846ad389e Mon Sep 17 00:00:00 2001
From: Govindraj.R <govindraj.raja@ti.com>
Date: Tue, 8 Nov 2011 16:57:22 +0530
Subject: [PATCH] OMAP2+: Fix Compilation error on current mainline
Fix below compilation failure on mainline kernel commit id
1ea6b8f48918282bdca0b32a34095504ee65bab5
arch/arm/mach-omap2/omap_l3_noc.c:240: error: expected ',' or ';'
before 'MODULE_DEVICE_TABLE'
Signed-off-by: Govindraj.R <govindraj.raja@ti.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
---
arch/arm/mach-omap2/omap_l3_noc.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/omap_l3_noc.c
b/arch/arm/mach-omap2/omap_l3_noc.c
index 6a66aa5..d15225f 100644
--- a/arch/arm/mach-omap2/omap_l3_noc.c
+++ b/arch/arm/mach-omap2/omap_l3_noc.c
@@ -237,7 +237,7 @@ static int __devexit omap4_l3_remove(struct
platform_device *pdev)
static const struct of_device_id l3_noc_match[] = {
{.compatible = "ti,omap4-l3-noc", },
{},
-}
+};
MODULE_DEVICE_TABLE(of, l3_noc_match);
#else
#define l3_noc_match NULL
--
1.7.4.1
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP2+: Fix Compilation error on current mainline
2011-11-08 11:59 ` Govindraj
@ 2011-11-08 15:25 ` Cousson, Benoit
2011-11-09 12:18 ` Govindraj
0 siblings, 1 reply; 10+ messages in thread
From: Cousson, Benoit @ 2011-11-08 15:25 UTC (permalink / raw)
To: Govindraj; +Cc: Santosh Shilimkar, Govindraj.R, linux-omap, Tony Lindgren
On 11/8/2011 12:59 PM, Govindraj wrote:
> On Tue, Nov 8, 2011 at 4:24 PM, Santosh Shilimkar
> <santosh.shilimkar@ti.com> wrote:
[...]
>> Opps. I missed that one while acking the patch.
>> Indeed the semi-colon one was agreed to be the left over
>> in the last patch discussion.
>
> Sorry, my bad sent the wrong patch here's the right one.[1]
>
> --
> Thanks,
> Govindraj.R
>
> [1]:
>
> From 9d244c0dea4de10ab9052fc1fe7bde8846ad389e Mon Sep 17 00:00:00 2001
> From: Govindraj.R<govindraj.raja@ti.com>
> Date: Tue, 8 Nov 2011 16:57:22 +0530
> Subject: [PATCH] OMAP2+: Fix Compilation error on current mainline
>
> Fix below compilation failure on mainline kernel commit id
> 1ea6b8f48918282bdca0b32a34095504ee65bab5
>
> arch/arm/mach-omap2/omap_l3_noc.c:240: error: expected ',' or ';'
> before 'MODULE_DEVICE_TABLE'
The issue is that 3.2-rc1 does not have that build error anymore...
It should appear only with this driver built as a module, and that's not
possible yet for the l3_noc.
The fix is still good to have, but the changelog should probably be updated.
Regards,
Benoit
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] OMAP2+: Fix Compilation error on current mainline
2011-11-08 15:25 ` Cousson, Benoit
@ 2011-11-09 12:18 ` Govindraj
0 siblings, 0 replies; 10+ messages in thread
From: Govindraj @ 2011-11-09 12:18 UTC (permalink / raw)
To: Cousson, Benoit; +Cc: Santosh Shilimkar, Govindraj.R, linux-omap, Tony Lindgren
On Tue, Nov 8, 2011 at 8:55 PM, Cousson, Benoit <b-cousson@ti.com> wrote:
> On 11/8/2011 12:59 PM, Govindraj wrote:
>>
>> On Tue, Nov 8, 2011 at 4:24 PM, Santosh Shilimkar
>> <santosh.shilimkar@ti.com> wrote:
>
> [...]
>
>>> Opps. I missed that one while acking the patch.
>>> Indeed the semi-colon one was agreed to be the left over
>>> in the last patch discussion.
>>
>> Sorry, my bad sent the wrong patch here's the right one.[1]
>>
>> --
>> Thanks,
>> Govindraj.R
>>
>> [1]:
>>
>> From 9d244c0dea4de10ab9052fc1fe7bde8846ad389e Mon Sep 17 00:00:00 2001
>> From: Govindraj.R<govindraj.raja@ti.com>
>> Date: Tue, 8 Nov 2011 16:57:22 +0530
>> Subject: [PATCH] OMAP2+: Fix Compilation error on current mainline
>>
>> Fix below compilation failure on mainline kernel commit id
>> 1ea6b8f48918282bdca0b32a34095504ee65bab5
>>
>> arch/arm/mach-omap2/omap_l3_noc.c:240: error: expected ',' or ';'
>> before 'MODULE_DEVICE_TABLE'
>
> The issue is that 3.2-rc1 does not have that build error anymore...
> It should appear only with this driver built as a module, and that's not
> possible yet for the l3_noc.
>
> The fix is still good to have, but the changelog should probably be updated.
>
Have posted a updated patch.
http://www.mail-archive.com/linux-omap@vger.kernel.org/msg58415.html
--
Thanks,
Govindraj.R
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2011-11-09 12:19 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-07 12:38 [PATCH] OMAP2+: Fix Compilation break on current mainline Govindraj.R
2011-11-07 12:59 ` Shilimkar, Santosh
2011-11-07 14:02 ` Cousson, Benoit
2011-11-08 6:28 ` [PATCH] OMAP2+: Fix Compilation error " Govindraj.R
2011-11-08 6:33 ` Shilimkar, Santosh
2011-11-08 10:46 ` Cousson, Benoit
2011-11-08 10:54 ` Santosh Shilimkar
2011-11-08 11:59 ` Govindraj
2011-11-08 15:25 ` Cousson, Benoit
2011-11-09 12:18 ` Govindraj
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox