Re: [PATCH] PCI: pciehp: Disable link and turn off slot power while removing

[Lukas Wunner <lukas@wunner.de>]

On Wed, May 06, 2026 at 02:16:10PM -0500, Mario Limonciello wrote:
> On 5/6/26 12:55, Lukas Wunner wrote:
> > Moreover there are products where Presence Detect is hardwired to zero,
> > see commit 80696f991424 ("PCI: pciehp: Tolerate Presence Detect hardwired
> > to zero").  So we have to rely on a link change to detect that a new device
> > has been plugged in after slot poweroff.  That won't work if the link is
> > disabled.
> 
> How would you feel about turning off link detect somewhere specific to the
> shutdown path?

I *think* shutdown is performed bottom-up in the hierarchy and so
this might work.  But it's not great to introduce such a change
for everyone just because a single product needs it.  It would
be better to root-cause the issue.  Maybe the vendor is able to
provide a BIOS update which resolves it?

Thanks,

Lukas