[PATCH v2 06/10] of/irq: Return errors from of_irq_to_resource()
From: Thierry Reding <hidden>
Date: 2013-09-18 13:26:35
Also in:
linux-arm-kernel, linux-devicetree, linux-mips, lkml, sparclinux
Subsystem:
linux for powerpc (32-bit and 64-bit), linux for powerpc embedded ppc8xx and ppc83xx, marvell mv643xx ethernet driver, networking drivers, open firmware and flattened device tree, the rest · Maintainers:
Madhavan Srinivasan, Christophe Leroy, Sebastian Hesselbarth, Andrew Lunn, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Rob Herring, Saravana Kannan, Linus Torvalds
Update of_irq_to_resource() to return 0 on success and a negative error code on failure. This allows the precise nature of the failure to be determined in the caller and errors to be propagated appropriately. While at it, make the index parameter unsigned. Accessing negative indices is invalid, so we might as well enforce that by using the right data type. Signed-off-by: Thierry Reding <redacted> --- Changes in v2: - convert existing callers instead of using compatible wrapper arch/powerpc/platforms/83xx/mpc832x_rdb.c | 2 +- drivers/net/ethernet/marvell/mv643xx_eth.c | 5 +++-- drivers/of/irq.c | 14 +++++++++++--- include/linux/of_irq.h | 2 +- 4 files changed, 16 insertions(+), 7 deletions(-)
diff --git a/arch/powerpc/platforms/83xx/mpc832x_rdb.c b/arch/powerpc/platforms/83xx/mpc832x_rdb.c
index eff5baa..b198e73 100644
--- a/arch/powerpc/platforms/83xx/mpc832x_rdb.c
+++ b/arch/powerpc/platforms/83xx/mpc832x_rdb.c@@ -89,7 +89,7 @@ static int __init of_fsl_spi_probe(char *type, char *compatible, u32 sysclk, goto err; ret = of_irq_to_resource(np, 0, &res[1]); - if (ret == NO_IRQ) + if (ret) goto err; pdev = platform_device_alloc("mpc83xx_spi", i);
diff --git a/drivers/net/ethernet/marvell/mv643xx_eth.c b/drivers/net/ethernet/marvell/mv643xx_eth.c
index 7fb5677..bd713bd 100644
--- a/drivers/net/ethernet/marvell/mv643xx_eth.c
+++ b/drivers/net/ethernet/marvell/mv643xx_eth.c@@ -2489,9 +2489,10 @@ static int mv643xx_eth_shared_of_add_port(struct platform_device *pdev, ppd.shared = pdev; memset(&res, 0, sizeof(res)); - if (!of_irq_to_resource(pnp, 0, &res)) { + ret = of_irq_to_resource(pnp, 0, &res); + if (ret) { dev_err(&pdev->dev, "missing interrupt on %s\n", pnp->name); - return -EINVAL; + return ret; } if (of_property_read_u32(pnp, "reg", &ppd.port_number)) {
diff --git a/drivers/of/irq.c b/drivers/of/irq.c
index 6ad46fd..e4f38c0 100644
--- a/drivers/of/irq.c
+++ b/drivers/of/irq.c@@ -341,10 +341,18 @@ EXPORT_SYMBOL_GPL(of_irq_map_one); * @dev: pointer to device tree node * @index: zero-based index of the irq * @r: pointer to resource structure to return result into. + * + * Returns zero on success or a negative error code on failure. */ -int of_irq_to_resource(struct device_node *dev, int index, struct resource *r) +int of_irq_to_resource(struct device_node *dev, unsigned int index, + struct resource *r) { - int irq = irq_of_parse_and_map(dev, index); + unsigned int irq; + int ret; + + ret = __irq_of_parse_and_map(dev, index, &irq); + if (ret) + return ret; /* Only dereference the resource if both the * resource and the irq are valid. */
@@ -364,7 +372,7 @@ int of_irq_to_resource(struct device_node *dev, int index, struct resource *r) r->name = name ? name : dev->full_name; } - return irq; + return 0; } EXPORT_SYMBOL_GPL(of_irq_to_resource);
diff --git a/include/linux/of_irq.h b/include/linux/of_irq.h
index 11da949..6d62b73 100644
--- a/include/linux/of_irq.h
+++ b/include/linux/of_irq.h@@ -67,7 +67,7 @@ extern int of_irq_map_one(struct device_node *device, int index, extern int irq_create_of_mapping(struct device_node *controller, const u32 *intspec, unsigned int intsize, unsigned int *virqp); -extern int of_irq_to_resource(struct device_node *dev, int index, +extern int of_irq_to_resource(struct device_node *dev, unsigned int index, struct resource *r); extern int of_irq_count(struct device_node *dev); extern int of_irq_to_resource_table(struct device_node *dev,
--
1.8.4