]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "PCI: armada8k: Add support for gpio controlled reset signal"
authorBaruch Siach <baruch@tkos.co.il>
Thu, 31 Jan 2019 21:00:18 +0000 (15:00 -0600)
committerBjorn Helgaas <bhelgaas@google.com>
Thu, 31 Jan 2019 21:07:29 +0000 (15:07 -0600)
commitbf4d333b1407b0a17b6bff5332eb6e822c4a5581
treeae61e6110f9bcecc24c77595fc69c51f2066f0a5
parentc49b2f1ecf118441059fc888b6e7521053058839
Revert "PCI: armada8k: Add support for gpio controlled reset signal"

Revert commit 6fd1cc7b76 ("PCI: armada8k: Add support for gpio controlled
reset signal").

That commit breaks boot on Macchiatobin board when a Mellanox NIC is
present in the PCIe slot.

It turns out that full reset cycle requires first comphy serdes
initialization. Reset signal toggle without comphy initialization makes
access to PCI configuration registers stall indefinitely. U-Boot toggles
the Macchiatobin PCIe reset line already at boot, after initializing the
comphy serdes.

So while commit 6fd1cc7b76 ("PCI: armada8k: Add support for gpio controlled
reset signal") enables PCIe on platforms that U-Boot does not touch the
reset line (like Clearfog GT-8K), it breaks PCIe (and boot) on the
Macchiatobin board.

Revert commit 6fd1cc7b76 ("PCI: armada8k: Add support for gpio controlled
reset signal") entirely to fix the Macchiatobin regression.

Reported-by: Sven Auhagen <sven.auhagen@voleatech.de>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
drivers/pci/controller/dwc/pcie-armada8k.c