Re: [PATCH V2] PCI: rcar: Add the initialization of PCIe link in resume_noirq

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Lorenzo, Marek,

On Fri, Mar 22, 2019 at 1:18 PM Lorenzo Pieralisi
<lorenzo.pieralisi@xxxxxxx> wrote:
> On Fri, Mar 22, 2019 at 01:09:13PM +0100, Marek Vasut wrote:
> > On 3/22/19 12:31 PM, Lorenzo Pieralisi wrote:
> > > On Sun, Feb 17, 2019 at 02:24:41PM +0100, marek.vasut@xxxxxxxxx wrote:
> > >> From: Kazufumi Ikeda <kaz-ikeda@xxxxxxxxxxxxx>
> > >>
> > >> Reestablish the PCIe link very early in the resume process in case it
> > >> went down to prevent PCI accesses from hanging the bus. Such accesses
> > >> can happen early in the PCI resume process, in the resume_noirq, thus
> > >> the link must be reestablished in the resume_noirq callback of the
> > >> driver.
> > >
> > > This looks like a fix (most likely fixing initial S2R support, please
> > > help me chase the commit ID), should we consider it for stable kernels ?
> > >
> > > Without it I understand S2R is actually broken on platforms with this
> > > host bridge.
> > I don't think this ever worked, so it's hard to find a Fixes: commit for
> > this.
>
> If we want to send it to stable kernels we have to select which versions
> we are covering. I think the only options for a Fixes: tag are either
> the initial S2R support commit for the platforms this driver runs on
> or the initial driver commit that harks back to v3.16 AFAICS.

This only started to become an issue when support for arm64 platforms
was added, where PSCI may power down the SoC, right?

Hence:
Fixes: e015f88c368da1e6 ("PCI: rcar: Add support for R-Car H3 to pcie-rcar")

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



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux