Re: [PATCH] PCI: cadence: Fixed cdns_pcie_host_link_setup return value.

From: Hans Zhang
Date: Fri Dec 20 2024 - 02:28:53 EST




On 12/19/24 08:35, Manivannan Sadhasivam wrote:

We have 5 PCIe controllers, and if a few of them are not connected to the
device. And it will affect the boot time.


Why are you enabling all controllers? Can't you just enable the ones you know
the endpoints are going to be connected? I'm just trying to see if we can avoid
having a quirk.

Our SOC has a PC product situation, and there may be PCIe slots on the PCB, but the device may not be plugged in. So we need to enable all ports.

If you do not know, then you need to introduce a quirk for your platform.
But that requires your controller driver to be upstreamed. We cannot provide
hooks for downstream drivers in upstream.

Our controller driver currently has no plans for upstream and needs to wait for notification from the boss.


Regards
Hans