Hello, On Thu, 24 Nov 2022 15:58:27 +0200 Vadym Kochan <vadym.kochan@xxxxxxxxxxx> wrote: > -static int armada8k_pcie_start_link(struct dw_pcie *pci) > -{ > - u32 reg; > - > - /* Start LTSSM */ > - reg = dw_pcie_readl_dbi(pci, PCIE_GLOBAL_CONTROL_REG); > - reg |= PCIE_APP_LTSSM_EN; > - dw_pcie_writel_dbi(pci, PCIE_GLOBAL_CONTROL_REG, reg); > - > - return 0; > -} So this code is going away, but I don't see it being re-added anywhere. I don't think anything in the code you are adding that sets the LTSSM_EN bit in PCIE_GLOBAL_CONTROL_REG. Am I missing something? Thomas -- Thomas Petazzoni, co-owner and CEO, Bootlin Embedded Linux and Kernel engineering and training https://bootlin.com