Re: [Regression] pcie_wait_for_link_delay (1132.853 ms @ 5039.414431)

[Mika Westerberg <mika.westerberg@linux.intel.com>]

On Tue, Aug 06, 2019 at 09:56:27PM +0200, Matthias Andree wrote:
> Mika,
> 
> reviewing the code in your commit, can we double-check the "!" in "if
> (!pdev->imm_ready)" here?
> 
> > static int get_downstream_delay(struct pci_bus *bus)
> > {
> >         struct pci_dev *pdev;
> >         int min_delay = 100;
> >         int max_delay = 0;
> >
> >         list_for_each_entry(pdev, &bus->devices, bus_list) {
> >                 if (!pdev->imm_ready)
> 
> I have zero clue of PCI (Express), but reading other parts of your patch,
> I think min_delay should be zeroed if imm_ready is 1 (currently it is
> zeroed if imm_ready == 0),
> unless the header file has a misleading comment.

Agreed it should be pdev->imm_ready without the negation.

I'll send a revert for that commit later today so the two reported
issues caused by it should get fixed.