[PATCH 1/4] ARM: davinci: Check for NULL return from irq_alloc_generic_chip

Nori, Sekhar nsekhar at ti.com
Thu Jul 14 01:20:38 EDT 2011


Hi Todd,

On Wed, Jul 13, 2011 at 04:01:50, Todd Poynor wrote:
> Signed-off-by: Todd Poynor <toddpoynor at google.com>

Can you please include some patch description
(even if it turns out to be a restatement of
patch headline)?

Also, in general, including some background on how
you discovered the issue and how it affected you
can help determine the priority of the fix.

Thanks,
Sekhar

> ---
>  arch/arm/mach-davinci/irq.c |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/mach-davinci/irq.c b/arch/arm/mach-davinci/irq.c
> index bfe68ec..ab7f688 100644
> --- a/arch/arm/mach-davinci/irq.c
> +++ b/arch/arm/mach-davinci/irq.c
> @@ -52,6 +52,13 @@ davinci_alloc_gc(void __iomem *base, unsigned int irq_start, unsigned int num)
>  	struct irq_chip_type *ct;
>  
>  	gc = irq_alloc_generic_chip("AINTC", 1, irq_start, base, handle_edge_irq);
> +
> +	if (!gc) {
> +		pr_err("%s: irq_alloc_generic_chip for IRQ %u failed\n",
> +		       __func__, irq_start);
> +		return;
> +	}
> +
>  	ct = gc->chip_types;
>  	ct->chip.irq_ack = irq_gc_ack;
>  	ct->chip.irq_mask = irq_gc_mask_clr_bit;
> -- 
> 1.7.3.1
> 
> 




More information about the linux-arm-kernel mailing list