gpiolib: cosmetic improvements for error handling in gpiochip_add()
Hopefully it makes the code look nicer and makes it easier to extend this function. Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Grant Likely <grant.likely@secretlab.ca> CC: devicetree-discuss@lists.ozlabs.org CC: linux-kernel@vger.kernel.org
This commit is contained in:
parent
ac80a51e2c
commit
cedb1881ba
|
@ -1101,14 +1101,20 @@ int gpiochip_add(struct gpio_chip *chip)
|
|||
|
||||
unlock:
|
||||
spin_unlock_irqrestore(&gpio_lock, flags);
|
||||
if (status == 0)
|
||||
status = gpiochip_export(chip);
|
||||
|
||||
if (status)
|
||||
goto fail;
|
||||
|
||||
status = gpiochip_export(chip);
|
||||
if (status)
|
||||
goto fail;
|
||||
|
||||
return 0;
|
||||
fail:
|
||||
/* failures here can mean systems won't boot... */
|
||||
if (status)
|
||||
pr_err("gpiochip_add: gpios %d..%d (%s) failed to register\n",
|
||||
chip->base, chip->base + chip->ngpio - 1,
|
||||
chip->label ? : "generic");
|
||||
pr_err("gpiochip_add: gpios %d..%d (%s) failed to register\n",
|
||||
chip->base, chip->base + chip->ngpio - 1,
|
||||
chip->label ? : "generic");
|
||||
return status;
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(gpiochip_add);
|
||||
|
|
Loading…
Reference in New Issue