Re: [PATCH 1/5] gpio/omap: free irq domain in probe() failure paths

2013-04-15 Thread Kevin Hilman
Jon Hunter jon-hun...@ti.com writes: On 04/10/2013 02:33 PM, Linus Walleij wrote: On Thu, Apr 4, 2013 at 10:16 PM, Jon Hunter jon-hun...@ti.com wrote: Currently the IRQ domain is not freed once allocated, in the case where omap_gpio_probe() fails. Therefore, ensure we free the domain if the

Re: [PATCH 1/5] gpio/omap: free irq domain in probe() failure paths

2013-04-10 Thread Linus Walleij
On Thu, Apr 4, 2013 at 10:16 PM, Jon Hunter jon-hun...@ti.com wrote: Currently the IRQ domain is not freed once allocated, in the case where omap_gpio_probe() fails. Therefore, ensure we free the domain if the probe does fail. Furthermore, the local variable ret is not needed and so remove

Re: [PATCH 1/5] gpio/omap: free irq domain in probe() failure paths

2013-04-10 Thread Jon Hunter
On 04/10/2013 02:33 PM, Linus Walleij wrote: On Thu, Apr 4, 2013 at 10:16 PM, Jon Hunter jon-hun...@ti.com wrote: Currently the IRQ domain is not freed once allocated, in the case where omap_gpio_probe() fails. Therefore, ensure we free the domain if the probe does fail. Furthermore, the

[PATCH 1/5] gpio/omap: free irq domain in probe() failure paths

2013-04-04 Thread Jon Hunter
Currently the IRQ domain is not freed once allocated, in the case where omap_gpio_probe() fails. Therefore, ensure we free the domain if the probe does fail. Furthermore, the local variable ret is not needed and so remove this. Signed-off-by: Jon Hunter jon-hun...@ti.com ---