Re: [PATCH] PCI: PCI_PROBE_ONLY clean-up

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

 



On Thu, Jun 23, 2016 at 05:16:47PM -0500, Bjorn Helgaas wrote:

> Lorenzo is changing the PCI_PROBE_ONLY case so the BARs and windows remain
> immutable, but we insert the resources into the iomem_resource tree.
> 
> The ideal thing would be to remove the use of PCI_PROBE_ONLY completely,
> and allow Linux to program BARs as necessary.  If the firmware *has*
> programmed the BARs, we don't change them unless we find something broken,
> so in most cases PCI_PROBE_ONLY is unnecessary.
> 
> There are several MIPS platforms (bcm1480, ip27, sb1250, virtio_guest, xlp,
> xlr) that set PCI_PROBE_ONLY for reasons I don't know.  These were added
> by:
> 
>   bcm1480
>     Andrew Isaacson <adi@xxxxxxxxxxxx>
>     dc41f94f7709 ("Support for the BCM1480 on-chip PCI-X bridge.")
> 
>   ip27
>     Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx>
>     96173a6c4ebc ("[MIPS] IP27: misc fixes")

Afair  I did originally implement the IP27 use of PCI_PROBE_ONLY.  The
problem is that it was not possible for the kernel to assign resources
properly on an IP27.  Also that would invalidate firmware configuration
information, so we had to live with whatever the firmware (mis)configured
for us.  Afair - it's a darn long time ...  I think the reasoning for
the BCM1480 was similar but Andy will hopefully recall the details.

  Ralf
--
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