[PATCH v2 00/40] PCI: dwc: Driver clean-ups
Kishon Vijay Abraham I
kishon at ti.com
Tue Sep 29 01:23:09 EDT 2020
Hi,
On 16/09/20 1:24 pm, Michael Walle wrote:
> Am 2020-09-16 00:02, schrieb Rob Herring:
>> Can you try this? The link up check seemed unnecessary as it is racy.
>> What happens if the link goes down right after checking? That's the
>> only thing in the change that sticks out.
>>
>> diff --git a/drivers/pci/controller/dwc/pcie-designware-host.c
>> b/drivers/pci/controller/dwc/pcie-designware-host.c
>> index 317ff512f8df..afee1a0e8883 100644
>> --- a/drivers/pci/controller/dwc/pcie-designware-host.c
>> +++ b/drivers/pci/controller/dwc/pcie-designware-host.c
>> @@ -441,6 +441,9 @@ static void __iomem
>> *dw_pcie_other_conf_map_bus(struct pci_bus *bus,
>> struct pcie_port *pp = bus->sysdata;
>> struct dw_pcie *pci = to_dw_pcie_from_pp(pp);
>>
>> + if (!dw_pcie_link_up(pci))
>> + return NULL;
>> +
>> busdev = PCIE_ATU_BUS(bus->number) |
>> PCIE_ATU_DEV(PCI_SLOT(devfn)) |
>> PCIE_ATU_FUNC(PCI_FUNC(devfn));
>
> This will fix the issue.
This fix is required to get DRA7 EVM booting again in linux-next.
Thanks
Kishon
More information about the linux-amlogic
mailing list