From: Vivek Gautam <gautam.vivek@samsung.com> To: Linux USB Mailing List <linux-usb@vger.kernel.org> Cc: "linux-samsung-soc@vger.kernel.org" <linux-samsung-soc@vger.kernel.org>, linux-tegra <linux-tegra@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, Greg KH <gregkh@linuxfoundation.org>, Alan Stern <stern@rowland.harvard.edu>, Stephen Warren <swarren@wwwdotorg.org>, Kukjin Kim <kgene.kim@samsung.com>, Thierry Reding <thierry.reding@gmail.com>, Alexander Shiyan <shc_work@mail.ru>, Vivek Gautam <gautam.vivek@samsung.com> Subject: Re: [PATCH v2 6/6] usb: host: ohci-exynos: Use devm_ioremap_resource instead of devm_ioremap Date: Wed, 4 Jun 2014 15:41:20 +0530 [thread overview] Message-ID: <CAFp+6iFufCh6SVEF-HQA62ZsJGJGSebDSs28eqcrMy1X8e5vjA@mail.gmail.com> (raw) In-Reply-To: <1399723210-21569-7-git-send-email-gautam.vivek@samsung.com> Hi, On Sat, May 10, 2014 at 5:30 PM, Vivek Gautam <gautam.vivek@samsung.com> wrote: > Using devm_ioremap_resource() API should actually be preferred over > devm_ioremap(), since the former request the mem region first and then > gives back the ioremap'ed memory pointer. > devm_ioremap_resource() calls request_mem_region(), therby preventing > other drivers to make any overlapping call to the same region. > > Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com> Although this patch and rest in the series are merged. But i have got a doubt, so making this thread alive. > --- > drivers/usb/host/ohci-exynos.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/drivers/usb/host/ohci-exynos.c b/drivers/usb/host/ohci-exynos.c > index 9cf80cb..dec691d 100644 > --- a/drivers/usb/host/ohci-exynos.c > +++ b/drivers/usb/host/ohci-exynos.c > @@ -120,10 +120,9 @@ skip_phy: > > hcd->rsrc_start = res->start; > hcd->rsrc_len = resource_size(res); > - hcd->regs = devm_ioremap(&pdev->dev, res->start, hcd->rsrc_len); > - if (!hcd->regs) { > - dev_err(&pdev->dev, "Failed to remap I/O memory\n"); > - err = -ENOMEM; > + hcd->regs = devm_ioremap_resource(&pdev->dev, res); Here, we replaced devm_ioremap() call with devm_ioremap_resource(), which internally requests the memory region and then does a "devm_ioremap()" or "devm_ioremap_nocache()" based on the check for IORESOURCE_CACHEABLE flag. But this flag is not set for the resource of this device. So should we be explicitly setting the flag in driver ? The query goes for other patches too in this series, wherein devm_ioremap() call is replaced with devm_ioremap_resource(). [snip] -- Best Regards Vivek Gautam Samsung R&D Institute, Bangalore India
WARNING: multiple messages have this Message-ID (diff)
From: gautam.vivek@samsung.com (Vivek Gautam) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v2 6/6] usb: host: ohci-exynos: Use devm_ioremap_resource instead of devm_ioremap Date: Wed, 4 Jun 2014 15:41:20 +0530 [thread overview] Message-ID: <CAFp+6iFufCh6SVEF-HQA62ZsJGJGSebDSs28eqcrMy1X8e5vjA@mail.gmail.com> (raw) In-Reply-To: <1399723210-21569-7-git-send-email-gautam.vivek@samsung.com> Hi, On Sat, May 10, 2014 at 5:30 PM, Vivek Gautam <gautam.vivek@samsung.com> wrote: > Using devm_ioremap_resource() API should actually be preferred over > devm_ioremap(), since the former request the mem region first and then > gives back the ioremap'ed memory pointer. > devm_ioremap_resource() calls request_mem_region(), therby preventing > other drivers to make any overlapping call to the same region. > > Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com> Although this patch and rest in the series are merged. But i have got a doubt, so making this thread alive. > --- > drivers/usb/host/ohci-exynos.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/drivers/usb/host/ohci-exynos.c b/drivers/usb/host/ohci-exynos.c > index 9cf80cb..dec691d 100644 > --- a/drivers/usb/host/ohci-exynos.c > +++ b/drivers/usb/host/ohci-exynos.c > @@ -120,10 +120,9 @@ skip_phy: > > hcd->rsrc_start = res->start; > hcd->rsrc_len = resource_size(res); > - hcd->regs = devm_ioremap(&pdev->dev, res->start, hcd->rsrc_len); > - if (!hcd->regs) { > - dev_err(&pdev->dev, "Failed to remap I/O memory\n"); > - err = -ENOMEM; > + hcd->regs = devm_ioremap_resource(&pdev->dev, res); Here, we replaced devm_ioremap() call with devm_ioremap_resource(), which internally requests the memory region and then does a "devm_ioremap()" or "devm_ioremap_nocache()" based on the check for IORESOURCE_CACHEABLE flag. But this flag is not set for the resource of this device. So should we be explicitly setting the flag in driver ? The query goes for other patches too in this series, wherein devm_ioremap() call is replaced with devm_ioremap_resource(). [snip] -- Best Regards Vivek Gautam Samsung R&D Institute, Bangalore India
next prev parent reply other threads:[~2014-06-04 10:11 UTC|newest] Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-05-10 12:00 [PATCH v2 0/6] usb: host: Cleanup for ioremap'ing hcd memory Vivek Gautam 2014-05-10 12:00 ` Vivek Gautam 2014-05-10 12:00 ` [PATCH v2 1/6] usb: host: ehci-exynos: Use devm_ioremap_resource instead of devm_ioremap Vivek Gautam 2014-05-10 12:00 ` Vivek Gautam 2014-05-10 12:00 ` [PATCH v2 2/6] usb: host: ehci-msm: " Vivek Gautam 2014-05-10 12:00 ` Vivek Gautam 2014-05-10 12:00 ` [PATCH v2 3/6] usb: host: ehci-mv: " Vivek Gautam 2014-05-10 12:00 ` Vivek Gautam 2014-05-10 12:00 ` [PATCH v2 4/6] usb: host: ehci-spear: " Vivek Gautam 2014-05-10 12:00 ` Vivek Gautam 2014-05-10 12:00 ` Vivek Gautam 2014-05-10 12:00 ` [PATCH v2 5/6] usb: host: ehci-tegra: " Vivek Gautam 2014-05-10 12:00 ` Vivek Gautam [not found] ` <1399723210-21569-6-git-send-email-gautam.vivek-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> 2014-05-12 16:58 ` Stephen Warren 2014-05-12 16:58 ` Stephen Warren 2014-05-12 16:58 ` Stephen Warren 2014-05-10 12:00 ` [PATCH v2 6/6] usb: host: ohci-exynos: " Vivek Gautam 2014-05-10 12:00 ` Vivek Gautam 2014-06-04 10:11 ` Vivek Gautam [this message] 2014-06-04 10:11 ` Vivek Gautam 2014-06-04 10:11 ` Vivek Gautam [not found] ` <CAFp+6iFufCh6SVEF-HQA62ZsJGJGSebDSs28eqcrMy1X8e5vjA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2014-06-04 13:13 ` Thierry Reding 2014-06-04 13:13 ` Thierry Reding 2014-06-04 13:13 ` Thierry Reding 2014-06-06 13:02 ` Vivek Gautam 2014-06-06 13:02 ` Vivek Gautam 2014-06-06 13:02 ` Vivek Gautam [not found] ` <CAFp+6iEe8fvxboG79=2s=oe4BkQJYAoVUPtYkPdjJHCe+bC=8Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2014-06-06 22:52 ` Thierry Reding 2014-06-06 22:52 ` Thierry Reding 2014-06-06 22:52 ` Thierry Reding 2014-06-24 5:49 ` Vivek Gautam 2014-06-24 5:49 ` Vivek Gautam 2014-06-24 5:49 ` Vivek Gautam 2014-05-10 14:25 ` [PATCH v2 0/6] usb: host: Cleanup for ioremap'ing hcd memory Alexander Shiyan 2014-05-10 14:25 ` Alexander Shiyan 2014-05-10 14:25 ` Alexander Shiyan 2014-05-11 14:45 ` Vivek Gautam 2014-05-11 14:45 ` Vivek Gautam 2014-05-11 14:45 ` Vivek Gautam [not found] ` <1399723210-21569-1-git-send-email-gautam.vivek-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> 2014-05-12 14:08 ` Alan Stern 2014-05-12 14:08 ` Alan Stern 2014-05-12 14:08 ` Alan Stern
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=CAFp+6iFufCh6SVEF-HQA62ZsJGJGSebDSs28eqcrMy1X8e5vjA@mail.gmail.com \ --to=gautam.vivek@samsung.com \ --cc=gregkh@linuxfoundation.org \ --cc=kgene.kim@samsung.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=linux-tegra@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=shc_work@mail.ru \ --cc=stern@rowland.harvard.edu \ --cc=swarren@wwwdotorg.org \ --cc=thierry.reding@gmail.com \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.