From 9d518ba397712add7fa06ff3ab056a26a3c46de6 Mon Sep 17 00:00:00 2001 From: Sascha Hauer Date: Thu, 31 Jul 2014 08:47:48 +0200 Subject: resource: Let dev_get_resource return an error pointer Signed-off-by: Sascha Hauer --- drivers/ata/intf_platform_ide.c | 6 +++--- drivers/base/driver.c | 6 +++--- drivers/mfd/syscon.c | 4 ++-- drivers/misc/sram.c | 2 ++ drivers/serial/serial_ns16550.c | 8 ++++---- 5 files changed, 14 insertions(+), 12 deletions(-) diff --git a/drivers/ata/intf_platform_ide.c b/drivers/ata/intf_platform_ide.c index db4c35611d..851f8f5c5b 100644 --- a/drivers/ata/intf_platform_ide.c +++ b/drivers/ata/intf_platform_ide.c @@ -97,8 +97,8 @@ static int platform_ide_probe(struct device_d *dev) alt_base = dev_request_mem_region(dev, 1); } else { reg = dev_get_resource(dev, IORESOURCE_IO, 0); - if (!reg) - return -ENODEV; + if (IS_ERR(reg)) + return PTR_ERR(reg); reg = request_ioport_region(dev_name(dev), reg->start, reg->end); @@ -108,7 +108,7 @@ static int platform_ide_probe(struct device_d *dev) reg_base = (void __force __iomem *) reg->start; alt = dev_get_resource(dev, IORESOURCE_IO, 1); - if (alt) { + if (!IS_ERR(alt)) { alt = request_ioport_region(dev_name(dev), alt->start, alt->end); diff --git a/drivers/base/driver.c b/drivers/base/driver.c index d90da4227a..524ed1c980 100644 --- a/drivers/base/driver.c +++ b/drivers/base/driver.c @@ -256,7 +256,7 @@ struct resource *dev_get_resource(struct device_d *dev, unsigned long type, } } - return NULL; + return ERR_PTR(-ENOENT); } void *dev_get_mem_region(struct device_d *dev, int num) @@ -264,7 +264,7 @@ void *dev_get_mem_region(struct device_d *dev, int num) struct resource *res; res = dev_get_resource(dev, IORESOURCE_MEM, num); - if (!res) + if (IS_ERR(res)) return NULL; return (void __force *)res->start; @@ -323,7 +323,7 @@ void __iomem *dev_request_mem_region(struct device_d *dev, int num) struct resource *res; res = dev_get_resource(dev, IORESOURCE_MEM, num); - if (!res) + if (IS_ERR(res)) return NULL; res = request_iomem_region(dev_name(dev), res->start, res->end); diff --git a/drivers/mfd/syscon.c b/drivers/mfd/syscon.c index 231f4fef97..9589a03f16 100644 --- a/drivers/mfd/syscon.c +++ b/drivers/mfd/syscon.c @@ -71,9 +71,9 @@ static int syscon_probe(struct device_d *dev) return -ENOMEM; res = dev_get_resource(dev, IORESOURCE_MEM, 0); - if (!res) { + if (IS_ERR(res)) { free(syscon); - return -ENOENT; + return PTR_ERR(res); } res = request_iomem_region(dev_name(dev), res->start, res->end); diff --git a/drivers/misc/sram.c b/drivers/misc/sram.c index 0466a15289..c89fe87eae 100644 --- a/drivers/misc/sram.c +++ b/drivers/misc/sram.c @@ -48,6 +48,8 @@ static int sram_probe(struct device_d *dev) cdev_find_free_index("sram")); res = dev_get_resource(dev, IORESOURCE_MEM, 0); + if (IS_ERR(res)) + return PTR_ERR(res); sram->cdev.size = (unsigned long)resource_size(res); sram->cdev.ops = &memops; diff --git a/drivers/serial/serial_ns16550.c b/drivers/serial/serial_ns16550.c index 09e6a6aba8..dd0986c326 100644 --- a/drivers/serial/serial_ns16550.c +++ b/drivers/serial/serial_ns16550.c @@ -317,8 +317,8 @@ static int ns16550_init_iomem(struct device_d *dev, struct ns16550_priv *priv) int width; res = dev_get_resource(dev, IORESOURCE_MEM, 0); - if (!res) - return -ENODEV; + if (IS_ERR(res)) + return PTR_ERR(res); priv->mmiobase = dev_request_mem_region(dev, 0); @@ -347,8 +347,8 @@ static int ns16550_init_ioport(struct device_d *dev, struct ns16550_priv *priv) int width; res = dev_get_resource(dev, IORESOURCE_IO, 0); - if (!res) - return -ENODEV; + if (IS_ERR(res)) + return PTR_ERR(res); res = request_ioport_region(dev_name(dev), res->start, res->end); if (!res) -- cgit v1.2.3