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 -- மணிவண்ணன் சதாசிவம்
