Re: [PATCH] drivers/base/platform.c: Add platform_get_irq_flags()

Daniel Mack
2009-03-15T12:13:51Z

On Fri, Mar 13, 2009 at 03:29:42PM +0100, Daniel Mack wrote:
> For drivers using platform_get_irq(), a function to get the irq flags
> supplied by the platform_device definition would be helpful. This patch
> adds it.

Any oppinion on that? Sorry I didn't Cc: anyone on that post, but I
couldn't figure out who's specifically in charge of maintaining that
code.

Daniel

> Signed-off-by: Daniel Mack <daniel@caiaq.de>
> ---
> drivers/base/platform.c | 13 +++++++++++++
> include/linux/platform_device.h | 1 +
> 2 files changed, 14 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/base/platform.c b/drivers/base/platform.c
> index 349a101..3d6a9f4 100644
> --- a/drivers/base/platform.c
> +++ b/drivers/base/platform.c
> @@ -63,6 +63,19 @@ int platform_get_irq(struct platform_device *dev, unsigned int num)
> EXPORT_SYMBOL_GPL(platform_get_irq);
>
> /**
> + * platform_get_irq_flags - get flags of an IRQ for a device
> + * @dev: platform device
> + * @num: IRQ number index
> + */
> +int platform_get_irq_flags(struct platform_device *dev, unsigned int num)
> +{
> + struct resource *r = platform_get_resource(dev, IORESOURCE_IRQ, num);
> +
> + return r ? r->flags : -ENXIO;
> +}
> +EXPORT_SYMBOL_GPL(platform_get_irq_flags);
> +
> +/**
> * platform_get_resource_byname - get a resource for a device by name
> * @dev: platform device
> * @type: resource type
> diff --git a/include/linux/platform_device.h b/include/linux/platform_device.h
> index 9a34269..a2c6c2c 100644
> --- a/include/linux/platform_device.h
> +++ b/include/linux/platform_device.h
> @@ -31,6 +31,7 @@ extern struct device platform_bus;
>
> extern struct resource *platform_get_resource(struct platform_device *, unsigned int, unsigned int);
> extern int platform_get_irq(struct platform_device *, unsigned int);
> +extern int platform_get_irq_flags(struct platform_device *, unsigned int);
> extern struct resource *platform_get_resource_byname(struct platform_device *, unsigned int, char *);
> extern int platform_get_irq_byname(struct platform_device *, char *);
> extern int platform_add_devices(struct platform_device **, int);
> --
> 1.6.2