All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Bjorn Helgaas <bhelgaas@google.com>,
	"Liviu.Dudau@arm.com" <Liviu.Dudau@arm.com>,
	nofooter <nofooter@xilinx.com>,
	"thomas.petazzoni@free-electrons.com" 
	<thomas.petazzoni@free-electrons.com>
Subject: RE: Purpose of pci_remap_iospace
Date: Wed, 13 Jul 2016 15:16:21 +0000	[thread overview]
Message-ID: <8520D5D51A55D047800579B094147198258B8B85@XAP-PVEXMBX01.xlnx.xilinx.com> (raw)
In-Reply-To: <4264647.cRxqIJ3z77@wuerfel>

> Subject: Re: Purpose of pci_remap_iospace
>
> On Wednesday, July 13, 2016 12:30:44 PM CEST Bharat Kumar Gogada wrote:
> >  > On Wednesday, July 13, 2016 8:11:56 AM CEST Bharat Kumar Gogada
> wrote:
> > > > > Subject: Re: Purpose of pci_remap_iospace
> > >
> > > I notice you have 1MB of I/O space here
> > >
> > > > Kernel Boot log:
> > > > [    2.345294] nwl-pcie fd0e0000.pcie: Link is UP
> > > > [    2.345339] PCI host bridge /amba/pcie@fd0e0000 ranges:
> > > > [    2.345356]   No bus range found for /amba/pcie@fd0e0000, using
> [bus
> > > 00-ff]
> > > > [    2.345382]    IO 0xe0000000..0xe00fffff -> 0x00000000
> > > > [    2.345401]   MEM 0xe0100000..0xeeffffff -> 0xe0100000
> > > > [    2.345498] nwl-pcie fd0e0000.pcie: PCI host bridge to bus 0000:00
> > > > [    2.345517] pci_bus 0000:00: root bus resource [bus 00-ff]
> > > > [    2.345533] pci_bus 0000:00: root bus resource [io  0x0000-0xfffff]
> > >
> > > and all of it gets mapped by the PCI core. Usually you only have 64K
> > > of I/O space per host bridge, and the PCI core should perhaps not
> > > try to map all of it, though I don't think this is actually your problem here.
> > >
> > > > [    2.345550] pci_bus 0000:00: root bus resource [mem 0xe0100000-
> > > 0xeeffffff]
> > > > [    2.345770] pci 0000:00:00.0: cannot attach to SMMU, is it on the same
> > > bus?
> > > > [    2.345786] iommu: Adding device 0000:00:00.0 to group 1
> > > > [    2.346142] pci 0000:01:00.0: cannot attach to SMMU, is it on the same
> > > bus?
> > > > [    2.346158] iommu: Adding device 0000:01:00.0 to group 1
> > > > [    2.346213] pci 0000:00:00.0: BAR 8: assigned [mem 0xe0100000-
> > > 0xe02fffff]
> > > > [    2.346234] pci 0000:01:00.0: BAR 0: assigned [mem 0xe0100000-
> 0xe01fffff
> > > 64bit]
> > > > [    2.346268] pci 0000:01:00.0: BAR 2: assigned [mem 0xe0200000-
> 0xe02fffff
> > > 64bit]
> > > > [    2.346300] pci 0000:01:00.0: BAR 4: no space for [io  size 0x0040]
> > > > [    2.346316] pci 0000:01:00.0: BAR 4: failed to assign [io  size 0x0040]
> > > > [    2.346333] pci 0000:00:00.0: PCI bridge to [bus 01-0c]
> > > > [    2.346350] pci 0000:00:00.0:   bridge window [mem 0xe0100000-
> > > 0xe02fffff]
> > > >
> > > > IO assignment fails.
> > >
> > > I would guess that the I/O space is not registered correctly. Is
> > > this drivers/pci/host/pcie-xilinx.c ? We have had problems with this
> > > in the past, since almost nobody uses I/O space and it requires
> > > several steps to all be done correctly.
> > >
> > Thanks Arnd.
> >
> > we are testing using drivers/pci/host/pcie-xilinx-nwl.c.
>
> According to Documentation/devicetree/bindings/pci/xilinx-nwl-pcie.txt,
> this hardware does not support I/O space.

We received a newer IP version with IO support, so we are trying to test this feature.
>
> Is this on ARM or microblaze?

It is ARM 64-bit.

> This has neither the PCI memory nor the I/O resource, it looks like you never
> call pci_add_resource_offset() to start with, or maybe it fails for some
> reason.

I see that above API is used in ARM drivers, do we need to do it in ARM64 also ?

Regards,
Bharat



This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.

  reply	other threads:[~2016-07-13 15:17 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-12  6:57 Purpose of pci_remap_iospace Bharat Kumar Gogada
2016-07-12  6:57 ` Bharat Kumar Gogada
2016-07-12  8:31 ` Arnd Bergmann
2016-07-12  8:31   ` Arnd Bergmann
2016-07-12  8:40   ` Bharat Kumar Gogada
2016-07-12  8:40     ` Bharat Kumar Gogada
2016-07-13  8:11   ` Bharat Kumar Gogada
2016-07-13  8:11     ` Bharat Kumar Gogada
2016-07-13  8:30     ` Arnd Bergmann
2016-07-13  8:30       ` Arnd Bergmann
2016-07-13 12:30       ` Bharat Kumar Gogada
2016-07-13 12:30         ` Bharat Kumar Gogada
2016-07-13 13:28         ` Arnd Bergmann
2016-07-13 13:28           ` Arnd Bergmann
2016-07-13 15:16           ` Bharat Kumar Gogada [this message]
2016-07-13 15:16             ` Bharat Kumar Gogada
2016-07-13 15:28             ` Arnd Bergmann
2016-07-13 15:28               ` Arnd Bergmann
2016-07-13 15:42               ` Liviu.Dudau
2016-07-13 15:42                 ` Liviu.Dudau
2016-07-13 16:13             ` Lorenzo Pieralisi
2016-07-13 16:13               ` Lorenzo Pieralisi
2016-07-13 13:46         ` Lorenzo Pieralisi
2016-07-13 13:46           ` Lorenzo Pieralisi
2016-07-14  6:03           ` Bharat Kumar Gogada
2016-07-14  6:03             ` Bharat Kumar Gogada
2016-07-14 13:32           ` Bharat Kumar Gogada
2016-07-14 13:32             ` Bharat Kumar Gogada
2016-07-14 14:56             ` Lorenzo Pieralisi
2016-07-14 14:56               ` Lorenzo Pieralisi
2016-07-14 15:05               ` Bharat Kumar Gogada
2016-07-14 15:05                 ` Bharat Kumar Gogada
2016-07-14 15:20                 ` Lorenzo Pieralisi
2016-07-14 15:20                   ` Lorenzo Pieralisi
2016-07-14 15:12               ` Arnd Bergmann
2016-07-14 15:12                 ` Arnd Bergmann
2016-07-14 15:27                 ` Lorenzo Pieralisi
2016-07-14 15:27                   ` Lorenzo Pieralisi
2016-07-15  5:21               ` Bharat Kumar Gogada
2016-07-15  5:21                 ` Bharat Kumar Gogada
2016-07-15  6:55                 ` Arnd Bergmann
2016-07-15  6:55                   ` Arnd Bergmann
2016-07-15  6:55                   ` Arnd Bergmann
2016-07-13 13:24     ` Liviu.Dudau
2016-07-13 13:24       ` Liviu.Dudau

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=8520D5D51A55D047800579B094147198258B8B85@XAP-PVEXMBX01.xlnx.xilinx.com \
    --to=bharat.kumar.gogada@xilinx.com \
    --cc=Liviu.Dudau@arm.com \
    --cc=arnd@arndb.de \
    --cc=bhelgaas@google.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=nofooter@xilinx.com \
    --cc=thomas.petazzoni@free-electrons.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: link
Be 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.