Re: [PATCH v2] PCI: mvebu - Support a bridge with no IO port window

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Oct 17, 2013 at 08:41:12AM -0600, Jason Gunthorpe wrote:
> On Thu, Oct 17, 2013 at 09:10:31AM -0400, Jason Cooper wrote:
> > On Tue, Oct 15, 2013 at 02:16:22PM -0600, Jason Gunthorpe wrote:
> > > Make pcie-io-aperture and the IO port MBUS ID in ranges optional.
> > > If not provided the bridge reports to Linux that IO space mapping is
> > > not supported and refuses to configure an IO mbus window.
> > > 
> > > This allows both complete disable (do not specify pcie-io-aperture) and
> > > per-port disable (do not specify a IO target ranges entry for the port)
> > > 
> > > Most PCIE devices these days do not require IO support to function,
> > > so having an option to disable it in the driver is useful.
> > > 
> > > Signed-off-by: Jason Gunthorpe <jgunthorpe@xxxxxxxxxxxxxxxxxxxx>
> > >  drivers/pci/host/pci-mvebu.c | 64 +++++++++++++++++++++++++++++---------------
> > >  1 file changed, 43 insertions(+), 21 deletions(-)
> > 
> > Applied to mvebu/drivers
> 
> If possible, Thomas should test this, IMHO. My hardware here
> completely lacks IO port windows so I cannot meaningfully test that
> aspect of the change.

Agreed, Thomas and I were just discussing this on #mvlinux.  I took this
in to get it as much coverage in -next as possible.  Since we are getting
close to the end of the merge window and Stephen Rothwell is on
vacation, so -next testing isn't as smooth and predictable as usual.

Once Thomas (and others, don't be shy!) has tested it, I'll either add
the Tested-by or drop it for a new version.

thx,

Jason.
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux