From: Andreas Larsson <andreas@gaisler.com>
To: nyushchenko@dev.rtsoft.ru, Grant Likely <grant.likely@linaro.org>,
Rob Herring <robh+dt@kernel.org>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>,
Thomas Gleixner <tglx@linutronix.de>,
devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org, lugovskoy@dev.rtsoft.ru
Subject: Re: [PATCH 20/21] usb: use devm_irq_of_parse_and_map() where appropriate
Date: Mon, 16 Jun 2014 11:35:43 +0200 [thread overview]
Message-ID: <539EBA6F.7060008@gaisler.com> (raw)
In-Reply-To: <1401880402-30091-21-git-send-email-nyushchenko@dev.rtsoft.ru>
On 2014-06-04 13:13, nyushchenko@dev.rtsoft.ru wrote:
> From: Nikita Yushchenko <nyushchenko@dev.rtsoft.ru>
>
> This avoids leak of IRQ mapping on error paths, and makes it possible
> to use devm_request_irq() without facing unmap-while-handler-installed
> issues.
>
> Signed-off-by: Nikita Yushchenko <nyushchenko@dev.rtsoft.ru>
> ---
> drivers/usb/gadget/gr_udc.c | 17 +++++++++++------
> drivers/usb/host/ehci-grlib.c | 11 ++++-------
> drivers/usb/host/ehci-orion.c | 3 ++-
> drivers/usb/host/ehci-ppc-of.c | 11 ++++-------
> drivers/usb/host/ehci-xilinx-of.c | 8 ++++----
> drivers/usb/host/ohci-ppc-of.c | 7 ++-----
> 6 files changed, 27 insertions(+), 30 deletions(-)
>
> diff --git a/drivers/usb/gadget/gr_udc.c b/drivers/usb/gadget/gr_udc.c
> index f984ee7..0a3e586 100644
> --- a/drivers/usb/gadget/gr_udc.c
> +++ b/drivers/usb/gadget/gr_udc.c
> @@ -2108,19 +2108,24 @@ static int gr_probe(struct platform_device *ofdev)
> if (IS_ERR(regs))
> return PTR_ERR(regs);
>
> - dev->irq = irq_of_parse_and_map(dev->dev->of_node, 0);
> - if (!dev->irq) {
> + dev->irq = devm_irq_of_parse_and_map(dev->dev, dev->dev->of_node, 0);
> + if (dev->irq <= 0) {
> dev_err(dev->dev, "No irq found\n");
> return -ENODEV;
> }
>
> /* Some core configurations has separate irqs for IN and OUT events */
> - dev->irqi = irq_of_parse_and_map(dev->dev->of_node, 1);
> + dev->irqi = devm_irq_of_parse_and_map(dev->dev, dev->dev->of_node, 1);
> + if (dev->irqi < 0) {
> + dev_err(dev->dev, "could not locate irq\n");
> + return dev->irqi;
> + }
> if (dev->irqi) {
> - dev->irqo = irq_of_parse_and_map(dev->dev->of_node, 2);
> - if (!dev->irqo) {
> + dev->irqo = devm_irq_of_parse_and_map(dev->dev,
> + dev->dev->of_node, 2);
> + if (dev->irqo <= 0) {
> dev_err(dev->dev, "Found irqi but not irqo\n");
> - return -ENODEV;
> + return dev->irqo ? dev->irqo : -ENODEV;
> }
> }
>
> diff --git a/drivers/usb/host/ehci-grlib.c b/drivers/usb/host/ehci-grlib.c
> index 495b6fb..666c03e 100644
> --- a/drivers/usb/host/ehci-grlib.c
> +++ b/drivers/usb/host/ehci-grlib.c
> @@ -111,11 +111,11 @@ static int ehci_hcd_grlib_probe(struct platform_device *op)
> hcd->rsrc_start = res.start;
> hcd->rsrc_len = resource_size(&res);
>
> - irq = irq_of_parse_and_map(dn, 0);
> - if (irq == NO_IRQ) {
> - dev_err(&op->dev, "%s: irq_of_parse_and_map failed\n",
> + irq = devm_irq_of_parse_and_map(&op->dev, dn, 0);
> + if (irq <= 0) {
> + dev_err(&op->dev, "%s: devm_irq_of_parse_and_map failed\n",
> __FILE__);
> - rv = -EBUSY;
> + rv = irq ? irq : -EINVAL;
Here and in more places below you change the return value from -EBUSY to
-EINVAL when irq == 0. These changes and the reason for them is not
something that is commented upon in the commit message. Maybe these
changes were not intended or should be in a separate patch?
> goto err_irq;
> }
>
> @@ -145,7 +145,6 @@ static int ehci_hcd_grlib_probe(struct platform_device *op)
> return 0;
>
> err_ioremap:
> - irq_dispose_mapping(irq);
> err_irq:
> usb_put_hcd(hcd);
>
> @@ -161,8 +160,6 @@ static int ehci_hcd_grlib_remove(struct platform_device *op)
>
> usb_remove_hcd(hcd);
>
> - irq_dispose_mapping(hcd->irq);
> -
> usb_put_hcd(hcd);
>
> return 0;
> diff --git a/drivers/usb/host/ehci-orion.c b/drivers/usb/host/ehci-orion.c
> index 30d35e5..c88177b 100644
> --- a/drivers/usb/host/ehci-orion.c
> +++ b/drivers/usb/host/ehci-orion.c
> @@ -155,7 +155,8 @@ static int ehci_orion_drv_probe(struct platform_device *pdev)
> pr_debug("Initializing Orion-SoC USB Host Controller\n");
>
> if (pdev->dev.of_node)
> - irq = irq_of_parse_and_map(pdev->dev.of_node, 0);
> + irq = devm_irq_of_parse_and_map(&pdev->dev,
> + pdev->dev.of_node, 0);
> else
> irq = platform_get_irq(pdev, 0);
> if (irq <= 0) {
> diff --git a/drivers/usb/host/ehci-ppc-of.c b/drivers/usb/host/ehci-ppc-of.c
> index 5479247..cc64210 100644
> --- a/drivers/usb/host/ehci-ppc-of.c
> +++ b/drivers/usb/host/ehci-ppc-of.c
> @@ -117,11 +117,11 @@ static int ehci_hcd_ppc_of_probe(struct platform_device *op)
> hcd->rsrc_start = res.start;
> hcd->rsrc_len = resource_size(&res);
>
> - irq = irq_of_parse_and_map(dn, 0);
> - if (irq == NO_IRQ) {
> - dev_err(&op->dev, "%s: irq_of_parse_and_map failed\n",
> + irq = devm_irq_of_parse_and_map(&op->dev, dn, 0);
> + if (irq <= 0) {
> + dev_err(&op->dev, "%s: devm_irq_of_parse_and_map failed\n",
> __FILE__);
> - rv = -EBUSY;
> + rv = irq ? irq : -EINVAL;
> goto err_irq;
> }
>
> @@ -174,7 +174,6 @@ static int ehci_hcd_ppc_of_probe(struct platform_device *op)
> return 0;
>
> err_ioremap:
> - irq_dispose_mapping(irq);
> err_irq:
> usb_put_hcd(hcd);
>
> @@ -194,8 +193,6 @@ static int ehci_hcd_ppc_of_remove(struct platform_device *op)
>
> usb_remove_hcd(hcd);
>
> - irq_dispose_mapping(hcd->irq);
> -
> /* use request_mem_region to test if the ohci driver is loaded. if so
> * ensure the ohci core is operational.
> */
> diff --git a/drivers/usb/host/ehci-xilinx-of.c b/drivers/usb/host/ehci-xilinx-of.c
> index fe57710..6c6f8c7 100644
> --- a/drivers/usb/host/ehci-xilinx-of.c
> +++ b/drivers/usb/host/ehci-xilinx-of.c
> @@ -153,11 +153,11 @@ static int ehci_hcd_xilinx_of_probe(struct platform_device *op)
> hcd->rsrc_start = res.start;
> hcd->rsrc_len = resource_size(&res);
>
> - irq = irq_of_parse_and_map(dn, 0);
> - if (!irq) {
> - dev_err(&op->dev, "%s: irq_of_parse_and_map failed\n",
> + irq = devm_irq_of_parse_and_map(&op->dev, dn, 0);
> + if (irq <= 0) {
> + dev_err(&op->dev, "%s: devm_irq_of_parse_and_map failed\n",
> __FILE__);
> - rv = -EBUSY;
> + rv = irq ? irq : -EINVAL;
> goto err_irq;
> }
>
> diff --git a/drivers/usb/host/ohci-ppc-of.c b/drivers/usb/host/ohci-ppc-of.c
> index 965e3e9..fcdd23a 100644
> --- a/drivers/usb/host/ohci-ppc-of.c
> +++ b/drivers/usb/host/ohci-ppc-of.c
> @@ -121,11 +121,11 @@ static int ohci_hcd_ppc_of_probe(struct platform_device *op)
> goto err_rmr;
> }
>
> - irq = irq_of_parse_and_map(dn, 0);
> + irq = devm_irq_of_parse_and_map(*op->dev, dn, 0);
> if (irq == NO_IRQ) {
> dev_err(&op->dev, "%s: irq_of_parse_and_map failed\n",
> __FILE__);
> - rv = -EBUSY;
> + rv = irq ? irq : -EINVAL;
> goto err_rmr;
> }
>
> @@ -170,7 +170,6 @@ static int ohci_hcd_ppc_of_probe(struct platform_device *op)
> pr_debug("%s: cannot get ehci offset from fdt\n", __FILE__);
> }
>
> - irq_dispose_mapping(irq);
> err_rmr:
> usb_put_hcd(hcd);
>
> @@ -185,8 +184,6 @@ static int ohci_hcd_ppc_of_remove(struct platform_device *op)
>
> usb_remove_hcd(hcd);
>
> - irq_dispose_mapping(hcd->irq);
> -
> usb_put_hcd(hcd);
>
> return 0;
>
next prev parent reply other threads:[~2014-06-16 9:37 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-04 11:13 [PATCH 00/21] add and use devm_irq_of_parse_and_map() nyushchenko
2014-06-04 11:13 ` [PATCH 01/21] irq: add devres version of OF IRQ mapping routines nyushchenko
2014-06-04 13:39 ` Thomas Gleixner
2014-06-04 11:13 ` [PATCH 02/21] ata: use devm_irq_of_parse_and_map() where appropriate nyushchenko
2014-06-04 11:13 ` [PATCH 03/21] exynos5440-cpufreq: use devm_irq_of_parse_and_map() nyushchenko
2014-06-04 11:13 ` [PATCH 04/21] omap-sham: " nyushchenko
2014-06-04 11:13 ` [PATCH 05/21] dma: use devm_irq_of_parse_and_map() where appropriate nyushchenko
2014-06-04 11:13 ` [PATCH 06/21] mpc85xx_edac: use devm_irq_of_parse_and_map() nyushchenko
2014-06-04 11:13 ` [PATCH 07/21] gpio: use devm_irq_of_parse_and_map() where appropriate nyushchenko
2014-06-04 11:13 ` [PATCH 08/21] i2c: " nyushchenko
2014-06-04 11:13 ` [PATCH 09/21] apbps2: use devm_irq_of_parse_and_map() nyushchenko
2014-06-04 11:13 ` [PATCH 10/21] media: use devm_irq_of_parse_and_map() where appropriate nyushchenko
2014-06-04 11:13 ` [PATCH 11/21] mfd: " nyushchenko
2014-06-17 15:12 ` Lee Jones
2014-06-17 15:36 ` Nikita Yushchenko
2014-06-18 8:48 ` Lee Jones
2014-06-18 11:31 ` Nikita Yushchenko
2014-06-18 12:20 ` Lee Jones
2014-06-04 11:13 ` [PATCH 12/21] mpc5121_nfc: use devm_irq_of_parse_and_map() nyushchenko
2014-06-04 11:13 ` [PATCH 13/21] net/can: use devm_irq_of_parse_and_map() where appropriate nyushchenko
2014-06-04 11:13 ` [PATCH 14/21] net/ethernet: " nyushchenko
2014-06-04 11:13 ` [PATCH 15/21] pinctrl: " nyushchenko
2014-06-04 11:13 ` [PATCH 16/21] bq24190_charger: use devm_irq_of_parse_and_map() nyushchenko
2014-06-04 11:13 ` [PATCH 17/21] rtc-mpc5121: " nyushchenko
2014-06-04 11:13 ` [PATCH 18/21] spi: use devm_irq_of_parse_and_map() where appropriate nyushchenko
2014-06-04 11:13 ` [PATCH 19/21] exynos_tmu: use devm_irq_of_parse_and_map() nyushchenko
2014-06-04 11:13 ` [PATCH 20/21] usb: use devm_irq_of_parse_and_map() where appropriate nyushchenko
2014-06-16 9:35 ` Andreas Larsson [this message]
2014-06-16 9:44 ` Nikita Yushchenko
2014-06-16 9:54 ` Andreas Larsson
2014-06-04 11:13 ` [PATCH 21/21] at91sam9_wdt: use devm_irq_of_parse_and_map() nyushchenko
2014-06-12 10:03 ` [PATCH 00/21] add and " Andreas Larsson
[not found] ` <5399AE1B.1080301@gaisler.com>
2014-06-12 19:02 ` Nikita Yushchenko
2014-06-16 8:23 ` Andreas Larsson
2014-06-16 8:36 ` Nikita Yushchenko
2014-06-16 9:29 ` Andreas Larsson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=539EBA6F.7060008@gaisler.com \
--to=andreas@gaisler.com \
--cc=benh@kernel.crashing.org \
--cc=devicetree@vger.kernel.org \
--cc=grant.likely@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lugovskoy@dev.rtsoft.ru \
--cc=nyushchenko@dev.rtsoft.ru \
--cc=robh+dt@kernel.org \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome