All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
@ 2011-05-26  9:35 ` Jesper Juhl
  0 siblings, 0 replies; 16+ messages in thread
From: Jesper Juhl @ 2011-05-26  9:35 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Andrew Lunn, trivial, linux-kernel, Russell King, Nicolas Pitre

Remove duplicate linux/serial_8250.h include from 
arch/arm/mach-dove/common.c that was introduced by commit 
5c60255149eece2a36ec9f5c99817b85f96fe8ec .

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
---
 common.c |    1 -
 1 file changed, 1 deletion(-)

 note: untested patch since I have no way to really build arm stuff.
       patch is against Linus' tree.

diff --git a/arch/arm/mach-dove/common.c b/arch/arm/mach-dove/common.c
index 5ed51b8..b6c3d6b 100644
--- a/arch/arm/mach-dove/common.c
+++ b/arch/arm/mach-dove/common.c
@@ -17,7 +17,6 @@
 #include <linux/clk.h>
 #include <linux/mbus.h>
 #include <linux/ata_platform.h>
-#include <linux/serial_8250.h>
 #include <linux/gpio.h>
 #include <asm/page.h>
 #include <asm/setup.h>


-- 
Jesper Juhl <jj@chaosbits.net>       http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.


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

* [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
@ 2011-05-26  9:35 ` Jesper Juhl
  0 siblings, 0 replies; 16+ messages in thread
From: Jesper Juhl @ 2011-05-26  9:35 UTC (permalink / raw)
  To: linux-arm-kernel

Remove duplicate linux/serial_8250.h include from 
arch/arm/mach-dove/common.c that was introduced by commit 
5c60255149eece2a36ec9f5c99817b85f96fe8ec .

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
---
 common.c |    1 -
 1 file changed, 1 deletion(-)

 note: untested patch since I have no way to really build arm stuff.
       patch is against Linus' tree.

diff --git a/arch/arm/mach-dove/common.c b/arch/arm/mach-dove/common.c
index 5ed51b8..b6c3d6b 100644
--- a/arch/arm/mach-dove/common.c
+++ b/arch/arm/mach-dove/common.c
@@ -17,7 +17,6 @@
 #include <linux/clk.h>
 #include <linux/mbus.h>
 #include <linux/ata_platform.h>
-#include <linux/serial_8250.h>
 #include <linux/gpio.h>
 #include <asm/page.h>
 #include <asm/setup.h>


-- 
Jesper Juhl <jj@chaosbits.net>       http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.

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

* Re: [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
  2011-05-26  9:35 ` Jesper Juhl
@ 2011-05-26 14:48   ` Andrew Lunn
  -1 siblings, 0 replies; 16+ messages in thread
From: Andrew Lunn @ 2011-05-26 14:48 UTC (permalink / raw)
  To: Jesper Juhl
  Cc: linux-arm-kernel, Andrew Lunn, trivial, linux-kernel,
	Russell King, Nicolas Pitre

Jesper Juhl pointed out there is a redundant include of
linux/serial_8250.h. However it turns out both are redundant.
This patch removes them both.

Only compile tested.

Reported-by:  Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
---
 arch/arm/mach-dove/common.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-dove/common.c b/arch/arm/mach-dove/common.c
index 5ed51b8..83dce85 100644
--- a/arch/arm/mach-dove/common.c
+++ b/arch/arm/mach-dove/common.c
@@ -13,11 +13,9 @@
 #include <linux/init.h>
 #include <linux/platform_device.h>
 #include <linux/pci.h>
-#include <linux/serial_8250.h>
 #include <linux/clk.h>
 #include <linux/mbus.h>
 #include <linux/ata_platform.h>
-#include <linux/serial_8250.h>
 #include <linux/gpio.h>
 #include <asm/page.h>
 #include <asm/setup.h>
-- 
1.7.4.4


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

* [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
@ 2011-05-26 14:48   ` Andrew Lunn
  0 siblings, 0 replies; 16+ messages in thread
From: Andrew Lunn @ 2011-05-26 14:48 UTC (permalink / raw)
  To: linux-arm-kernel

Jesper Juhl pointed out there is a redundant include of
linux/serial_8250.h. However it turns out both are redundant.
This patch removes them both.

Only compile tested.

Reported-by:  Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
---
 arch/arm/mach-dove/common.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-dove/common.c b/arch/arm/mach-dove/common.c
index 5ed51b8..83dce85 100644
--- a/arch/arm/mach-dove/common.c
+++ b/arch/arm/mach-dove/common.c
@@ -13,11 +13,9 @@
 #include <linux/init.h>
 #include <linux/platform_device.h>
 #include <linux/pci.h>
-#include <linux/serial_8250.h>
 #include <linux/clk.h>
 #include <linux/mbus.h>
 #include <linux/ata_platform.h>
-#include <linux/serial_8250.h>
 #include <linux/gpio.h>
 #include <asm/page.h>
 #include <asm/setup.h>
-- 
1.7.4.4

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

* Re: [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
  2011-05-26  9:35 ` Jesper Juhl
@ 2011-05-27 15:36   ` Sergei Shtylyov
  -1 siblings, 0 replies; 16+ messages in thread
From: Sergei Shtylyov @ 2011-05-27 15:36 UTC (permalink / raw)
  To: Jesper Juhl
  Cc: linux-arm-kernel, Andrew Lunn, Nicolas Pitre, trivial,
	linux-kernel, Russell King

Hello.

Jesper Juhl wrote:

> Remove duplicate linux/serial_8250.h include from 
> arch/arm/mach-dove/common.c that was introduced by commit 
> 5c60255149eece2a36ec9f5c99817b85f96fe8ec .

   Please also specify that commit's summary -- for the human readers.

> Signed-off-by: Jesper Juhl <jj@chaosbits.net>

WBR, Sergei

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

* [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
@ 2011-05-27 15:36   ` Sergei Shtylyov
  0 siblings, 0 replies; 16+ messages in thread
From: Sergei Shtylyov @ 2011-05-27 15:36 UTC (permalink / raw)
  To: linux-arm-kernel

Hello.

Jesper Juhl wrote:

> Remove duplicate linux/serial_8250.h include from 
> arch/arm/mach-dove/common.c that was introduced by commit 
> 5c60255149eece2a36ec9f5c99817b85f96fe8ec .

   Please also specify that commit's summary -- for the human readers.

> Signed-off-by: Jesper Juhl <jj@chaosbits.net>

WBR, Sergei

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

* Re: [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
  2011-05-26 14:48   ` Andrew Lunn
@ 2011-06-16 21:51     ` Jesper Juhl
  -1 siblings, 0 replies; 16+ messages in thread
From: Jesper Juhl @ 2011-06-16 21:51 UTC (permalink / raw)
  To: Andrew Lunn
  Cc: linux-arm-kernel, trivial, linux-kernel, Russell King, Nicolas Pitre

On Thu, 26 May 2011, Andrew Lunn wrote:

> Jesper Juhl pointed out there is a redundant include of
> linux/serial_8250.h. However it turns out both are redundant.
> This patch removes them both.
> 
> Only compile tested.
> 
> Reported-by:  Jesper Juhl <jj@chaosbits.net>
> Signed-off-by: Andrew Lunn <andrew@lunn.ch>

Seems you are right.

Reviewed-by: Jesper Juhl <jj@chaosbits.net>

-- 
Jesper Juhl <jj@chaosbits.net>       http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.


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

* [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
@ 2011-06-16 21:51     ` Jesper Juhl
  0 siblings, 0 replies; 16+ messages in thread
From: Jesper Juhl @ 2011-06-16 21:51 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, 26 May 2011, Andrew Lunn wrote:

> Jesper Juhl pointed out there is a redundant include of
> linux/serial_8250.h. However it turns out both are redundant.
> This patch removes them both.
> 
> Only compile tested.
> 
> Reported-by:  Jesper Juhl <jj@chaosbits.net>
> Signed-off-by: Andrew Lunn <andrew@lunn.ch>

Seems you are right.

Reviewed-by: Jesper Juhl <jj@chaosbits.net>

-- 
Jesper Juhl <jj@chaosbits.net>       http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.

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

* Re: [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
  2011-06-16 21:51     ` Jesper Juhl
@ 2011-06-17 10:01       ` Jiri Kosina
  -1 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2011-06-17 10:01 UTC (permalink / raw)
  To: Jesper Juhl
  Cc: Andrew Lunn, linux-arm-kernel, linux-kernel, Russell King, Nicolas Pitre

On Thu, 16 Jun 2011, Jesper Juhl wrote:

> > Jesper Juhl pointed out there is a redundant include of
> > linux/serial_8250.h. However it turns out both are redundant.
> > This patch removes them both.
> > 
> > Only compile tested.
> > 
> > Reported-by:  Jesper Juhl <jj@chaosbits.net>
> > Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> 
> Seems you are right.
> 
> Reviewed-by: Jesper Juhl <jj@chaosbits.net>

I don't seem to be able to find the patch anywhere during a quick search 
of the archives. Could you please bounce it to me (in case it hasn't been 
already applied elsewhere)?

Thanks,

-- 
Jiri Kosina
SUSE Labs

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

* [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
@ 2011-06-17 10:01       ` Jiri Kosina
  0 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2011-06-17 10:01 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, 16 Jun 2011, Jesper Juhl wrote:

> > Jesper Juhl pointed out there is a redundant include of
> > linux/serial_8250.h. However it turns out both are redundant.
> > This patch removes them both.
> > 
> > Only compile tested.
> > 
> > Reported-by:  Jesper Juhl <jj@chaosbits.net>
> > Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> 
> Seems you are right.
> 
> Reviewed-by: Jesper Juhl <jj@chaosbits.net>

I don't seem to be able to find the patch anywhere during a quick search 
of the archives. Could you please bounce it to me (in case it hasn't been 
already applied elsewhere)?

Thanks,

-- 
Jiri Kosina
SUSE Labs

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

* Re: [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
  2011-06-16 21:51     ` Jesper Juhl
@ 2011-06-17 13:14       ` Jiri Kosina
  -1 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2011-06-17 13:14 UTC (permalink / raw)
  To: Jesper Juhl
  Cc: Andrew Lunn, linux-arm-kernel, linux-kernel, Russell King, Nicolas Pitre

On Thu, 16 Jun 2011, Jesper Juhl wrote:

> On Thu, 26 May 2011, Andrew Lunn wrote:
> 
> > Jesper Juhl pointed out there is a redundant include of
> > linux/serial_8250.h. However it turns out both are redundant.
> > This patch removes them both.
> > 
> > Only compile tested.
> > 
> > Reported-by:  Jesper Juhl <jj@chaosbits.net>
> > Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> 
> Seems you are right.
> 
> Reviewed-by: Jesper Juhl <jj@chaosbits.net>

Doesn't seem to be in linux-next as of today. I have picked it up.

-- 
Jiri Kosina
SUSE Labs

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

* [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
@ 2011-06-17 13:14       ` Jiri Kosina
  0 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2011-06-17 13:14 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, 16 Jun 2011, Jesper Juhl wrote:

> On Thu, 26 May 2011, Andrew Lunn wrote:
> 
> > Jesper Juhl pointed out there is a redundant include of
> > linux/serial_8250.h. However it turns out both are redundant.
> > This patch removes them both.
> > 
> > Only compile tested.
> > 
> > Reported-by:  Jesper Juhl <jj@chaosbits.net>
> > Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> 
> Seems you are right.
> 
> Reviewed-by: Jesper Juhl <jj@chaosbits.net>

Doesn't seem to be in linux-next as of today. I have picked it up.

-- 
Jiri Kosina
SUSE Labs

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

* Re: [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
  2011-06-16 21:51     ` Jesper Juhl
@ 2011-06-20  3:20       ` Nicolas Pitre
  -1 siblings, 0 replies; 16+ messages in thread
From: Nicolas Pitre @ 2011-06-20  3:20 UTC (permalink / raw)
  To: Jesper Juhl; +Cc: Andrew Lunn, linux-arm-kernel, trivial, lkml, Russell King

On Thu, 16 Jun 2011, Jesper Juhl wrote:

> On Thu, 26 May 2011, Andrew Lunn wrote:
> 
> > Jesper Juhl pointed out there is a redundant include of
> > linux/serial_8250.h. However it turns out both are redundant.
> > This patch removes them both.
> > 
> > Only compile tested.
> > 
> > Reported-by:  Jesper Juhl <jj@chaosbits.net>
> > Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> 
> Seems you are right.
> 
> Reviewed-by: Jesper Juhl <jj@chaosbits.net>

Could this patch be sent to RMK's patch system if no one else 
picks it up please?
Thanks.


Nicolas

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

* [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
@ 2011-06-20  3:20       ` Nicolas Pitre
  0 siblings, 0 replies; 16+ messages in thread
From: Nicolas Pitre @ 2011-06-20  3:20 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, 16 Jun 2011, Jesper Juhl wrote:

> On Thu, 26 May 2011, Andrew Lunn wrote:
> 
> > Jesper Juhl pointed out there is a redundant include of
> > linux/serial_8250.h. However it turns out both are redundant.
> > This patch removes them both.
> > 
> > Only compile tested.
> > 
> > Reported-by:  Jesper Juhl <jj@chaosbits.net>
> > Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> 
> Seems you are right.
> 
> Reviewed-by: Jesper Juhl <jj@chaosbits.net>

Could this patch be sent to RMK's patch system if no one else 
picks it up please?
Thanks.


Nicolas

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

* Re: [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
  2011-06-20  3:20       ` Nicolas Pitre
@ 2011-06-20  8:40         ` Jiri Kosina
  -1 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2011-06-20  8:40 UTC (permalink / raw)
  To: Nicolas Pitre
  Cc: Jesper Juhl, Andrew Lunn, linux-arm-kernel, lkml, Russell King

On Sun, 19 Jun 2011, Nicolas Pitre wrote:

> > > Jesper Juhl pointed out there is a redundant include of
> > > linux/serial_8250.h. However it turns out both are redundant.
> > > This patch removes them both.
> > > 
> > > Only compile tested.
> > > 
> > > Reported-by:  Jesper Juhl <jj@chaosbits.net>
> > > Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> > 
> > Seems you are right.
> > 
> > Reviewed-by: Jesper Juhl <jj@chaosbits.net>
> 
> Could this patch be sent to RMK's patch system if no one else 
> picks it up please?

I have picked it up last week.

-- 
Jiri Kosina
SUSE Labs

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

* [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c
@ 2011-06-20  8:40         ` Jiri Kosina
  0 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2011-06-20  8:40 UTC (permalink / raw)
  To: linux-arm-kernel

On Sun, 19 Jun 2011, Nicolas Pitre wrote:

> > > Jesper Juhl pointed out there is a redundant include of
> > > linux/serial_8250.h. However it turns out both are redundant.
> > > This patch removes them both.
> > > 
> > > Only compile tested.
> > > 
> > > Reported-by:  Jesper Juhl <jj@chaosbits.net>
> > > Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> > 
> > Seems you are right.
> > 
> > Reviewed-by: Jesper Juhl <jj@chaosbits.net>
> 
> Could this patch be sent to RMK's patch system if no one else 
> picks it up please?

I have picked it up last week.

-- 
Jiri Kosina
SUSE Labs

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

end of thread, other threads:[~2011-06-20  8:41 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-26  9:35 [PATCH][Trivial] ARM: remove duplicate include from arch/arm/mach-dove/common.c Jesper Juhl
2011-05-26  9:35 ` Jesper Juhl
2011-05-26 14:48 ` Andrew Lunn
2011-05-26 14:48   ` Andrew Lunn
2011-06-16 21:51   ` Jesper Juhl
2011-06-16 21:51     ` Jesper Juhl
2011-06-17 10:01     ` Jiri Kosina
2011-06-17 10:01       ` Jiri Kosina
2011-06-17 13:14     ` Jiri Kosina
2011-06-17 13:14       ` Jiri Kosina
2011-06-20  3:20     ` Nicolas Pitre
2011-06-20  3:20       ` Nicolas Pitre
2011-06-20  8:40       ` Jiri Kosina
2011-06-20  8:40         ` Jiri Kosina
2011-05-27 15:36 ` Sergei Shtylyov
2011-05-27 15:36   ` Sergei Shtylyov

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.