Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-28 Thread Thierry Reding
On Tue, Aug 27, 2013 at 12:03:01PM -0600, Bjorn Helgaas wrote: > On Tue, Aug 27, 2013 at 10:11 AM, Stephen Warren > wrote: > > On 08/27/2013 02:14 AM, Thierry Reding wrote: > >> On Mon, Aug 19, 2013 at 02:49:07PM -0600, Bjorn Helgaas wrote: > >>> On Mon, Aug 19, 2013 at 2:12 PM, Thierry Reding >

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-28 Thread Thierry Reding
On Wed, Aug 28, 2013 at 07:02:16AM -0600, Bjorn Helgaas wrote: > On Tue, Aug 27, 2013 at 12:03 PM, Bjorn Helgaas wrote: > >> On 08/27/2013 02:14 AM, Thierry Reding wrote: > > >>> If Stephen's fine with it I suppose we could take pci-tegra.c > >>> driver changes through the Tegra tree. But I think

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-28 Thread Bjorn Helgaas
On Tue, Aug 27, 2013 at 12:03 PM, Bjorn Helgaas wrote: >> On 08/27/2013 02:14 AM, Thierry Reding wrote: >>> If Stephen's fine with it I suppose we could take pci-tegra.c >>> driver changes through the Tegra tree. But I think it'd be good if >>> we could still Cc you on patches so you're aware of

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-27 Thread Bjorn Helgaas
On Tue, Aug 27, 2013 at 10:11 AM, Stephen Warren wrote: > On 08/27/2013 02:14 AM, Thierry Reding wrote: >> On Mon, Aug 19, 2013 at 02:49:07PM -0600, Bjorn Helgaas wrote: >>> On Mon, Aug 19, 2013 at 2:12 PM, Thierry Reding >>> wrote: On Mon, Aug 19, 2013 at 02:04:24PM -0600, Bjorn Helgaas wro

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-27 Thread Stephen Warren
On 08/27/2013 02:14 AM, Thierry Reding wrote: > On Mon, Aug 19, 2013 at 02:49:07PM -0600, Bjorn Helgaas wrote: >> On Mon, Aug 19, 2013 at 2:12 PM, Thierry Reding >> wrote: >>> On Mon, Aug 19, 2013 at 02:04:24PM -0600, Bjorn Helgaas wrote: On Mon, Aug 19, 2013 at 6:02 AM, Thierry Reding

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-27 Thread Thierry Reding
On Mon, Aug 19, 2013 at 02:49:07PM -0600, Bjorn Helgaas wrote: > On Mon, Aug 19, 2013 at 2:12 PM, Thierry Reding > wrote: > > On Mon, Aug 19, 2013 at 02:04:24PM -0600, Bjorn Helgaas wrote: > >> On Mon, Aug 19, 2013 at 6:02 AM, Thierry Reding > >> wrote: > >> > >> > Bjorn, how do you want to handl

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-20 Thread Thierry Reding
On Tue, Aug 20, 2013 at 12:53:32PM -0600, Stephen Warren wrote: > On 08/20/2013 10:26 AM, Bjorn Helgaas wrote: > > On Tue, Aug 20, 2013 at 3:13 AM, Julia Lawall wrote: > >> From: Julia Lawall > >> > >> Use devm_ioremap_resource instead of devm_request_and_ioremap. > >> > >> This was done using th

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-20 Thread Stephen Warren
On 08/20/2013 10:26 AM, Bjorn Helgaas wrote: > On Tue, Aug 20, 2013 at 3:13 AM, Julia Lawall wrote: >> From: Julia Lawall >> >> Use devm_ioremap_resource instead of devm_request_and_ioremap. >> >> This was done using the semantic patch >> scripts/coccinelle/api/devm_ioremap_resource.cocci >> >> E

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-20 Thread Bjorn Helgaas
On Tue, Aug 20, 2013 at 3:13 AM, Julia Lawall wrote: > From: Julia Lawall > > Use devm_ioremap_resource instead of devm_request_and_ioremap. > > This was done using the semantic patch > scripts/coccinelle/api/devm_ioremap_resource.cocci > > Error-handling code was manually removed from the associ

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-20 Thread Julia Lawall
From: Julia Lawall Use devm_ioremap_resource instead of devm_request_and_ioremap. This was done using the semantic patch scripts/coccinelle/api/devm_ioremap_resource.cocci Error-handling code was manually removed from the associated calls to platform_get_resource. Adjust the comment at the thi

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Stephen Warren
On 08/19/2013 01:33 PM, Thierry Reding wrote: > On Mon, Aug 19, 2013 at 02:15:39PM +0200, Julia Lawall wrote: >> >> >> On Mon, 19 Aug 2013, Thierry Reding wrote: >> >>> On Mon, Aug 19, 2013 at 02:07:54PM +0200, Julia Lawall wrote: On Mon, 19 Aug 2013, Thierry Reding wrote:

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Bjorn Helgaas
On Mon, Aug 19, 2013 at 2:12 PM, Thierry Reding wrote: > On Mon, Aug 19, 2013 at 02:04:24PM -0600, Bjorn Helgaas wrote: >> On Mon, Aug 19, 2013 at 6:02 AM, Thierry Reding >> wrote: >> >> > Bjorn, how do you want to handle patches to the Tegra PCIe driver in the >> > future? Do you want me to prep

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Thierry Reding
On Mon, Aug 19, 2013 at 02:04:24PM -0600, Bjorn Helgaas wrote: > On Mon, Aug 19, 2013 at 6:02 AM, Thierry Reding > wrote: > > > Bjorn, how do you want to handle patches to the Tegra PCIe driver in the > > future? Do you want me to prepare a branch and pull from that or would > > you rather just t

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Bjorn Helgaas
On Mon, Aug 19, 2013 at 6:02 AM, Thierry Reding wrote: > Bjorn, how do you want to handle patches to the Tegra PCIe driver in the > future? Do you want me to prepare a branch and pull from that or would > you rather just take simple patches? I'm in the habit of applying patches from email, so th

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Thierry Reding
On Mon, Aug 19, 2013 at 02:15:39PM +0200, Julia Lawall wrote: > > > On Mon, 19 Aug 2013, Thierry Reding wrote: > > > On Mon, Aug 19, 2013 at 02:07:54PM +0200, Julia Lawall wrote: > > > > > > > > > On Mon, 19 Aug 2013, Thierry Reding wrote: > > > > > > > On Mon, Aug 19, 2013 at 01:20:35PM +0200,

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Julia Lawall
On Mon, 19 Aug 2013, Thierry Reding wrote: > On Mon, Aug 19, 2013 at 02:07:54PM +0200, Julia Lawall wrote: > > > > > > On Mon, 19 Aug 2013, Thierry Reding wrote: > > > > > On Mon, Aug 19, 2013 at 01:20:35PM +0200, Julia Lawall wrote: > > > > From: Julia Lawall > > > > > > > > Use devm_ioremap_r

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Thierry Reding
On Mon, Aug 19, 2013 at 02:07:54PM +0200, Julia Lawall wrote: > > > On Mon, 19 Aug 2013, Thierry Reding wrote: > > > On Mon, Aug 19, 2013 at 01:20:35PM +0200, Julia Lawall wrote: > > > From: Julia Lawall > > > > > > Use devm_ioremap_resource instead of devm_request_and_ioremap. > > > > > > This

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Julia Lawall
On Mon, 19 Aug 2013, Thierry Reding wrote: > On Mon, Aug 19, 2013 at 01:20:35PM +0200, Julia Lawall wrote: > > From: Julia Lawall > > > > Use devm_ioremap_resource instead of devm_request_and_ioremap. > > > > This was done using the semantic patch > > scripts/coccinelle/api/devm_ioremap_resourc

Re: [PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Thierry Reding
On Mon, Aug 19, 2013 at 01:20:35PM +0200, Julia Lawall wrote: > From: Julia Lawall > > Use devm_ioremap_resource instead of devm_request_and_ioremap. > > This was done using the semantic patch > scripts/coccinelle/api/devm_ioremap_resource.cocci > > Error-handling code was manually removed from

[PATCH 1/7] PCI: tegra: replace devm_request_and_ioremap by devm_ioremap_resource

2013-08-19 Thread Julia Lawall
From: Julia Lawall Use devm_ioremap_resource instead of devm_request_and_ioremap. This was done using the semantic patch scripts/coccinelle/api/devm_ioremap_resource.cocci Error-handling code was manually removed from the associated calls to platform_get_resource. Signed-off-by: Julia Lawall