Re: [PATCH v7 0/4] PCI: Add support for resetting the Root Ports in a platform specific way

Manivannan Sadhasivam
2026-03-11T15:14:32Z

On Wed, Mar 11, 2026 at 08:09:53PM +0530, Manivannan Sadhasivam wrote:
> On Wed, Mar 11, 2026 at 12:05:15PM +0100, Niklas Cassel wrote:
> > On Tue, Mar 10, 2026 at 07:31:58PM +0530, Manivannan Sadhasivam via B4 Relay wrote:
> > > Changes in v7:
> > > - Dropped Rockchip Root port reset patch due to reported issues. But the series
> > > works on other platforms as tested by others.
> >
> > Are you referring to
> >
> > ## On EP side:
> > # echo 0 > /sys/kernel/config/pci_ep/controllers/a40000000.pcie-ep/start && \
> > sleep 0.1 && echo 1 > /sys/kernel/config/pci_ep/controllers/a40000000.pcie-ep/start
> >
> > Then running pcitest only having 7 / 16 tests passed ?
> >
> > If so, isn't that a problem also for qcom?
> >
>
> No, tests are passing on my setup after link up.
>
> >
> > There is no chance that the patch:
> > "misc: pci_endpoint_test: Add AER error handlers"
> > improves things in this regard?
> >
> > Or will it simply avoid the "AER: device recovery failed" print?
> >
>
> Yes, as mentioned in the commit message, it just avoids the AER recovery failure
> message.
>

I also realized that Endpoint state is not saved in all the code paths. So the
pci_endpoint_test driver has to save/restore the state also. But it is still not
clear why that didn't help you.

Can you share the snapshot of the entire config space before and after reset
using 'lspci -xxxx -s "0000:01:00"'?

- Mani

--
மணிவண்ணன் சதாசிவம்