diff options
author | Sascha Hauer <s.hauer@pengutronix.de> | 2014-07-31 10:38:08 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2014-09-16 08:32:10 +0200 |
commit | ed6e965824303255cacc1c1a195d3684caa26bce (patch) | |
tree | 26e2b1b78498675ceae4dd2e990836c26ec1d5ec /drivers/gpio/gpio-clps711x.c | |
parent | 5bdc82c54a3306f2ae151a00f2df54f9240395b8 (diff) | |
download | barebox-ed6e965824303255cacc1c1a195d3684caa26bce.tar.gz barebox-ed6e965824303255cacc1c1a195d3684caa26bce.tar.xz |
resource: Let dev_request_mem_region return an error pointer
For all users fix or add the error check.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers/gpio/gpio-clps711x.c')
-rw-r--r-- | drivers/gpio/gpio-clps711x.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/drivers/gpio/gpio-clps711x.c b/drivers/gpio/gpio-clps711x.c index 2f12439e0f..7d3f1ffb25 100644 --- a/drivers/gpio/gpio-clps711x.c +++ b/drivers/gpio/gpio-clps711x.c @@ -26,18 +26,22 @@ static int clps711x_gpio_probe(struct device_d *dev) return -ENODEV; dat = dev_request_mem_region(dev, 0); + if (IS_ERR(dat)) + return PTR_ERR(dat); + switch (id) { case 3: dir_inv = dev_request_mem_region(dev, 1); + if (IS_ERR(dir_inv)) + return PTR_ERR(dir_inv); break; default: dir = dev_request_mem_region(dev, 1); + if (IS_ERR(dir)) + return PTR_ERR(dir); break; } - if (!dat || (!dir && !dir_inv)) - return -EINVAL; - bgc = xzalloc(sizeof(struct bgpio_chip)); if (!bgc) return -ENOMEM; |