[PATCH 0/2] Add irq extra flags to 8250 serial driver

[PATCH 0/2] Add irq extra flags to 8250 serial driver

[Bryan Wu]

Hi Alan,

Thanks Javier for updating this patch according to you review.
Do you think it is ok? If yes, I'd like to add these patches to
Blackfin git-tree and will invite linus to pull later.

-Bryan

[PATCH 1/2] [8250 Serial Driver]: Added support for 8250-class UARTs in HV Sistemas H8606 board

[Bryan Wu]

From: Javier Herrero <jherrero@hvsistemas.es>

Added support for 8250-class UARTs in HV Sistemas H8606 board,
modification in 8250.c driver for correct compilation with Blackfin

Besides, I think that there is more people using 8250-class UARTs
with a different hardware than the H8606 board. This code can be shared
by them.

Signed-off-by: Javier Herrero <jherrero@hvsistemas.es>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Signed-off-by: Bryan Wu <cooloney@kernel.org>
---
 drivers/serial/8250.c |    5 +++--
 drivers/serial/8250.h |    5 +++++
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/drivers/serial/8250.c b/drivers/serial/8250.c
index a1ca9b7..1400ea6 100644
--- a/drivers/serial/8250.c
+++ b/drivers/serial/8250.c
@@ -43,6 +43,7 @@
 
 #include <asm/io.h>
 #include <asm/irq.h>
+#include <asm/serial.h>
 
 #include "8250.h"
 
@@ -92,8 +93,6 @@ static unsigned int nr_uarts = CONFIG_SERIAL_8250_RUNTIME_UARTS;
  */
 #define CONFIG_HUB6 1
 
-#include <asm/serial.h>
-
 /*
  * SERIAL_PORT_DFNS tells us about built-in ports that have no
  * standard enumeration mechanism.   Platforms that can find all
@@ -1548,6 +1547,8 @@ static int serial_link_irq_chain(struct uart_8250_port *up)
 		i->head = &up->list;
 		spin_unlock_irq(&i->lock);
 
+		irq_flags |= SERIAL_EXTRA_IRQ_FLAGS;
+
 		ret = request_irq(up->port.irq, serial8250_interrupt,
 				  irq_flags, "serial", i);
 		if (ret < 0)
diff --git a/drivers/serial/8250.h b/drivers/serial/8250.h
index 91bd28f..a10a40c 100644
--- a/drivers/serial/8250.h
+++ b/drivers/serial/8250.h
@@ -78,3 +78,8 @@ struct serial8250_config {
 #else
 #define ALPHA_KLUDGE_MCR 0
 #endif
+
+#ifndef SERIAL_EXTRA_IRQ_FLAGS
+#define SERIAL_EXTRA_IRQ_FLAGS 0
+#endif
+
-- 
1.5.5

[PATCH 2/2] [Blackfin] serial driver: add extra IRQ flag for 8250 serial driver

[Bryan Wu]

From: Javier Herrero <jherrero@hvsistemas.es>

Signed-off-by: Javier Herrero <jherrero@hvsistemas.es>
Signed-off-by: Bryan Wu <cooloney@kernel.org>
---
 include/asm-blackfin/serial.h |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
 create mode 100644 include/asm-blackfin/serial.h

diff --git a/include/asm-blackfin/serial.h b/include/asm-blackfin/serial.h
new file mode 100644
index 0000000..994dd86
--- /dev/null
+++ b/include/asm-blackfin/serial.h
@@ -0,0 +1,5 @@
+/*
+ * include/asm-blackfin/serial.h
+ */
+
+#define SERIAL_EXTRA_IRQ_FLAGS IRQF_TRIGGER_HIGH
-- 
1.5.5

Re: [PATCH 1/2] [8250 Serial Driver]: Added support for 8250-class UARTs in HV Sistemas H8606 board

[Alan Cox]

On Tue, 13 May 2008 12:14:28 +0800
Bryan Wu <cooloney@kernel.org> wrote:

> From: Javier Herrero <jherrero@hvsistemas.es>
> 
> Added support for 8250-class UARTs in HV Sistemas H8606 board,
> modification in 8250.c driver for correct compilation with Blackfin
> 
> Besides, I think that there is more people using 8250-class UARTs
> with a different hardware than the H8606 board. This code can be shared
> by them.
> 
> Signed-off-by: Javier Herrero <jherrero@hvsistemas.es>
> Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
> Signed-off-by: Bryan Wu <cooloney@kernel.org>

Acked-by: Alan Cox <alan@redhat.com>

Re: [PATCH 1/2] [8250 Serial Driver]: Added support for 8250-class UARTs in HV Sistemas H8606 board

[Bryan Wu]

On Tue, May 13, 2008 at 5:50 PM, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
> On Tue, 13 May 2008 12:14:28 +0800
>  Bryan Wu <cooloney@kernel.org> wrote:
>
>  > From: Javier Herrero <jherrero@hvsistemas.es>
>  >
>  > Added support for 8250-class UARTs in HV Sistemas H8606 board,
>  > modification in 8250.c driver for correct compilation with Blackfin
>  >
>  > Besides, I think that there is more people using 8250-class UARTs
>  > with a different hardware than the H8606 board. This code can be shared
>  > by them.
>  >
>  > Signed-off-by: Javier Herrero <jherrero@hvsistemas.es>
>  > Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
>  > Signed-off-by: Bryan Wu <cooloney@kernel.org>
>
>  Acked-by: Alan Cox <alan@redhat.com>

Thanks, I will apply this patch to Blackfin git-tree

-Bryan