* [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver
@ 2009-06-22 23:25 Vikram Pandita
2009-08-10 16:39 ` Tony Lindgren
2009-08-11 12:57 ` [APPLIED] " Tony Lindgren
0 siblings, 2 replies; 6+ messages in thread
From: Vikram Pandita @ 2009-06-22 23:25 UTC (permalink / raw)
To: linux-omap; +Cc: Vikram Pandita
Pass irqflags to 8250 driver from zoom2 board file
Zoom2 has IRQF_TRIGGER_RISING requirement for the 8250 GPIO irq
This patch is dependent on 8250 driver changes getting accepted upstream:
Refer: http://patchwork.kernel.org/patch/31884/
Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
---
Warning:
could cause compilation break if 8250 patch is not integrated first
arch/arm/mach-omap2/board-zoom-debugboard.c | 2 ++
arch/arm/mach-omap2/serial.c | 2 +-
2 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-debugboard.c
index bac5c43..f546063 100644
--- a/arch/arm/mach-omap2/board-zoom-debugboard.c
+++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
@@ -12,6 +12,7 @@
#include <linux/gpio.h>
#include <linux/serial_8250.h>
#include <linux/smsc911x.h>
+#include <linux/interrupt.h>
#include <mach/gpmc.h>
@@ -84,6 +85,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
.mapbase = 0x10000000,
.irq = OMAP_GPIO_IRQ(102),
.flags = UPF_BOOT_AUTOCONF|UPF_IOREMAP|UPF_SHARE_IRQ,
+ .irqflags = IRQF_SHARED | IRQF_TRIGGER_RISING,
.iotype = UPIO_MEM,
.regshift = 1,
.uartclk = QUART_CLK,
diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
index b094c15..45d4884 100644
--- a/arch/arm/mach-omap2/serial.c
+++ b/arch/arm/mach-omap2/serial.c
@@ -429,7 +429,7 @@ static void omap_uart_idle_init(struct omap_uart_state *uart)
omap_ctrl_writew(v, uart->padconf);
}
- p->flags |= UPF_SHARE_IRQ;
+ p->irqflags |= IRQF_SHARED;
ret = request_irq(p->irq, omap_uart_interrupt, IRQF_SHARED,
"serial idle", (void *)uart);
WARN_ON(ret);
--
1.6.0.3.613.g9f8f13
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver
2009-06-22 23:25 [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver Vikram Pandita
@ 2009-08-10 16:39 ` Tony Lindgren
2009-08-10 16:44 ` Pandita, Vikram
2009-08-12 19:31 ` Pandita, Vikram
2009-08-11 12:57 ` [APPLIED] " Tony Lindgren
1 sibling, 2 replies; 6+ messages in thread
From: Tony Lindgren @ 2009-08-10 16:39 UTC (permalink / raw)
To: Vikram Pandita; +Cc: linux-omap
* Vikram Pandita <vikram.pandita@ti.com> [090623 02:29]:
> Pass irqflags to 8250 driver from zoom2 board file
> Zoom2 has IRQF_TRIGGER_RISING requirement for the 8250 GPIO irq
>
> This patch is dependent on 8250 driver changes getting accepted upstream:
> Refer: http://patchwork.kernel.org/patch/31884/
FYI, I'll archive this for now, please resubmit once the related changes
are integrated. Or if resubmitting the related changes, please submit this
too via the serial list.
Acked-by: Tony Lindgren <tony@atomide.com>
> Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
> ---
>
> Warning:
> could cause compilation break if 8250 patch is not integrated first
>
> arch/arm/mach-omap2/board-zoom-debugboard.c | 2 ++
> arch/arm/mach-omap2/serial.c | 2 +-
> 2 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-debugboard.c
> index bac5c43..f546063 100644
> --- a/arch/arm/mach-omap2/board-zoom-debugboard.c
> +++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
> @@ -12,6 +12,7 @@
> #include <linux/gpio.h>
> #include <linux/serial_8250.h>
> #include <linux/smsc911x.h>
> +#include <linux/interrupt.h>
>
> #include <mach/gpmc.h>
>
> @@ -84,6 +85,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
> .mapbase = 0x10000000,
> .irq = OMAP_GPIO_IRQ(102),
> .flags = UPF_BOOT_AUTOCONF|UPF_IOREMAP|UPF_SHARE_IRQ,
> + .irqflags = IRQF_SHARED | IRQF_TRIGGER_RISING,
> .iotype = UPIO_MEM,
> .regshift = 1,
> .uartclk = QUART_CLK,
> diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
> index b094c15..45d4884 100644
> --- a/arch/arm/mach-omap2/serial.c
> +++ b/arch/arm/mach-omap2/serial.c
> @@ -429,7 +429,7 @@ static void omap_uart_idle_init(struct omap_uart_state *uart)
> omap_ctrl_writew(v, uart->padconf);
> }
>
> - p->flags |= UPF_SHARE_IRQ;
> + p->irqflags |= IRQF_SHARED;
> ret = request_irq(p->irq, omap_uart_interrupt, IRQF_SHARED,
> "serial idle", (void *)uart);
> WARN_ON(ret);
> --
> 1.6.0.3.613.g9f8f13
>
> --
> 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] 6+ messages in thread
* RE: [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver
2009-08-10 16:39 ` Tony Lindgren
@ 2009-08-10 16:44 ` Pandita, Vikram
2009-08-11 6:44 ` Tony Lindgren
2009-08-12 19:31 ` Pandita, Vikram
1 sibling, 1 reply; 6+ messages in thread
From: Pandita, Vikram @ 2009-08-10 16:44 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap
>-----Original Message-----
>From: Tony Lindgren [mailto:tony@atomide.com]
>Sent: Monday, August 10, 2009 11:39 AM
>To: Pandita, Vikram
>Cc: linux-omap@vger.kernel.org
>Subject: Re: [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver
>
>* Vikram Pandita <vikram.pandita@ti.com> [090623 02:29]:
>> Pass irqflags to 8250 driver from zoom2 board file
>> Zoom2 has IRQF_TRIGGER_RISING requirement for the 8250 GPIO irq
>>
>> This patch is dependent on 8250 driver changes getting accepted upstream:
>> Refer: http://patchwork.kernel.org/patch/31884/
>
>FYI, I'll archive this for now, please resubmit once the related changes
>are integrated. Or if resubmitting the related changes, please submit this
>too via the serial list.
Looks like the dependency patch is already in linux-next tree at [1].
The dependency patch is already lined up stream.
So looks like you can line this irqflag change for zoom2.
[1] http://git.kernel.org/?p=linux/kernel/git/sfr/linux-next.git;a=commit;h=7053133124d5cdf207c1168c7a0c582a18e12ea7
>
>Acked-by: Tony Lindgren <tony@atomide.com>
>
>
>> Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
>> ---
>>
>> Warning:
>> could cause compilation break if 8250 patch is not integrated first
>>
>> arch/arm/mach-omap2/board-zoom-debugboard.c | 2 ++
>> arch/arm/mach-omap2/serial.c | 2 +-
>> 2 files changed, 3 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-
>debugboard.c
>> index bac5c43..f546063 100644
>> --- a/arch/arm/mach-omap2/board-zoom-debugboard.c
>> +++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
>> @@ -12,6 +12,7 @@
>> #include <linux/gpio.h>
>> #include <linux/serial_8250.h>
>> #include <linux/smsc911x.h>
>> +#include <linux/interrupt.h>
>>
>> #include <mach/gpmc.h>
>>
>> @@ -84,6 +85,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
>> .mapbase = 0x10000000,
>> .irq = OMAP_GPIO_IRQ(102),
>> .flags = UPF_BOOT_AUTOCONF|UPF_IOREMAP|UPF_SHARE_IRQ,
>> + .irqflags = IRQF_SHARED | IRQF_TRIGGER_RISING,
>> .iotype = UPIO_MEM,
>> .regshift = 1,
>> .uartclk = QUART_CLK,
>> diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
>> index b094c15..45d4884 100644
>> --- a/arch/arm/mach-omap2/serial.c
>> +++ b/arch/arm/mach-omap2/serial.c
>> @@ -429,7 +429,7 @@ static void omap_uart_idle_init(struct omap_uart_state *uart)
>> omap_ctrl_writew(v, uart->padconf);
>> }
>>
>> - p->flags |= UPF_SHARE_IRQ;
>> + p->irqflags |= IRQF_SHARED;
>> ret = request_irq(p->irq, omap_uart_interrupt, IRQF_SHARED,
>> "serial idle", (void *)uart);
>> WARN_ON(ret);
>> --
>> 1.6.0.3.613.g9f8f13
>>
>> --
>> 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] 6+ messages in thread
* Re: [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver
2009-08-10 16:44 ` Pandita, Vikram
@ 2009-08-11 6:44 ` Tony Lindgren
0 siblings, 0 replies; 6+ messages in thread
From: Tony Lindgren @ 2009-08-11 6:44 UTC (permalink / raw)
To: Pandita, Vikram; +Cc: linux-omap
* Pandita, Vikram <vikram.pandita@ti.com> [090810 23:20]:
>
>
> >-----Original Message-----
> >From: Tony Lindgren [mailto:tony@atomide.com]
> >Sent: Monday, August 10, 2009 11:39 AM
> >To: Pandita, Vikram
> >Cc: linux-omap@vger.kernel.org
> >Subject: Re: [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver
> >
> >* Vikram Pandita <vikram.pandita@ti.com> [090623 02:29]:
> >> Pass irqflags to 8250 driver from zoom2 board file
> >> Zoom2 has IRQF_TRIGGER_RISING requirement for the 8250 GPIO irq
> >>
> >> This patch is dependent on 8250 driver changes getting accepted upstream:
> >> Refer: http://patchwork.kernel.org/patch/31884/
> >
> >FYI, I'll archive this for now, please resubmit once the related changes
> >are integrated. Or if resubmitting the related changes, please submit this
> >too via the serial list.
>
> Looks like the dependency patch is already in linux-next tree at [1].
> The dependency patch is already lined up stream.
>
> So looks like you can line this irqflag change for zoom2.
OK, thanks, will queue to omap3-upstream queue.
Tony
>
> [1] http://git.kernel.org/?p=linux/kernel/git/sfr/linux-next.git;a=commit;h=7053133124d5cdf207c1168c7a0c582a18e12ea7
>
> >
> >Acked-by: Tony Lindgren <tony@atomide.com>
> >
> >
> >> Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
> >> ---
> >>
> >> Warning:
> >> could cause compilation break if 8250 patch is not integrated first
> >>
> >> arch/arm/mach-omap2/board-zoom-debugboard.c | 2 ++
> >> arch/arm/mach-omap2/serial.c | 2 +-
> >> 2 files changed, 3 insertions(+), 1 deletions(-)
> >>
> >> diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-
> >debugboard.c
> >> index bac5c43..f546063 100644
> >> --- a/arch/arm/mach-omap2/board-zoom-debugboard.c
> >> +++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
> >> @@ -12,6 +12,7 @@
> >> #include <linux/gpio.h>
> >> #include <linux/serial_8250.h>
> >> #include <linux/smsc911x.h>
> >> +#include <linux/interrupt.h>
> >>
> >> #include <mach/gpmc.h>
> >>
> >> @@ -84,6 +85,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
> >> .mapbase = 0x10000000,
> >> .irq = OMAP_GPIO_IRQ(102),
> >> .flags = UPF_BOOT_AUTOCONF|UPF_IOREMAP|UPF_SHARE_IRQ,
> >> + .irqflags = IRQF_SHARED | IRQF_TRIGGER_RISING,
> >> .iotype = UPIO_MEM,
> >> .regshift = 1,
> >> .uartclk = QUART_CLK,
> >> diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
> >> index b094c15..45d4884 100644
> >> --- a/arch/arm/mach-omap2/serial.c
> >> +++ b/arch/arm/mach-omap2/serial.c
> >> @@ -429,7 +429,7 @@ static void omap_uart_idle_init(struct omap_uart_state *uart)
> >> omap_ctrl_writew(v, uart->padconf);
> >> }
> >>
> >> - p->flags |= UPF_SHARE_IRQ;
> >> + p->irqflags |= IRQF_SHARED;
> >> ret = request_irq(p->irq, omap_uart_interrupt, IRQF_SHARED,
> >> "serial idle", (void *)uart);
> >> WARN_ON(ret);
> >> --
> >> 1.6.0.3.613.g9f8f13
> >>
> >> --
> >> 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] 6+ messages in thread
* [APPLIED] [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver
2009-06-22 23:25 [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver Vikram Pandita
2009-08-10 16:39 ` Tony Lindgren
@ 2009-08-11 12:57 ` Tony Lindgren
1 sibling, 0 replies; 6+ messages in thread
From: Tony Lindgren @ 2009-08-11 12:57 UTC (permalink / raw)
To: linux-omap
This patch has been applied to the linux-omap
by youw fwiendly patch wobot.
Branch in linux-omap: omap3-upstream
Initial commit ID (Likely to change): 577736be2190198b88c9f3868b75f8de3b98e3e6
PatchWorks
http://patchwork.kernel.org/patch/31888/
Git (Likely to change, and takes a while to get mirrored)
http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=577736be2190198b88c9f3868b75f8de3b98e3e6
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver
2009-08-10 16:39 ` Tony Lindgren
2009-08-10 16:44 ` Pandita, Vikram
@ 2009-08-12 19:31 ` Pandita, Vikram
1 sibling, 0 replies; 6+ messages in thread
From: Pandita, Vikram @ 2009-08-12 19:31 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap
tony
>-----Original Message-----
>From: Pandita, Vikram
>>-----Original Message-----
>>From: Tony Lindgren [mailto:tony@atomide.com]
>>Subject: Re: [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver
>>
>>* Vikram Pandita <vikram.pandita@ti.com> [090623 02:29]:
>>> Pass irqflags to 8250 driver from zoom2 board file
>>> Zoom2 has IRQF_TRIGGER_RISING requirement for the 8250 GPIO irq
>>>
>>> This patch is dependent on 8250 driver changes getting accepted upstream:
>>> Refer: http://patchwork.kernel.org/patch/31884/
>>
>>FYI, I'll archive this for now, please resubmit once the related changes
>>are integrated. Or if resubmitting the related changes, please submit this
>>too via the serial list.
>
>Looks like the dependency patch is already in linux-next tree at [1].
>The dependency patch is already lined up stream.
>
>So looks like you can line this irqflag change for zoom2.
>
>[1] http://git.kernel.org/?p=linux/kernel/git/sfr/linux-
>next.git;a=commit;h=7053133124d5cdf207c1168c7a0c582a18e12ea7
>
>>
>>Acked-by: Tony Lindgren <tony@atomide.com>
Would you be able to line this zoom2 patch as well in your omap3-boards branch?
This patch is missing from your branch.
>>
>>
>>> Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
>>> ---
>>>
>>> Warning:
>>> could cause compilation break if 8250 patch is not integrated first
>>>
>>> arch/arm/mach-omap2/board-zoom-debugboard.c | 2 ++
>>> arch/arm/mach-omap2/serial.c | 2 +-
>>> 2 files changed, 3 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-
>>debugboard.c
>>> index bac5c43..f546063 100644
>>> --- a/arch/arm/mach-omap2/board-zoom-debugboard.c
>>> +++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
>>> @@ -12,6 +12,7 @@
>>> #include <linux/gpio.h>
>>> #include <linux/serial_8250.h>
>>> #include <linux/smsc911x.h>
>>> +#include <linux/interrupt.h>
>>>
>>> #include <mach/gpmc.h>
>>>
>>> @@ -84,6 +85,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
>>> .mapbase = 0x10000000,
>>> .irq = OMAP_GPIO_IRQ(102),
>>> .flags = UPF_BOOT_AUTOCONF|UPF_IOREMAP|UPF_SHARE_IRQ,
>>> + .irqflags = IRQF_SHARED | IRQF_TRIGGER_RISING,
>>> .iotype = UPIO_MEM,
>>> .regshift = 1,
>>> .uartclk = QUART_CLK,
>>> diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
>>> index b094c15..45d4884 100644
>>> --- a/arch/arm/mach-omap2/serial.c
>>> +++ b/arch/arm/mach-omap2/serial.c
>>> @@ -429,7 +429,7 @@ static void omap_uart_idle_init(struct omap_uart_state *uart)
>>> omap_ctrl_writew(v, uart->padconf);
>>> }
>>>
>>> - p->flags |= UPF_SHARE_IRQ;
>>> + p->irqflags |= IRQF_SHARED;
>>> ret = request_irq(p->irq, omap_uart_interrupt, IRQF_SHARED,
>>> "serial idle", (void *)uart);
>>> WARN_ON(ret);
>>> --
>>> 1.6.0.3.613.g9f8f13
>>>
>>> --
>>> 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] 6+ messages in thread
end of thread, other threads:[~2009-08-12 19:31 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-06-22 23:25 [PATCH v2] OMAP2: Zoom2: Pass irqflags to 8250 driver Vikram Pandita
2009-08-10 16:39 ` Tony Lindgren
2009-08-10 16:44 ` Pandita, Vikram
2009-08-11 6:44 ` Tony Lindgren
2009-08-12 19:31 ` Pandita, Vikram
2009-08-11 12:57 ` [APPLIED] " Tony Lindgren
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.