Hi Geert-san, > From: Geert Uytterhoeven, Sent: Wednesday, October 9, 2019 8:31 PM > > Hi Shimoda-san, > > On Wed, Oct 9, 2019 at 1:05 PM Yoshihiro Shimoda > <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote: <snip> > > @@ -615,6 +616,8 @@ static int rcar_pcie_hw_init(struct rcar_pcie *pcie) > > if (IS_ENABLED(CONFIG_PCI_MSI)) > > rcar_pci_write_reg(pcie, 0x801f0000, PCIEMSITXR); > > > > + rcar_rmw32(pcie, MACCTLR, MACCTLR_RESERVED, 0); > > + > > /* Finish initialization - establish a PCI Express link */ > > rcar_pci_write_reg(pcie, CFINIT, PCIETCTLR); > > I guess the same should be added to rcar_pcie_resume_noirq(), > as s2ram on R-Car Gen3 powers down the SoC? Thank you for the pointed it out! You're correct. So, I'll update the patch. Best regards, Yoshihiro Shimoda > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds