Re: [PATCH v2 0/3] Marvell PCIe driver improvements

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

 



On Thu, May 23, 2013 at 11:01:10AM -0600, Bjorn Helgaas wrote:
> On Thu, May 23, 2013 at 8:32 AM, Thomas Petazzoni
> <thomas.petazzoni@xxxxxxxxxxxxxxxxxx> wrote:
> > Bjorn, Jason, Andrew, Gregory,
> >
> > Here is an updated version of the PCIe improvements.
> >
> > Changes since v1:
> >
> >  * Fix some typos in the comments of PATCH 2, noticed by Bjorn
> >    Helgaas.
> >
> >  * Rework the two last patches (first one was fixing the capability
> >    list, the other one the emulation of the devsel bits in the status
> >    register), after the comments of Bjorn. In fact, it turns out that
> >    the status register is composed of read-only bits for which we want
> >    to emulate a value of zero, and the other bits are write-1-to-clear
> >    bits that expose error conditions that we do not handle in the
> >    emulated bridge. So, in summary, emulating a 0 status register on
> >    reads, and ignoring writes to the status register is the simplest
> >    solution. If we later on implement a capability list, we can use a
> >    more clever emulation of the status register, but for the moment,
> >    this simple solution is enough. Therefore, the two last patches are
> >    now one single, simpler patch.
> >
> > I'd like those improvements to be kept separated from the original
> > PCIe driver itself: while the PCIe driver has been around and reviewed
> > for a long time, those improvements are newer. And I clearly do not
> > want the PCIe driver to miss 3.11 because of any problem that could be
> > found in those additional improvements.
> >
> > The main improvement being brought here is that the PCI-to-PCI bridge
> > logic is fixed/extended to properly support physical PCIe bridges that
> > are connected on a PCIe interface of a Marvell board. Without this
> > improvement, only the devices connected directly to the PCIe
> > interfaces of the board are properly enumerated. Any device that would
> > sit beyond a physical bridge is not visible.
> >
> > Bjorn, with your Acked-by, could the Marvell maintainers include those
> > patches in their branch, merged through arm-soc? They already have the
> > Marvell PCIe driver itself, so I believe it makes sense to merge those
> > improvements through the same path.
> >
> > Jason, those patches have been prepared on top of my marvell-pcie-v10
> > branch, I hope that's ok for you. If you want me to rebase them on
> > some other branch in which you have integrated the PCIe driver, don't
> > hesitate to tell me to do so. That said, since those patches are only
> > touching the driver itself and no other file, they should not cause
> > any conflict with other changes.
> >
> > Thanks,
> >
> > Thomas
> >
> > Thomas Petazzoni (3):
> >   pci: mvebu: no longer fake the slot location of downstream devices
> >   pci: mvebu: allow the enumeration of devices beyond physical bridges
> >   pci: mvebu: fix the emulation of the status register
> >
> >  drivers/pci/host/pci-mvebu.c |   55 +++++++++++++++++++++++++++++++++---------
> >  1 file changed, 44 insertions(+), 11 deletions(-)
> 
> For all three:
> 
> Acked-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
> 
> Feel free to merge these through the arm-soc tree or whatever makes
> sense for you.  Nobody else is working on this file, so there
> shouldn't be any conflicts.

Thanks, Bjorn!

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