All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH]arch:avr32:kernel:time.c: Removed IRQF_DISABLED
@ 2013-08-09  3:29 ` Kumar Gaurav
  0 siblings, 0 replies; 6+ messages in thread
From: Kumar Gaurav @ 2013-08-09  3:17 UTC (permalink / raw)
  To: hskinnemoen, egtvedt, tglx, srivatsa.bhat
  Cc: linux-kernel, kernel-janitors, Kumar Gaurav

Removed IRQF_DISABLED as it's no-op and should be removed

Signed-off-by: Kumar Gaurav <kumargauravgupta3@gmail.com>
---
 arch/avr32/kernel/time.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/avr32/kernel/time.c b/arch/avr32/kernel/time.c
index 869a1c6..0566c81 100644
--- a/arch/avr32/kernel/time.c
+++ b/arch/avr32/kernel/time.c
@@ -59,7 +59,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
 static struct irqaction timer_irqaction = {
 	.handler	= timer_interrupt,
 	/* Oprofile uses the same irq as the timer, so allow it to be shared */
-	.flags		= IRQF_TIMER | IRQF_DISABLED | IRQF_SHARED,
+	.flags		= IRQF_TIMER | IRQF_TRIGGER_NONE | IRQF_SHARED,
 	.name		= "avr32_comparator",
 };
 
-- 
1.7.9.5


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

* Re: [PATCH]arch:avr32:kernel:time.c: Removed IRQF_DISABLED
  2013-08-09  3:29 ` Kumar Gaurav
@ 2013-08-09  3:31   ` Kumar Gaurav
  -1 siblings, 0 replies; 6+ messages in thread
From: Kumar Gaurav @ 2013-08-09  3:19 UTC (permalink / raw)
  To: Kumar Gaurav
  Cc: hskinnemoen, egtvedt, tglx, srivatsa.bhat, linux-kernel,
	kernel-janitors

Please ignore the below patch. I'll be resending this with 0 instead of 
IRQF_TRIGGER_NONE
On Friday 09 August 2013 08:47 AM, Kumar Gaurav wrote:
> Removed IRQF_DISABLED as it's no-op and should be removed
>
> Signed-off-by: Kumar Gaurav <kumargauravgupta3@gmail.com>
> ---
>   arch/avr32/kernel/time.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/avr32/kernel/time.c b/arch/avr32/kernel/time.c
> index 869a1c6..0566c81 100644
> --- a/arch/avr32/kernel/time.c
> +++ b/arch/avr32/kernel/time.c
> @@ -59,7 +59,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
>   static struct irqaction timer_irqaction = {
>   	.handler	= timer_interrupt,
>   	/* Oprofile uses the same irq as the timer, so allow it to be shared */
> -	.flags		= IRQF_TIMER | IRQF_DISABLED | IRQF_SHARED,
> +	.flags		= IRQF_TIMER | IRQF_TRIGGER_NONE | IRQF_SHARED,
>   	.name		= "avr32_comparator",
>   };
>   


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

* [PATCH]arch:avr32:kernel:time.c: Removed IRQF_DISABLED
@ 2013-08-09  3:29 ` Kumar Gaurav
  0 siblings, 0 replies; 6+ messages in thread
From: Kumar Gaurav @ 2013-08-09  3:29 UTC (permalink / raw)
  To: hskinnemoen, egtvedt, tglx, srivatsa.bhat
  Cc: linux-kernel, kernel-janitors, Kumar Gaurav

Removed IRQF_DISABLED as it's no-op and should be removed

Signed-off-by: Kumar Gaurav <kumargauravgupta3@gmail.com>
---
 arch/avr32/kernel/time.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/avr32/kernel/time.c b/arch/avr32/kernel/time.c
index 869a1c6..0566c81 100644
--- a/arch/avr32/kernel/time.c
+++ b/arch/avr32/kernel/time.c
@@ -59,7 +59,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
 static struct irqaction timer_irqaction = {
 	.handler	= timer_interrupt,
 	/* Oprofile uses the same irq as the timer, so allow it to be shared */
-	.flags		= IRQF_TIMER | IRQF_DISABLED | IRQF_SHARED,
+	.flags		= IRQF_TIMER | IRQF_TRIGGER_NONE | IRQF_SHARED,
 	.name		= "avr32_comparator",
 };
 
-- 
1.7.9.5


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

* Re: [PATCH]arch:avr32:kernel:time.c: Removed IRQF_DISABLED
  2013-08-09  3:31   ` Kumar Gaurav
@ 2013-08-09  3:30     ` Joe Perches
  -1 siblings, 0 replies; 6+ messages in thread
From: Joe Perches @ 2013-08-09  3:30 UTC (permalink / raw)
  To: Kumar Gaurav
  Cc: hskinnemoen, egtvedt, tglx, srivatsa.bhat, linux-kernel,
	kernel-janitors

On Fri, 2013-08-09 at 08:49 +0530, Kumar Gaurav wrote:
> Please ignore the below patch. I'll be resending this with 0 instead of 
> IRQF_TRIGGER_NONE

Hey again Kumar.

This is one of those patches that should just have
IRQF_DISABLED completely elided.

> On Friday 09 August 2013 08:47 AM, Kumar Gaurav wrote:
[]
> > diff --git a/arch/avr32/kernel/time.c b/arch/avr32/kernel/time.c
[]
> > @@ -59,7 +59,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
[]
> > -	.flags		= IRQF_TIMER | IRQF_DISABLED | IRQF_SHARED,
> > +	.flags		= IRQF_TIMER | IRQF_TRIGGER_NONE | IRQF_SHARED,

	.flags		= IRQF_TIMER | IRQF_SHARED,



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

* Re: [PATCH]arch:avr32:kernel:time.c: Removed IRQF_DISABLED
@ 2013-08-09  3:30     ` Joe Perches
  0 siblings, 0 replies; 6+ messages in thread
From: Joe Perches @ 2013-08-09  3:30 UTC (permalink / raw)
  To: Kumar Gaurav
  Cc: hskinnemoen, egtvedt, tglx, srivatsa.bhat, linux-kernel,
	kernel-janitors

On Fri, 2013-08-09 at 08:49 +0530, Kumar Gaurav wrote:
> Please ignore the below patch. I'll be resending this with 0 instead of 
> IRQF_TRIGGER_NONE

Hey again Kumar.

This is one of those patches that should just have
IRQF_DISABLED completely elided.

> On Friday 09 August 2013 08:47 AM, Kumar Gaurav wrote:
[]
> > diff --git a/arch/avr32/kernel/time.c b/arch/avr32/kernel/time.c
[]
> > @@ -59,7 +59,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
[]
> > -	.flags		= IRQF_TIMER | IRQF_DISABLED | IRQF_SHARED,
> > +	.flags		= IRQF_TIMER | IRQF_TRIGGER_NONE | IRQF_SHARED,

	.flags		= IRQF_TIMER | IRQF_SHARED,



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

* Re: [PATCH]arch:avr32:kernel:time.c: Removed IRQF_DISABLED
@ 2013-08-09  3:31   ` Kumar Gaurav
  0 siblings, 0 replies; 6+ messages in thread
From: Kumar Gaurav @ 2013-08-09  3:31 UTC (permalink / raw)
  To: Kumar Gaurav
  Cc: hskinnemoen, egtvedt, tglx, srivatsa.bhat, linux-kernel,
	kernel-janitors

Please ignore the below patch. I'll be resending this with 0 instead of 
IRQF_TRIGGER_NONE
On Friday 09 August 2013 08:47 AM, Kumar Gaurav wrote:
> Removed IRQF_DISABLED as it's no-op and should be removed
>
> Signed-off-by: Kumar Gaurav <kumargauravgupta3@gmail.com>
> ---
>   arch/avr32/kernel/time.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/avr32/kernel/time.c b/arch/avr32/kernel/time.c
> index 869a1c6..0566c81 100644
> --- a/arch/avr32/kernel/time.c
> +++ b/arch/avr32/kernel/time.c
> @@ -59,7 +59,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
>   static struct irqaction timer_irqaction = {
>   	.handler	= timer_interrupt,
>   	/* Oprofile uses the same irq as the timer, so allow it to be shared */
> -	.flags		= IRQF_TIMER | IRQF_DISABLED | IRQF_SHARED,
> +	.flags		= IRQF_TIMER | IRQF_TRIGGER_NONE | IRQF_SHARED,
>   	.name		= "avr32_comparator",
>   };
>   


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

end of thread, other threads:[~2013-08-09  3:31 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-09  3:17 [PATCH]arch:avr32:kernel:time.c: Removed IRQF_DISABLED Kumar Gaurav
2013-08-09  3:29 ` Kumar Gaurav
2013-08-09  3:19 ` Kumar Gaurav
2013-08-09  3:31   ` Kumar Gaurav
2013-08-09  3:30   ` Joe Perches
2013-08-09  3:30     ` Joe Perches

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.