On Wed, Jan 17, 2018 at 08:27:39AM -0800, Sinan Kaya wrote: > On 1/17/2018 5:37 AM, Oza Pawandeep wrote: > > +static bool dpc_wait_link_active(struct pci_dev *pdev) > > +{ > > I think you can also make this function common instead of making another copy here. > Of course, this would be another patch. It is actually very similar to __pcie_wait_link_active in pciehp_hpc.c, so there's some opprotunity to make even more common code.