[PATCH] ARM: imx: drop support for irq priorisation

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Mon Nov 4 04:48:10 EST 2013


Hello Shawn,

I didn't add you to the list of recipents for this patch after
consulting the MAINTAINERS file. Sascha now told me that it's probably
you who takes this patch?! I suggest to fix MAINTAINERS accordingly.
Maybe just merge the entries "ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
and "ARM/FREESCALE IMX6"?

Best regards
Uwe

On Mon, Nov 04, 2013 at 09:41:29AM +0100, Sascha Hauer wrote:
> On Mon, Nov 04, 2013 at 09:17:02AM +0100, Uwe Kleine-König wrote:
> > The code for irq priorisation support doesn't have any in-tree users and
> > the Kconfig description does wrong promises because nowadays irq
> > handlers are called with irqs disabled, so no high prioritized irq can
> > interrupt a lower prioritized handler.
> > 
> > Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> 
> Acked-by: Sascha Hauer <s.hauer at pengutronix.de>
> 
> Sascha
> 
> > ---
> >  arch/arm/mach-imx/Kconfig      | 10 ----------
> >  arch/arm/mach-imx/avic.c       | 25 -------------------------
> >  arch/arm/mach-imx/irq-common.h |  1 -
> >  3 files changed, 36 deletions(-)
> > 
> > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> > index 15c9ae3..9431729 100644
> > --- a/arch/arm/mach-imx/Kconfig
> > +++ b/arch/arm/mach-imx/Kconfig
> > @@ -20,16 +20,6 @@ config ARCH_MXC
> >  menu "Freescale i.MX support"
> >  	depends on ARCH_MXC
> >  
> > -config MXC_IRQ_PRIOR
> > -	bool "Use IRQ priority"
> > -	help
> > -	  Select this if you want to use prioritized IRQ handling.
> > -	  This feature prevents higher priority ISR to be interrupted
> > -	  by lower priority IRQ.
> > -	  This may be useful in embedded applications, where are strong
> > -	  requirements for timing.
> > -	  Say N here, unless you have a specialized requirement.
> > -
> >  config MXC_TZIC
> >  	bool
> >  
> > diff --git a/arch/arm/mach-imx/avic.c b/arch/arm/mach-imx/avic.c
> > index e163ec7..8d1df2e 100644
> > --- a/arch/arm/mach-imx/avic.c
> > +++ b/arch/arm/mach-imx/avic.c
> > @@ -54,28 +54,6 @@
> >  static void __iomem *avic_base;
> >  static struct irq_domain *domain;
> >  
> > -#ifdef CONFIG_MXC_IRQ_PRIOR
> > -static int avic_irq_set_priority(unsigned char irq, unsigned char prio)
> > -{
> > -	struct irq_data *d = irq_get_irq_data(irq);
> > -	unsigned int temp;
> > -	unsigned int mask = 0x0F << irq % 8 * 4;
> > -
> > -	irq = d->hwirq;
> > -
> > -	if (irq >= AVIC_NUM_IRQS)
> > -		return -EINVAL;
> > -
> > -	temp = __raw_readl(avic_base + AVIC_NIPRIORITY(irq / 8));
> > -	temp &= ~mask;
> > -	temp |= prio & mask;
> > -
> > -	__raw_writel(temp, avic_base + AVIC_NIPRIORITY(irq / 8));
> > -
> > -	return 0;
> > -}
> > -#endif
> > -
> >  #ifdef CONFIG_FIQ
> >  static int avic_set_irq_fiq(unsigned int irq, unsigned int type)
> >  {
> > @@ -102,9 +80,6 @@ static int avic_set_irq_fiq(unsigned int irq, unsigned int type)
> >  
> >  
> >  static struct mxc_extra_irq avic_extra_irq = {
> > -#ifdef CONFIG_MXC_IRQ_PRIOR
> > -	.set_priority = avic_irq_set_priority,
> > -#endif
> >  #ifdef CONFIG_FIQ
> >  	.set_irq_fiq = avic_set_irq_fiq,
> >  #endif
> > diff --git a/arch/arm/mach-imx/irq-common.h b/arch/arm/mach-imx/irq-common.h
> > index 5b2dabb..6e3175d 100644
> > --- a/arch/arm/mach-imx/irq-common.h
> > +++ b/arch/arm/mach-imx/irq-common.h
> > @@ -24,7 +24,6 @@
> >  
> >  struct mxc_extra_irq
> >  {
> > -	int (*set_priority)(unsigned char irq, unsigned char prio);
> >  	int (*set_irq_fiq)(unsigned int irq, unsigned int type);
> >  };
> >  
> > -- 
> > 1.8.4.rc3
> > 
> > 
> 
> -- 
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
> 

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |



More information about the linux-arm-kernel mailing list