[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <s5hv9sidyu5.wl-tiwai@suse.de>
Date: Mon, 21 Oct 2019 15:15:14 +0200
From: Takashi Iwai <tiwai@...e.de>
To: Kai-Heng Feng <kai.heng.feng@...onical.com>
Cc: bhelgaas@...gle.com, linux-pci@...r.kernel.org,
alsa-devel@...a-project.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6 1/2] PCI: Add a helper to check Power Resource Requirements _PR3 existence
On Fri, 18 Oct 2019 10:45:43 +0200,
Takashi Iwai wrote:
>
> On Fri, 18 Oct 2019 10:30:11 +0200,
> Kai-Heng Feng wrote:
> >
> >
> >
> > > On Oct 18, 2019, at 16:18, Takashi Iwai <tiwai@...e.de> wrote:
> > >
> > > On Fri, 18 Oct 2019 09:38:47 +0200,
> > > Kai-Heng Feng wrote:
> > >>
> > >> A driver may want to know the existence of _PR3, to choose different
> > >> runtime suspend behavior. A user will be add in next patch.
> > >>
> > >> This is mostly the same as nouveau_pr3_present().
> > >>
> > >> Reported-by: kbuild test robot <lkp@...el.com>
> > >
> > > It's confusing -- this particular change isn't reported by the test
> > > bot, but only about the lack of the CONFIG_ACPI ifdef.
> >
> > Hmm, because the test bot asked to add the tag.
>
> Yes, but it's only if you add a new fix patch on top of it.
> You can write some their credit, too, but basically it'd be enough to
> point to the Link tag to the thread.
>
> > If it's not appropriate will you drop it? I can also send a v7.
>
> I can modify in my side.
Now I merged both patches to for-next branch.
thanks,
Takashi
Powered by blists - more mailing lists