From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fabio Estevam Subject: Re: [PATCH v6 3/6] power: Add support for DA9150 Fuel-Gauge Date: Mon, 5 Oct 2015 13:10:12 -0300 Message-ID: References: <80d467703ba19bef12e9bcdd246e1de7bd33a8b1.1444059987.git.Adam.Thomson.Opensource@diasemi.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <80d467703ba19bef12e9bcdd246e1de7bd33a8b1.1444059987.git.Adam.Thomson.Opensource@diasemi.com> Sender: linux-kernel-owner@vger.kernel.org To: Adam Thomson Cc: Lee Jones , Samuel Ortiz , Sebastian Reichel , Dmitry Eremin-Solenikov , David Woodhouse , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , "linux-pm@vger.kernel.org" , "devicetree@vger.kernel.org" , linux-kernel , Support Opensource List-Id: devicetree@vger.kernel.org On Mon, Oct 5, 2015 at 1:03 PM, Adam Thomson wrote: > + /* Register IRQ */ > + irq = platform_get_irq_byname(pdev, "FG"); > + if (irq < 0) { > + dev_err(dev, "Failed to get IRQ FG: %d\n", irq); You need to do "ret = irq" here. Otherwise you will return success in irq_fail. > + goto irq_fail;