Re: [PATCH] irq: uninline generic_handle_irq, export handle_simple_irq,irq_modify_status, generic_handle_irq, irq_alloc_descs and irq_free_descs

From: Jonathan Cameron
Date: Wed May 18 2011 - 05:31:02 EST


Sorry, scratch this. Should be EXPORT_SYMBOL_GPL for generic_handle_irq.
New version in a few moments.


> V3: stop exporting irq_to_desc and instead take generic_handle_irq out of the header.
> As suggested by Thomas Gleixner.
>
> Patch was previously:
> irq: export handle_simple_irq, irq_modify_status, irq_to_desc, irq_alloc_descs and irq_free_descs
>
> V2: add irq_alloc_descs and irq_free_descs for dynamic pool allocation.
>
> This is to allow their use in IIO. We have complete control of
> both the irq_chip creation and requesting of irqs so would like
> to be able to build as a module. That requires these exports.
>
> Signed-off-by: Jonathan Cameron <jic23@xxxxxxxxx>
> ---
> Hi Thomas,
>
> Looking a bit more at possible locations it is clear that CONFIG_GENERIC_HARDIRQS
> only exists as an ifdef in irqdesc.c so I guess that is the least intrusive place
> to move this to.
>
> Lots stacked behind this, so thought I'd keep the ball rolling rather than
> waiting for your reply to my question of where best to put generic_handle_irq.
>
> Thanks,
>
> Jonathan
>
> include/linux/irqdesc.h | 5 +----
> kernel/irq/chip.c | 2 ++
> kernel/irq/irqdesc.c | 8 ++++++++
> 3 files changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/include/linux/irqdesc.h b/include/linux/irqdesc.h
> index a082905..27b971d 100644
> --- a/include/linux/irqdesc.h
> +++ b/include/linux/irqdesc.h
> @@ -109,10 +109,7 @@ static inline void generic_handle_irq_desc(unsigned int irq, struct irq_desc *de
> desc->handle_irq(irq, desc);
> }
>
> -static inline void generic_handle_irq(unsigned int irq)
> -{
> - generic_handle_irq_desc(irq, irq_to_desc(irq));
> -}
> +void generic_handle_irq(unsigned int irq);
>
> /* Test to see if a driver has successfully requested an irq */
> static inline int irq_has_action(unsigned int irq)
> diff --git a/kernel/irq/chip.c b/kernel/irq/chip.c
> index 4af1e2b..ad409d3 100644
> --- a/kernel/irq/chip.c
> +++ b/kernel/irq/chip.c
> @@ -310,6 +310,7 @@ handle_simple_irq(unsigned int irq, struct irq_desc *desc)
> out_unlock:
> raw_spin_unlock(&desc->lock);
> }
> +EXPORT_SYMBOL_GPL(handle_simple_irq);
>
> /**
> * handle_level_irq - Level type irq handler
> @@ -612,6 +613,7 @@ void irq_modify_status(unsigned int irq, unsigned long clr, unsigned long set)
>
> irq_put_desc_unlock(desc, flags);
> }
> +EXPORT_SYMBOL_GPL(irq_modify_status);
>
> /**
> * irq_cpu_online - Invoke all irq_cpu_online functions.
> diff --git a/kernel/irq/irqdesc.c b/kernel/irq/irqdesc.c
> index 2c039c9..dbffb95 100644
> --- a/kernel/irq/irqdesc.c
> +++ b/kernel/irq/irqdesc.c
> @@ -311,6 +311,7 @@ void irq_free_descs(unsigned int from, unsigned int cnt)
> bitmap_clear(allocated_irqs, from, cnt);
> mutex_unlock(&sparse_irq_lock);
> }
> +EXPORT_SYMBOL_GPL(irq_free_descs);
>
> /**
> * irq_alloc_descs - allocate and initialize a range of irq descriptors
> @@ -351,6 +352,7 @@ err:
> mutex_unlock(&sparse_irq_lock);
> return ret;
> }
> +EXPORT_SYMBOL_GPL(irq_alloc_descs);
>
> /**
> * irq_reserve_irqs - mark irqs allocated
> @@ -443,4 +445,10 @@ unsigned int kstat_irqs(unsigned int irq)
> sum += *per_cpu_ptr(desc->kstat_irqs, cpu);
> return sum;
> }
> +
> +void generic_handle_irq(unsigned int irq)
> +{
> + generic_handle_irq_desc(irq, irq_to_desc(irq));
> +}
> +EXPORT_SYMBOL(generic_handle_irq);
> #endif /* CONFIG_GENERIC_HARDIRQS */

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/