From patchwork Wed Sep 18 13:24:49 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Thierry Reding X-Patchwork-Id: 2907021 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id BB8F9BFF05 for ; Wed, 18 Sep 2013 13:55:10 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 90B962034F for ; Wed, 18 Sep 2013 13:55:09 +0000 (UTC) Received: from casper.infradead.org (casper.infradead.org [85.118.1.10]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5192E20326 for ; Wed, 18 Sep 2013 13:55:08 +0000 (UTC) Received: from merlin.infradead.org ([2001:4978:20e::2]) by casper.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VMHo8-0006B3-VH; Wed, 18 Sep 2013 13:28:43 +0000 Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1VMHnS-0002Yr-Gs; Wed, 18 Sep 2013 13:27:58 +0000 Received: from mail-bk0-f52.google.com ([209.85.214.52]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VMHmb-0002PG-NL for linux-arm-kernel@lists.infradead.org; Wed, 18 Sep 2013 13:27:07 +0000 Received: by mail-bk0-f52.google.com with SMTP id e11so2755129bkh.39 for ; Wed, 18 Sep 2013 06:26:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=roErhqjyuKZG51q3QNRf29gDF/GEW6RKpdhfYGEJUKc=; b=FEryFd9IwNHvZk9gHVX7NYWl5GZTa2yv6MBr06bp4ElAzjcM51Mn7Pgli4bMKgTJal ixHHSksihFdx6hvcHYE3Z3j0xulu6J1BFC3if+NxR7pbgYWMIK088qj75Y9fSM2BeJbf ERekz96uwhO2Hr2v+5TKolGeoKWF7lsLMADkhhYfBAKBRDwdcpLdQtAIoYsQn2AWm2eR c6bSzmhHysGn9X8bfhoPp/T+lPAPsAYyfUxIBAWytSYHmVumOCGOrJHKxWsPl4eTbRzo x9VaVoSvQF1IXQRqx70sytgExIXOM89aZwFu4smEAeylMUJxwdyW1Y/U84z489Vf/pJ+ aYRg== X-Received: by 10.204.111.196 with SMTP id t4mr1424660bkp.38.1379510788809; Wed, 18 Sep 2013 06:26:28 -0700 (PDT) Received: from localhost (port-55509.pppoe.wtnet.de. [46.59.217.135]) by mx.google.com with ESMTPSA id pk7sm904986bkb.2.1969.12.31.16.00.00 (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Wed, 18 Sep 2013 06:26:28 -0700 (PDT) From: Thierry Reding To: Rob Herring , Grant Likely , Greg Kroah-Hartman , Thomas Gleixner Subject: [PATCH v2 07/10] of/irq: Propagate errors in of_irq_to_resource_table() Date: Wed, 18 Sep 2013 15:24:49 +0200 Message-Id: <1379510692-32435-8-git-send-email-treding@nvidia.com> X-Mailer: git-send-email 1.8.4 In-Reply-To: <1379510692-32435-1-git-send-email-treding@nvidia.com> References: <1379510692-32435-1-git-send-email-treding@nvidia.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20130918_092705_921258_9125FB1C X-CRM114-Status: GOOD ( 16.18 ) X-Spam-Score: 0.9 (/) Cc: linux-mips@linux-mips.org, Russell King , devicetree@vger.kernel.org, Benjamin Herrenschmidt , linux-kernel@vger.kernel.org, Ralf Baechle , sparclinux@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, KHOP_BIG_TO_CC, RCVD_IN_DNSWL_MED, RP_MATCHES_RCVD, T_DKIM_INVALID,UNPARSEABLE_RELAY autolearn=no version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Now that all helpers return precise error codes, this function can propagate these errors to the caller properly. Signed-off-by: Thierry Reding Acked-by: Ralf Baechle --- Changes in v2: - return 0 on success or a negative error code on failure - convert callers to new calling convention arch/mips/lantiq/irq.c | 2 +- arch/mips/lantiq/xway/gptu.c | 6 ++++-- drivers/of/irq.c | 14 ++++++++------ drivers/tty/serial/lantiq.c | 2 +- 4 files changed, 14 insertions(+), 10 deletions(-) diff --git a/arch/mips/lantiq/irq.c b/arch/mips/lantiq/irq.c index eb3e186..5bb7ee6 100644 --- a/arch/mips/lantiq/irq.c +++ b/arch/mips/lantiq/irq.c @@ -389,7 +389,7 @@ int __init icu_of_init(struct device_node *node, struct device_node *parent) ret = of_irq_to_resource_table(eiu_node, ltq_eiu_irq, exin_avail); - if (ret != exin_avail) + if (ret < 0) panic("failed to load external irq resources\n"); if (request_mem_region(res.start, resource_size(&res), diff --git a/arch/mips/lantiq/xway/gptu.c b/arch/mips/lantiq/xway/gptu.c index 850821d..0c4b134 100644 --- a/arch/mips/lantiq/xway/gptu.c +++ b/arch/mips/lantiq/xway/gptu.c @@ -137,10 +137,12 @@ static int gptu_probe(struct platform_device *pdev) { struct clk *clk; struct resource *res; + int ret; - if (of_irq_to_resource_table(pdev->dev.of_node, irqres, 6) != 6) { + ret = of_irq_to_resource_table(pdev->dev.of_node, irqres, 6); + if (ret < 0) { dev_err(&pdev->dev, "Failed to get IRQ list\n"); - return -EINVAL; + return ret; } res = platform_get_resource(pdev, IORESOURCE_MEM, 0); diff --git a/drivers/of/irq.c b/drivers/of/irq.c index e4f38c0..6d7f824 100644 --- a/drivers/of/irq.c +++ b/drivers/of/irq.c @@ -397,18 +397,20 @@ int of_irq_count(struct device_node *dev) * @res: array of resources to fill in * @nr_irqs: the number of IRQs (and upper bound for num of @res elements) * - * Returns the size of the filled in table (up to @nr_irqs). + * Returns 0 on success or a negative error code on failure. */ int of_irq_to_resource_table(struct device_node *dev, struct resource *res, int nr_irqs) { - int i; + int i, ret; - for (i = 0; i < nr_irqs; i++, res++) - if (!of_irq_to_resource(dev, i, res)) - break; + for (i = 0; i < nr_irqs; i++, res++) { + ret = of_irq_to_resource(dev, i, res); + if (ret < 0) + return ret; + } - return i; + return 0; } EXPORT_SYMBOL_GPL(of_irq_to_resource_table); diff --git a/drivers/tty/serial/lantiq.c b/drivers/tty/serial/lantiq.c index 88d01e0..e59efdc 100644 --- a/drivers/tty/serial/lantiq.c +++ b/drivers/tty/serial/lantiq.c @@ -686,7 +686,7 @@ lqasc_probe(struct platform_device *pdev) mmres = platform_get_resource(pdev, IORESOURCE_MEM, 0); ret = of_irq_to_resource_table(node, irqres, 3); - if (!mmres || (ret != 3)) { + if (!mmres || (ret < 0)) { dev_err(&pdev->dev, "failed to get memory/irq for serial port\n"); return -ENODEV;