From b8a808f063985641477c39b257fdbae41fbf4ab1 Mon Sep 17 00:00:00 2001 From: Gavin Shan Date: Mon, 20 Feb 2017 10:57:51 +1100 Subject: astbmc/p8dnu: Enable PCI slot's power supply on PEX8718 in hot-add path This issue is reported from superMicro's "p8dnu" platform. PEX8718 is connected to PHB direct slot. We create dynamic PCI slots for its (2) downstream ports and all of them support surprise hotplug capability. The problem is power supply lost on hot-remove and it isn't turned on automatically on hot-add. It means the PCIe link behind the slot isn't up and the PCI adapter behind the slot can't be probed successfully. This fixes the issue by forcing to turn on the power supply on hardware when user (kernel) requests to do so, as we did for PEX9733. Reported-by: Hank Chang Signed-off-by: Gavin Shan Tested-by: Willie Liauw Signed-off-by: Stewart Smith --- platforms/astbmc/slots.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/platforms/astbmc/slots.c b/platforms/astbmc/slots.c index f28ad7d..8be7426 100644 --- a/platforms/astbmc/slots.c +++ b/platforms/astbmc/slots.c @@ -175,9 +175,11 @@ static void create_dynamic_slot(struct phb *phb, struct pci_device *pd) * isn't turned on automatically when inserting PCI adapter to * the slot at later point. We set a flag to the slot here, to * turn on the power supply in (suprise or managed) hot-add path. + * + * We have same issue with PEX8718 as above on "p8dnu" platform. */ - if (dt_node_is_compatible(dt_root, "supermicro,p8dnu") && - slot->pd && slot->pd->vdid == 0x973310b5) + if (dt_node_is_compatible(dt_root, "supermicro,p8dnu") && slot->pd && + (slot->pd->vdid == 0x973310b5 || slot->pd->vdid == 0x871810b5)) pci_slot_add_flags(slot, PCI_SLOT_FLAG_FORCE_POWERON); } -- cgit v1.1