From: Lukas Wunner <lukas@wunner.de>
To: Daniel Vetter <daniel@ffwll.ch>,
Alex Deucher <alexander.deucher@amd.com>,
Sean Paul <seanpaul@chromium.org>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Gustavo Padovan <gustavo@padovan.org>,
Bjorn Helgaas <bhelgaas@google.com>
Cc: alsa-devel@alsa-project.org, Martin Lopatar <lopin@dataplex.cz>,
nouveau@lists.freedesktop.org, Maik Freudenberg <hhfeuer@gmx.de>,
"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
Denis Lisov <dennis.lissov@gmail.com>,
dri-devel@lists.freedesktop.org,
Daniel Drake <dan@reactivated.net>,
Hans de Goede <hdegoede@redhat.com>,
Kai-Heng Feng <kai.heng.feng@canonical.com>,
Peter Wu <peter@lekensteyn.nl>,
linux-pci@vger.kernel.org, Takashi Iwai <tiwai@suse.com>
Subject: Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA
Date: Sun, 11 Mar 2018 16:55:49 +0100 [thread overview]
Message-ID: <20180311155549.GA11806@wunner.de> (raw)
In-Reply-To: <20180306102940.GG22212@phenom.ffwll.local>
On Tue, Mar 06, 2018 at 11:29:40AM +0100, Daniel Vetter wrote:
> On Sat, Mar 03, 2018 at 10:53:24AM +0100, Lukas Wunner wrote:
> > Modernize vga_switcheroo by using a device link to enforce a runtime PM
> > dependency from an HDA controller to the GPU it's integrated into, v2.
> >
> > https://github.com/l1k/linux/commits/switcheroo_devlink_v2
>
> This all looks really reasonable and like a good cleanup, but it's a bit
> too much detail so I'll punt review to someone else with more clue.
Patches [3/7] to [7/7] were reviewed by Peter Wu, the HDA bits in
patch [5/7] additionally by Takashi.
Patch [2/7] was acked by Bjorn. There was no ack for patch [1/7]
(authored by Rafael), but it adressed the objection Bjorn raised
against my original patch, so I'm assuming Bjorn is okay with it.
(Bjorn, please let me know if that isn't the case.)
The series has been tested on 5 systems, which raises the confidence:
2x AMD PowerXpress (Mike Lothian, Kai Heng Feng)
2x Nvidia Optimus (Denis Lisov, Peter Wu)
1x MacBook Pro
The issues found during Peter Wu's thorough testing appear to all
be unrelated to this series, as per my e-mail yesterday.
If there are no objections, I plan to push the series to
drm-misc-next by the middle of the coming week so that it
would still catch the last train to 4.17.
Thanks,
Lukas
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2018-03-11 15:55 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-03 9:53 [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA Lukas Wunner
[not found] ` <cover.1520068884.git.lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
2018-03-03 9:53 ` [PATCH v2 1/7] PCI: Restore config space on runtime resume despite being unbound Lukas Wunner
[not found] ` <92fb6e6ae2730915eb733c08e2f76c6a313e3860.1520068884.git.lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org>
2018-03-13 17:34 ` Bjorn Helgaas
2018-03-03 9:53 ` [PATCH v2 5/7] vga_switcheroo: Use device link for HDA controller Lukas Wunner
2018-03-03 9:53 ` [PATCH v2 6/7] vga_switcheroo: Let HDA autosuspend on mux change Lukas Wunner
2018-03-03 9:53 ` [PATCH v2 2/7] PCI: Make pci_wakeup_bus() & pci_bus_set_current_state() public Lukas Wunner
2018-03-03 9:53 ` [PATCH v2 3/7] vga_switcheroo: Update PCI current_state on power change Lukas Wunner
2018-03-03 9:53 ` [PATCH v2 7/7] drm/nouveau: Runtime suspend despite HDA being unbound Lukas Wunner
2018-03-05 20:58 ` [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA Peter Wu
2018-03-10 6:09 ` Lukas Wunner
2018-03-03 9:53 ` [PATCH v2 4/7] vga_switcheroo: Deduplicate power state tracking Lukas Wunner
2018-03-06 10:29 ` [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA Daniel Vetter
2018-03-11 15:55 ` Lukas Wunner [this message]
2018-03-12 16:54 ` Daniel Vetter
2018-03-12 20:55 ` Lukas Wunner
2018-03-12 21:25 ` Takashi Iwai
2018-03-13 15:05 ` Daniel Vetter
2018-03-13 17:36 ` Bjorn Helgaas
2018-03-14 6:07 ` Lukas Wunner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180311155549.GA11806@wunner.de \
--to=lukas@wunner.de \
--cc=alexander.deucher@amd.com \
--cc=alsa-devel@alsa-project.org \
--cc=bhelgaas@google.com \
--cc=dan@reactivated.net \
--cc=daniel@ffwll.ch \
--cc=dennis.lissov@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=gustavo@padovan.org \
--cc=hdegoede@redhat.com \
--cc=hhfeuer@gmx.de \
--cc=kai.heng.feng@canonical.com \
--cc=linux-pci@vger.kernel.org \
--cc=lopin@dataplex.cz \
--cc=maarten.lankhorst@linux.intel.com \
--cc=nouveau@lists.freedesktop.org \
--cc=peter@lekensteyn.nl \
--cc=rafael.j.wysocki@intel.com \
--cc=seanpaul@chromium.org \
--cc=tiwai@suse.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox