From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v2 2/2] PCI: of: Ignore resources with failed translation Date: Tue, 13 Oct 2015 15:10:57 -0500 Message-ID: References: <3604931244806840dd12ff9663671b52d2acd951.1444288748.git.p.fedin@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <3604931244806840dd12ff9663671b52d2acd951.1444288748.git.p.fedin@samsung.com> Sender: linux-pci-owner@vger.kernel.org To: Pavel Fedin Cc: "linux-pci@vger.kernel.org" , "devicetree@vger.kernel.org" , Frank Rowand , Grant Likely List-Id: devicetree@vger.kernel.org On Thu, Oct 8, 2015 at 2:24 AM, Pavel Fedin wrote: > This patch allows PCI host controller to function even if part of resources > is unusable for some reason. An example is non-LPAE kernel on a machine > which has some 64-bit resources. Unusable resources will be just skipped > instead of a complete failure. > > Signed-off-by: Pavel Fedin Applied, thanks. Rob > --- > drivers/of/of_pci.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/of/of_pci.c b/drivers/of/of_pci.c > index 5751dc5..ea7c2b6 100644 > --- a/drivers/of/of_pci.c > +++ b/drivers/of/of_pci.c > @@ -223,8 +223,10 @@ int of_pci_get_host_bridge_resources(struct device_node *dev, > } > > err = of_pci_range_to_resource(&range, dev, res); > - if (err) > - goto conversion_failed; > + if (err) { > + kfree(res); > + continue; > + } > > if (resource_type(res) == IORESOURCE_IO) { > if (!io_base) { > -- > 2.4.4 >